* [FFmpeg-devel] [PATCH] avcodec/mjpegdec: add frame threading for mjpeg decoder
@ 2023-09-07 16:44 Paul B Mahol
2023-09-07 17:07 ` Andreas Rheinhardt
2023-09-07 21:17 ` Michael Niedermayer
0 siblings, 2 replies; 7+ messages in thread
From: Paul B Mahol @ 2023-09-07 16:44 UTC (permalink / raw)
To: FFmpeg development discussions and patches
[-- Attachment #1: Type: text/plain, Size: 10 bytes --]
Attached.
[-- Attachment #2: 0001-avcodec-mjpegdec-add-frame-threading-for-mjpeg-decod.patch --]
[-- Type: text/x-patch, Size: 1742 bytes --]
From 2cff86b07b56d2923f60923e7dd21a546c2cc6fa Mon Sep 17 00:00:00 2001
From: Paul B Mahol <onemda@gmail.com>
Date: Thu, 7 Sep 2023 18:42:03 +0200
Subject: [PATCH] avcodec/mjpegdec: add frame threading for mjpeg decoder
Signed-off-by: Paul B Mahol <onemda@gmail.com>
---
libavcodec/mjpegdec.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/libavcodec/mjpegdec.c b/libavcodec/mjpegdec.c
index 29f281231c..cc38b6c4a9 100644
--- a/libavcodec/mjpegdec.c
+++ b/libavcodec/mjpegdec.c
@@ -52,12 +52,12 @@
#include "jpeglsdec.h"
#include "profiles.h"
#include "put_bits.h"
+#include "thread.h"
#include "tiff.h"
#include "exif.h"
#include "bytestream.h"
#include "tiff_common.h"
-
static int init_default_huffman_tables(MJpegDecodeContext *s)
{
static const struct {
@@ -753,8 +753,8 @@ int ff_mjpeg_decode_sof(MJpegDecodeContext *s)
}
av_frame_unref(s->picture_ptr);
- if (ff_get_buffer(s->avctx, s->picture_ptr, AV_GET_BUFFER_FLAG_REF) < 0)
- return -1;
+ if ((ret = ff_thread_get_buffer(s->avctx, s->picture_ptr, AV_GET_BUFFER_FLAG_REF)) < 0)
+ return ret;
s->picture_ptr->pict_type = AV_PICTURE_TYPE_I;
s->picture_ptr->flags |= AV_FRAME_FLAG_KEY;
s->got_picture = 1;
@@ -3006,7 +3006,7 @@ const FFCodec ff_mjpeg_decoder = {
.close = ff_mjpeg_decode_end,
FF_CODEC_DECODE_CB(ff_mjpeg_decode_frame),
.flush = decode_flush,
- .p.capabilities = AV_CODEC_CAP_DR1,
+ .p.capabilities = AV_CODEC_CAP_DR1 | AV_CODEC_CAP_FRAME_THREADS,
.p.max_lowres = 3,
.p.priv_class = &mjpegdec_class,
.p.profiles = NULL_IF_CONFIG_SMALL(ff_mjpeg_profiles),
--
2.39.1
[-- Attachment #3: Type: text/plain, Size: 251 bytes --]
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [FFmpeg-devel] [PATCH] avcodec/mjpegdec: add frame threading for mjpeg decoder
2023-09-07 16:44 [FFmpeg-devel] [PATCH] avcodec/mjpegdec: add frame threading for mjpeg decoder Paul B Mahol
@ 2023-09-07 17:07 ` Andreas Rheinhardt
2023-09-07 17:34 ` Paul B Mahol
2023-09-07 21:17 ` Michael Niedermayer
1 sibling, 1 reply; 7+ messages in thread
From: Andreas Rheinhardt @ 2023-09-07 17:07 UTC (permalink / raw)
To: ffmpeg-devel
Paul B Mahol:
> Attached.
>
>
Haven't you sent a patch exactly like this before, which led to Michael
Niedermayer providing a command line where this changes the output?
- Andreas
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [FFmpeg-devel] [PATCH] avcodec/mjpegdec: add frame threading for mjpeg decoder
2023-09-07 17:07 ` Andreas Rheinhardt
@ 2023-09-07 17:34 ` Paul B Mahol
2023-09-07 17:48 ` Paul B Mahol
0 siblings, 1 reply; 7+ messages in thread
From: Paul B Mahol @ 2023-09-07 17:34 UTC (permalink / raw)
To: FFmpeg development discussions and patches
On Thu, Sep 7, 2023 at 7:21 PM Andreas Rheinhardt <
andreas.rheinhardt@outlook.com> wrote:
> Paul B Mahol:
> > Attached.
> >
> >
>
> Haven't you sent a patch exactly like this before, which led to Michael
> Niedermayer providing a command line where this changes the output?
>
That was not me.
But I know what thread you are referring to.
> - Andreas
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [FFmpeg-devel] [PATCH] avcodec/mjpegdec: add frame threading for mjpeg decoder
2023-09-07 17:34 ` Paul B Mahol
@ 2023-09-07 17:48 ` Paul B Mahol
0 siblings, 0 replies; 7+ messages in thread
From: Paul B Mahol @ 2023-09-07 17:48 UTC (permalink / raw)
To: FFmpeg development discussions and patches
On Thu, Sep 7, 2023 at 7:34 PM Paul B Mahol <onemda@gmail.com> wrote:
>
>
> On Thu, Sep 7, 2023 at 7:21 PM Andreas Rheinhardt <
> andreas.rheinhardt@outlook.com> wrote:
>
>> Paul B Mahol:
>> > Attached.
>> >
>> >
>>
>> Haven't you sent a patch exactly like this before, which led to Michael
>> Niedermayer providing a command line where this changes the output?
>>
>
> That was not me.
>
> But I know what thread you are referring to.
>
Checked that file, my patch does not fix nor break playing of that .avi
That AVI with mjpeg codec is not decoding with latest FFmpeg master code.
Even if I extract mjpeg losslessly from avi It still decodes just fine
without and with this patch applied.
>
>
>> - Andreas
>>
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel@ffmpeg.org
>> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>> To unsubscribe, visit link above, or email
>> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
>>
>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [FFmpeg-devel] [PATCH] avcodec/mjpegdec: add frame threading for mjpeg decoder
2023-09-07 16:44 [FFmpeg-devel] [PATCH] avcodec/mjpegdec: add frame threading for mjpeg decoder Paul B Mahol
2023-09-07 17:07 ` Andreas Rheinhardt
@ 2023-09-07 21:17 ` Michael Niedermayer
2023-09-07 21:39 ` Paul B Mahol
1 sibling, 1 reply; 7+ messages in thread
From: Michael Niedermayer @ 2023-09-07 21:17 UTC (permalink / raw)
To: FFmpeg development discussions and patches
[-- Attachment #1.1: Type: text/plain, Size: 2774 bytes --]
On Thu, Sep 07, 2023 at 06:44:40PM +0200, Paul B Mahol wrote:
> Attached.
> mjpegdec.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
> f65de11bf58ea36b071c3501d4fa700d242edf21 0001-avcodec-mjpegdec-add-frame-threading-for-mjpeg-decod.patch
> From 2cff86b07b56d2923f60923e7dd21a546c2cc6fa Mon Sep 17 00:00:00 2001
> From: Paul B Mahol <onemda@gmail.com>
> Date: Thu, 7 Sep 2023 18:42:03 +0200
> Subject: [PATCH] avcodec/mjpegdec: add frame threading for mjpeg decoder
>
> Signed-off-by: Paul B Mahol <onemda@gmail.com>
> ---
> libavcodec/mjpegdec.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
./ffmpeg -v -1 -i tickets/1915/m_noint.avi -f framecrc -an -t 1 -
Here the timestamps change and the files output does not look healthy
(looks as if one field is behind)
@@ -645,18 +645,22 @@
#codec_id 0: rawvideo
#dimensions 0: 640x480
#sar 0: 0/1
-0, 0, 0, 1, 614400, 0x51734de4
-0, 2, 2, 1, 614400, 0x9dfe6e90
-0, 4, 4, 1, 614400, 0x16d1f0d5
-0, 6, 6, 1, 614400, 0x67cbb0f0
-0, 8, 8, 1, 614400, 0x40f5223e
-0, 10, 10, 1, 614400, 0xef9a9ba8
-0, 12, 12, 1, 614400, 0x415cb69a
-0, 14, 14, 1, 614400, 0xbebf1993
-0, 16, 16, 1, 614400, 0xe4495f0f
-0, 18, 18, 1, 614400, 0x05a06480
-0, 20, 20, 1, 614400, 0x323bb5eb
-0, 22, 22, 1, 614400, 0x941b44ff
+0, 0, 0, 1, 614400, 0xb7f88313
+0, 1, 1, 1, 614400, 0xeda529e0
+0, 2, 2, 1, 614400, 0x29d53819
+0, 3, 3, 1, 614400, 0x95619af7
+0, 4, 4, 1, 614400, 0xfadb0437
+0, 5, 5, 1, 614400, 0xb2efa298
+0, 6, 6, 1, 614400, 0x8f4312c6
+0, 7, 7, 1, 614400, 0x8968e329
+0, 8, 8, 1, 614400, 0x545954c5
+0, 9, 9, 1, 614400, 0x144cd0b3
+0, 10, 10, 1, 614400, 0x5cd10fe0
+0, 11, 11, 1, 614400, 0x81b8473c
+0, 12, 12, 1, 614400, 0x58808c89
+0, 13, 13, 1, 614400, 0x21371f9f
+0, 14, 14, 1, 614400, 0x7533cd2a
+0, 15, 15, 1, 614400, 0xc2e8ec09
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
I have often repented speaking, but never of holding my tongue.
-- Xenocrates
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]
[-- Attachment #2: Type: text/plain, Size: 251 bytes --]
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [FFmpeg-devel] [PATCH] avcodec/mjpegdec: add frame threading for mjpeg decoder
2023-09-07 21:17 ` Michael Niedermayer
@ 2023-09-07 21:39 ` Paul B Mahol
2023-09-07 21:57 ` Michael Niedermayer
0 siblings, 1 reply; 7+ messages in thread
From: Paul B Mahol @ 2023-09-07 21:39 UTC (permalink / raw)
To: FFmpeg development discussions and patches
Sorry but I do not have such file, file i have only does 1 frame.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [FFmpeg-devel] [PATCH] avcodec/mjpegdec: add frame threading for mjpeg decoder
2023-09-07 21:39 ` Paul B Mahol
@ 2023-09-07 21:57 ` Michael Niedermayer
0 siblings, 0 replies; 7+ messages in thread
From: Michael Niedermayer @ 2023-09-07 21:57 UTC (permalink / raw)
To: FFmpeg development discussions and patches
[-- Attachment #1.1: Type: text/plain, Size: 412 bytes --]
On Thu, Sep 07, 2023 at 11:39:41PM +0200, Paul B Mahol wrote:
> Sorry but I do not have such file, file i have only does 1 frame.
https://samples.ffmpeg.org/ffmpeg-bugs/trac/ticket1915/
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Good people do not need laws to tell them to act responsibly, while bad
people will find a way around the laws. -- Plato
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]
[-- Attachment #2: Type: text/plain, Size: 251 bytes --]
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2023-09-07 21:57 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-09-07 16:44 [FFmpeg-devel] [PATCH] avcodec/mjpegdec: add frame threading for mjpeg decoder Paul B Mahol
2023-09-07 17:07 ` Andreas Rheinhardt
2023-09-07 17:34 ` Paul B Mahol
2023-09-07 17:48 ` Paul B Mahol
2023-09-07 21:17 ` Michael Niedermayer
2023-09-07 21:39 ` Paul B Mahol
2023-09-07 21:57 ` Michael Niedermayer
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
This inbox may be cloned and mirrored by anyone:
git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
ffmpegdev@gitmailbox.com
public-inbox-index ffmpegdev
Example config snippet for mirrors.
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git