Re: [FFmpeg-devel] [PATCH v5 3/3] lavc/dxvenc: improve compatibility with Resolume products

2025-04-30 Thread Emma Worley
Given the lack of feedback I'd like to request this changeset be merged :) ___ 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 w

[FFmpeg-devel] [PATCH] hwcontext_vulkan: fix exporting multi-plane DRM modifiers

2025-04-30 Thread Russell Greene
From: Russell Greene Previously, it was assumed that `drmFormatModifierPlaneCount` was one for all modifiers when exporting, which is not always the case, in particular for AMD GPUs and maybe others. Fetch the number of memory planes and fill the structs appropriately in this situation. The en

[FFmpeg-devel] [PATCH] avcodec/exr: add support for half-float DWAA/B compression

2025-04-30 Thread James Almer
Fixes ticket #11555. Signed-off-by: James Almer --- libavcodec/exr.c | 56 +++- 1 file changed, 41 insertions(+), 15 deletions(-) diff --git a/libavcodec/exr.c b/libavcodec/exr.c index a16dd96502..1b3aad96ea 100644 --- a/libavcodec/exr.c +++ b/libavco

Re: [FFmpeg-devel] [PATCH] avformat/mpegts: update stream info when PMT ES stream_type changes

2025-04-30 Thread Pavel Koshevoy
On Sat, Apr 26, 2025 at 5:20 AM Michael Niedermayer wrote: > On Fri, Apr 25, 2025 at 11:45:08AM -0600, Pavel Koshevoy wrote: > > On Fri, Apr 25, 2025 at 7:40 AM Michael Niedermayer < > mich...@niedermayer.cc> > > wrote: > > > > > On Tue, Apr 22, 2025 at 06:51:55PM -0600, Pavel Koshevoy wrote: > >

[FFmpeg-devel] [PATCH] postproc/tests: Add test for temporal denoise

2025-04-30 Thread Michael Niedermayer
Sponsored-by: Sovereign Tech Fund Signed-off-by: Michael Niedermayer --- libpostproc/Makefile | 1 + libpostproc/tests/temptest.c | 120 + tests/fate/libpostproc.mak | 4 + tests/ref/fate/temptest | 336 +++ 4 files changed, 461 inser

Re: [FFmpeg-devel] [PATCH v2] avcodec/cbs_apv: don't return an error when reading empty buffers

2025-04-30 Thread Mark Thompson
On 30/04/2025 22:22, James Almer wrote: > The output will be a fragment with zero units, which is a lot more user > friendly > than making them think something went wrong, as it already happens with > cbs_av1. > > Signed-off-by: James Almer > --- > libavcodec/cbs_apv.c | 4 ++-- > 1 file chang

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

2025-04-30 Thread Mark Thompson
On 27/04/2025 20:58, James Almer wrote: > 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

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

2025-04-30 Thread Mark Thompson
On 27/04/2025 23:21, James Almer wrote: > 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 change

Re: [FFmpeg-devel] [PATCH 1/5] avformat/apvdec: Use ffio_read_size()

2025-04-30 Thread Andreas Rheinhardt
Mark Thompson: > On 28/04/2025 10:39, Andreas Rheinhardt wrote: >> Patches attached. >> >> - Andreas >> > >> [PATCH 1/5] avformat/apvdec: Use ffio_read_size() > > LGTM. > >> [PATCH 2/5] avformat/apvdec: Check before access > > LGTM. > >> pkt->flags= AV_PKT_FLAG_KEY; > > Not sure w

Re: [FFmpeg-devel] [PATCH] avcodec/apv_dsp: Fix left-shift of negative value

2025-04-30 Thread Andreas Rheinhardt
Mark Thompson: > On 30/04/2025 19:26, Andreas Rheinhardt wrote: >> Patch attached. >> >> - Andreas > > Hmm, the spec does write exactly this but also defines << as arithmetic left > shift of a two's complement representation so it works. > > Seems fair, LGTM. > > More generally this line is not

Re: [FFmpeg-devel] [PATCH] avcodec/apv_dsp: Fix left-shift of negative value

2025-04-30 Thread Mark Thompson
On 30/04/2025 19:26, Andreas Rheinhardt wrote: > Patch attached. > > - Andreas Hmm, the spec does write exactly this but also defines << as arithmetic left shift of a two's complement representation so it works. Seems fair, LGTM. More generally this line is not robust to extreme values (can in

[FFmpeg-devel] [PATCH v2] avcodec/cbs_apv: don't return an error when reading empty buffers

2025-04-30 Thread James Almer
The output will be a fragment with zero units, which is a lot more user friendly than making them think something went wrong, as it already happens with cbs_av1. Signed-off-by: James Almer --- libavcodec/cbs_apv.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec

Re: [FFmpeg-devel] [PATCH] avcodec/cbs_apv: don't return an error when reading empty buffers

2025-04-30 Thread James Almer
On 4/30/2025 6:17 PM, Mark Thompson wrote: On 29/04/2025 20:57, James Almer wrote: The output will be a fragment with zero units, which is a lot more user friendly than making them think something went wrong, as it already happens with cbs_av1. Signed-off-by: James Almer --- libavcodec/cbs_a

Re: [FFmpeg-devel] [PATCH] avcodec/cbs_apv: don't return an error when reading empty buffers

2025-04-30 Thread Mark Thompson
On 29/04/2025 20:57, James Almer wrote: > The output will be a fragment with zero units, which is a lot more user > friendly > than making them think something went wrong, as it already happens with > cbs_av1. > > Signed-off-by: James Almer > --- > libavcodec/cbs_apv.c | 2 +- > 1 file changed

Re: [FFmpeg-devel] [PATCH 1/5] avformat/apvdec: Use ffio_read_size()

2025-04-30 Thread Mark Thompson
On 28/04/2025 10:39, Andreas Rheinhardt wrote: > Patches attached. > > - Andreas > > [PATCH 1/5] avformat/apvdec: Use ffio_read_size() LGTM. > [PATCH 2/5] avformat/apvdec: Check before access LGTM. > pkt->flags= AV_PKT_FLAG_KEY; Not sure where the funny alignment on this line ca

[FFmpeg-devel] [PATCH] lavc: Add unit test for APV entropy decode

2025-04-30 Thread Mark Thompson
--- Cleaned up a bit from use earlier. This program allows testing of changes to entropy decode in a form much easier to debug than a whole stream. I had a more complete randomised test of transform here as well along with test code for checking intermediates, though it seems like the checkasm

[FFmpeg-devel] [PATCH] postproc/postprocess_template: Fix left shift of negative numbers

2025-04-30 Thread Andreas Rheinhardt
Patch attached. - Andreas From 54ceac12363b666ad8005f282f6f9f08b6334b21 Mon Sep 17 00:00:00 2001 From: Andreas Rheinhardt Date: Wed, 30 Apr 2025 20:30:53 +0200 Subject: [PATCH] postproc/postprocess_template: Fix left shift of negative numbers Affects the blocktest test tool. Signed-off-by: And

[FFmpeg-devel] [PATCH] avcodec/apv_dsp: Fix left-shift of negative value

2025-04-30 Thread Andreas Rheinhardt
Patch attached. - Andreas From a33080ab9e500432e816808fbdc42d15b47e7237 Mon Sep 17 00:00:00 2001 From: Andreas Rheinhardt Date: Wed, 30 Apr 2025 20:07:35 +0200 Subject: [PATCH] avcodec/apv_dsp: Fix left-shift of negative value Signed-off-by: Andreas Rheinhardt --- libavcodec/apv_dsp.c | 2 +-

Re: [FFmpeg-devel] [PATCH 1/5] avformat/apvdec: Use ffio_read_size()

2025-04-30 Thread Andreas Rheinhardt
Andreas Rheinhardt: > Patches attached. > > - Andreas > Will apply this patchset tomorrow unless there are objections. - Andreas ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, vi

[FFmpeg-devel] [PATCH] tests/api/api-dump-stream-meta-test: Fix leaks

2025-04-30 Thread Andreas Rheinhardt
Patch attached. - Andreas From 6d4ff45d38b97ed55feda84276d31db25156dcea Mon Sep 17 00:00:00 2001 From: Andreas Rheinhardt Date: Wed, 30 Apr 2025 19:31:50 +0200 Subject: [PATCH] tests/api/api-dump-stream-meta-test: Fix leaks Signed-off-by: Andreas Rheinhardt --- tests/api/api-dump-stream-meta-t

Re: [FFmpeg-devel] [PATCH 11/17] swscale/x86: add SIMD backend

2025-04-30 Thread Niklas Haas
On Tue, 29 Apr 2025 15:00:50 +0200 Michael Niedermayer wrote: > On Sat, Apr 26, 2025 at 07:41:15PM +0200, Niklas Haas wrote: > > From: Niklas Haas > > > > This covers most 8-bit and 16-bit ops, and some 32-bit ops. It also covers > > all > > floating point operations. While this is not yet 100%

Re: [FFmpeg-devel] [PATCH] avcodec: add APV encoder using liboapv

2025-04-30 Thread James Almer
On 4/30/2025 11:03 AM, Dawid Kozinski/Multimedia (PLT) /SRPOL/Staff Engineer/Samsung Electronics wrote: -Original Message- From: ffmpeg-devel On Behalf Of James Almer Sent: środa, 30 kwietnia 2025 02:40 To:ffmpeg-devel@ffmpeg.org Subject: [FFmpeg-devel] [PATCH] avcodec: add APV encoder us

Re: [FFmpeg-devel] [PATCH] avcodec: add APV encoder using liboapv

2025-04-30 Thread Dawid Kozinski/Multimedia (PLT) /SRPOL/Staff Engineer/Samsung Electronics
> -Original Message- > From: ffmpeg-devel On Behalf Of James > Almer > Sent: środa, 30 kwietnia 2025 02:40 > To: ffmpeg-devel@ffmpeg.org > Subject: [FFmpeg-devel] [PATCH] avcodec: add APV encoder using liboapv > > From: Dawid Kozinski > > Co-authored-by: James Almer > Signed-off-by

Re: [FFmpeg-devel] [PATCH v2 4/4] avcodec/apv_decoder: Provided support for APV decoder

2025-04-30 Thread Lynne
On 30/04/2025 10:11, Dawid Kozinski wrote: - Added APV decoder wrapper - Changes in project configuration file and libavcodec Makefile - Added documentation for APV decoder wrapper Signed-off-by: Dawid Kozinski --- configure | 1 + doc/decoders.texi | 27 ++ libavcodec/

Re: [FFmpeg-devel] [PATCH] avfilter/vf_setparams: Fix chroma_location being cleared

2025-04-30 Thread Tobias Rapp
On 30/04/2025 14:09, Tobias Rapp wrote: Fix chroma_location being cleared by setrange and setfield filters. This was forgotten in 201f1cba150d44de6fedfeee4e8647170ed5fbca. Signed-off-by: Tobias Rapp --- libavfilter/vf_setparams.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavfi

Re: [FFmpeg-devel] [PATCH v2 3/4] avcodec/apv_encoder: Provided support for APV encoder

2025-04-30 Thread James Almer
On 4/30/2025 5:11 AM, Dawid Kozinski wrote: - Added APV encoder wrapper - Changes in project configuration file and libavcodec Makefile - Added documentation for oapv wrapper Signed-off-by: Dawid Kozinski --- configure | 4 + doc/encoders.texi | 41 +++ doc/general

[FFmpeg-devel] [PATCH] avfilter/vf_setparams: Fix chroma_location being cleared

2025-04-30 Thread Tobias Rapp
Fix chroma_location being cleared by setrange and setfield filters. This was forgotten in 201f1cba150d44de6fedfeee4e8647170ed5fbca. Signed-off-by: Tobias Rapp --- libavfilter/vf_setparams.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavfilter/vf_setparams.c b/libavfilter/vf_setparam

[FFmpeg-devel] [PATCH v2 4/4] avcodec/apv_decoder: Provided support for APV decoder

2025-04-30 Thread Dawid Kozinski
- Added APV decoder wrapper - Changes in project configuration file and libavcodec Makefile - Added documentation for APV decoder wrapper Signed-off-by: Dawid Kozinski --- configure | 1 + doc/decoders.texi | 27 ++ libavcodec/Makefile| 1 + libavcodec/libapvdec.c | 56

[FFmpeg-devel] [PATCH v2 3/4] avcodec/apv_encoder: Provided support for APV encoder

2025-04-30 Thread Dawid Kozinski
- Added APV encoder wrapper - Changes in project configuration file and libavcodec Makefile - Added documentation for oapv wrapper Signed-off-by: Dawid Kozinski --- configure | 4 + doc/encoders.texi | 41 +++ doc/general_contents.texi | 8 + libavcodec/Makefile

[FFmpeg-devel] [PATCH v2 2/4] avformat/mov_demuxer: Extended MOV demuxer to handle APV video content

2025-04-30 Thread Dawid Kozinski
- Added apv extension to the list of extensions for ff_mov_demuxer Signed-off-by: Dawid Kozinski --- libavformat/demux.c | 1 + libavformat/mov.c | 2 ++ 2 files changed, 3 insertions(+) diff --git a/libavformat/demux.c b/libavformat/demux.c index 2795863567..4e76de0629 100644 --- a/libavform

[FFmpeg-devel] [PATCH v2 1/4] avformat/mov_muxer: Extended MOV muxer to handle APV video content

2025-04-30 Thread Dawid Kozinski
- Changes in mov_write_video_tag function to handle APV elementary stream - Provided structure APVDecoderConfigurationRecord that specifies the decoder configuration information for APV video content Signed-off-by: Dawid Kozinski --- libavformat/Makefile| 2 +- libavformat/apv.c | 8