Re: [FFmpeg-devel] [PATCH 14/18] avcodec/hevcdec: Don't allocate redundant HEVCContexts

2022-07-22 Thread Andreas Rheinhardt
Andreas Rheinhardt: > Michael Niedermayer: >> On Sat, Jul 02, 2022 at 08:32:06AM +0200, Andreas Rheinhardt wrote: >>> Michael Niedermayer: On Fri, Jul 01, 2022 at 12:29:45AM +0200, Andreas Rheinhardt wrote: > The HEVC decoder has both HEVCContext and HEVCLocalContext > structures. The

[FFmpeg-devel] [PATCH 5/5] avcodec/vp8, vp9: Avoid using VP56mv and VP56Frame in VP8/9

2022-07-22 Thread Andreas Rheinhardt
Instead replace VP56mv by new and identical structures VP8mv and VP9mv. Also replace VP56Frame by VP8FrameType in vp8.h and use that in VP8 code. Also remove VP56_FRAME_GOLDEN2, as this has only been used by VP8. This allows to remove all inclusions of vp56.h from everything that is not VP5/6. Thi

[FFmpeg-devel] [PATCH 4/5] avcodec/vp56: Move VP5-9 range coder functions to a header of their own

2022-07-22 Thread Andreas Rheinhardt
Also use a vpx prefix for them. Signed-off-by: Andreas Rheinhardt --- libavcodec/Makefile | 12 +- libavcodec/arm/vp8.h | 4 +- libavcodec/arm/vp8_armv6.S | 2 +- libavcodec/arm/{vp56_arith.h => vpx_arith.h} | 18 +- libavc

[FFmpeg-devel] [PATCH 3/5] avcodec/vp56: Move VP8/9-only rac functions to a header of their own

2022-07-22 Thread Andreas Rheinhardt
Also rename these functions from vp8_rac_* to vp89_rac_*. Signed-off-by: Andreas Rheinhardt --- libavcodec/vp56.h | 31 libavcodec/vp8.c | 166 +- libavcodec/vp89_rac.h | 66 + libavcodec/vp9.c | 49 +++--

[FFmpeg-devel] [PATCH 2/5] avcodec/vp56.h: Move VP8-only functions to vp8.c

2022-07-22 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavcodec/vp56.h | 36 +--- libavcodec/vp8.c | 34 ++ 2 files changed, 35 insertions(+), 35 deletions(-) diff --git a/libavcodec/vp56.h b/libavcodec/vp56.h index d2e6ad3ed0..08172e1987 100644

[FFmpeg-devel] [PATCH 1/5] avcodec/hq_hqa: Remove transient GetByteContext from context

2022-07-22 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavcodec/hq_hqa.c | 48 +++-- libavcodec/hq_hqa.h | 2 -- 2 files changed, 25 insertions(+), 25 deletions(-) diff --git a/libavcodec/hq_hqa.c b/libavcodec/hq_hqa.c index 9a07d83114..a17fa18bf8 100644 --- a/libavcode

Re: [FFmpeg-devel] [FFmpeg-cvslog] [ffmpeg-web] branch master updated. 42a0288 web/download: Add FFmpeg 5.1

2022-07-22 Thread Gyan Doshi
On 2022-07-23 02:16 am, Michael Niedermayer wrote: On Sat, Jul 23, 2022 at 12:06:29AM +0530, Gyan Doshi wrote: Is the News entry forthcoming? sounds like you want to write the news entry? Not exactly. For 4.3, it was never posted. I did it 10 months after the release. So just checking if

Re: [FFmpeg-devel] [PATCH] avcodec/aacdec: don't force HE-AACv2 profile if no PS info is present

2022-07-22 Thread James Almer
On 7/22/2022 8:00 PM, Alex Converse wrote: On Fri, Jul 22, 2022 at 8:37 AM James Almer wrote: On 7/22/2022 12:14 PM, Andreas Rheinhardt wrote: James Almer: On 7/22/2022 11:56 AM, Andreas Rheinhardt wrote: James Almer: On 7/22/2022 11:23 AM, Andreas Rheinhardt wrote: James Almer: On 7/18/2

Re: [FFmpeg-devel] [PATCH] avcodec/aacdec: don't force HE-AACv2 profile if no PS info is present

2022-07-22 Thread Alex Converse
On Fri, Jul 22, 2022 at 8:37 AM James Almer wrote: > On 7/22/2022 12:14 PM, Andreas Rheinhardt wrote: > > James Almer: > >> On 7/22/2022 11:56 AM, Andreas Rheinhardt wrote: > >>> James Almer: > On 7/22/2022 11:23 AM, Andreas Rheinhardt wrote: > > James Almer: > >> On 7/18/2022 10:57 A

Re: [FFmpeg-devel] [FFmpeg-cvslog] [ffmpeg-web] branch master updated. 42a0288 web/download: Add FFmpeg 5.1

2022-07-22 Thread Michael Niedermayer
On Sat, Jul 23, 2022 at 12:06:29AM +0530, Gyan Doshi wrote: > Is the News entry forthcoming? sounds like you want to write the news entry? Please go ahead thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Everything should be made as simple as possible, but n

Re: [FFmpeg-devel] [PATCH 17/18] avcodec/pthread_slice: Reuse buffer if possible

2022-07-22 Thread Andreas Rheinhardt
Tomas Härdin: > fre 2022-07-01 klockan 00:29 +0200 skrev Andreas Rheinhardt: >> Signed-off-by: Andreas Rheinhardt >> --- >>  libavcodec/pthread_slice.c | 6 -- >>  1 file changed, 4 insertions(+), 2 deletions(-) >> >> diff --git a/libavcodec/pthread_slice.c b/libavcodec/pthread_slice.c >> index

[FFmpeg-devel] [PATCH] avcodec/hevcdec: Output MD5-message in one piece

2022-07-22 Thread Andreas Rheinhardt
Otherwise, there is no guarantee that the various av_log-messages are not interrupted by another log statement. The latter may originate from anywhere else, even the HEVC decoder itself, as happens when one uses frame-threading to decode the BUMPING_A_ericsson_1.bit sample from the FATE-suite. Fur

Re: [FFmpeg-devel] [FFmpeg-cvslog] [ffmpeg-web] branch master updated. 42a0288 web/download: Add FFmpeg 5.1

2022-07-22 Thread Gyan Doshi
Is the News entry forthcoming? On 2022-07-23 12:00 am, ffmpeg-...@ffmpeg.org wrote: The branch, master has been updated via 42a0288d963853aa3c7043946feb4ac3f5242fd4 (commit) from d4a25b141f0a4dadea3b621c6da8c48dde16f608 (commit) - Log --

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mov: Rework the AVIF parser to handle multiple items

2022-07-22 Thread Vignesh Venkatasubramanian
On Wed, Jul 13, 2022 at 9:12 AM Vignesh Venkatasubramanian wrote: > > On Mon, Jul 11, 2022 at 3:25 PM James Zern > wrote: > > > > On Thu, Jun 30, 2022 at 2:04 PM Vignesh Venkatasubramanian > > wrote: > > > > > > Stores the item ids of all the items found in the file and > > > processes the prima

[FFmpeg-devel] [PATCH] qsv: Update ffmpeg qsv_errors to recognize GPU hang

2022-07-22 Thread Dmitry Rogozhkin
GPU hang is one of the most typical errors on Intel GPUs in case something goes wrong. It's important to recognize it explicitly for easier bugs triage. Also, this error code can be used to trigger GPU recovery path in self-written applications. Signed-off-by: Hon Wai Chow Signed-off-by: Dmitry R

Re: [FFmpeg-devel] [PATCH 22/35] fftools: add an object pool

2022-07-22 Thread Andreas Rheinhardt
Anton Khirnov: > Hi, > Quoting Andreas Rheinhardt (2022-06-16 23:41:36) >> AVFifos are often used with non-POD elements that need custom init, >> reset (unref) and free callbacks (in addition to the move callbacks >> already supported). So why not add it to AVFifo? The only drawback to >> this that

Re: [FFmpeg-devel] [PATCH] avcodec/pthread_frame: update the main avctx from the current, ThreadContext

2022-07-22 Thread Michael Niedermayer
On Mon, Jul 18, 2022 at 08:56:32PM +0200, Michael Niedermayer wrote: > On Sat, Jul 09, 2022 at 08:45:31AM +0200, Steve Lhomme wrote: > > Patch attached in the email. > > > > In some cases, the submit packet can result in configurations changes of the > > hardware decoders. The previous HW context

Re: [FFmpeg-devel] [PATCH 22/35] fftools: add an object pool

2022-07-22 Thread Anton Khirnov
Hi, Quoting Andreas Rheinhardt (2022-06-16 23:41:36) > AVFifos are often used with non-POD elements that need custom init, > reset (unref) and free callbacks (in addition to the move callbacks > already supported). So why not add it to AVFifo? The only drawback to > this that I see is that the pool

Re: [FFmpeg-devel] [PATCH] avcodec/aacdec: don't force HE-AACv2 profile if no PS info is present

2022-07-22 Thread James Almer
On 7/22/2022 12:14 PM, Andreas Rheinhardt wrote: James Almer: On 7/22/2022 11:56 AM, Andreas Rheinhardt wrote: James Almer: On 7/22/2022 11:23 AM, Andreas Rheinhardt wrote: James Almer: On 7/18/2022 10:57 AM, Andreas Rheinhardt wrote: James Almer: On 7/14/2022 9:10 AM, Andreas Rheinhardt w

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/tiff: Check tile_length and tile_width

2022-07-22 Thread Michael Niedermayer
On Fri, Jul 22, 2022 at 08:55:49AM +0200, Michael Niedermayer wrote: > Fixes: Division by 0 > Fixes: > 49235/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_TIFF_fuzzer-5495613847896064 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg >

Re: [FFmpeg-devel] [PATCH] avcodec/aacdec: don't force HE-AACv2 profile if no PS info is present

2022-07-22 Thread Andreas Rheinhardt
James Almer: > On 7/22/2022 11:56 AM, Andreas Rheinhardt wrote: >> James Almer: >>> On 7/22/2022 11:23 AM, Andreas Rheinhardt wrote: James Almer: > On 7/18/2022 10:57 AM, Andreas Rheinhardt wrote: >> James Almer: >>> On 7/14/2022 9:10 AM, Andreas Rheinhardt wrote: James Al

Re: [FFmpeg-devel] [PATCH] avcodec/aacdec: don't force HE-AACv2 profile if no PS info is present

2022-07-22 Thread James Almer
On 7/22/2022 11:56 AM, Andreas Rheinhardt wrote: James Almer: On 7/22/2022 11:23 AM, Andreas Rheinhardt wrote: James Almer: On 7/18/2022 10:57 AM, Andreas Rheinhardt wrote: James Almer: On 7/14/2022 9:10 AM, Andreas Rheinhardt wrote: James Almer: Should fix ticket #3361 Signed-off-by: Jam

Re: [FFmpeg-devel] [PATCH] avcodec/aacdec: don't force HE-AACv2 profile if no PS info is present

2022-07-22 Thread Andreas Rheinhardt
James Almer: > On 7/22/2022 11:23 AM, Andreas Rheinhardt wrote: >> James Almer: >>> On 7/18/2022 10:57 AM, Andreas Rheinhardt wrote: James Almer: > On 7/14/2022 9:10 AM, Andreas Rheinhardt wrote: >> James Almer: >>> Should fix ticket #3361 >>> >>> Signed-off-by: James Almer

Re: [FFmpeg-devel] [PATCH] avcodec/aacdec: don't force HE-AACv2 profile if no PS info is present

2022-07-22 Thread James Almer
On 7/22/2022 11:23 AM, Andreas Rheinhardt wrote: James Almer: On 7/18/2022 10:57 AM, Andreas Rheinhardt wrote: James Almer: On 7/14/2022 9:10 AM, Andreas Rheinhardt wrote: James Almer: Should fix ticket #3361 Signed-off-by: James Almer --- This also needs an update to some fate ref samples

Re: [FFmpeg-devel] [PATCH] avcodec/aacdec: don't force HE-AACv2 profile if no PS info is present

2022-07-22 Thread Andreas Rheinhardt
James Almer: > On 7/18/2022 10:57 AM, Andreas Rheinhardt wrote: >> James Almer: >>> On 7/14/2022 9:10 AM, Andreas Rheinhardt wrote: James Almer: > Should fix ticket #3361 > > Signed-off-by: James Almer > --- > This also needs an update to some fate ref samples i'll upload

[FFmpeg-devel] [PATCH v2 2/2] ffmpeg: add video heartbeat capability to fix_sub_duration

2022-07-22 Thread Jan Ekström
From: Jan Ekström Splits the currently handled subtitle at random access point packets that can be configured to follow a specific output stream. This way the subtitle - which is known to be shown at this time can be split and passed to muxer before its full duration is yet known. Co-authored-b

[FFmpeg-devel] [PATCH v2 1/2] ffmpeg: refactor post-decoding steps for subtitles into a function

2022-07-22 Thread Jan Ekström
From: Jan Ekström This enables us to later call this when generating additional subtitles for splitting purposes. Co-authored-by: Andrzej Nadachowski Signed-off-by: Jan Ekström --- fftools/ffmpeg.c | 50 1 file changed, 29 insertions(+), 21 de

Re: [FFmpeg-devel] [PATCH v2 3/3] hwcontext_vaapi: #if guard VAAPI_DRM specifics

2022-07-22 Thread Emil Velikov
On Thu, 21 Jul 2022 at 22:06, Mark Thompson wrote: > > On 20/07/2022 11:56, Emil Velikov wrote: > > From: Emil Velikov > > > > Similar to the VAAPI_X11 bits, guard all the VAAPI_DRM parts behind a > > compiler guard. > > > > Signed-off-by: Emil Velikov > > --- > > libavutil/hwcontext_vaapi.c |

Re: [FFmpeg-devel] [PATCH] avcodec/aacdec: don't force HE-AACv2 profile if no PS info is present

2022-07-22 Thread James Almer
On 7/18/2022 10:57 AM, Andreas Rheinhardt wrote: James Almer: On 7/14/2022 9:10 AM, Andreas Rheinhardt wrote: James Almer: Should fix ticket #3361 Signed-off-by: James Almer --- This also needs an update to some fate ref samples i'll upload before pushing (fate-aac-al_sbr_ps_04_ur and fate-a