Re: [FFmpeg-devel] [PATCH 4/4] vaapi_encode_h265: Enable 4:2:2 support

2020-07-23 Thread Linjie Fu
On Mon, Jul 20, 2020 at 10:32 PM Linjie Fu wrote: > > On Fri, May 15, 2020 at 3:21 PM Fu, Linjie wrote: > > > > > From: ffmpeg-devel On Behalf Of > > > Mark Thompson > > > Sent: Sunday, March 8, 2020 00:15 > > > To: ffmpeg-devel@ffmpeg.org > > > Subject: Re: [FFmpeg-devel] [PATCH 4/4] vaapi_enco

Re: [FFmpeg-devel] [PATCH] avfilter: add radial and circular blur video filter

2020-07-23 Thread Paul B Mahol
On 7/23/20, Michael Niedermayer wrote: > On Sun, Jul 19, 2020 at 09:45:44PM +0200, Paul B Mahol wrote: >> Signed-off-by: Paul B Mahol >> --- >> doc/filters.texi | 46 >> libavfilter/Makefile | 2 + >> libavfilter/allfilters.c | 2 + >> libavfilter/vf_rblur.c | 558 +++

[FFmpeg-devel] [PATCH] avformat/smacker: Support seeking to first frame

2020-07-23 Thread Andreas Rheinhardt
From: Timotej Lazar Add .read_seek function to the smacker demuxer for the special case of seeking to ts=0. This is useful because smacker – like bink, with a similar implementation – was mostly used to encode clips in video games, where random seeks are rare but looping media are common. Signed

Re: [FFmpeg-devel] [PATCH v6 0/6] MIPS MSA & MMI Runtime detection support

2020-07-23 Thread Shiyou Yin
>-Original Message- >From: ffmpeg-devel-boun...@ffmpeg.org [mailto:ffmpeg-devel-boun...@ffmpeg.org] >On Behalf Of >Michael Niedermayer >Sent: Thursday, July 23, 2020 11:24 PM >To: FFmpeg development discussions and patches >Subject: Re: [FFmpeg-devel] [PATCH v6 0/6] MIPS MSA & MMI Runtime

[FFmpeg-devel] [PATCH] hwcontext_vaapi: remove duplicate formats from sw_format list

2020-07-23 Thread Haihao Xiang
hwcontext_vaapi maps different VA fourcc to the same pix_fmt for U/V plane swap cases, however duplicate formats are not expected in sw_format list when merging formats. For example: ffmpeg -loglevel debug -init_hw_device vaapi -filter_hw_device vaapi0 \ -f lavfi -i smptebars -vf \ "hwupload=deriv

Re: [FFmpeg-devel] [PATCH 01/11] avformat/webmdashenc: Fix segfault when no filename is given when live

2020-07-23 Thread Andreas Rheinhardt
Andreas Rheinhardt: > by checking a bit earlier. > > Signed-off-by: Andreas Rheinhardt > --- > libavformat/webmdashenc.c | 8 +--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/libavformat/webmdashenc.c b/libavformat/webmdashenc.c > index eb286cab99..3eefd6df8b 100644 >

Re: [FFmpeg-devel] [PATCH 1/2] lavc/aac_ac3_parser: fix the potential overflow

2020-07-23 Thread myp...@gmail.com
On Fri, Jul 24, 2020 at 4:43 AM Alexander Strasser wrote: > > On 2020-07-01 21:05 +0200, Alexander Strasser wrote: > > On 2020-07-01 16:23 +0200, Anton Khirnov wrote: > > > Quoting Jun Zhao (2020-06-29 15:23:10) > > > > From: Jun Zhao > > > > > > > > Fix the potential overflow. > > > > > > > > Su

[FFmpeg-devel] [PATCH v2 2/2] avutil/timecode: cosmetics on av_timecode_get_smpte

2020-07-23 Thread Marton Balint
Signed-off-by: Marton Balint --- libavutil/timecode.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/libavutil/timecode.c b/libavutil/timecode.c index cb916970ef..b528e4a510 100644 --- a/libavutil/timecode.c +++ b/libavutil/timecode.c @@ -71,7 +71,7 @@ uint32_t av_timec

[FFmpeg-devel] [PATCH v2 1/2] avutil/timecode: fix av_timecode_get_smpte_from_framenum with 50/60 fps

2020-07-23 Thread Marton Balint
SMPTE 12M timecode can only count frames up to 39, because the tens-of-frames value is stored in 2 bit. In order to resolve this 50/60 fps SMPTE timecode is using the field bit (which is the same bit as the phase correction bit) to signal the least significant bit of a 50/60 fps timecode. See SMPTE

Re: [FFmpeg-devel] [PATCH v2 2/2] libavcodec/libaomenc.c: Add command-line options for inter-coding tools

2020-07-23 Thread James Zern
On Wed, Jul 22, 2020 at 4:11 PM Wang Cao wrote: > > From: Wang Cao > > Signed-off-by: Wang Cao > --- > doc/encoders.texi | 36 + > libavcodec/libaomenc.c | 60 ++ > libavcodec/version.h | 2 +- > 3 files changed, 97 inserti

[FFmpeg-devel] [PATCH] Support HDR10+ metadata for HEVC.

2020-07-23 Thread Mohammad Izadi
From: Mohammad Izadi HDR10+ is dynamic metadata (A/341 Amendment - SMPTE2094-40) that needs to be decoded from ITU-T T.35 in HEVC bitstream. The HDR10+ is transferred to side data packet to be used or passed through. --- libavcodec/avpacket.c | 1 + libavcodec/decode.c | 1 + libavcodec/

Re: [FFmpeg-devel] [PATCH] Support HDR10+ metadata for HEVC

2020-07-23 Thread Mohammad Izadi
I am not sure if you received the patch in reply to the thread? I used: git send-email 0001-Support-HDR10-metadata-for-HEVC.patch --to= ffmpeg-devel@ffmpeg.org --in-reply-to= 422719c5-f010-6b39-6415-b3bf46dcb...@rothenpieler.org It seems it created a new thread. Did you receive it? Thanks, Moha

[FFmpeg-devel] [PATCH] Support HDR10+ metadata for HEVC.

2020-07-23 Thread Mohammad Izadi
From: Mohammad Izadi HDR10+ is dynamic metadata (A/341 Amendment - SMPTE2094-40) that needs to be decoded from ITU-T T.35 in HEVC bitstream. The HDR10+ is transferred to side data packet to be used or passed through. --- libavcodec/avpacket.c | 1 + libavcodec/decode.c | 1 + libavcodec/

Re: [FFmpeg-devel] [PATCH v2 1/2] libavcodec/libaomenc.c: Add command-line options for tx tools.

2020-07-23 Thread James Zern
On Wed, Jul 22, 2020 at 4:12 PM Wang Cao wrote: > > From: Wang Cao > > Signed-off-by: Wang Cao > --- > doc/encoders.texi | 20 > libavcodec/libaomenc.c | 30 ++ > libavcodec/version.h | 2 +- > 3 files changed, 51 insertions(+), 1 deletio

Re: [FFmpeg-devel] [PATCH] Support HDR10+ metadata for HEVC

2020-07-23 Thread Mohammad Izadi
Thanks, Mohammad On Thu, Jul 23, 2020 at 1:14 AM zhilizhao wrote: > > > > On Jul 17, 2020, at 5:47 AM, Steinar H. Gunderson < > steinar+ffm...@gunderson.no> wrote: > > > > On Thu, Jul 16, 2020 at 06:34:31PM -0300, James Almer wrote: > >>> static AVMutex codec_mutex = AV_MUTEX_INITIALIZER; > >>>

Re: [FFmpeg-devel] [PATCH] Support HDR10+ metadata for HEVC

2020-07-23 Thread Mohammad Izadi
Please see my answers inline: On Thu, Jul 16, 2020 at 2:34 PM James Almer wrote: > On 7/16/2020 4:23 PM, Mohammad Izadi wrote: > > From: Mohammad Izadi > > > > --- > > libavcodec/avpacket.c | 1 + > > libavcodec/decode.c | 1 + > > libavcodec/hevc_sei.c | 40 +++--- > > libavcodec/h

[FFmpeg-devel] [PATCH 2/2] avcodec/alac: Check decorr_shift to avoid invalid shift

2020-07-23 Thread Michael Niedermayer
Fixes: shift exponent 128 is too large for 32-bit type 'int' Fixes: 23860/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ALAC_fuzzer-5751138914402304 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- li

[FFmpeg-devel] [PATCH 1/2] avcodec/alacdsp: fix integer overflow in decorrelate_stereo()

2020-07-23 Thread Michael Niedermayer
Fixes: signed integer overflow: -16777216 * 131 cannot be represented in type 'int' Fixes: 23835/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ALAC_fuzzer-5669943160078336 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Mich

Re: [FFmpeg-devel] [PATCH v2 1/3] libavutil/imgutils: add utility to get plane sizes

2020-07-23 Thread Alexander Strasser
On 2020-07-22 11:56 +0200, Nicolas George wrote: > James Almer (12020-07-20): > > No, i'll push v3 soon if my argumentation below was not enough to > > convince Nicolas or Michael. My intention is to use ints for the new > > function, not to postpone committing it in any form indefinitely. > > Sorr

Re: [FFmpeg-devel] [PATCH 1/2] lavc/aac_ac3_parser: fix the potential overflow

2020-07-23 Thread Alexander Strasser
On 2020-07-01 21:05 +0200, Alexander Strasser wrote: > On 2020-07-01 16:23 +0200, Anton Khirnov wrote: > > Quoting Jun Zhao (2020-06-29 15:23:10) > > > From: Jun Zhao > > > > > > Fix the potential overflow. > > > > > > Suggested-by: Alexander Strasser > > > Signed-off-by: Jun Zhao > > > --- > >

Re: [FFmpeg-devel] [PATCH 8/8] swscale/yuv2rgb: cosmetics

2020-07-23 Thread Michael Niedermayer
On Wed, Jul 22, 2020 at 11:27:39PM +0800, lance.lmw...@gmail.com wrote: > From: Limin Wang > > Signed-off-by: Limin Wang > --- > libswscale/yuv2rgb.c | 7 --- > 1 file changed, 4 insertions(+), 3 deletions(-) ok thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC78

Re: [FFmpeg-devel] [PATCH 14/14] [inline assembly] add mmx clobbers to cavsdsp

2020-07-23 Thread Michael Niedermayer
On Thu, Jul 23, 2020 at 04:00:35PM +0200, FRÉDÉRIC RECOULES wrote: > Hi, > > I just realized that I have been unsubscribed from the mailing list (is there > a inactivity timeout?). there is no inactivity timeout but occasionally something gets misclassified as spam by some mail providers. For e

[FFmpeg-devel] [PATCH v4 2/2] libavcodec/jpeg2000dec: Support for PPM marker

2020-07-23 Thread gautamramk
From: Gautam Ramakrishnan This patch adds support for PPM marker for JPEG2000 decoder. It allows the samples p1_03.j2k and p1_05.j2k to be decoded. --- libavcodec/jpeg2000dec.c | 107 +++ 1 file changed, 97 insertions(+), 10 deletions(-) diff --git a/libavcod

[FFmpeg-devel] [PATCH v4 1/2] libavcodec/jpeg2000dec: Fix codeblock decode check

2020-07-23 Thread gautamramk
From: Gautam Ramakrishnan The codeblock decoder checks whether the mqc decoder has decoded the right number of bytes. However, this check does not account for the fact that the mqc encoder's flush routine adds 2 bytes of data which does not have to be read by the decoder. The check is modified to

Re: [FFmpeg-devel] [PATCH v3 1/3] libavcodec/jpeg2000dec: Fix codeblock decode check

2020-07-23 Thread Gautam Ramakrishnan
On Thu, Jul 23, 2020 at 7:56 PM Michael Niedermayer wrote: > > On Tue, Jul 21, 2020 at 11:37:13PM +0530, gautamr...@gmail.com wrote: > > From: Gautam Ramakrishnan > > > > The codeblock decoder checks whether the mqc decoder > > has decoded the right number of bytes. However, this > > check does n

Re: [FFmpeg-devel] [PATCH v6 0/6] MIPS MSA & MMI Runtime detection support

2020-07-23 Thread Michael Niedermayer
On Thu, Jul 23, 2020 at 04:18:32PM +0200, Michael Niedermayer wrote: > On Tue, Jul 21, 2020 at 03:40:25PM +0800, Shiyou Yin wrote: > > >-Original Message- > > >From: ffmpeg-devel-boun...@ffmpeg.org > > >[mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > > >Jiaxun Yang > > >Sent: Satur

Re: [FFmpeg-devel] [PATCH] avfilter: add radial and circular blur video filter

2020-07-23 Thread Michael Niedermayer
On Sun, Jul 19, 2020 at 09:45:44PM +0200, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > doc/filters.texi | 46 > libavfilter/Makefile | 2 + > libavfilter/allfilters.c | 2 + > libavfilter/vf_rblur.c | 558 +++ > 4 files chan

Re: [FFmpeg-devel] [PATCH v3 2/3] libavcodec/jpeg2000dec: Add check when done with main header markers

2020-07-23 Thread Paul B Mahol
On 7/23/20, Michael Niedermayer wrote: > On Tue, Jul 21, 2020 at 11:37:14PM +0530, gautamr...@gmail.com wrote: >> From: Gautam Ramakrishnan >> >> This patch sets a flag when the processing of the >> main header is complete. >> --- >> libavcodec/jpeg2000dec.c | 4 >> 1 file changed, 4 insert

Re: [FFmpeg-devel] [PATCH v3 2/3] libavcodec/jpeg2000dec: Add check when done with main header markers

2020-07-23 Thread Michael Niedermayer
On Tue, Jul 21, 2020 at 11:37:14PM +0530, gautamr...@gmail.com wrote: > From: Gautam Ramakrishnan > > This patch sets a flag when the processing of the > main header is complete. > --- > libavcodec/jpeg2000dec.c | 4 > 1 file changed, 4 insertions(+) will apply thx [...] -- Michael

Re: [FFmpeg-devel] [PATCH v3 1/3] libavcodec/jpeg2000dec: Fix codeblock decode check

2020-07-23 Thread Michael Niedermayer
On Tue, Jul 21, 2020 at 11:37:13PM +0530, gautamr...@gmail.com wrote: > From: Gautam Ramakrishnan > > The codeblock decoder checks whether the mqc decoder > has decoded the right number of bytes. However, this > check does not account for the fact that the mqc encoder's > flush routine adds 2 byt

Re: [FFmpeg-devel] [PATCH v6 0/6] MIPS MSA & MMI Runtime detection support

2020-07-23 Thread Michael Niedermayer
On Tue, Jul 21, 2020 at 03:40:25PM +0800, Shiyou Yin wrote: > >-Original Message- > >From: ffmpeg-devel-boun...@ffmpeg.org > >[mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > >Jiaxun Yang > >Sent: Saturday, July 18, 2020 11:36 PM > >To: ffmpeg-devel@ffmpeg.org > >Cc: Jiaxun Yang > >

Re: [FFmpeg-devel] [PATCH 7/7] avfilter: add afwtdn filter

2020-07-23 Thread Valentin Schweitzer
On 23/07/2020 14:55, Paul B Mahol wrote: Site have javascript copy to copy coefficients, and yes for almost all wavelets You're right, I missed that I used that site as source. You could link it above the variable declaration, I think this might be useful for anyone looking for a source on

Re: [FFmpeg-devel] [PATCH 14/14] [inline assembly] add mmx clobbers to cavsdsp

2020-07-23 Thread FRÉDÉRIC RECOULES
Hi, I just realized that I have been unsubscribed from the mailing list (is there a inactivity timeout?). Thus, I do not know if there was some news about the review of the submitted patches. Yet, I could well imagine that some conflicts have appeared since the submission in May, so if it is

Re: [FFmpeg-devel] [PATCH 7/7] avfilter: add afwtdn filter

2020-07-23 Thread Paul B Mahol
On 7/23/20, Valentin Schweitzer wrote: > On 12/06/2020 00:58, Nicolas George wrote: >> Paul B Mahol (12020-06-12): >>> FFmpeg is not math encyclopedia. >> You did not type these numbers, you took them from somewhere. Include it >> in the source code, where it belongs. > > I am not familiar with th

Re: [FFmpeg-devel] [PATCH 7/7] avfilter: add afwtdn filter

2020-07-23 Thread Valentin Schweitzer
On 12/06/2020 00:58, Nicolas George wrote: Paul B Mahol (12020-06-12): FFmpeg is not math encyclopedia. You did not type these numbers, you took them from somewhere. Include it in the source code, where it belongs. I am not familiar with the entirety of this commit, but this seems to concern

Re: [FFmpeg-devel] [PATCH] Support HDR10+ metadata for HEVC

2020-07-23 Thread Timo Rothenpieler
On 23/07/2020 10:13, zhilizhao wrote: On Jul 17, 2020, at 5:47 AM, Steinar H. Gunderson wrote: On Thu, Jul 16, 2020 at 06:34:31PM -0300, James Almer wrote: static AVMutex codec_mutex = AV_MUTEX_INITIALIZER; +static const uint8_t usa_country_code = 0xB5; +static const uint16_t smpte_provide

Re: [FFmpeg-devel] [PATCH] Support HDR10+ metadata for HEVC

2020-07-23 Thread zhilizhao
> On Jul 17, 2020, at 5:47 AM, Steinar H. Gunderson > wrote: > > On Thu, Jul 16, 2020 at 06:34:31PM -0300, James Almer wrote: >>> static AVMutex codec_mutex = AV_MUTEX_INITIALIZER; >>> +static const uint8_t usa_country_code = 0xB5; >>> +static const uint16_t smpte_provider_code = 0x003C; >>> +

Re: [FFmpeg-devel] [PATCH] avfilter: add radial and circular blur video filter

2020-07-23 Thread Paul B Mahol
On 7/19/20, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > doc/filters.texi | 46 > libavfilter/Makefile | 2 + > libavfilter/allfilters.c | 2 + > libavfilter/vf_rblur.c | 558 +++ > 4 files changed, 608 insertions(+) > cre