Re: [FFmpeg-devel] [PATCH v7 2/3] libavformat: Add DFPWM raw format

2022-03-14 Thread Tomas Härdin
+static const AVOption dfpwm_options[] = { +{ "sample_rate", "", offsetof(DFPWMAudioDemuxerContext, sample_rate), AV_OPT_TYPE_INT, {.i64 = 48000}, 0, INT_MAX, AV_OPT_FLAG_DECODING_PARAM }, +{ "channels","", offsetof(DFPWMAudioDemuxerContext, channels), AV_OPT_TYPE_INT, {.i64 = 1}, 0, IN

Re: [FFmpeg-devel] [PATCH v7 2/3] libavformat: Add DFPWM raw format

2022-03-14 Thread Paul B Mahol
On Mon, Mar 14, 2022 at 10:47 AM Tomas Härdin wrote: > +static const AVOption dfpwm_options[] = { > +{ "sample_rate", "", offsetof(DFPWMAudioDemuxerContext, > sample_rate), AV_OPT_TYPE_INT, {.i64 = 48000}, 0, INT_MAX, > AV_OPT_FLAG_DECODING_PARAM }, > +{ "channels","", offsetof(DFPWMA

Re: [FFmpeg-devel] [PATCH v6 1/2] libavcodec: Added DFPWM1a codec

2022-03-14 Thread Tomas Härdin
mån 2022-03-07 klockan 22:04 -0500 skrev Jack Bruienne: > On 3/7/22 06:03, Tomas Härdin wrote: > > > tor 2022-03-03 klockan 10:44 -0500 skrev Jack Bruienne: > > >   From the wiki page (https://wiki.vexatos.com/dfpwm): > > > > DFPWM (Dynamic Filter Pulse Width Modulation) is an audio codec > > > >

Re: [FFmpeg-devel] [PATCH v7 2/3] libavformat: Add DFPWM raw format

2022-03-14 Thread Tomas Härdin
mån 2022-03-14 klockan 10:56 +0100 skrev Paul B Mahol: > On Mon, Mar 14, 2022 at 10:47 AM Tomas Härdin > wrote: > > > +static const AVOption dfpwm_options[] = { > > +    { "sample_rate", "", offsetof(DFPWMAudioDemuxerContext, > > sample_rate), AV_OPT_TYPE_INT, {.i64 = 48000}, 0, INT_MAX, > > AV_O

Re: [FFmpeg-devel] [PATCH v4 1/4] lavc/vaapi_encode_h265: Add GPB frame support for hevc_vaapi

2022-03-14 Thread Wang, Fei W
> -Original Message- > From: ffmpeg-devel On Behalf Of Xiang, > Haihao > Sent: Monday, March 14, 2022 10:15 AM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH v4 1/4] lavc/vaapi_encode_h265: Add GPB > frame support for hevc_vaapi > > On Sun, 2022-03-13 at 20:45 +

[FFmpeg-devel] [PATCH 1/2] avcodec: add null encoders

2022-03-14 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavcodec/Makefile | 2 ++ libavcodec/allcodecs.c | 2 ++ libavcodec/codec_desc.c | 14 ++ libavcodec/codec_id.h | 2 ++ libavcodec/nullenc.c| 61 + 5 files changed, 81 insertions(+) create mode 100644

[FFmpeg-devel] [PATCH 2/2] avformat/nullenc: use null encoders instead

2022-03-14 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavformat/nullenc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/nullenc.c b/libavformat/nullenc.c index d4769d5920..a83ed72039 100644 --- a/libavformat/nullenc.c +++ b/libavformat/nullenc.c @@ -30,8 +30,8 @@ static int null

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: add null encoders

2022-03-14 Thread James Almer
On 3/14/2022 8:56 AM, Paul B Mahol wrote: Signed-off-by: Paul B Mahol --- libavcodec/Makefile | 2 ++ libavcodec/allcodecs.c | 2 ++ libavcodec/codec_desc.c | 14 ++ libavcodec/codec_id.h | 2 ++ libavcodec/nullenc.c| 61 +

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: add null encoders

2022-03-14 Thread Paul B Mahol
On Mon, Mar 14, 2022 at 1:12 PM James Almer wrote: > > > On 3/14/2022 8:56 AM, Paul B Mahol wrote: > > Signed-off-by: Paul B Mahol > > --- > > libavcodec/Makefile | 2 ++ > > libavcodec/allcodecs.c | 2 ++ > > libavcodec/codec_desc.c | 14 ++ > > libavcodec/codec_id.h | 2

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: add null encoders

2022-03-14 Thread James Almer
On 3/14/2022 9:17 AM, Paul B Mahol wrote: On Mon, Mar 14, 2022 at 1:12 PM James Almer wrote: On 3/14/2022 8:56 AM, Paul B Mahol wrote: Signed-off-by: Paul B Mahol --- libavcodec/Makefile | 2 ++ libavcodec/allcodecs.c | 2 ++ libavcodec/codec_desc.c | 14 ++ liba

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: add null encoders

2022-03-14 Thread Paul B Mahol
On Mon, Mar 14, 2022 at 1:17 PM James Almer wrote: > > > On 3/14/2022 9:17 AM, Paul B Mahol wrote: > > On Mon, Mar 14, 2022 at 1:12 PM James Almer wrote: > > > >> > >> > >> On 3/14/2022 8:56 AM, Paul B Mahol wrote: > >>> Signed-off-by: Paul B Mahol > >>> --- > >>>libavcodec/Makefile |

[FFmpeg-devel] [PATCH] Allow to modify max qp configuration parameter in libvpx without reseting the encoder

2022-03-14 Thread Danil Chapovalov
--- libavcodec/libvpxenc.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/libavcodec/libvpxenc.c b/libavcodec/libvpxenc.c index 8f94ba15dc..45baeed435 100644 --- a/libavcodec/libvpxenc.c +++ b/libavcodec/libvpxenc.c @@ -1658,6 +1658,13 @@ static int vpx_encode(AVCodecContext *avctx, AV

[FFmpeg-devel] [PATCH] ffmpeg: print proper statistics for video output when using wrapped avframe encoder

2022-03-14 Thread James Almer
pkt->size contains the size of an AVFrame struct plus some padding, which is meaningless. Use instead the size of a raw video frame, as if it was the rawvideo encoder. Signed-off-by: James Almer --- fftools/ffmpeg.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a

Re: [FFmpeg-devel] [PATCH 3/4] avformat/mxfdec: Check for avio_read() failure in mxf_read_strong_ref_array()

2022-03-14 Thread Michael Niedermayer
On Sun, Mar 13, 2022 at 04:52:25PM +0100, Marton Balint wrote: > > > On Sun, 13 Mar 2022, Michael Niedermayer wrote: > > > Signed-off-by: Michael Niedermayer > > --- > > libavformat/mxfdec.c | 8 +++- > > 1 file changed, 7 insertions(+), 1 deletion(-) > > > > diff --git a/libavformat/mxfdec

Re: [FFmpeg-devel] [PATCH 4/4] avformat/mxfdec: Do not clear array in mxf_read_strong_ref_array() before writing

2022-03-14 Thread Michael Niedermayer
On Sun, Mar 13, 2022 at 04:53:29PM +0100, Marton Balint wrote: > > > On Sun, 13 Mar 2022, Michael Niedermayer wrote: > > > Signed-off-by: Michael Niedermayer > > --- > > libavformat/mxfdec.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/libavformat/mxfdec.c b/li

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/vp9_superframe_split_bsf: Check in size

2022-03-14 Thread Michael Niedermayer
On Sun, Mar 13, 2022 at 04:03:42PM -0300, James Almer wrote: > > > On 3/12/2022 8:52 PM, Michael Niedermayer wrote: > > Fixes: Out of array read > > Fixes: > > 45137/clusterfuzz-testcase-minimized-ffmpeg_BSF_VP9_SUPERFRAME_SPLIT_fuzzer-4984270639202304 > > > > Found-by: continuous fuzzing proce

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/vp9_superframe_split_bsf: Check in size

2022-03-14 Thread James Almer
On 3/14/2022 11:04 AM, Michael Niedermayer wrote: On Sun, Mar 13, 2022 at 04:03:42PM -0300, James Almer wrote: On 3/12/2022 8:52 PM, Michael Niedermayer wrote: Fixes: Out of array read Fixes: 45137/clusterfuzz-testcase-minimized-ffmpeg_BSF_VP9_SUPERFRAME_SPLIT_fuzzer-4984270639202304 Fou

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/vp9_superframe_split_bsf: Check in size

2022-03-14 Thread Michael Niedermayer
On Mon, Mar 14, 2022 at 11:07:38AM -0300, James Almer wrote: > > > On 3/14/2022 11:04 AM, Michael Niedermayer wrote: > > On Sun, Mar 13, 2022 at 04:03:42PM -0300, James Almer wrote: > > > > > > > > > On 3/12/2022 8:52 PM, Michael Niedermayer wrote: > > > > Fixes: Out of array read > > > > Fixes

[FFmpeg-devel] [PATCH] ffmpeg: add packet duration to AVPacket logging

2022-03-14 Thread Jan Ekström
From: Jaakko Perttilä Especially useful when debugging subtitle output, but also shows if values are set or not for demux and encoding. Co-authored-by: Jan Ekström Signed-off-by: Jan Ekström --- fftools/ffmpeg.c | 27 ++- 1 file changed, 18 insertions(+), 9 deletions(

Re: [FFmpeg-devel] [PATCH 00/13] [RFC] Reduce unnecessary recompilation

2022-03-14 Thread Michael Niedermayer
On Fri, Mar 11, 2022 at 02:17:42PM +0200, Martin Storsjö wrote: > On Wed, 23 Feb 2022, Martin Storsjö wrote: > > > When updating the ffmpeg source, one quite often ends up in a situation > > where practically all of the codebase (or all of a library) gets rebuilt, > > due to updates to headers tha

Re: [FFmpeg-devel] [PATCH] ffmpeg: add packet duration to AVPacket logging

2022-03-14 Thread Paul B Mahol
LGTM, could also be refactored/deduplicated. ___ 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] Allow to modify max qp configuration parameter in libvpx without reseting the encoder

2022-03-14 Thread Jan Ekström
On Mon, Mar 14, 2022 at 3:05 PM Danil Chapovalov wrote: > > --- Probably something a la avcodec/libvpxenc: enable dynamic quantizer reconfiguration ? > libavcodec/libvpxenc.c | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/libavcodec/libvpxenc.c b/libavcodec/libvpxenc.c > inde

[FFmpeg-devel] [PATCH v2 1/2] avformat/movenc: initialize pts/dts of timecode packet

2022-03-14 Thread lance . lmwang
From: Limin Wang Fix below error message when timecode packet is written. "Application provided duration: -9223372036854775808 / timestamp: -9223372036854775808 is out of range for mov/mp4 format" try to reproduce by: ffmpeg -y -f lavfi -i color -metadata "timecode=00:00:00:00" -t 1 test.mov N

[FFmpeg-devel] [PATCH v2 2/2] avformat/movenc: use warning log level and small adjustment for the log

2022-03-14 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- libavformat/movenc.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 436ceb8..d0072f5 100644 --- a/libavformat/movenc.c +++ b/libavformat/movenc.c @@ -5638,9 +5638,8 @@ stat

Re: [FFmpeg-devel] [PATCH v2] avcodec: Add dv marker bsf

2022-03-14 Thread Dave Rice
> On Mar 12, 2022, at 1:09 PM, Michael Niedermayer > wrote: > > On Sat, Mar 12, 2022 at 10:11:52AM -0500, Dave Rice wrote: >> >> >>> On Mar 10, 2022, at 4:41 AM, Tobias Rapp wrote: >>> >>> On 09/03/2022 19:18, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer --- >>

Re: [FFmpeg-devel] [PATCH 3/4] avformat/mxfdec: Check for avio_read() failure in mxf_read_strong_ref_array()

2022-03-14 Thread Marton Balint
On Mon, 14 Mar 2022, Michael Niedermayer wrote: On Sun, Mar 13, 2022 at 04:52:25PM +0100, Marton Balint wrote: On Sun, 13 Mar 2022, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer --- libavformat/mxfdec.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --

[FFmpeg-devel] [PATCH 1/2] avformat/mxfenc: allow more bits for variable part in uuid generation

2022-03-14 Thread Marton Balint
Also make sure we do not change the product UID. Signed-off-by: Marton Balint --- libavformat/mxfenc.c| 9 + tests/ref/fate/copy-trac4914| 2 +- tests/ref/fate/mxf-d10-user-comments| 6 +++--- tests/ref/fate/mxf-opatom-user-comments | 2 +- tests/ref/f

[FFmpeg-devel] [PATCH 2/2] avformat/mxfenc: do not write index tables with the same InstanceUID

2022-03-14 Thread Marton Balint
Only index tables repeating previous index tables should use the same InstaceUID. Use the index start position when generating the InstanceUID to fix this. Signed-off-by: Marton Balint --- libavformat/mxfenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mxfenc

Re: [FFmpeg-devel] [PATCH v2] avcodec: Add dv marker bsf

2022-03-14 Thread Michael Niedermayer
On Mon, Mar 14, 2022 at 12:04:46PM -0400, Dave Rice wrote: > > > > On Mar 12, 2022, at 1:09 PM, Michael Niedermayer > > wrote: > > > > On Sat, Mar 12, 2022 at 10:11:52AM -0500, Dave Rice wrote: > >> > >> > >>> On Mar 10, 2022, at 4:41 AM, Tobias Rapp wrote: > >>> > >>> On 09/03/2022 19:18,

Re: [FFmpeg-devel] [PATCH 2/3] tools/target_dec_fuzzer: Adjust threshold for DFA

2022-03-14 Thread Michael Niedermayer
On Fri, Mar 11, 2022 at 09:20:24PM +0100, Michael Niedermayer wrote: > Fixes: Timeout > Fixes: > 45351/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_DFA_fuzzer-5768895011618816 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Signed-

[FFmpeg-devel] [PATCH] avfilter/vf_subtitles: pass storage size to libass

2022-03-14 Thread Oneric
Due to a quirk of the ASS format some tags depend on the exact storage resolution of the video, so tell libass via ass_set_storage_size. --- ass_set_storage_size exists since libass 0.10.2; ffmpeg since 5.0 already requires 0.11.0. This resolution dependences of ASS was already recognised when th

Re: [FFmpeg-devel] [PATCH 2/4] avformat/mxfdec: Check count in mxf_read_strong_ref_array()

2022-03-14 Thread Tomas Härdin
sön 2022-03-13 klockan 00:52 +0100 skrev Michael Niedermayer: > Signed-off-by: Michael Niedermayer > --- >  libavformat/mxfdec.c | 8 +++- >  1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c > index b85c10bf19..d7cdd22c8a 100644 > ---

Re: [FFmpeg-devel] [PATCH 3/4] avformat/mxfdec: Check for avio_read() failure in mxf_read_strong_ref_array()

2022-03-14 Thread Tomas Härdin
sön 2022-03-13 klockan 00:52 +0100 skrev Michael Niedermayer: > Signed-off-by: Michael Niedermayer > --- >  libavformat/mxfdec.c | 8 +++- >  1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c > index d7cdd22c8a..828fc0f9f1 100644 > ---

Re: [FFmpeg-devel] [PATCH] avfilter/vf_subtitles: pass storage size to libass

2022-03-14 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of Oneric > Sent: Monday, March 14, 2022 8:07 PM > To: ffmpeg-devel@ffmpeg.org > Cc: Oneric > Subject: [FFmpeg-devel] [PATCH] avfilter/vf_subtitles: pass storage size > to libass > > Due to a quirk of the ASS format some tags depend

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mxfenc: allow more bits for variable part in uuid generation

2022-03-14 Thread Tomas Härdin
mån 2022-03-14 klockan 19:49 +0100 skrev Marton Balint: > Also make sure we do not change the product UID. > > Signed-off-by: Marton Balint > --- >  libavformat/mxfenc.c    | 9 + >  tests/ref/fate/copy-trac4914    | 2 +- >  tests/ref/fate/mxf-d10-user-comments    |

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mxfenc: do not write index tables with the same InstanceUID

2022-03-14 Thread Tomas Härdin
mån 2022-03-14 klockan 19:49 +0100 skrev Marton Balint: > Only index tables repeating previous index tables should use the same > InstaceUID. Use the index start position when generating the > InstanceUID to fix > this. > > Signed-off-by: Marton Balint > --- >  libavformat/mxfenc.c | 2 +- >  1 fi

Re: [FFmpeg-devel] [PATCH] avfilter/vf_subtitles: pass storage size to libass

2022-03-14 Thread Oneric
On Mon, Mar 14, 2022 at 19:35:36 +, Soft Works wrote: > > I've been at the same point some time ago where I wondered why ffmpeg is > not setting this, but then I had found that it is overridden by the call > to ass_set_pixel_aspect(). > > ass_set_pixel_aspect() is setting settings.par and if

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mxfenc: do not write index tables with the same InstanceUID

2022-03-14 Thread Marton Balint
On Mon, 14 Mar 2022, Tomas Härdin wrote: mån 2022-03-14 klockan 19:49 +0100 skrev Marton Balint: Only index tables repeating previous index tables should use the same InstaceUID. Use the index start position when generating the InstanceUID to fix this. Signed-off-by: Marton Balint ---  liba

Re: [FFmpeg-devel] [PATCH] avfilter/vf_subtitles: pass storage size to libass

2022-03-14 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of Oneric > Sent: Monday, March 14, 2022 8:50 PM > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH] avfilter/vf_subtitles: pass storage > size to libass > > On Mon, Mar 14, 2022 at 19:35:36 +, S

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mxfenc: allow more bits for variable part in uuid generation

2022-03-14 Thread Marton Balint
On Mon, 14 Mar 2022, Tomas Härdin wrote: mån 2022-03-14 klockan 19:49 +0100 skrev Marton Balint: Also make sure we do not change the product UID. Signed-off-by: Marton Balint ---  libavformat/mxfenc.c    | 9 +  tests/ref/fate/copy-trac4914    | 2 +-  tests/re

Re: [FFmpeg-devel] [PATCH] avfilter/vf_subtitles: pass storage size to libass

2022-03-14 Thread Oneric
On Mon, Mar 14, 2022 at 19:57:05 +, Soft Works wrote: > > > ass_set_pixel_aspect() is setting settings.par and if I'm not mistaken, > > > an existing par setting leads to the storage size setting to be ignored: > > > > It’s not overridden. Only the explicit PAR is currently preferd over the >

[FFmpeg-devel] [PATCH] fate: add chromashift tests

2022-03-14 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- tests/fate/filter-video.mak | 4 tests/ref/fate/filter-chromashift-smear | 10 ++ tests/ref/fate/filter-chromashift-wrap | 10 ++ 3 files changed, 24 insertions(+) create mode 100644 tests/ref/fate/filter-chromashift-smear creat

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mxfenc: allow more bits for variable part in uuid generation

2022-03-14 Thread Tomas Härdin
mån 2022-03-14 klockan 20:57 +0100 skrev Marton Balint: > > > On Mon, 14 Mar 2022, Tomas Härdin wrote: > > > mån 2022-03-14 klockan 19:49 +0100 skrev Marton Balint: > > > Also make sure we do not change the product UID. > > > > > > Signed-off-by: Marton Balint > > > --- > > >  libavformat/mxfe

Re: [FFmpeg-devel] [PATCH] avfilter/vf_subtitles: pass storage size to libass

2022-03-14 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of Oneric > Sent: Monday, March 14, 2022 9:08 PM > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH] avfilter/vf_subtitles: pass storage > size to libass > > On Mon, Mar 14, 2022 at 19:57:05 +, S

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mxfenc: do not write index tables with the same InstanceUID

2022-03-14 Thread Tomas Härdin
mån 2022-03-14 klockan 20:54 +0100 skrev Marton Balint: > > > On Mon, 14 Mar 2022, Tomas Härdin wrote: > > > mån 2022-03-14 klockan 19:49 +0100 skrev Marton Balint: > > > Only index tables repeating previous index tables should use the > > > same > > > InstaceUID. Use the index start position wh

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mxfenc: do not write index tables with the same InstanceUID

2022-03-14 Thread Marton Balint
On Mon, 14 Mar 2022, Tomas Härdin wrote: mån 2022-03-14 klockan 20:54 +0100 skrev Marton Balint: On Mon, 14 Mar 2022, Tomas Härdin wrote: > mån 2022-03-14 klockan 19:49 +0100 skrev Marton Balint: > > Only index tables repeating previous index tables should use the > > same > > InstaceUID.

[FFmpeg-devel] [PATCH 1/4] avfilter/x86/vf_blend: use unaligned movs for output

2022-03-14 Thread Marton Balint
Fixes crashes with: ffmpeg -f lavfi -i allyuv=d=1 -vf tblend=difference128,pad=5000:ih:1 -f null x Signed-off-by: Marton Balint --- libavfilter/x86/vf_blend.asm | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/libavfilter/x86/vf_blend.asm b/libavfilte

[FFmpeg-devel] [PATCH 2/4] avfilter/vf_zscale: fix number of threads

2022-03-14 Thread Marton Balint
Make sure it is between [1, MAX_THERADS] and also take into account the outlink size in order not to request zero height output from zscale. Signed-off-by: Marton Balint --- libavfilter/vf_zscale.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/vf_zscale.c b/liba

[FFmpeg-devel] [PATCH 3/4] avfilter/vf_zscale: properly check return value of slice threads

2022-03-14 Thread Marton Balint
Signed-off-by: Marton Balint --- libavfilter/vf_zscale.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/libavfilter/vf_zscale.c b/libavfilter/vf_zscale.c index bab87b0c94..ceefc95224 100644 --- a/libavfilter/vf_zscale.c +++ b/libavfilter/vf_zscale.c @@ -117,6 +117

[FFmpeg-devel] [PATCH 4/4] avfilter/vf_zscale: realign output buffer if needed

2022-03-14 Thread Marton Balint
Output buffer alignment might be different to ZIMG_ALIGNMENT or it may not be aligned at all if a downstream filter (e.g. vf_pad) intentionally misaligns it. Or maybe we should unconditionally always allocate output with av_frame_get_buffer() instead of ff_get_video_buffer()? Signed-off-by: Marto

Re: [FFmpeg-devel] [PATCH 2/4] avfilter/vf_zscale: fix number of threads

2022-03-14 Thread Paul B Mahol
lgtm ___ 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 3/4] avfilter/vf_zscale: properly check return value of slice threads

2022-03-14 Thread Paul B Mahol
lgtm ___ 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 4/4] avfilter/vf_zscale: realign output buffer if needed

2022-03-14 Thread Paul B Mahol
lgtm ___ 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".

[FFmpeg-devel] [PATCH] fate: add epx tests

2022-03-14 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- tests/fate/filter-video.mak | 6 ++ tests/ref/fate/filter-ep2x | 7 +++ tests/ref/fate/filter-ep3x | 7 +++ 3 files changed, 20 insertions(+) create mode 100644 tests/ref/fate/filter-ep2x create mode 100644 tests/ref/fate/filter-ep3x diff --git a/t

Re: [FFmpeg-devel] [PATCH] lavc/aarch64: add some neon pix_abs functions

2022-03-14 Thread Martin Storsjö
On Mon, 7 Mar 2022, Swinney, Jonathan wrote: - ff_pix_abs16_neon - ff_pix_abs16_xy2_neon In direct micro benchmarks of these ff functions verses their C implementations, these functions performed as follows on AWS Graviton 2: ff_pix_abs16_neon: c: benchmark ran 10 iterations in 0.955383 s

Re: [FFmpeg-devel] [PATCH] lavc/aarch64: add some neon pix_abs functions

2022-03-14 Thread Martin Storsjö
On Mon, 7 Mar 2022, Pop, Sebastian wrote: Here are a few suggestions: +add d18, d17, d18 // add to the end result register [...] +mov w0, v18.S[0]// copy result to general purpose register I think you can use 32-bit register s18 instead

Re: [FFmpeg-devel] [PATCH] aarch64: Only emit the PAC/BTI note section when targeting ELF

2022-03-14 Thread Martin Storsjö
On Wed, 9 Mar 2022, Martin Storsjö wrote: This avoids build errors if such features are enabled while targeting another binary format. (Using such features on other platforms might require some other form of signaling/setup though, but the ELF specific .note section isn't applicable at least.)

Re: [FFmpeg-devel] [PATCH v4 1/4] lavc/vaapi_encode_h265: Add GPB frame support for hevc_vaapi

2022-03-14 Thread Mark Thompson
On 14/03/2022 02:15, Xiang, Haihao wrote: On Sun, 2022-03-13 at 20:45 +, Mark Thompson wrote: On 11/03/2022 09:00, Fei Wang wrote: From: Linjie Fu Use GPB frames to replace regular P/B frames if backend driver does not support it. - GPB: Generalized P and B picture. Regular P/B fra

Re: [FFmpeg-devel] [PATCH v4 1/4] lavc/vaapi_encode_h265: Add GPB frame support for hevc_vaapi

2022-03-14 Thread Mark Thompson
On 14/03/2022 11:07, Wang, Fei W wrote: -Original Message- From: ffmpeg-devel On Behalf Of Xiang, Haihao Sent: Monday, March 14, 2022 10:15 AM To: ffmpeg-devel@ffmpeg.org Subject: Re: [FFmpeg-devel] [PATCH v4 1/4] lavc/vaapi_encode_h265: Add GPB frame support for hevc_vaapi On Sun, 2022

Re: [FFmpeg-devel] [PATCH v3] avcodec/pngenc: support writing iCCP chunks

2022-03-14 Thread Andreas Rheinhardt
Niklas Haas: > From: Niklas Haas > > encode_zbuf is mostly a mirror image of decode_zbuf. Other than that, > the code is pretty straightforward. Special care needs to be taken to > avoid writing more than 79 characters of the profile description (the > maximum supported). > > To write the (dynam