Re: [FFmpeg-devel] [PATCH v2] libavcodec/vp8dec: fix the multi-thread HWAccel decode error

2019-05-27 Thread Xiang, Haihao
On Thu, 2019-03-28 at 13:28 -0400, Shaofei Wang wrote: > Fix the issue: https://github.com/intel/media-driver/issues/317 > > the root cause is update_dimensions will be called multple times > when decoder thread number is not only 1, but update_dimensions > call get_pixel_format in each decode

[FFmpeg-devel] [PATCH v2 2/3] avcodec/nvenc: add master display and light level sei for HDR10

2019-05-27 Thread lance . lmwang
From: Limin Wang The testing command for the HDR10 output with nvenc: $ ./ffmpeg_g -y -i 4K.mp4 -c:v hevc_nvenc -color_primaries bt2020 -colorspace bt2020_ncl -color_trc smpte2084 -sei hdr10 \ -master_display "G(13250,34500)B(7500,3000)R(34000,16000)WP(15635,16450)L(1000,50)"

[FFmpeg-devel] [PATCH v2 3/3] avcodec/nvenc: add more sei support for SDR HLG

2019-05-27 Thread lance . lmwang
From: Limin Wang The testing command for the atc HLG output with nvenc: ./ffmpeg_g -y -i 4K.mp4 -c:v hevc_nvenc -g 7 -color_primaries bt2020 -colorspace bt2020_ncl -color_trc arib-std-b67 -sei hlg test.ts Signed-off-by: Limin Wang --- libavcodec/nvenc.c | 22 ++

[FFmpeg-devel] [PATCH v2 1/3] avcodec/nvenc: add more sei data support

2019-05-27 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- libavcodec/nvenc.c | 36 ++-- 1 file changed, 22 insertions(+), 14 deletions(-) diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c index 3fe64bba8b..75dda6d689 100644 --- a/libavcodec/nvenc.c +++

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/nvenc: add master display and light level sei for HDR10

2019-05-27 Thread James Almer
On 5/27/2019 7:13 PM, Timo Rothenpieler wrote: > On 27.05.2019 23:49, Lance Wang wrote: >> On Saturday, May 25, 2019, James Almer wrote: >> >>> On 5/22/2019 3:59 AM, lance.lmw...@gmail.com wrote: From: Limin Wang The testing command for the HDR10 output with nvenc: $

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/nvenc: add master display and light level sei for HDR10

2019-05-27 Thread Timo Rothenpieler
On 27.05.2019 23:49, Lance Wang wrote: On Saturday, May 25, 2019, James Almer wrote: On 5/22/2019 3:59 AM, lance.lmw...@gmail.com wrote: From: Limin Wang The testing command for the HDR10 output with nvenc: $ ./ffmpeg_g -y -i 4K.mp4 -c:v hevc_nvenc -g 7 -color_primaries bt2020 -colorspace

[FFmpeg-devel] [PATCH] tools: Add fuzzer for demuxers

2019-05-27 Thread Michael Niedermayer
This is based on target_dec_fuzzer Signed-off-by: Michael Niedermayer --- Makefile | 3 + tools/Makefile| 3 + tools/target_dem_fuzzer.c | 163 ++ 3 files changed, 169 insertions(+) create mode 100644

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/nvenc: add master display and light level sei for HDR10

2019-05-27 Thread Lance Wang
On Saturday, May 25, 2019, James Almer wrote: > On 5/22/2019 3:59 AM, lance.lmw...@gmail.com wrote: > > From: Limin Wang > > > > The testing command for the HDR10 output with nvenc: > > $ ./ffmpeg_g -y -i 4K.mp4 -c:v hevc_nvenc -g 7 -color_primaries bt2020 > -colorspace bt2020_ncl -color_trc

Re: [FFmpeg-devel] CPIA

2019-05-27 Thread Stephan Hilb
Michael Niedermayer wrote on 27.05.2019 at 00:22: > ideally some seperation between frames would be kept > for example each frame in its own file might be the easiest > or maybe just add teh codec id to nut and try to store it in that > with stream copy Added the codec id but getting invalid

Re: [FFmpeg-devel] [PATCH v2] avfilter/avf_aphasemeter: Add out-of-phase and mono detection

2019-05-27 Thread Moritz Barsnick
On Mon, May 20, 2019 at 14:40:24 +0200, Romane Lafon wrote: > +float tolerance = 1.0 - s->tolerance; Strictly speaking 1.0f (or 1.f). > +float angle = cos(s->angle/180.0*PI); If you want a float as result, use cosf(s->angle / 180.f * M_PI). (I'm aware PI and M_PI aren't explicitly

Re: [FFmpeg-devel] [PATCH v2] avfilter/avf_aphasemeter: Add out-of-phase and mono detection

2019-05-27 Thread Paul B Mahol
On 5/20/19, Romane Lafon wrote: > I have updated the patch that extends aphasemeter filter. > It allows to get metadata for out-of-phase or mono sequences of stereo > streams. > It displays start, end and duration as for silencedetect filter. > Please use M_PI, and do not invent own constants.

Re: [FFmpeg-devel] [PATCH] avcodec/ffv1dec_template: Optimize common case in run mode

2019-05-27 Thread Michael Niedermayer
On Thu, May 23, 2019 at 10:36:10PM +0200, Michael Niedermayer wrote: > On Thu, May 23, 2019 at 10:57:14AM +0200, Carl Eugen Hoyos wrote: > > Am Do., 23. Mai 2019 um 01:47 Uhr schrieb Michael Niedermayer > > : > > > > > > Fixes: Timeout (14sec -> 9sec) > > > Fixes: > > >

Re: [FFmpeg-devel] PATCH: doc/swscaler: explain default Lanczos parameter

2019-05-27 Thread Gyan
On 26-05-2019 10:32 PM, Gyan wrote: On 26-05-2019 10:08 PM, Werner Robitza wrote: On Sun, May 26, 2019 at 6:09 PM Gyan wrote: Do you want to update it for all algorithms with missing parameter details? Not at this stage – would be a lot of guesswork since I didn't implement the filters.

Re: [FFmpeg-devel] [PATCH] avcodec/truemotion2: Fix several integer overflows in tm2_update_block()

2019-05-27 Thread Michael Niedermayer
On Sun, May 05, 2019 at 12:57:50AM +0200, Michael Niedermayer wrote: > Fixes: signed integer overflow: -1877966852 + -469491713 cannot be > represented in type 'int' > Fixes: > 14561/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_TRUEMOTION2_fuzzer-5167608359288832 > > Found-by: continuous

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/diracdec: Check for arith decoder errors in dirac_unpack_block_motion_data()

2019-05-27 Thread Michael Niedermayer
On Sun, May 12, 2019 at 11:21:48PM +0200, Michael Niedermayer wrote: > Fixes: Timeout (54sec -> 188ms) > Fixes: > 14585/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_DIRAC_fuzzer-5649933052411904 > > Found-by: continuous fuzzing process >

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mp3enc: Avoid SEEK_END as it is unsupported

2019-05-27 Thread Michael Niedermayer
On Tue, May 14, 2019 at 01:04:52PM +0200, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavformat/mp3enc.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) will apply [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Let us

Re: [FFmpeg-devel] [PATCH v2] avcodec/hevcdec: set the SEI parameters early on the AVCodecContext

2019-05-27 Thread James Almer
On 5/27/2019 11:06 AM, Steve Lhomme wrote: > It's better to do it before the buffers are actually created. At least in VLC > we currently don't support changing some parameters dynamically easily so we > don't use the information if it comes after the buffer are created. > > Co-authored-by: James

Re: [FFmpeg-devel] [PATCH v1 2/2] fftools/ffmpeg: add exif orientation support per frame's metadata

2019-05-27 Thread Nicolas George
Jun Li (12019-05-25): > I believe it has been enabled by default, the 'autorotate' value is true by > default. Yes, but it is currently enabled for constant rotation. Variable rotation is more tricky, so the question of enabling it by default must be asked again. > Do you mean the orientation

Re: [FFmpeg-devel] [PATCH v1 1/2] lavf/vf_transpose: add exif orientation support

2019-05-27 Thread Nicolas George
Jun Li (12019-05-27): > I compared the perf between vflip and this patch before, they take almost > the same time, around 9ms for a frame. > Hflip is a little special, but surely I can update the patch to avoid frame > copy, and it will be as fast as hflip. > > But still this is duplicated

Re: [FFmpeg-devel] [PATCH v1 1/2] lavf/vf_transpose: add exif orientation support

2019-05-27 Thread Paul B Mahol
On 5/27/19, Jun Li wrote: > On Mon, May 27, 2019 at 1:39 AM Paul B Mahol wrote: > >> On 5/27/19, Jun Li wrote: >> > On Mon, May 27, 2019 at 12:32 AM Paul B Mahol wrote: >> > >> >> On 5/27/19, Jun Li wrote: >> >> > On Sun, May 26, 2019 at 2:09 AM Paul B Mahol >> wrote: >> >> > >> >> >> On

[FFmpeg-devel] [PATCH v2] avcodec/hevcdec: set the SEI parameters early on the AVCodecContext

2019-05-27 Thread Steve Lhomme
It's better to do it before the buffers are actually created. At least in VLC we currently don't support changing some parameters dynamically easily so we don't use the information if it comes after the buffer are created. Co-authored-by: James Almer --- The same problem may exist with H264

Re: [FFmpeg-devel] [PATCH v1 1/2] lavf/vf_transpose: add exif orientation support

2019-05-27 Thread Jun Li
On Mon, May 27, 2019 at 1:39 AM Paul B Mahol wrote: > On 5/27/19, Jun Li wrote: > > On Mon, May 27, 2019 at 12:32 AM Paul B Mahol wrote: > > > >> On 5/27/19, Jun Li wrote: > >> > On Sun, May 26, 2019 at 2:09 AM Paul B Mahol > wrote: > >> > > >> >> On 5/26/19, Jun Li wrote: > >> >> > On Sun,

Re: [FFmpeg-devel] [PATCH] avcodec/v210dec: Fix alignment check for AVX2

2019-05-27 Thread Michael Niedermayer
On Sat, May 18, 2019 at 10:32:42PM +0200, Michael Niedermayer wrote: > On Sat, May 18, 2019 at 01:05:01PM +0200, Reimar Döffinger wrote: > > On 18.05.2019, at 12:15, Michael Niedermayer wrote: > > > > > On Sat, May 18, 2019 at 12:02:55PM +0200, James Darnley wrote: > > >> On 2019-05-18 09:39,

Re: [FFmpeg-devel] [PATCH v1 1/2] lavf/vf_transpose: add exif orientation support

2019-05-27 Thread Paul B Mahol
On 5/27/19, Jun Li wrote: > On Mon, May 27, 2019 at 12:32 AM Paul B Mahol wrote: > >> On 5/27/19, Jun Li wrote: >> > On Sun, May 26, 2019 at 2:09 AM Paul B Mahol wrote: >> > >> >> On 5/26/19, Jun Li wrote: >> >> > On Sun, May 26, 2019 at 1:16 AM Paul B Mahol >> wrote: >> >> > >> >> >> On

Re: [FFmpeg-devel] [PATCH v1 1/2] lavf/vf_transpose: add exif orientation support

2019-05-27 Thread Jun Li
On Mon, May 27, 2019 at 12:32 AM Paul B Mahol wrote: > On 5/27/19, Jun Li wrote: > > On Sun, May 26, 2019 at 2:09 AM Paul B Mahol wrote: > > > >> On 5/26/19, Jun Li wrote: > >> > On Sun, May 26, 2019 at 1:16 AM Paul B Mahol > wrote: > >> > > >> >> On 5/25/19, Jun Li wrote: > >> >> > Add

Re: [FFmpeg-devel] [PATCH v1 1/2] lavf/vf_transpose: add exif orientation support

2019-05-27 Thread Paul B Mahol
On 5/27/19, Jun Li wrote: > On Sun, May 26, 2019 at 2:09 AM Paul B Mahol wrote: > >> On 5/26/19, Jun Li wrote: >> > On Sun, May 26, 2019 at 1:16 AM Paul B Mahol wrote: >> > >> >> On 5/25/19, Jun Li wrote: >> >> > Add exif orientation support and expose an option. >> >> > --- >> >> >

Re: [FFmpeg-devel] [PATCH] avcodec/vp3data: combine eob_run_base and eob_run_get_bits tables

2019-05-27 Thread Peter Ross
On Sun, May 26, 2019 at 09:27:55PM +0200, Reimar Döffinger wrote: > On Sat, May 25, 2019 at 12:04:49PM +1000, Peter Ross wrote: > > --- > > This provides a small readability improvement. > > I observe no performance change on x86_64 or arm6. > > Looks good to me, but probably the maintainer

[FFmpeg-devel] [PATCH 2/2] libavfilter/vf_blend.c: remove duplicate code with macro-defined function

2019-05-27 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- libavfilter/vf_blend.c | 188 +++-- 1 file changed, 48 insertions(+), 140 deletions(-) diff --git a/libavfilter/vf_blend.c b/libavfilter/vf_blend.c index 46e0518d26..cf71200773 100644 ---

[FFmpeg-devel] [PATCH 1/2] libavfilter/vf_blend.c: remove duplicate code with same funcion

2019-05-27 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- libavfilter/vf_blend.c | 21 + 1 file changed, 9 insertions(+), 12 deletions(-) diff --git a/libavfilter/vf_blend.c b/libavfilter/vf_blend.c index 83f3d591a8..46e0518d26 100644 --- a/libavfilter/vf_blend.c +++