Re: [FFmpeg-devel] [PATCH V2] lavc/vaapi_encode_h264: add "coder" option support

2017-08-14 Thread Steven Liu
2017-08-15 11:33 GMT+08:00 Steven Liu : > 2017-08-15 11:23 GMT+08:00 Jun Zhao : >> >> >> On 2017/8/15 10:41, Steven Liu wrote: >>> 2017-08-15 8:48 GMT+08:00 Jun Zhao : ping? >>> No Documentation for the coder options? >> >> Do you means man page or the other docs ? > doc/encoders.texi > This

Re: [FFmpeg-devel] [PATCH V2] lavc/vaapi_encode_h264: add "coder" option support

2017-08-14 Thread Steven Liu
2017-08-15 11:23 GMT+08:00 Jun Zhao : > > > On 2017/8/15 10:41, Steven Liu wrote: >> 2017-08-15 8:48 GMT+08:00 Jun Zhao : >>> ping? >> No Documentation for the coder options? > > Do you means man page or the other docs ? doc/encoders.texi This one. > >>> >>> On 2017/8/9 9:39, Jun Zhao wrote:

Re: [FFmpeg-devel] [PATCH V2] lavc/vaapi_encode_h264: add "coder" option support

2017-08-14 Thread Jun Zhao
On 2017/8/15 10:41, Steven Liu wrote: > 2017-08-15 8:48 GMT+08:00 Jun Zhao : >> ping? > No Documentation for the coder options? Do you means man page or the other docs ? >> >> On 2017/8/9 9:39, Jun Zhao wrote: >>> V2: Follow libx264 "coder" option style, base on Hendrik Leppkes code >>> review

Re: [FFmpeg-devel] [PATCH 6/7] avutil/opencl: convert to stdatomic

2017-08-14 Thread Wei Gao
2017-03-23 7:34 GMT+08:00 James Almer : > Signed-off-by: James Almer > --- > libavutil/opencl.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/libavutil/opencl.c b/libavutil/opencl.c > index af35770e06..9b6c8d10d8 100644 > --- a/libavutil/opencl.c > +++ b/libavutil

Re: [FFmpeg-devel] [PATCH V2] lavc/vaapi_encode_h264: add "coder" option support

2017-08-14 Thread Steven Liu
2017-08-15 8:48 GMT+08:00 Jun Zhao : > ping? No Documentation for the coder options? > > On 2017/8/9 9:39, Jun Zhao wrote: >> V2: Follow libx264 "coder" option style, base on Hendrik Leppkes code review. >> > ___ > ffmpeg-devel mailing list > ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/rangecoder: Do not increase the pointer beyond the buffer

2017-08-14 Thread James Almer
On 8/14/2017 8:07 PM, Michael Niedermayer wrote: > On Sun, Aug 13, 2017 at 07:18:11PM -0300, James Almer wrote: >> On 8/13/2017 7:15 PM, Michael Niedermayer wrote: >>> Fixes: undefined behavior >>> >>> Signed-off-by: Michael Niedermayer >>> --- >>> libavcodec/rangecoder.c | 1 + >>> libavcodec/ra

[FFmpeg-devel] [PATCH 2/2] avcodec/diracdec: Fixes integer overflow

2017-08-14 Thread Michael Niedermayer
Fixes: runtime error: signed integer overflow: 340018243 * 27 cannot be represented in type 'int' Fixes: 2861/clusterfuzz-testcase-minimized-5361070510178304 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --

[FFmpeg-devel] [PATCH 1/2] avcodec/diracdec: Check perspective_exp and zrs_exp.

2017-08-14 Thread Michael Niedermayer
Fixes: undefined shift Fixes: runtime error: shift exponent 264 is too large for 32-bit type 'int' Fixes: 2860/clusterfuzz-testcase-minimized-4672811689836544 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --

Re: [FFmpeg-devel] [PATCH V2] lavc/vaapi_encode_h264: add "coder" option support

2017-08-14 Thread Jun Zhao
ping? On 2017/8/9 9:39, Jun Zhao wrote: > V2: Follow libx264 "coder" option style, base on Hendrik Leppkes code review. > ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/cavsdec: Check I frame mb decode for errors

2017-08-14 Thread Michael Niedermayer
On Sun, Aug 13, 2017 at 08:38:38PM +0200, Michael Niedermayer wrote: > Fixes: timeout > Fixes: 2943/clusterfuzz-testcase-5430257156882432 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Signed-off-by: Michael Niedermayer > --- > libavc

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/rangecoder: Do not increase the pointer beyond the buffer

2017-08-14 Thread Michael Niedermayer
On Sun, Aug 13, 2017 at 07:18:11PM -0300, James Almer wrote: > On 8/13/2017 7:15 PM, Michael Niedermayer wrote: > > Fixes: undefined behavior > > > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/rangecoder.c | 1 + > > libavcodec/rangecoder.h | 8 ++-- > > 2 files changed, 7 ins

Re: [FFmpeg-devel] [PATCH] libavfilter/af_ambisonic.c Added File for Ambisonic Filter

2017-08-14 Thread Sanchit Sinha
On Mon, Mar 20, 2017 at 2:16 PM, Carl Eugen Hoyos wrote: > 2017-03-13 10:15 GMT+01:00 Sanchit Sinha : > > Sir, I cannot find any tabs in the code. > > For the future: > There is a script in tools/patcheck that helps you with some basic > issues. > > Carl Eugen > __

Re: [FFmpeg-devel] [PATCH 1/1] libavdevice/decklink: configurablity to set max queue size

2017-08-14 Thread Marton Balint
On Mon, 14 Aug 2017, Patagar, Ravindra wrote: Configurablity to set max queue size so that worst case latency can be controlled. Signed-off-by: Ravindra Patagar --- libavdevice/decklink_common.h | 2 ++ libavdevice/decklink_common_c.h | 2 ++ libavdevice/decklink_dec.cpp| 7 +-- libav

Re: [FFmpeg-devel] [PATCH] Add tonemap filter

2017-08-14 Thread Vittorio Giovara
>Hi, > >On 8/11/17, Vittorio Giovara wrote: >[...] >> +/* do the tone map */ >> +for (y = 0; y < in->height; y++) >> +for (x = 0; x < in->width; x++) >> +tonemap(s, out, in, desc, x, y, peak); >> + >> +/* copy/generate alpha if needed */ >> +if (desc->flags & AV

Re: [FFmpeg-devel] [PATCH 1/3] Add single precision planar RGB pixel formats

2017-08-14 Thread Paul B Mahol
On 8/11/17, Vittorio Giovara wrote: > Signed-off-by: Vittorio Giovara > --- > libavutil/pixdesc.c | 54 > > libavutil/pixdesc.h | 5 > libavutil/pixfmt.h | 7 ++ > tests/ref/fate/sws-pixdesc-query | 20 ++

Re: [FFmpeg-devel] [PATCH 2/3] zscale: Enable single precision input/ouput filtering

2017-08-14 Thread Paul B Mahol
Hi, On 8/11/17, Vittorio Giovara wrote: > --- > libavfilter/vf_zscale.c | 7 --- > 1 file changed, 4 insertions(+), 3 deletions(-) > LGTM ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] Add tonemap filter

2017-08-14 Thread Paul B Mahol
Hi, On 8/11/17, Vittorio Giovara wrote: [...] > +/* do the tone map */ > +for (y = 0; y < in->height; y++) > +for (x = 0; x < in->width; x++) > +tonemap(s, out, in, desc, x, y, peak); > + > +/* copy/generate alpha if needed */ > +if (desc->flags & AV_PIX_FMT_FL

Re: [FFmpeg-devel] [Patch] scale_npp : Fix pass through mode

2017-08-14 Thread Yogender Gupta
The better simplification may be : ctx->outputs[0]->hw_frames_ctx = s->stages[last_stage > 0] ? s->stages[last_stage].frames_ctx : ctx->inputs[0]->hw_frames_ctx; -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of Timo Rothenpieler Sent: Monday,

Re: [FFmpeg-devel] [Patch] scale_npp : Fix pass through mode

2017-08-14 Thread Timo Rothenpieler
Am 14.08.2017 um 15:42 schrieb Yogender Gupta: >>> Applied with a slightly changed logic to achieve the same with fewer lines. > > I just tested, the modified logic doesn't work as none of the stages are > initialized. > True, I mis-read the array name as being identical. -ctx->outputs[0]-

Re: [FFmpeg-devel] [Patch] scale_npp : Fix pass through mode

2017-08-14 Thread Yogender Gupta
>> Applied with a slightly changed logic to achieve the same with fewer lines. I just tested, the modified logic doesn't work as none of the stages are initialized. Thanks, Yogender -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of Timo Rothenp

Re: [FFmpeg-devel] [PATCH 1/2] libavcodec: Add support for QSV screen capture plugin

2017-08-14 Thread Alexander Bilyak
Hi, Mark, I am not satisfied with this approach neither. But I thought it would be better this way since we have already well-written code for encoder/decoder, surface management, etc. Sharing this functions between lavc and lavd will be a big mess up and produce a lot of avpriv_ functions (I've tr

Re: [FFmpeg-devel] [PATCH 1/2] libavcodec: Add support for QSV screen capture plugin

2017-08-14 Thread Mark Thompson
On 11/08/17 10:10, Alexander Bilyak wrote: > Intel QSV SDK provide screen capture plugin starting from API ver 1.17 > as runtime loadable plugin for QSV decoder. > > * add API version selection while initialization of QSV context > (default is still 1.1 for usual encoding/decoding) > --- > config

Re: [FFmpeg-devel] [Patch] scale_npp : Fix pass through mode

2017-08-14 Thread Timo Rothenpieler
Applied with a slightly changed logic to achieve the same with fewer lines. (Which I forgot to commit before pushing, so there's two patches now) ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/3] Add single precision planar RGB pixel formats

2017-08-14 Thread Carl Eugen Hoyos
2017-08-11 16:11 GMT+02:00 Vittorio Giovara : > [...] Can't you use the functions from FFmpeg's exr implementation to provide basic libswscale support? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/

Re: [FFmpeg-devel] [Patch] scale_npp : Fix pass through mode

2017-08-14 Thread Mark Thompson
On 14/08/17 12:20, Yogender Gupta wrote: > Pass through mode (input resolution = output resolution) mode is broken in > scale_npp. This patch fixes the same. > > Thanks, > Yogender > > From 6ab2dee97b0eef4c06e9ba8a24a9be01a8c3e33d Mon Sep 17 00:00:00 2001 > From: Yogender Gupta > Date: Mon, 14

[FFmpeg-devel] [Patch] scale_npp : Fix pass through mode

2017-08-14 Thread Yogender Gupta
Pass through mode (input resolution = output resolution) mode is broken in scale_npp. This patch fixes the same. Thanks, Yogender --- This email message is for the sole use of the intended recipient(s) and may cont