Re: [FFmpeg-devel] [PATCH 1/3] configure: add cuda-sdk for things requiring full CUDA sdk

>>+cuda_sdk >> libnpp IMO, Libnpp is part of the CUDA SDK, and we can remove all the libnpp related changes and just replace it by cuda-sdk in the configure/make files. Thanks, Yogender --- This email

Re: [FFmpeg-devel] [PATCH] lavc/vaapi_encode_h264: Enable MB rate control

On 2017/5/11 19:39, Mark Thompson wrote: > On 11/05/17 01:29, Jun Zhao wrote: >> Enable the MB level rate control, verified in i965 driver master branch with >> Skylake. > > I think it makes sense to expose this, but can you explain a bit more what > this actually does? All I can see in the

Re: [FFmpeg-devel] [PATCH] avcodec/avcodec: Limit the number of side data elements per packet

On Thu, 11 May 2017 23:27:37 +0200 Michael Niedermayer wrote: > On Thu, May 11, 2017 at 06:54:16PM +0200, wm4 wrote: > > On Thu, 11 May 2017 13:01:36 +0200 > > Michael Niedermayer wrote: > > > > > Fixes:

[FFmpeg-devel] [PATCH] avcodec/aacsbr_template: Do not change bs_num_env before its checked

Fixes: 1489/clusterfuzz-testcase-minimized-5075102901207040 Fixes: out of array access Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/aacsbr_template.c | 28

[FFmpeg-devel] Why do we use `strip -wN` instead of the more general `strip -x`?

I want to discuss why we use this and argue that we should be using `strip -x` all the time anyway. The man page for strip says that -x removes all non-global symbols. -wN is a combination of -w for wildcard matching and -N to remove a given symbol. -wN gets ..@* as an argument. Together they

Re: [FFmpeg-devel] [PATCH 1/3] configure: add cuda-sdk for things requiring full CUDA sdk

On Thu, 11 May 2017 22:59:19 +0200 Timo Rothenpieler wrote: > --- > configure | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/configure b/configure > index c347336426..ef2134828b 100755 > --- a/configure > +++ b/configure > @@ -297,6 +297,7

Re: [FFmpeg-devel] [PATCH 2/3] build: add support for building .cu files via nvcc

On Thu, 11 May 2017 22:59:20 +0200 Timo Rothenpieler wrote: > Original work by Yogender Gupta > --- > .gitignore | 2 ++ > Makefile | 2 ++ > compat/cuda/ptx2c.sh | 35 +++ > configure

[FFmpeg-devel] [PATCH] avcodec/dcaenc: Do not abort process in case of bitrate deficit

Hi. This patch removes assert which can cause abort process in case of limited bitrate: Old behaviour - abort if at least one subband has 1bit quantizer and consumed_bits still greater than frame_bits size. It was a bit strange - we still could reduce bits consumption by reducing SNR for other

Re: [FFmpeg-devel] [PATCH] avcodec/avcodec: Limit the number of side data elements per packet

On Thu, May 11, 2017 at 06:54:16PM +0200, wm4 wrote: > On Thu, 11 May 2017 13:01:36 +0200 > Michael Niedermayer wrote: > > > Fixes: 1293/clusterfuzz-testcase-minimized-6054752074858496 > > > > Found-by: continuous fuzzing process > >

[FFmpeg-devel] [PATCH 3/3] avfilter/scale_cuda: add CUDA scale filter

From: Yogender Gupta --- libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/vf_scale_cuda.c | 555 +++ libavfilter/vf_scale_cuda.cu | 212 + 4 files changed, 769 insertions(+)

[FFmpeg-devel] [PATCH 1/3] configure: add cuda-sdk for things requiring full CUDA sdk

--- configure | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/configure b/configure index c347336426..ef2134828b 100755 --- a/configure +++ b/configure @@ -297,6 +297,7 @@ External library support: The following libraries provide various hardware acceleration

[FFmpeg-devel] [PATCH 2/3] build: add support for building .cu files via nvcc

Original work by Yogender Gupta --- .gitignore | 2 ++ Makefile | 2 ++ compat/cuda/ptx2c.sh | 35 +++ configure| 24 +++- ffbuild/common.mak | 15 --- 5 files changed, 74

Re: [FFmpeg-devel] libavcodec/exr : cosmetics : rename variable

2017-05-05 22:32 GMT+02:00 Martin Vignali : > 2017-03-25 13:08 GMT+01:00 Martin Vignali : > >> >> Hello, >> >> In attach a patch who rename tile variable to better follow ffmpeg coding >> style >> > > Ping for apply > ping for apply

Re: [FFmpeg-devel] libavcodec/exr : simplify reorder_pixels

2017-05-05 22:28 GMT+02:00 Martin Vignali : > Hello, > > in attach a patch who simplify the reorder_pixels func > > reorder_pixels is call by rle_uncompress and zip_uncompress > with size == uncompress_size > > uncompress_size is a multiple of 2 (because exr store data

Re: [FFmpeg-devel] [PATCH] This fixes ISO date formatissue when manifest created by this muxer is not playable in most players. This ensures compatibility with dash standard. Tested on many players (d

Hello, Does anybody know what happened with this patch? -- Původní e-mail -- Od: MFojtak Komu: Aaron Levinson , FFmpeg development discussions and patches Datum: 2. 5. 2017 8:06:29 Předmět: Re: [FFmpeg-devel]

[FFmpeg-devel] configure - "Check 'sdl2-config' first, then 'sdl2.pc'."

See https://trac.ffmpeg.org/ticket/6377. From 3f9c065f96815ffefaa39cda7bacd8132bfdf66c Mon Sep 17 00:00:00 2001 From: Reino Wijnsma Date: Wed, 10 May 2017 00:50:30 +0200 Subject: [PATCH] Check 'sdl2-config' first, then 'sdl2.pc'. --- configure | 8 1 file changed, 4

Re: [FFmpeg-devel] [PATCH] CUDA Scale Filter

Am 10.05.2017 um 19:26 schrieb Yogender Gupta: Please find attached a patch for CUDA Scale Filter. I have modified the makefiles as suggested during the previous reviews to have all PTX generated during compile time. Common.mak has changes for PTX generation Ptx2c.sh has been added in ffbuild

Re: [FFmpeg-devel] [PATCH 2/4] h264dec: be more explicit in handling container cropping

On 5/11/2017 9:56 AM, Michael Niedermayer wrote: > On Wed, May 10, 2017 at 11:06:52PM -0300, James Almer wrote: >> On 5/10/2017 9:55 PM, Michael Niedermayer wrote: >>> On Wed, May 10, 2017 at 10:41:30AM -0300, James Almer wrote: On 5/9/2017 11:56 PM, Michael Niedermayer wrote: > On Mon,

Re: [FFmpeg-devel] [PATCH] avcodec/avcodec: Limit the number of side data elements per packet

On Thu, 11 May 2017 13:01:36 +0200 Michael Niedermayer wrote: > Fixes: 1293/clusterfuzz-testcase-minimized-6054752074858496 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg > Signed-off-by: Michael Niedermayer

Re: [FFmpeg-devel] FFMpeg GPL license Violations by Android apps

2017-05-11 15:55 GMT+02:00 riya khanna : > We are a group of PhD students at Georgia Tech carrying out a research > project that explores free software violations by mobile apps. We found > that over 100 Android apps use GPL version of FFMpeg. > > We can share a list of

Re: [FFmpeg-devel] configure: Prioritize 'sdl2-config'-check

On 11-5-2017 13:56, Stephen Hutchinson wrote: > On 5/11/2017 7:43 AM, Reino Wijnsma wrote: >> https://trac.ffmpeg.org/ticket/6377#comment:2: >>> Inadequate for building FFPlay, yes. But perhaps not for other >>> software. >>> Are there any downsides to my proposition? >> I could

Re: [FFmpeg-devel] FFMpeg GPL license Violations by Android apps

Hi, On Thu, May 11, 2017 at 10:24 AM, Ronald S. Bultje wrote: > Hi, > > On Thu, May 11, 2017 at 9:55 AM, riya khanna > wrote: > >> Hi, >> >> We are a group of PhD students at Georgia Tech carrying out a research >> project that explores free

Re: [FFmpeg-devel] FFMpeg GPL license Violations by Android apps

Hi, On Thu, May 11, 2017 at 9:55 AM, riya khanna wrote: > Hi, > > We are a group of PhD students at Georgia Tech carrying out a research > project that explores free software violations by mobile apps. We found > that over 100 Android apps use GPL version of FFMpeg. >

[FFmpeg-devel] FFMpeg GPL license Violations by Android apps

Hi, We are a group of PhD students at Georgia Tech carrying out a research project that explores free software violations by mobile apps. We found that over 100 Android apps use GPL version of FFMpeg. We can share a list of all the apps if you are interested in details. Please let us know.

Re: [FFmpeg-devel] ffmpeg using MediaCodec, avcodec_open2 says error code:-1 (0xffffffff) text: “Operation not permitted”

Thanks a lot for the note, I'm going to move the question to the correct forum. Apologizes -- Miguel del Amor Herrera Engineering Manager [1477648663875_image001.png] T: +34 868076105/868074570 Traperia 19 3º D Murcia www.displaynote.com

Re: [FFmpeg-devel] [PATCH 2/4] h264dec: be more explicit in handling container cropping

On Wed, May 10, 2017 at 11:06:52PM -0300, James Almer wrote: > On 5/10/2017 9:55 PM, Michael Niedermayer wrote: > > On Wed, May 10, 2017 at 10:41:30AM -0300, James Almer wrote: > >> On 5/9/2017 11:56 PM, Michael Niedermayer wrote: > >>> On Mon, May 08, 2017 at 03:46:23PM -0300, James Almer wrote:

[FFmpeg-devel] [PATCH] lavf/mov: make invalid mdhd time_scale default to 1 instead of erroring out

Some samples have their metadata track time_scale incorrectly set to 0 and the check introduced by a398f054fdb9b0f0b5a91c231fba6ce014143f71 prevents playback of those samples. Setting the time_scale to 1 fixes playback. --- libavformat/mov.c | 4 ++-- 1 file changed, 2 insertions(+), 2

Re: [FFmpeg-devel] [PATCH] lavc/aarch64/simple_idct: separate macro arguments with commas

On Wed, May 10, 2017 at 08:23:02PM +0200, Matthieu Bouron wrote: > On Tue, May 09, 2017 at 11:08:48PM +0200, Matthieu Bouron wrote: > > On Sun, May 7, 2017 at 11:05 AM, Matthieu Bouron > > wrote: > > > > > > > > > > > Le 2 mai 2017 12:01 PM, "Benoit Fouet"

Re: [FFmpeg-devel] configure: Prioritize 'sdl2-config'-check

On 5/11/2017 7:43 AM, Reino Wijnsma wrote: https://trac.ffmpeg.org/ticket/6377#comment:2: Inadequate for building FFPlay, yes. But perhaps not for other software. Are there any downsides to my proposition? I could of course contact the sdl2 developers, but when 'sdl2-config' already has the

Re: [FFmpeg-devel] ffmpeg using MediaCodec, avcodec_open2 says error code:-1 (0xffffffff) text: “Operation not permitted”

On Thu, May 11, 2017 at 08:10:37AM +, Miguel del Amor wrote: > > Qt version: 5.7.1 > ffmpeg version: n3.3 > Android version: 5.1.1 ( but I've tried on some different devices ) > > I'm trying to use the Android API MediaCoded that have been supported by > ffmpeg but while I try to open the

Re: [FFmpeg-devel] [PATCH] lavc/vaapi_encode_h264: Enable MB rate control

On 11/05/17 01:29, Jun Zhao wrote: > Enable the MB level rate control, verified in i965 driver master branch with > Skylake. I think it makes sense to expose this, but can you explain a bit more what this actually does? All I can see in the i965 driver is that it allocates an extra buffer

Re: [FFmpeg-devel] configure: Prioritize 'sdl2-config'-check

https://trac.ffmpeg.org/ticket/6377#comment:2: > Inadequate for building FFPlay, yes. But perhaps not for other software. > Are there any downsides to my proposition? I could of course contact the sdl2 developers, but when 'sdl2-config' already has the librarylist setup right to build FFPlay, why

Re: [FFmpeg-devel] [PATCH] libavformat/avio: fix fill_buffer return error eof when offset1 == buffer_size

On Wed, May 10, 2017 at 03:49:34PM +0800, raymond zheng wrote: > ping... > > 2017-05-08 18:57 GMT+08:00 raymondzheng : > > > --- > > libavformat/aviobuf.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) Where is this patch ? i dont see it locally nor on

Re: [FFmpeg-devel] configure: Prioritize 'sdl2-config'-check

On Thu, 11 May 2017 12:53:33 +0200 Reino Wijnsma wrote: > See https://trac.ffmpeg.org/ticket/6377. > By prioritizing the 'sdl2-config'-check the librarylist that's needed to > build FFPlay is setup automatically and one doesn't have to patch > 'sdl2.pc' anymore. Wouldn't it

[FFmpeg-devel] [PATCH] avcodec/avcodec: Limit the number of side data elements per packet

Fixes: 1293/clusterfuzz-testcase-minimized-6054752074858496 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/avcodec.h | 8 libavcodec/avpacket.c | 5 - 2

[FFmpeg-devel] configure: Prioritize 'sdl2-config'-check

See https://trac.ffmpeg.org/ticket/6377. By prioritizing the 'sdl2-config'-check the librarylist that's needed to build FFPlay is setup automatically and one doesn't have to patch 'sdl2.pc' anymore. From 3f9c065f96815ffefaa39cda7bacd8132bfdf66c Mon Sep 17 00:00:00 2001 From: Reino Wijnsma

Re: [FFmpeg-devel] [PATCH] avfilter/af_pan: ignore named channels not in current layout

On 11 May 2017 at 08:49, Nicolas George wrote: > Le duodi 22 floréal, an CCXXV, Marton Balint a écrit : > > You still have to use a different command line for stereo and 7.1, so I > > don't quite see how this simplifies your use case. Even if your use case > > were justified,

Re: [FFmpeg-devel] [PATCH 1/2] cmdutils_opencl: Fix read of uninitialized pointer

On Mon, Apr 17, 2017 at 04:17:17AM +0200, Michael Niedermayer wrote: > Fixes: CID1396856 > > Signed-off-by: Michael Niedermayer > --- > cmdutils_opencl.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) patchset applied [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH] avcodec/aacdec_fixed: Fix various integer overflows

On Sat, May 06, 2017 at 10:09:59PM +0200, Michael Niedermayer wrote: > Fixes: 1377/clusterfuzz-testcase-minimized-5487049807233024 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg > Signed-off-by: Michael Niedermayer

Re: [FFmpeg-devel] [PATCH] CUDA Scale Filter

Am 10.05.2017 um 19:26 schrieb Yogender Gupta: > Please find attached a patch for CUDA Scale Filter. I have modified the > makefiles as suggested during the previous reviews to have all PTX generated > during compile time. > > Common.mak has changes for PTX generation > Ptx2c.sh has been added

Re: [FFmpeg-devel] [PATCH] ffprobe: discard non-selected streams

On Tue, May 09, 2017 at 10:56:32AM +0200, Clément Bœsch wrote: > From: Clément Bœsch > > --- > ffprobe.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/ffprobe.c b/ffprobe.c > index 8129bcedf0..f2a3cc7a73 100644 > --- a/ffprobe.c > +++ b/ffprobe.c > @@ -2886,6

[FFmpeg-devel] ffmpeg using MediaCodec, avcodec_open2 says error code:-1 (0xffffffff) text: “Operation not permitted”

Qt version: 5.7.1 ffmpeg version: n3.3 Android version: 5.1.1 ( but I've tried on some different devices ) I'm trying to use the Android API MediaCoded that have been supported by ffmpeg but while I try to open the codec I got "Operation not permitted" all times, I've tried some changes and

Re: [FFmpeg-devel] [PATCH 1/3] lavu/frame: add av_frame_check_align().

On Tue, May 9, 2017 at 8:19 PM, Nicolas George wrote: > diff --git a/libavutil/frame.c b/libavutil/frame.c > index 24d5d5f184..e8467a1cd6 100644 > --- a/libavutil/frame.c > +++ b/libavutil/frame.c > @@ -781,3 +781,21 @@ const char *av_frame_side_data_name(enum >

Re: [FFmpeg-devel] [PATCH] avfilter/af_pan: ignore named channels not in current layout

Le duodi 22 floréal, an CCXXV, Marton Balint a écrit : > You still have to use a different command line for stereo and 7.1, so I > don't quite see how this simplifies your use case. Even if your use case > were justified, I'd still think the current behaviour should be preserved as > a default. I

Re: [FFmpeg-devel] [PATCH v10] - Added Turing codec interface for ffmpeg

Dear all, First of all thank you very much for all the help you provided with this patch. We are now at the tenth iteration, and the patch greatly improved since the first time we submitted it. In fact most of the lines of code in the current version of the patch were suggested by you or

Re: [FFmpeg-devel] [PATCH] avcodec/xpmdec: Fix multiple pointer/memory issues

On 5/11/17, Michael Niedermayer wrote: > Most of these were found through code review in response to > fixing 1466/clusterfuzz-testcase-minimized-5961584419536896 > There is thus no testcase for most of this. > The initial issue was Found-by: continuous fuzzing process >