Re: [FFmpeg-devel] [PATCH] avformat/apvdec: export color information

James Almer: > apv_read_header() reads enough information that the generic demux code doesn't > attempt to read a frame to fill missing fields in codecpar, so make sure it's > set here. > > Signed-off-by: James Almer > --- > libavformat/apvdec.c | 32 +--- > 1 file ch

[FFmpeg-devel] [PATCH v3] avformat/apvdec: export color information

apv_read_header() reads enough information that the generic demux code doesn't attempt to read a frame to fill missing fields in codecpar, so make sure it's set here. Signed-off-by: James Almer --- libavformat/apvdec.c | 40 ++-- 1 file changed, 38 insertions(

[FFmpeg-devel] [PATCH] apv_decode: Replace division with shift

The compiler can't see that this should be a shift and generates a real division which is slow enough to appear in profiles on its own. --- libavcodec/apv_decode.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/libavcodec/apv_decode.c b/libavcodec/apv_decode.c ind

Re: [FFmpeg-devel] [PATCH] fate: Add test for APV 422-10 profile

On 4/27/2025 3:13 PM, Mark Thompson wrote: Bitstream generated using the reference encoder, then edited to fix the colour description and an extra metadata block added. FFmpeg decoder output is identical to the reference decoder output. The content used is the first three frames of "Waterfall"

[FFmpeg-devel] [PATCH 1/2] avcodec: add an APV parser

Signed-off-by: James Almer --- libavcodec/Makefile | 1 + libavcodec/apv_parser.c | 165 libavcodec/parsers.c| 1 + 3 files changed, 167 insertions(+) create mode 100644 libavcodec/apv_parser.c diff --git a/libavcodec/Makefile b/libavcodec/Ma

[FFmpeg-devel] [PATCH 2/2] avformat/apvdec: don't fill container level fields with codec level info

This is a raw demuxer, it should not read codec level information and export it as container level information. The generic demux code will use the recently introduced parser to take care of that. Signed-off-by: James Almer --- configure| 1 + libavformat/apvdec.c | 46 ++---

[FFmpeg-devel] [PATCH v2 1/2] avcodec: add an APV parser

Signed-off-by: James Almer --- Now working if the first AU is not a primary frame. configure | 1 + libavcodec/Makefile | 1 + libavcodec/apv_parser.c | 133 libavcodec/parsers.c| 1 + 4 files changed, 136 insertions(+) create

[FFmpeg-devel] [PATCH v2 03/20] avcodec/vvc: support decoding prefix and suffix nal units

From: Wu Jianhua Signed-off-by: Wu Jianhua --- configure | 4 ++- libavcodec/Makefile | 1 + libavcodec/vvc/Makefile | 1 + libavcodec/vvc/dec.c| 32 libavcodec/vvc/dec.h| 2 ++ libavcodec/vvc/sei.c| 55 +

[FFmpeg-devel] [PATCH v2 04/20] avcodec/vvc/sei: add decode_film_grain_characteristics

From: Wu Jianhua Signed-off-by: Wu Jianhua --- libavcodec/vvc/sei.c | 66 1 file changed, 66 insertions(+) diff --git a/libavcodec/vvc/sei.c b/libavcodec/vvc/sei.c index 2842862a36..365f815950 100644 --- a/libavcodec/vvc/sei.c +++ b/libavcodec/vvc/s

[FFmpeg-devel] [PATCH v2 02/20] avcodec/cbs_sei_syntax_template: add sei message film_grain_characteristics

From: Wu Jianhua Signed-off-by: Wu Jianhua --- libavcodec/cbs_h2645.c | 6 libavcodec/cbs_sei.h | 21 +++ libavcodec/cbs_sei_syntax_template.c | 53 3 files changed, 80 insertions(+) diff --git a/libavcodec/cbs_h2645.c b/

[FFmpeg-devel] [PATCH v2 01/20] avcodec/cbs_h2645: add cbs_sei_h274_types

From: Wu Jianhua Signed-off-by: Wu Jianhua --- libavcodec/cbs_h2645.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/libavcodec/cbs_h2645.c b/libavcodec/cbs_h2645.c index 369e3ac876..b0cf6e143a 100644 --- a/libavcodec/cbs_h2645.c +++ b/libavcodec/cbs_h2645.c @@ -2310,6 +2310,1

[FFmpeg-devel] [PATCH v2 06/20] avcodec/vvc/dec: support applying film grain by the decoder

From: Wu Jianhua Signed-off-by: Wu Jianhua --- libavcodec/Makefile | 2 +- libavcodec/vvc/dec.c | 89 +-- libavcodec/vvc/dec.h | 8 +++- libavcodec/vvc/refs.c | 11 +- 4 files changed, 104 insertions(+), 6 deletions(-) diff --git a/libavcodec/M

[FFmpeg-devel] [PATCH v2 09/20] avcodec/vvc/sei: add decode_decoded_picture_hash

From: Wu Jianhua Signed-off-by: Wu Jianhua --- libavcodec/vvc/sei.c | 19 +++ libavcodec/vvc/sei.h | 2 ++ 2 files changed, 21 insertions(+) diff --git a/libavcodec/vvc/sei.c b/libavcodec/vvc/sei.c index 365f815950..2d81ae5908 100644 --- a/libavcodec/vvc/sei.c +++ b/libavcodec

[FFmpeg-devel] [PATCH v2 10/20] avcodec/h274: add ff_h274_hash functions

From: Wu Jianhua Signed-off-by: Wu Jianhua --- libavcodec/h274.c | 155 ++ libavcodec/h274.h | 7 +++ 2 files changed, 162 insertions(+) diff --git a/libavcodec/h274.c b/libavcodec/h274.c index 5709200322..e46926e4cc 100644 --- a/libavcodec/h274.c

[FFmpeg-devel] [PATCH v2 08/20] avcodec/h274: add H274SEIPictureHash struct

From: Wu Jianhua Signed-off-by: Wu Jianhua --- libavcodec/h274.h | 10 ++ 1 file changed, 10 insertions(+) diff --git a/libavcodec/h274.h b/libavcodec/h274.h index cebc8becb3..e1803edaf3 100644 --- a/libavcodec/h274.h +++ b/libavcodec/h274.h @@ -64,4 +64,14 @@ int ff_h274_apply_film_gr

[FFmpeg-devel] [PATCH v2 05/20] avcodec/vvc/dec: export sei to the frame when the frame starts

From: Wu Jianhua Signed-off-by: Wu Jianhua --- libavcodec/h2645_sei.c | 9 + libavcodec/h2645_sei.h | 2 +- libavcodec/vvc/dec.c | 12 3 files changed, 18 insertions(+), 5 deletions(-) diff --git a/libavcodec/h2645_sei.c b/libavcodec/h2645_sei.c index 2494daaf3c..78d9d

[FFmpeg-devel] [PATCH v2 13/20] avcodec/vvc/sei: add decode_display_orientation

From: Wu Jianhua Signed-off-by: Wu Jianhua --- libavcodec/vvc/sei.c | 27 +++ 1 file changed, 27 insertions(+) diff --git a/libavcodec/vvc/sei.c b/libavcodec/vvc/sei.c index 2d81ae5908..7073762d49 100644 --- a/libavcodec/vvc/sei.c +++ b/libavcodec/vvc/sei.c @@ -94,6 +94

[FFmpeg-devel] [PATCH v2 12/20] avcodec/cbs_sei_syntax_template: add sei message sei_display_orientation

From: Wu Jianhua Signed-off-by: Wu Jianhua --- libavcodec/cbs_h2645.c | 6 ++ libavcodec/cbs_sei.h | 7 +++ libavcodec/cbs_sei_syntax_template.c | 18 ++ 3 files changed, 31 insertions(+) diff --git a/libavcodec/cbs_h2645.c b/libavcodec/c

[FFmpeg-devel] [PATCH v2 17/20] avcodec/vvc/sei: add decode_frame_field_info

From: Wu Jianhua Signed-off-by: Wu Jianhua --- libavcodec/vvc/sei.c | 26 ++ libavcodec/vvc/sei.h | 1 + 2 files changed, 27 insertions(+) diff --git a/libavcodec/vvc/sei.c b/libavcodec/vvc/sei.c index 49a7961c63..47edd94fb2 100644 --- a/libavcodec/vvc/sei.c +++ b/liba

[FFmpeg-devel] [PATCH v2 16/20] avcodec/h274: add H274SEIFrameFieldInfo struct

From: Wu Jianhua Signed-off-by: Wu Jianhua --- libavcodec/h274.h | 8 1 file changed, 8 insertions(+) diff --git a/libavcodec/h274.h b/libavcodec/h274.h index 0ca5f0cbab..055dd591d2 100644 --- a/libavcodec/h274.h +++ b/libavcodec/h274.h @@ -81,4 +81,12 @@ int ff_h274_hash_verify(H274H

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/decode: sync side_data_pref_mask across worker threads

James Almer: > side_data_pref_mask is set during ff_decode_preinit() and then never copied to > worker threads. As such, any decoder calling one of the side data helpers that > rely on side_data_pref() in frame threaded scenarios will ignore the user set > list of side data types. > > Signed-off-b

Re: [FFmpeg-devel] [PATCH] avformat/apvdec: export color information

On 4/27/2025 3:29 PM, Mark Thompson wrote: On 27/04/2025 18:53, James Almer wrote: apv_read_header() reads enough information that the generic demux code doesn't attempt to read a frame to fill missing fields in codecpar, so make sure it's set here. Signed-off-by: James Almer --- libavformat

Re: [FFmpeg-devel] [PATCH] apv_decode: Replace division with shift

On 4/27/2025 3:49 PM, Mark Thompson wrote: The compiler can't see that this should be a shift and generates a real division which is slow enough to appear in profiles on its own. --- libavcodec/apv_decode.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/libavc

[FFmpeg-devel] [PATCH v2] avformat/apvdec: export color information

apv_read_header() reads enough information that the generic demux code doesn't attempt to read a frame to fill missing fields in codecpar, so make sure it's set here. Signed-off-by: James Almer --- libavformat/apvdec.c | 38 +- 1 file changed, 37 insertions(+)

[FFmpeg-devel] [PATCH 1/7] avformat/apvenc: Only allow APV

Patches attached. - Andreas From c0dbda4a3a63a451618e4578c779cd999d365819 Mon Sep 17 00:00:00 2001 From: Andreas Rheinhardt Date: Sun, 27 Apr 2025 20:11:50 +0200 Subject: [PATCH 1/7] avformat/apvenc: Only allow APV Signed-off-by: Andreas Rheinhardt --- libavformat/apvenc.c | 2 +- 1 file chang

Re: [FFmpeg-devel] [PATCH] avcodec/h264dec: export x264_build value

James Almer: > It's a value the caller may care about. Why? (Theoretically, the caller "may" care about all values.) > > Signed-off-by: James Almer > --- > libavcodec/h264_slice.c | 1 + > libavcodec/h264dec.c | 2 +- > tests/ref/fate/flv-demux | 2 +- > tests/ref/fate/mov-

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/decode: sync side_data_pref_mask across worker threads

On 4/27/2025 3:52 PM, Andreas Rheinhardt wrote: James Almer: side_data_pref_mask is set during ff_decode_preinit() and then never copied to worker threads. As such, any decoder calling one of the side data helpers that rely on side_data_pref() in frame threaded scenarios will ignore the user set

Re: [FFmpeg-devel] [PATCH v3] avformat/apvdec: export color information

James Almer: > apv_read_header() reads enough information that the generic demux code doesn't > attempt to read a frame to fill missing fields in codecpar, so make sure it's > set here. > > Signed-off-by: James Almer > --- > libavformat/apvdec.c | 40 ++-- > 1

Re: [FFmpeg-devel] [PATCH v3] avformat/apvdec: export color information

On 4/27/2025 5:50 PM, Andreas Rheinhardt wrote: James Almer: apv_read_header() reads enough information that the generic demux code doesn't attempt to read a frame to fill missing fields in codecpar, so make sure it's set here. Signed-off-by: James Almer --- libavformat/apvdec.c | 40 +++

Re: [FFmpeg-devel] Addition of encoder functionality in libavcodec/vpx_rac.h and corresponding commit in vpx_rac.c

Hi Amogh, On Sun, Apr 27, 2025 at 10:38:57PM +0530, Amogh Kumar Sinha wrote: >Dear Sir, >I am sending this patch for adding encoder functionality, as the >Qualification Task for GSoC 2025. Kindly look into it: Ok, this "diff" comes at the last minute and its not quite ready for inclus

[FFmpeg-devel] [PATCH] aacdec_usac: correct Mps212 parsing location

It gets parsed after SBR, even if there is no SBR. --- libavcodec/aac/aacdec_usac.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/aac/aacdec_usac.c b/libavcodec/aac/aacdec_usac.c index ccdf58bc8e..ef0c115aa0 100644 --- a/libavcodec/aac/aacdec_usac.c +++ b/l

[FFmpeg-devel] [PATCH v2 20/20] avcodec/vvc/sei: add decode_mastering_display_colour_volume

From: Wu Jianhua Signed-off-by: Wu Jianhua --- libavcodec/vvc/sei.c | 21 + 1 file changed, 21 insertions(+) diff --git a/libavcodec/vvc/sei.c b/libavcodec/vvc/sei.c index 0692a99fc8..d8ab2bf245 100644 --- a/libavcodec/vvc/sei.c +++ b/libavcodec/vvc/sei.c @@ -158,6 +158,24

[FFmpeg-devel] [PATCH v2 07/20] avcodec/vvc/dec: support removing film grain params from side data

From: Wu Jianhua Signed-off-by: Wu Jianhua --- libavcodec/vvc/refs.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/vvc/refs.c b/libavcodec/vvc/refs.c index 4cf44bcf46..cfbc2977c0 100644 --- a/libavcodec/vvc/refs.c +++ b/libavcodec/vvc/refs.c @@ -295,6 +295,9 @@ int ff_vvc_ou

[FFmpeg-devel] [PATCH v2 14/20] avcodec/vvc/sei: add decode_content_light_level_info

From: Wu Jianhua Signed-off-by: Wu Jianhua --- libavcodec/vvc/sei.c | 12 1 file changed, 12 insertions(+) diff --git a/libavcodec/vvc/sei.c b/libavcodec/vvc/sei.c index 7073762d49..49a7961c63 100644 --- a/libavcodec/vvc/sei.c +++ b/libavcodec/vvc/sei.c @@ -118,6 +118,15 @@ static

[FFmpeg-devel] [PATCH v2 19/20] avcodec/vvc/sei: add decode_ambient_viewing_environment

From: Wu Jianhua Signed-off-by: Wu Jianhua --- libavcodec/vvc/sei.c | 13 + 1 file changed, 13 insertions(+) diff --git a/libavcodec/vvc/sei.c b/libavcodec/vvc/sei.c index 47edd94fb2..0692a99fc8 100644 --- a/libavcodec/vvc/sei.c +++ b/libavcodec/vvc/sei.c @@ -148,6 +148,16 @@ stati

[FFmpeg-devel] [PATCH v2 18/20] avcodec/vvc: support fields

From: Wu Jianhua passed files: FIELD_A_Panasonic_4.bit FIELD_B_Panasonic_2.bit Signed-off-by: Wu Jianhua --- libavcodec/vvc/refs.c | 8 1 file changed, 8 insertions(+) diff --git a/libavcodec/vvc/refs.c b/libavcodec/vvc/refs.c index cfbc2977c0..79967b77d3 100644 --- a/libavco

[FFmpeg-devel] [PATCH v2 15/20] avcodec/cbs_sei_syntax_template: add sei message frame_field_information

From: Wu Jianhua Signed-off-by: Wu Jianhua --- libavcodec/cbs_h2645.c | 6 ++ libavcodec/cbs_sei.h | 12 libavcodec/cbs_sei_syntax_template.c | 26 ++ 3 files changed, 44 insertions(+) diff --git a/libavcodec/cbs_h2645.c b

[FFmpeg-devel] [PATCH v2 11/20] avcodec/vvcdec: verify picture hash

From: Wu Jianhua Signed-off-by: Wu Jianhua --- libavcodec/vvc/dec.c | 21 - libavcodec/vvc/dec.h | 2 ++ 2 files changed, 22 insertions(+), 1 deletion(-) diff --git a/libavcodec/vvc/dec.c b/libavcodec/vvc/dec.c index 4c0d08440a..11d653e684 100644 --- a/libavcodec/vvc/dec.c

Re: [FFmpeg-devel] [PATCH v4 0/7] APV support

On 27/04/2025 01:15, James Almer wrote: > On 4/26/2025 5:49 PM, Mark Thompson wrote: >> v4: minor changes from previous version reflecting review comments. >> >> This seems to be converging so will consider pushing soon and then working >> on top of it (the patches from Dawid Kozinski for ISOBMFF

Re: [FFmpeg-devel] [TC] "Future Log Output Default"

Le lauantaina 26. huhtikuuta 2025, 18.10.27 Itä-Euroopan kesäaika Michael Niedermayer a écrit : > This is just an announcement that the TC has been asked to look into > avutil/log: Add log flag to control printing of memory addresses > GitHub:https://github.com/ffstaging/FFmpeg/pull/59

[FFmpeg-devel] [PATCH v2] lavc/vvc: Detect subpic overlaps at CTU level

In d5dbcc00d889fb17948b025a468b00ddbea9e058, it was hoped that detection of subpicture overlaps could be performed at the tile level, so as to avoid introducing per-CTU checks. Unfortunately since that patch, fuzzing has indicated there are some structures involving pps_subpic_one_or_more_tiles_sli

Re: [FFmpeg-devel] [PATCH] lavc/vvc: Detect subpic overlaps at CTU level

On 27/04/2025 03:42, Nuo Mi wrote: > On Fri, Apr 18, 2025 at 10:40 PM Frank Plowman > wrote: > >> In d5dbcc00d889fb17948b025a468b00ddbea9e058, it was hoped that detection >> of subpicture overlaps could be performed at the tile level, so as to >> avoid introducing per-CTU checks. Unfortunately si

Re: [FFmpeg-devel] [RFC] Shaping the AVTextFormat API Surface

On date Friday 2025-04-25 13:16:59 +, softworkz . wrote: [...] > Tell me what I should check for and what not in those four groups of > functions and for those things which should be checked, tell me which > way (return error, return silently, allow segfault or use an assertion). > > Then I'l

Re: [FFmpeg-devel] [PATCH v7 01/13] fftools/textformat: Formatting and whitespace changes

On date Friday 2025-04-25 23:30:56 +, softworkz wrote: > From: softworkz > @@ -484,15 +481,14 @@ void avtext_print_time(AVTextFormatContext *tctx, const > char *key, > > void avtext_print_ts(AVTextFormatContext *tctx, const char *key, int64_t ts, > int is_duration) > { > -if ((!is_du

[FFmpeg-devel] [PATCH v2] avcodec: Add A2DP LATM audio support.

From: zhanquan cen This commit introduces three key components for Bluetooth A2DP LATM streams: 1.A2DP-specific LATM decoder (aac_latm_a2dp) New codec ID AV_CODEC_ID_AAC_LATM_A2DP Inherits LATMContext with A2DP extensions Attaches "a2dp_rechunk" bitstream filter. 2.Parser enhancement Ex

Re: [FFmpeg-devel] [RFC] Shaping the AVTextFormat API Surface

On date Thursday 2025-04-24 19:12:09 +0200, Nicolas George wrote: > softworkz . (HE12025-04-22): [...] > ffprobe has a concept of sections, and no more. XML does not have a > concept of sections. JSON does not have a concept of sections. CSV does > not have a concept of sections. Other parts of FFm

[FFmpeg-devel] [PATCH] avformat/apvdec: export color information

apv_read_header() reads enough information that the generic demux code doesn't attempt to read a frame to fill missing fields in codecpar, so make sure it's set here. Signed-off-by: James Almer --- libavformat/apvdec.c | 32 +--- 1 file changed, 29 insertions(+), 3 de

Re: [FFmpeg-devel] [RFC] Shaping the AVTextFormat API Surface

> -Original Message- > From: ffmpeg-devel On Behalf Of > Stefano Sabatini > Sent: Sonntag, 27. April 2025 12:42 > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [RFC] Shaping the AVTextFormat API Surface > > On date Friday 2025-04-25 13:1

[FFmpeg-devel] [PATCH] fate: add target to remove sample files that are no longer on the server

Also add a target that shows what would be deleted Signed-off-by: Michael Niedermayer --- doc/fate.texi | 3 +++ tests/Makefile | 4 2 files changed, 7 insertions(+) diff --git a/doc/fate.texi b/doc/fate.texi index 7a2e0edfcc6..0a90a6401be 100644 --- a/doc/fate.texi +++ b/doc/fate.texi @@

[FFmpeg-devel] [PATCH] fate: Add test for APV 422-10 profile

Bitstream generated using the reference encoder, then edited to fix the colour description and an extra metadata block added. FFmpeg decoder output is identical to the reference decoder output. The content used is the first three frames of "Waterfall" from the SVT Open Content Video Test Suite 20

Re: [FFmpeg-devel] [PATCH] avformat/apvdec: export color information

On 27/04/2025 18:53, James Almer wrote: > apv_read_header() reads enough information that the generic demux code doesn't > attempt to read a frame to fill missing fields in codecpar, so make sure it's > set here. > > Signed-off-by: James Almer > --- > libavformat/apvdec.c | 32 ++

Re: [FFmpeg-devel] [PATCH 1/7] avformat/apvenc: Only allow APV

On 27/04/2025 19:42, Andreas Rheinhardt wrote: > Patches attached. > > - Andreas > [PATCH 1/7] avformat/apvenc: Only allow APV Oops, sure. > [PATCH 2/7] avformat/apvenc: Remove unused header Sure. > [PATCH 3/7] avformat/apvdec: Use ffio_read_size() Is there a cleaner way to discard very shor

Re: [FFmpeg-devel] [PATCH 1/7] avformat/apvenc: Only allow APV

Mark Thompson: > On 27/04/2025 19:42, Andreas Rheinhardt wrote: >> Patches attached. >> >> - Andreas > >> [PATCH 1/7] avformat/apvenc: Only allow APV > > Oops, sure. > >> [PATCH 2/7] avformat/apvenc: Remove unused header > > Sure. > >> [PATCH 3/7] avformat/apvdec: Use ffio_read_size() > > Is

[FFmpeg-devel] [PATCH] avcodec/pthread_frame: don't call ff_decode_internal_sync() during per-frame thread synchonization

The values it copies across contexts are set during preinit() and never changed. Signed-off-by: James Almer --- libavcodec/pthread_frame.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/pthread_frame.c b/libavcodec/pthread_frame.c index 7019108af6..e283eb2941 100644 --- a/libavcod