Re: [FFmpeg-devel] [PATCH] checkasm/hevc_deblock: Initialize buffer

2024-03-07 Thread Andreas Rheinhardt
Andreas Rheinhardt: > Fixes the hevc_deblock checkasm test with Valgrind. > > Signed-off-by: Andreas Rheinhardt > --- > tests/checkasm/hevc_deblock.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/tests/checkasm/hevc_deblock.c b/tests/checkasm/hevc_deblock.c > index 91e57f5cf5..04cf9d

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/vdpau: Remove outdated comment

2024-03-07 Thread Andreas Rheinhardt
Andreas Rheinhardt: > Forgotten in b773a8d8c1dfe4cfc6eabf509e26ab011270b9ed. > > Signed-off-by: Andreas Rheinhardt > --- > libavcodec/vdpau.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/libavcodec/vdpau.c b/libavcodec/vdpau.c > index 2b9b78d8d0..c56ee1f44c 100644 > --- a/libavcod

Re: [FFmpeg-devel] [PATCH] avutil/dict: Deduplicate freeing dictionary

2024-03-07 Thread Andreas Rheinhardt
Andreas Rheinhardt: > Signed-off-by: Andreas Rheinhardt > --- > libavutil/dict.c | 10 -- > 1 file changed, 4 insertions(+), 6 deletions(-) > > diff --git a/libavutil/dict.c b/libavutil/dict.c > index 7f23d5336a..6fb09399ba 100644 > --- a/libavutil/dict.c > +++ b/libavutil/dict.c > @@ -1

Re: [FFmpeg-devel] [PATCH] avcodec/ccaption_dec: Avoid relocations for strings

2024-03-07 Thread Andreas Rheinhardt
Andreas Rheinhardt: > The longest string here takes four bytes, so using an array > of pointers is wasteful even when ignoring the cost of relocations; > the lack of relocations also implies that this array > will now be put into .rodata and not into .data.rel.ro. > > Static asserts are used to en

Re: [FFmpeg-devel] [PATCH 01/29] lavu/opt: factor per-type dispatch out of av_opt_get()

2024-03-07 Thread Anton Khirnov
Will push the set tomorrow if nobody has further objections. -- Anton Khirnov ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.

Re: [FFmpeg-devel] [PATCH v5 1/3] avformat/flvdec: support enhanced flv PacketTypeMetadata

2024-03-07 Thread Steven Liu
zhupengfei via ffmpeg-devel 于2024年3月4日周一 21:52写道: > > From: Zhu Pengfei <411294...@qq.com> > > Signed-off-by: Zhu Pengfei <411294...@qq.com> > --- > libavformat/flvdec.c | 177 ++- > 1 file changed, 176 insertions(+), 1 deletion(-) > > diff --git a/libavfor

Re: [FFmpeg-devel] [PATCH] avutil/dict: Deduplicate freeing dictionary

2024-03-07 Thread epirat07
On 5 Mar 2024, at 16:53, Andreas Rheinhardt wrote: > Signed-off-by: Andreas Rheinhardt > --- > libavutil/dict.c | 10 -- > 1 file changed, 4 insertions(+), 6 deletions(-) > > diff --git a/libavutil/dict.c b/libavutil/dict.c > index 7f23d5336a..6fb09399ba 100644 > --- a/libavutil/dict.c >

Re: [FFmpeg-devel] [PATCH 1/3] avformat/dvdvideodec: add CLUT utilities and subtitle palette support

2024-03-07 Thread Anton Khirnov
Quoting Marth64 (2024-03-06 16:36:32) > Good morning, > > DVD subtitle palettes, which are natively YUV, are currently carried as > a hex string in their respective subtitle streams and have > no concept of colorspace tagging (even predating dvd demuxer). The > convention is to convert > them to R

Re: [FFmpeg-devel] [PATCH 1/3] avformat/dvdvideodec: add CLUT utilities and subtitle palette support

2024-03-07 Thread Andreas Rheinhardt
Anton Khirnov: > Quoting Marth64 (2024-03-06 16:36:32) >> Good morning, >> >> DVD subtitle palettes, which are natively YUV, are currently carried as >> a hex string in their respective subtitle streams and have >> no concept of colorspace tagging (even predating dvd demuxer). The >> convention is

Re: [FFmpeg-devel] [PATCH 1/4] lavc/vp9dsp: R-V V ipred dc

2024-03-07 Thread flow gg
updated it in the reply flow gg 于2024年3月3日周日 23:31写道: > > As noted eaerlier, I don't understand why you have two size parameters. > It > seems that \size is always either the same as (1 << (\size2 - 1)) a.k.a. > ((1 > << \size2) / 2), or unused. The assembler *can* compute arithmetic > constants

Re: [FFmpeg-devel] [PATCH 23/29] avcodec/mpeg12dec: use ff_frame_new_side_data

2024-03-07 Thread Andreas Rheinhardt
Anton Khirnov: > Quoting Andreas Rheinhardt (2024-03-04 14:36:09) >> Anton Khirnov: >>> From: Niklas Haas >>> >>> For consistency, even though this cannot be overriden at the packet >>> level. >>> --- >>> libavcodec/mpeg12dec.c | 18 ++ >>> 1 file changed, 10 insertions(+), 8 dele

Re: [FFmpeg-devel] [PATCH 23/29] avcodec/mpeg12dec: use ff_frame_new_side_data

2024-03-07 Thread Anton Khirnov
Quoting Andreas Rheinhardt (2024-03-07 12:19:28) > Anton Khirnov: > > Quoting Andreas Rheinhardt (2024-03-04 14:36:09) > >> Anton Khirnov: > >>> From: Niklas Haas > >>> > >>> For consistency, even though this cannot be overriden at the packet > >>> level. > >>> --- > >>> libavcodec/mpeg12dec.c |

Re: [FFmpeg-devel] [PATCH 23/29] avcodec/mpeg12dec: use ff_frame_new_side_data

2024-03-07 Thread James Almer
On 3/7/2024 9:18 AM, Anton Khirnov wrote: Quoting Andreas Rheinhardt (2024-03-07 12:19:28) Anton Khirnov: Quoting Andreas Rheinhardt (2024-03-04 14:36:09) Anton Khirnov: From: Niklas Haas For consistency, even though this cannot be overriden at the packet level. --- libavcodec/mpeg12dec.c

Re: [FFmpeg-devel] [PATCH 23/29] avcodec/mpeg12dec: use ff_frame_new_side_data

2024-03-07 Thread Anton Khirnov
Quoting James Almer (2024-03-07 13:25:07) > I wouldn't. It adds an extra layer of abstraction for no real gain. The gain is simpler API. What's the gain in forcing decoder authors to call a different function to accomplish the same task, just because frame threading happens to be used. > And the

Re: [FFmpeg-devel] [RFC] clarifying the TC conflict of interest rule

2024-03-07 Thread Vittorio Giovara
On Thu, Mar 7, 2024 at 12:25 AM Michael Niedermayer via ffmpeg-devel < ffmpeg-devel@ffmpeg.org> wrote: > > instead of backroom deals, for a > > change. > > iam sorry, but these accusations are not acceptable > The application was and is on a public wiki > the SoWs where collected by pierre and it

[FFmpeg-devel] [PATCH] fftools/ffmpeg_filter: do not assume av_buffersrc_get_nb_failed_requests()>0

2024-03-07 Thread Anton Khirnov
Apparently it can happen that avfilter_graph_request_oldest() returns EAGAIN, yet av_buffersrc_get_nb_failed_requests() returns 0 for every input. Fixes #10795 --- fftools/ffmpeg_filter.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fftools/ffmpeg_filter.c b/fftools/ffmpeg_

[FFmpeg-devel] [PATCH v2 17/18] fftools/ffmpeg: add loopback decoding

2024-03-07 Thread Anton Khirnov
This allows to send an encoder's output back to decoding and feed the result into a complex filtergraph. --- Now using [dec:X] instead of [decX] in filtergraph link labels, as suggested by Marvin on IRC. --- Changelog | 1 + doc/ffmpeg.texi | 49 ++-- fftools/cmdut

Re: [FFmpeg-devel] [PATCH 06/29] lavu/opt: add array options

2024-03-07 Thread Andreas Rheinhardt
Andreas Rheinhardt: > Anton Khirnov: >> +/** >> + * Must be set as default_val for AV_OPT_TYPE_FLAG_ARRAY options. >> + */ >> +typedef struct AVOptionArrayDef { >> +/** >> + * Must be set to sizeof(AVOptionArrayDef), in order to allow extending >> this >> + * struct without breaking AB

[FFmpeg-devel] Patch which requires a new library

2024-03-07 Thread Sergio Garcia Murillo
Hi all! I would like to contribute a patch that adds sig v4 authentication ( https://docs.aws.amazon.com/AmazonS3/latest/API/sig-v4-authenticating-requests.html) to the http module. This would allow for example to play/store an MPEG DASH content stored on an S3 Bucket which requires authenticatio

[FFmpeg-devel] [PATCH] avcodec/xvmc: Remove header

2024-03-07 Thread Andreas Rheinhardt
Forgotten in a12cd3be98e8aba6e74274192ec6532988aa9444. Signed-off-by: Andreas Rheinhardt --- libavcodec/Makefile | 2 - libavcodec/xvmc.h | 171 2 files changed, 173 deletions(-) delete mode 100644 libavcodec/xvmc.h diff --git a/libavcodec/Makef

Re: [FFmpeg-devel] [PATCH] avcodec/xvmc: Remove header

2024-03-07 Thread James Almer
On 3/7/2024 12:04 PM, Andreas Rheinhardt wrote: Forgotten in a12cd3be98e8aba6e74274192ec6532988aa9444. Signed-off-by: Andreas Rheinhardt --- libavcodec/Makefile | 2 - libavcodec/xvmc.h | 171 2 files changed, 173 deletions(-) delete mode 1

[FFmpeg-devel] [PATCH v2 06/29] lavu/opt: add array options

2024-03-07 Thread Anton Khirnov
--- doc/APIchanges| 3 + libavutil/opt.c | 360 +- libavutil/opt.h | 56 ++- libavutil/tests/opt.c | 49 ++ tests/ref/fate/opt| 35 +++- 5 files changed, 458 insertions(+), 45 deletions(-) diff --git a/doc/APIchanges b/d

[FFmpeg-devel] [PATCH v2] avcodec/libx264: fix extradata when config annexb=0

2024-03-07 Thread Zhao Zhili
From: Zhao Zhili --- configure| 2 +- libavcodec/libx264.c | 134 +++ 2 files changed, 111 insertions(+), 25 deletions(-) diff --git a/configure b/configure index db7dc89755..24cb897d28 100755 --- a/configure +++ b/configure @@ -3491,7 +3491

Re: [FFmpeg-devel] Patch which requires a new library

2024-03-07 Thread Kieran Kunhya
On Thu, 7 Mar 2024 at 15:07, Sergio Garcia Murillo < sergio.garcia.muri...@gmail.com> wrote: > > Could anyone give me any pointers on what is the best way of doing this? > You should use a well known crypto library to implement this in FFmpeg (e.g libgcrypt, openssl etc). Kieran

[FFmpeg-devel] [PATCH] avcodec/libdav1d: Stop mangling AVPacket.opaque

2024-03-07 Thread Andreas Rheinhardt
Unnecessary since 67e7f0b0537eee1357769038270fda08fe32 as there are no longer two opaque fields. Signed-off-by: Andreas Rheinhardt --- libavcodec/libdav1d.c | 27 --- 1 file changed, 27 deletions(-) diff --git a/libavcodec/libdav1d.c b/libavcodec/libdav1d.c index 794

Re: [FFmpeg-devel] [PATCH 1/2 v2] fftools/ffprobe: export Tile Grid Stream Group parameters

2024-03-07 Thread Stefano Sabatini
On date Wednesday 2024-03-06 21:49:51 -0300, James Almer wrote: > Signed-off-by: James Almer > --- > fftools/ffprobe.c | 53 --- > tests/fate/mov.mak| 6 +-- > .../ref/fate/mov-heic-demux-still-image-grid | 29 ++ >

[FFmpeg-devel] [PATCH] avcodec/librav1e: Don't unnecessarily create new references

2024-03-07 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- Absolutely untested (not even compiled). libavcodec/librav1e.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/libavcodec/librav1e.c b/libavcodec/librav1e.c index dbd728a408..2a6d8bfbed 100644 --- a/libavcodec/librav1e.c +++ b/lib

Re: [FFmpeg-devel] [PATCH 2/2] fftools/ffprobe: export IAMF Stream Group parameters

2024-03-07 Thread Stefano Sabatini
On date Wednesday 2024-03-06 21:49:52 -0300, James Almer wrote: > Signed-off-by: James Almer > --- > fftools/ffprobe.c | 157 +++- > tests/fate/iamf.mak | 8 +- > tests/fate/mov.mak | 8 +- > tests/ref/fate/iam

Re: [FFmpeg-devel] [PATCH 1/2 v2] fftools/ffprobe: export Tile Grid Stream Group parameters

2024-03-07 Thread Stefano Sabatini
On date Thursday 2024-03-07 17:17:06 +0100, Stefano Sabatini wrote: > On date Wednesday 2024-03-06 21:49:51 -0300, James Almer wrote: > > Signed-off-by: James Almer > > --- > > fftools/ffprobe.c | 53 --- > > tests/fate/mov.mak

Re: [FFmpeg-devel] Patch which requires a new library

2024-03-07 Thread Sergio Garcia Murillo
El jue, 7 mar 2024, 16:30, Kieran Kunhya escribió: > On Thu, 7 Mar 2024 at 15:07, Sergio Garcia Murillo < > sergio.garcia.muri...@gmail.com> wrote: > > > > > Could anyone give me any pointers on what is the best way of doing this? > > > > You should use a well known crypto library to implement th

Re: [FFmpeg-devel] Patch which requires a new library

2024-03-07 Thread Kieran Kunhya
On Thu, 7 Mar 2024, 17:16 Sergio Garcia Murillo, < sergio.garcia.muri...@gmail.com> wrote: > El jue, 7 mar 2024, 16:30, Kieran Kunhya escribió: > > > On Thu, 7 Mar 2024 at 15:07, Sergio Garcia Murillo < > > sergio.garcia.muri...@gmail.com> wrote: > > > > > > > > Could anyone give me any pointers

Re: [FFmpeg-devel] Patch which requires a new library

2024-03-07 Thread Sergio Garcia Murillo
El jue, 7 mar 2024, 18:21, Kieran Kunhya escribió: > On Thu, 7 Mar 2024, 17:16 Sergio Garcia Murillo, < > sergio.garcia.muri...@gmail.com> wrote: > > > El jue, 7 mar 2024, 16:30, Kieran Kunhya escribió: > > > > > On Thu, 7 Mar 2024 at 15:07, Sergio Garcia Murillo < > > > sergio.garcia.muri...@gm

Re: [FFmpeg-devel] [PATCH 2/2] fftools/ffprobe: export IAMF Stream Group parameters

2024-03-07 Thread James Almer
On 3/7/2024 1:45 PM, Stefano Sabatini wrote: On date Wednesday 2024-03-06 21:49:52 -0300, James Almer wrote: Signed-off-by: James Almer --- fftools/ffprobe.c | 157 +++- tests/fate/iamf.mak | 8 +- tests/fate/mov.mak

Re: [FFmpeg-devel] [PATCH 1/2 v2] fftools/ffprobe: export Tile Grid Stream Group parameters

2024-03-07 Thread James Almer
On 3/7/2024 1:17 PM, Stefano Sabatini wrote: On date Wednesday 2024-03-06 21:49:51 -0300, James Almer wrote: Signed-off-by: James Almer --- fftools/ffprobe.c | 53 --- tests/fate/mov.mak| 6 +-- .../ref/fate/mov-heic-d

Re: [FFmpeg-devel] [PATCH 1/2 v2] fftools/ffprobe: export Tile Grid Stream Group parameters

2024-03-07 Thread James Almer
On 3/7/2024 1:46 PM, Stefano Sabatini wrote: On date Thursday 2024-03-07 17:17:06 +0100, Stefano Sabatini wrote: On date Wednesday 2024-03-06 21:49:51 -0300, James Almer wrote: Signed-off-by: James Almer --- fftools/ffprobe.c | 53 --- tests/fate/

Re: [FFmpeg-devel] Patch which requires a new library

2024-03-07 Thread Rémi Denis-Courmont
Le torstaina 7. maaliskuuta 2024, 19.30.46 EET Sergio Garcia Murillo a écrit : > > The point is we don't want to use the external lib. > > For what? This is aws lib implementing the aws s3 signatures and using > ffmpeg crypto libs. For not depending on an external library, especially one so small

Re: [FFmpeg-devel] [PATCH 2/2] lavc/vc1dsp: R-V V mspel_pixels

2024-03-07 Thread Rémi Denis-Courmont
Le lauantaina 2. maaliskuuta 2024, 14.06.13 EET flow gg a écrit : > Here adjusting the order, rather than simply using .rept, will be 13%-24% > faster. Isn't it also faster to max LMUL for the adds here? Also this might not be much noticeable on C908, but avoiding sequential dependencies on the

Re: [FFmpeg-devel] [PATCH 23/29] avcodec/mpeg12dec: use ff_frame_new_side_data

2024-03-07 Thread Niklas Haas
On Tue, 05 Mar 2024 11:00:02 +0100 Anton Khirnov wrote: > I prefer consistency here, otherwise the decoder authors have to choose > which function to use, and they are often not aware of the precise > implications of thise choice. Better to always use just one function. Regardless of my personal

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/dovi_rpu: implement support for profile 10

2024-03-07 Thread Niklas Haas
Will merge tomorrow without further comments. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] Patch which requires a new library

2024-03-07 Thread Sergio Garcia Murillo
On Thu, Mar 7, 2024 at 7:02 PM Rémi Denis-Courmont wrote: > Le torstaina 7. maaliskuuta 2024, 19.30.46 EET Sergio Garcia Murillo a > écrit : > > > The point is we don't want to use the external lib. > > > > For what? This is aws lib implementing the aws s3 signatures and using > > ffmpeg crypto l

Re: [FFmpeg-devel] [PATCH 02/18] fftools/ffmpeg_filter: refactor setting input timebase

2024-03-07 Thread Michael Niedermayer
On Wed, Mar 06, 2024 at 12:03:03PM +0100, Anton Khirnov wrote: > Treat it analogously to stream parameters like format/dimensions/etc. > This is functionally different from previous code in 2 ways: > * for non-CFR video, the frame timebase (set by the decoder) is used > rather than the demuxer ti

Re: [FFmpeg-devel] Patch which requires a new library

2024-03-07 Thread Kieran Kunhya
On Thu, 7 Mar 2024, 20:16 Sergio Garcia Murillo, < sergio.garcia.muri...@gmail.com> wrote: > On Thu, Mar 7, 2024 at 7:02 PM Rémi Denis-Courmont > wrote: > > > Le torstaina 7. maaliskuuta 2024, 19.30.46 EET Sergio Garcia Murillo a > > écrit : > > > > The point is we don't want to use the external

[FFmpeg-devel] [PATCH] af_tempo.c: fix checking of samples and zero frame counts

2024-03-07 Thread Rajiv Harlalka
Check for zeros equal to the total samples early, because in case the check is true we would already be leaving the first few frames out. Signed-off-by: Rajiv Harlalka #10692 --- libavfilter/af_atempo.c | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/libavfilter/a

[FFmpeg-devel] [PATCH] af_tempo.c: fix checking of samples and zero frame counts

2024-03-07 Thread Rajiv Harlalka
Check for zeros equal to the total samples early, because in case the check is true we would already be leaving the first few frames out. Signed-off-by: Rajiv Harlalka #10692 --- libavfilter/af_atempo.c | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/libavfilt

[FFmpeg-devel] [PATCH 29/29] tests/fate/matroska: add tests for side data preference

2024-03-07 Thread Andreas Rheinhardt
From: Anton Khirnov Cf. #10857 --- Updated in light of the updated sample. tests/fate/matroska.mak | 6 + tests/ref/fate/matroska-side-data-pref-codec | 348 ++ tests/ref/fate/matroska-side-data-pref-global | 348 ++ 3 files changed, 702

Re: [FFmpeg-devel] [PATCH] avcodec/xbmenc: Avoid snprintf() for data->hex conversion

2024-03-07 Thread Andreas Rheinhardt
Andreas Rheinhardt: > Use a small LUT instead. Improves performance. > > Signed-off-by: Andreas Rheinhardt > --- > libavcodec/xbmenc.c | 21 +++-- > 1 file changed, 15 insertions(+), 6 deletions(-) > > diff --git a/libavcodec/xbmenc.c b/libavcodec/xbmenc.c > index cd8b73afa3..52

Re: [FFmpeg-devel] [PATCH] avcodec/xpmdec: Avoid unnecessary size_t->int conversion

2024-03-07 Thread Andreas Rheinhardt
Andreas Rheinhardt: > Signed-off-by: Andreas Rheinhardt > --- > libavcodec/xpmdec.c | 7 +++ > 1 file changed, 3 insertions(+), 4 deletions(-) > > diff --git a/libavcodec/xpmdec.c b/libavcodec/xpmdec.c > index 5bc02378c8..e7f8a90fdc 100644 > --- a/libavcodec/xpmdec.c > +++ b/libavcodec/xpmde

[FFmpeg-devel] [PATCH] avformat/tests/movenc: Constify write functions

2024-03-07 Thread Andreas Rheinhardt
Forgotten in 2a68d945cd74265bb71c3d38b7a2e7f7d7e87be5. Signed-off-by: Andreas Rheinhardt --- Sorry for this. Will apply this soon to fix FATE. libavformat/tests/movenc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/tests/movenc.c b/libavformat/tests/movenc

Re: [FFmpeg-devel] [PATCH 2/2] lavc/vc1dsp: R-V V mspel_pixels

2024-03-07 Thread flow gg
> Isn't it also faster to max LMUL for the adds here? It requires the use of one more vset, making the time slightly longer: 147.7 (m1), 148.7 (m8 + vset). Also this might not be much noticeable on C908, but avoiding sequential dependencies on the address registers may help. I mean, avoid using a

Re: [FFmpeg-devel] [PATCH] avcodec/xbmenc: Avoid snprintf() for data->hex conversion

2024-03-07 Thread Marton Balint
On Fri, 8 Mar 2024, Andreas Rheinhardt wrote: Andreas Rheinhardt: Use a small LUT instead. Improves performance. Signed-off-by: Andreas Rheinhardt --- libavcodec/xbmenc.c | 21 +++-- 1 file changed, 15 insertions(+), 6 deletions(-) diff --git a/libavcodec/xbmenc.c b/libav

[FFmpeg-devel] [PATCH v2 1/9] avcodec/bsf/pcm_rechunk: reorder supported codec list

2024-03-07 Thread Marton Balint
Use lexical order. Signed-off-by: Marton Balint --- libavcodec/bsf/pcm_rechunk.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/libavcodec/bsf/pcm_rechunk.c b/libavcodec/bsf/pcm_rechunk.c index 28b5722ac9..423c414d76 100644 --- a/libavcodec/bsf/pcm_rec

[FFmpeg-devel] [PATCH v2 2/9] avcodec/bsf/pcm_rechunk: add some more supported PCM formats

2024-03-07 Thread Marton Balint
Signed-off-by: Marton Balint --- libavcodec/bsf/pcm_rechunk.c | 20 +++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/libavcodec/bsf/pcm_rechunk.c b/libavcodec/bsf/pcm_rechunk.c index 423c414d76..b1b57f96a9 100644 --- a/libavcodec/bsf/pcm_rechunk.c +++ b/libavcode

[FFmpeg-devel] [PATCH v2 3/9] avformat/daudenc: force 2000 sample packet size with a bsf

2024-03-07 Thread Marton Balint
The samples I found all have 2000 sample packets, and by forcing the packet size with a bsf we could automagically make muxing work for packets containing more than 3640 samples. Signed-off-by: Marton Balint --- configure | 1 + doc/muxers.texi | 16 +--

[FFmpeg-devel] [PATCH v2 4/9] fate: make filter-channelsplit test use a fixed frame size

2024-03-07 Thread Marton Balint
Muxing multiple streams to raw files is allowed but the packets are interleaved, so the output is dependant of packet size. Signed-off-by: Marton Balint --- tests/fate/filter-audio.mak | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/fate/filter-audio.mak b/tests/fat

[FFmpeg-devel] [PATCH v1 1/2] lavc/vvcdec: Add missed chroma sampling factor for crop offset

2024-03-07 Thread fei . w . wang-at-intel . com
From: Fei Wang Signed-off-by: Fei Wang --- libavcodec/vvc/vvc_refs.c | 8 libavcodec/vvc/vvcdec.c | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/libavcodec/vvc/vvc_refs.c b/libavcodec/vvc/vvc_refs.c index 99f2dcf3ec..afcfc09da7 100644 --- a/libavcodec/vvc/vv

[FFmpeg-devel] [PATCH v2 5/9] fate: use a fixed wav demux packet size for amix tests

2024-03-07 Thread Marton Balint
The dropout transition feature of the amix filter depends on the incoming packet size. Signed-off-by: Marton Balint --- tests/fate/filter-audio.mak | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/fate/filter-audio.mak b/tests/fate/filter-audio.mak index 5b5d741f06

[FFmpeg-devel] [PATCH v2 6/9] fate: use atrim filter instead of -frames:a 20 for fate-filter-tremolo

2024-03-07 Thread Marton Balint
To make it independent of incoming wav demuxer packet size. Signed-off-by: Marton Balint --- tests/fate/filter-audio.mak | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/fate/filter-audio.mak b/tests/fate/filter-audio.mak index 78fec28b04..6ddc0baf57 100644 --- a/tes

[FFmpeg-devel] [PATCH v2 7/9] avformat/pcm: factorize and improve determining the default packet size

2024-03-07 Thread Marton Balint
- Remove the 1024 cap on the number of samples, for high sample rate audio it was suboptimal, calculate the low neighbour power of two for the number of samples (audio blocks) instead. - Make the function work correctly for non-pcm codecs by using av_get_audio_frame_duration2() to esimate the

[FFmpeg-devel] [PATCH v1 2/2] lavc/vvc_ps: Correct NoOutputBeforeRecoveryFlag of IDR

2024-03-07 Thread fei . w . wang-at-intel . com
From: Fei Wang Signed-off-by: Fei Wang --- libavcodec/vvc/vvc_ps.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/vvc/vvc_ps.c b/libavcodec/vvc/vvc_ps.c index e6e46d2039..7972803da6 100644 --- a/libavcodec/vvc/vvc_ps.c +++ b/libavcodec/vvc/vvc_ps.c @@ -742,7 +742

Re: [FFmpeg-devel] [PATCH] avcodec/xbmenc: Avoid snprintf() for data->hex conversion

2024-03-07 Thread Andreas Rheinhardt
Marton Balint: > > > On Fri, 8 Mar 2024, Andreas Rheinhardt wrote: > >> Andreas Rheinhardt: >>> Use a small LUT instead. Improves performance. >>> >>> Signed-off-by: Andreas Rheinhardt >>> --- >>>  libavcodec/xbmenc.c | 21 +++-- >>>  1 file changed, 15 insertions(+), 6 deletions

Re: [FFmpeg-devel] [RFC] fateserver

2024-03-07 Thread Marth64
Hi Michael, all, While awaiting direction I will go ahead and study this process further. From an overall view, it is something I am comfortable to help maintain. Perhaps my focus can be to get the current version in a stable place, and work on a long term improvement strategy to the rewrite versi

Re: [FFmpeg-devel] [PATCH] avcodec/xbmenc: Avoid snprintf() for data->hex conversion

2024-03-07 Thread Marton Balint
On Fri, 8 Mar 2024, Andreas Rheinhardt wrote: Marton Balint: On Fri, 8 Mar 2024, Andreas Rheinhardt wrote: Andreas Rheinhardt: Use a small LUT instead. Improves performance. Signed-off-by: Andreas Rheinhardt ---  libavcodec/xbmenc.c | 21 +++--  1 file changed, 15 insert

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: add h264_mp4toannexb bitstream filter if needed when muxing h264

2024-03-07 Thread Marton Balint
On Wed, 6 Mar 2024, Tomas Härdin wrote: lör 2024-02-24 klockan 15:13 +0100 skrev Andreas Rheinhardt: Tomas Härdin: > > > > +static int mxf_check_bitstream(AVFormatContext *s, AVStream > > > > *st, > > > > const AVPacket *pkt) > > > > +{ > > > > +    if (st->codecpar->codec_id == AV_CODEC_ID_H

Re: [FFmpeg-devel] [PATCH] lavfi/vulkan_filter: fix input format

2024-03-07 Thread Xiang, Haihao
On Do, 2024-03-07 at 05:05 +0100, Lynne wrote: > Mar 6, 2024, 06:26 by haihao.xiang-at-intel@ffmpeg.org: > > > From: Haihao Xiang > > > > Otherwise s->input_format is always yuv420p. > > > > This fixes invalid output format for hwframe download in the command > > below: > > ./ffmpeg -init_h

[FFmpeg-devel] [PATCH] avcodec/hevc_cabac: Let compiler count offsets

2024-03-07 Thread Andreas Rheinhardt
This is easily possible with an X macro. Using an enum for the offsets also allows to remove two arrays which are not really needed and will typically be optimized away by the compiler: The first just exists to count the number of syntax elements*, the second one exists to get offset[CONSTANT]. The

Re: [FFmpeg-devel] [PATCH 02/18] fftools/ffmpeg_filter: refactor setting input timebase

2024-03-07 Thread Anton Khirnov
Quoting Michael Niedermayer (2024-03-07 21:37:39) > On Wed, Mar 06, 2024 at 12:03:03PM +0100, Anton Khirnov wrote: > > Treat it analogously to stream parameters like format/dimensions/etc. > > This is functionally different from previous code in 2 ways: > > * for non-CFR video, the frame timebase (

[FFmpeg-devel] [PATCH] lavu/hwcontext_vulkan: check both vendor and PCI IDs

2024-03-07 Thread Xiang, Haihao
From: Haihao Xiang Otherwise the derived device and the source device might have different PCI ID or vendor ID in a multiple-device system. Signed-off-by: Haihao Xiang --- libavutil/hwcontext_vulkan.c | 31 +-- 1 file changed, 29 insertions(+), 2 deletions(-) diff