Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-07 Thread Timo Rothenpieler
Am 07.09.2016 um 13:27 schrieb Carl Eugen Hoyos: > 2016-09-07 12:50 GMT+02:00 Sven C. Dack <sven.c.d...@sky.com>: >> On 07/09/16 11:25, Carl Eugen Hoyos wrote: >>> >>>> Am 07.09.2016 um 11:40 schrieb "Sven C. Dack" <sven.c.d...@sky.com>:

Re: [FFmpeg-devel] Possible incomplete commit "avcodec/nvenc: support RGB input"

2016-09-08 Thread Timo Rothenpieler
>> for fmt in yuv420p nv12 bgr0 rgb0; do >> ./ffmpeg -f lavfi -i "testsrc=size=1920x1080:duration=10:rate=30" >> -c:v h264_nvenc -global_quality 20 -pix_fmt "$fmt" -y out_"${fmt}".mkv >> done > > You feed to nvenc only rgb? what testsrc only supports. Use testsrc2. pix_fmt should make sure

Re: [FFmpeg-devel] Possible incomplete commit "avcodec/nvenc: support RGB input"

2016-09-08 Thread Timo Rothenpieler
Am 08.09.2016 um 02:29 schrieb Sven C. Dack: > On 08/09/16 00:57, Hendrik Leppkes wrote: >> The image copying code was refactored in an earlier patch to be >> generic and not rely on hard-coding format info, hence the second part >> is not needed anymore. >> > > This is not quite accurate. It

Re: [FFmpeg-devel] Possible incomplete commit "avcodec/nvenc: support RGB input"

2016-09-08 Thread Timo Rothenpieler
Am 08.09.2016 um 02:29 schrieb Sven C. Dack: > On 08/09/16 00:57, Hendrik Leppkes wrote: >> The image copying code was refactored in an earlier patch to be >> generic and not rely on hard-coding format info, hence the second part >> is not needed anymore. >> > > This is not quite accurate. It

Re: [FFmpeg-devel] [PATCH v2] swscale: add unscaled conversion from yuv420p to p010

2016-09-04 Thread Timo Rothenpieler
> Finally, with the change, the function can also be used > for P016, note that I tried to object to P010: It does not > serve any real purpose, if I remember correctly, the > explanation for the commit was that there is a bug in > FFmpeg's pix_fmt decision routine that needed to > be

Re: [FFmpeg-devel] [PATCH v2] swscale: add unscaled conversion from yuv420p to p010

2016-09-04 Thread Timo Rothenpieler
On 9/4/2016 4:06 PM, Carl Eugen Hoyos wrote: > 2016-09-04 16:02 GMT+02:00 Timo Rothenpieler <t...@rothenpieler.org>: >> The purpose of this patch is to make conversion from >> yuv420p (8 bit) to p010 (10 bit) fast. > > Do I understand you correctly that your patch is &g

Re: [FFmpeg-devel] high bitdepth support and the location of the zero padding

2016-09-04 Thread Timo Rothenpieler
On 9/4/2016 3:01 PM, Carl Eugen Hoyos wrote: > Hi! > > 2016-09-04 14:55 GMT+02:00 Wilbert Dijkhof > : >> I hope this is the right place for this question. If not i hope you >> can point me to a place where they can help us with further. > > No, libav-user

Re: [FFmpeg-devel] [PATCH 2/2] swscale: add unscaled conversion from yuv420p to p010

2016-09-03 Thread Timo Rothenpieler
On 9/3/2016 1:47 PM, Carl Eugen Hoyos wrote: > 2016-09-03 0:06 GMT+02:00 Timo Rothenpieler <t...@rothenpieler.org>: >> On 9/2/2016 7:16 PM, Carl Eugen Hoyos wrote: >>> 2016-09-02 16:36 GMT+02:00 Timo Rothenpieler <t...@rothenpieler.org>: >>> >&

Re: [FFmpeg-devel] [PATCH] avcodec/nvenc: Include nvEncodeAPI v7 SDK header

2016-08-29 Thread Timo Rothenpieler
On 8/29/2016 8:43 PM, James Almer wrote: > On 8/27/2016 9:58 AM, Timo Rothenpieler wrote: >> @@ -5996,6 +5992,22 @@ enabled vdpau && enabled xlib && >> check_lib2 "vdpau/vdpau.h vdpau/vdpau_x11.h" vdp_device_create_x11 >> -lvdpau &

[FFmpeg-devel] [PATCH] swscale: add support for P010LE/BE output

2016-08-29 Thread Timo Rothenpieler
--- libswscale/output.c | 98 +++- libswscale/utils.c | 4 +- libswscale/x86/swscale.c | 4 +- tests/ref/fate/filter-pixdesc-p010be | 1 + tests/ref/fate/filter-pixdesc-p010le | 1 +

Re: [FFmpeg-devel] [PATCH 1/2] configure: fix nvenc detection logic

2016-08-31 Thread Timo Rothenpieler
On 8/31/2016 5:42 PM, Carl Eugen Hoyos wrote: > 2016-08-31 17:32 GMT+02:00 James Almer <jamr...@gmail.com>: >> On 8/31/2016 11:58 AM, Carl Eugen Hoyos wrote: >>> 2016-08-31 16:42 GMT+02:00 Timo Rothenpieler <t...@rothenpieler.org>: >>> >>>&

Re: [FFmpeg-devel] [PATCH] swscale: add unscaled copy from yuv420p10 to p010

2016-09-01 Thread Timo Rothenpieler
On 9/1/2016 6:20 PM, Michael Niedermayer wrote: > On Thu, Sep 01, 2016 at 05:23:04PM +0200, Timo Rothenpieler wrote: >> --- >> libswscale/swscale_unscaled.c | 39 +++ >> 1 file changed, 39 insertions(+) >> >> diff --git

Re: [FFmpeg-devel] Performance of P010LE/BE pixel convertion

2016-09-01 Thread Timo Rothenpieler
Can you test again with this patch applied: https://github.com/BtbN/FFmpeg/commit/54cf5500720c9b701d4fe16c2c6ff2e3cc1508d7.patch ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] swscale: add unscaled copy from yuv420p10 to p010

2016-09-01 Thread Timo Rothenpieler
--- libswscale/swscale_unscaled.c | 39 +++ 1 file changed, 39 insertions(+) diff --git a/libswscale/swscale_unscaled.c b/libswscale/swscale_unscaled.c index b231abe..51768fa 100644 --- a/libswscale/swscale_unscaled.c +++ b/libswscale/swscale_unscaled.c @@

Re: [FFmpeg-devel] [PATCH] configure: improve logic and checks for nvenc

2016-08-31 Thread Timo Rothenpieler
>> +echo 'NV_ENCODE_API_FUNCTION_LIST flist;' >> +echo 'void f(void) { struct { const GUID guid; } s[] = { { >> NV_ENC_PRESET_HQ_GUID } }; }' > > This will most likely prevent nvenc from being enabled for msvc 2012, but not > old > mingw32, which is failing with the error: > >

[FFmpeg-devel] [PATCH] configure: improve logic and checks for nvenc

2016-08-30 Thread Timo Rothenpieler
--- configure | 37 + 1 file changed, 25 insertions(+), 12 deletions(-) diff --git a/configure b/configure index 52931c3..bcfc9a8 100755 --- a/configure +++ b/configure @@ -5992,20 +5992,33 @@ enabled vdpau && enabled xlib && check_lib2 "vdpau/vdpau.h

[FFmpeg-devel] [PATCH 3/3] swscale: add support for P010LE/BE output

2016-08-31 Thread Timo Rothenpieler
--- libswscale/output.c | 98 +++- libswscale/utils.c | 4 +- libswscale/x86/swscale.c | 4 +- tests/ref/fate/filter-pixdesc-p010be | 1 + tests/ref/fate/filter-pixdesc-p010le | 1 +

[FFmpeg-devel] [PATCH 1/3] avfilter/drawutils: P010 is not supported

2016-08-31 Thread Timo Rothenpieler
--- libavfilter/drawutils.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavfilter/drawutils.c b/libavfilter/drawutils.c index 8153fde..a3710db 100644 --- a/libavfilter/drawutils.c +++ b/libavfilter/drawutils.c @@ -184,6 +184,8 @@ int ff_draw_init(FFDrawContext *draw, enum

[FFmpeg-devel] [PATCH 2/3] avfilter/drawutils: honor shift for color component description

2016-08-31 Thread Timo Rothenpieler
--- libavfilter/drawutils.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavfilter/drawutils.c b/libavfilter/drawutils.c index a3710db..f6760be 100644 --- a/libavfilter/drawutils.c +++ b/libavfilter/drawutils.c @@ -253,7 +253,8 @@ void ff_draw_color(FFDrawContext *draw,

Re: [FFmpeg-devel] [PATCH] configure: improve logic and checks for nvenc

2016-08-31 Thread Timo Rothenpieler
> 2016-08-31 15:03 GMT+02:00 Timo Rothenpieler <t...@rothenpieler.org>: >> Forgot this, the idea with my approach is to handle the case where >> --enable-nvenc is requested, but the compile-check fails. > >> Just silently disabling it then seems wrong. > >

Re: [FFmpeg-devel] [PATCH] configure: improve logic and checks for nvenc

2016-08-31 Thread Timo Rothenpieler
Forgot this, the idea with my approach is to handle the case where --enable-nvenc is requested, but the compile-check fails. Just silently disabling it then seems wrong. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] configure: improve logic and checks for nvenc

2016-08-31 Thread Timo Rothenpieler
> 2016-08-31 15:26 GMT+02:00 Timo Rothenpieler <t...@rothenpieler.org>: >>> 2016-08-31 15:03 GMT+02:00 Timo Rothenpieler <t...@rothenpieler.org>: >>>> Forgot this, the idea with my approach is to handle the case where >>>> --enable-nvenc is reque

[FFmpeg-devel] [PATCH 2/2] configure: fix ldl dependency for new nvenc encoder names

2016-08-31 Thread Timo Rothenpieler
--- configure | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/configure b/configure index e30ddd2..f2c8492 100755 --- a/configure +++ b/configure @@ -5388,7 +5388,8 @@ decklink_indev_extralibs="$decklink_indev_extralibs $ldl" frei0r_filter_extralibs='$ldl'

[FFmpeg-devel] [PATCH 1/2] configure: fix nvenc detection logic

2016-08-31 Thread Timo Rothenpieler
--- configure | 34 +++--- 1 file changed, 19 insertions(+), 15 deletions(-) diff --git a/configure b/configure index 52931c3..e30ddd2 100755 --- a/configure +++ b/configure @@ -3205,7 +3205,7 @@ enable audiotoolbox enable d3d11va dxva2 vaapi vda vdpau

Re: [FFmpeg-devel] Patch for fixing of nvenc.c compilation using msvc tools

2016-09-14 Thread Timo Rothenpieler
On 9/14/2016 3:43 PM, Yogender Kumar Gupta wrote: > Attached is a patch for SDK 7_0 for NVENC. This adds other features > available in SDK 7_0 as well as fixes an issue with HEVC profile > I'd very much dislike applying this change. It makes the list very hard to read. While it could be

Re: [FFmpeg-devel] [PATCH] Patch for SDK 7.0 for NVENC

2016-09-14 Thread Timo Rothenpieler
On 9/14/2016 6:30 PM, Carl Eugen Hoyos wrote: > 2016-09-14 18:26 GMT+02:00 Timo Rothenpieler <t...@rothenpieler.org>: >> On 9/14/2016 3:43 PM, Yogender Kumar Gupta wrote: >>> Attached is a patch for SDK 7_0 for NVENC. This adds other features >>> available in

Re: [FFmpeg-devel] [PATCH] Patch for SDK 7.0 for NVENC

2016-09-14 Thread Timo Rothenpieler
On 9/14/2016 3:43 PM, Yogender Kumar Gupta wrote: > Attached is a patch for SDK 7_0 for NVENC. This adds other features > available in SDK 7_0 as well as fixes an issue with HEVC profile > What carl said. Also, a some of the added options are not used anywhere: zeroReorderDelay, enableNonRefP

Re: [FFmpeg-devel] [PATCH] Patch for SDK 7.0 for NVENC

2016-09-14 Thread Timo Rothenpieler
On 9/14/2016 3:43 PM, Yogender Kumar Gupta wrote: > Attached is a patch for SDK 7_0 for NVENC. This adds other features > available in SDK 7_0 as well as fixes an issue with HEVC profile > I'd very much dislike applying this change. It makes the list very hard to read. While it could be

Re: [FFmpeg-devel] [PATCH] configure: don't build ffserver unless explicitly enabled

2016-09-10 Thread Timo Rothenpieler
On 9/11/2016 1:22 AM, Carl Eugen Hoyos wrote: > 2016-09-10 23:25 GMT+02:00 Timo Rothenpieler <t...@rothenpieler.org>: > >> - --disable-ffserver disable ffserver build >> + --enable-ffserverenable ffserver build ffserver is unmaintained for a very

Re: [FFmpeg-devel] [PATCH] cuvid: Always check for internal errors during parsing

2016-09-10 Thread Timo Rothenpieler
On 9/10/2016 9:51 PM, Philip Langdale wrote: > The cuvid parser is basically undocumented, and although you'd > think that a failed callback would result in the overall parse > call returning an error, that is not true. > > So, we end up silently trying to keep going as if nothing is wrong, >

Re: [FFmpeg-devel] [PATCH] configure: don't build ffserver unless explicitly enabled

2016-09-10 Thread Timo Rothenpieler
On 9/10/2016 11:40 PM, Josh de Kock wrote: > On 10/09/2016 22:25, Timo Rothenpieler wrote: >> [...] >> +DEPRECATED_PROGRAM_LIST=" >> +ffserver >> +" >> [...] > > I don't really see the point of this, the other programs are unlikely to > be

[FFmpeg-devel] [PATCH] configure: don't build ffserver unless explicitly enabled

2016-09-10 Thread Timo Rothenpieler
--- configure | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/configure b/configure index b11ca7f..d67d8a2 100755 --- a/configure +++ b/configure @@ -116,7 +116,7 @@ Program options: --disable-ffmpeg disable ffmpeg build --disable-ffplay

Re: [FFmpeg-devel] [PATCH] avcodec/mpegvideo_enc: fix memory leak

2016-09-26 Thread Timo Rothenpieler
> Will push later today, unless someone has a better idea how to address > this issue. > Maybe some way to not use av_frame_copy_props in the first place? applied ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] avcodec/nvenc: use AVERROR_BUFFER_TOO_SMALL instead of ENOBUFS

2016-09-24 Thread Timo Rothenpieler
On 9/24/2016 8:31 PM, James Almer wrote: > Should fix compilation with mingw32. > > Signed-off-by: James Almer > --- > libavcodec/nvenc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c > index e3edd74..fc5253a

Re: [FFmpeg-devel] [PATCH 2/3] ffmpeg: use new decode API

2016-09-30 Thread Timo Rothenpieler
> Will push this patch in 24 hours unless other problems are pointed out. +1 for pushing the entire series. Changes like this are almost impossible to do without breaking some corner cases. Delaying it until all of those are fixed won't get the project anywhere. So far I haven't found anything

Re: [FFmpeg-devel] [PATCH] lavc: remove libfaac wrapper

2016-09-30 Thread Timo Rothenpieler
> dont remember if it was specific to libfaac but some issues in the > mov edit list patches about initial padding and trailing padding > where found using libfaac as encoder. > if libfaac support is droped, muxers wont be tested against it anymore > most likely. It could be done with command line

Re: [FFmpeg-devel] [PATCH 2/3] ffmpeg: use new decode API

2016-09-30 Thread Timo Rothenpieler
Am 30.09.2016 um 10:43 schrieb wm4: > On Fri, 30 Sep 2016 10:38:41 +0200 > Timo Rothenpieler <t...@rothenpieler.org> wrote: > >>> Will push this patch in 24 hours unless other problems are pointed out. >> >> +1 for pushing the entire series. >> Cha

[FFmpeg-devel] [PATCH] configure: define _GNU_SOURCE on cygwin

2016-09-30 Thread Timo Rothenpieler
Some function definitions are missing without it, for example sigaction. Fixes ticket #5868 --- configure | 1 + 1 file changed, 1 insertion(+) diff --git a/configure b/configure index 899057d..0e2f459 100755 --- a/configure +++ b/configure @@ -4805,6 +4805,7 @@ case $target_os in

Re: [FFmpeg-devel] Developer needed for applying watermark onto video on server

2016-10-03 Thread Timo Rothenpieler
On 10/3/2016 11:09 PM, chuck wrote: > Hi, I am looking for a developer for hire to help us with an issue > applying a watermark to videos after they are uploaded and converted on > the server. I am the project manager and do not have much technical > information but you will be able to speak with

Re: [FFmpeg-devel] [PATCH 0/3] Include headers for cuvid

2016-09-21 Thread Timo Rothenpieler
Am 21.09.2016 um 11:58 schrieb Hendrik Leppkes: > On Wed, Sep 21, 2016 at 10:23 AM, Timo Rothenpieler > <t...@rothenpieler.org> wrote: >>> Well its just that both cuvid and cuda are both currently flagged as >>> nonfree in FFmpeg which limits there availability. So

Re: [FFmpeg-devel] [PATCH 0/3] Include headers for cuvid

2016-09-21 Thread Timo Rothenpieler
> Well its just that both cuvid and cuda are both currently flagged as > nonfree in FFmpeg which limits there availability. So I was just wondering > what needed to be done to make them gpl compatible as I would like to see > cuvid be more available. GPL conformant CUDA headers. Someone would

Re: [FFmpeg-devel] [PATCH 3/3] cuvid: Use the compat headers for nvcuvid

2016-09-21 Thread Timo Rothenpieler
On 9/21/2016 6:38 AM, Philip Langdale wrote: > Signed-off-by: Philip Langdale > --- > libavcodec/cuvid.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/cuvid.c b/libavcodec/cuvid.c > index f2e92cf..7fd0b0d 100644 > --- a/libavcodec/cuvid.c

[FFmpeg-devel] [PATCH] avformat/utils: force native h264 decoder for probing

2016-09-21 Thread Timo Rothenpieler
--- libavformat/utils.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/libavformat/utils.c b/libavformat/utils.c index a9bd034..4c5340b 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -164,6 +164,13 @@ int ff_copy_whiteblacklists(AVFormatContext *dst, const

[FFmpeg-devel] [PATCH] avformat/utils: only call h264 decoder private function if h264 decoder is in use

2016-09-18 Thread Timo Rothenpieler
Fixes a crash when decoding with for example h264_cuvid, as avpriv_h264_has_num_reorder_frames assumes the AVCodecContext->priv_data to be a H264Context. --- libavformat/utils.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index

Re: [FFmpeg-devel] [PATCH 1/3] avcodec: add new AVOID_PROBING capability

2016-09-23 Thread Timo Rothenpieler
series applied ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] avformat/utils: force native h264 decoder for probing

2016-09-22 Thread Timo Rothenpieler
--- libavformat/utils.c | 18 +++--- 1 file changed, 15 insertions(+), 3 deletions(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index a9bd034..05d2315 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -186,6 +186,18 @@ FF_ENABLE_DEPRECATION_WARNINGS

Re: [FFmpeg-devel] [RFC FIX] Build error with ffmpeg 3.1.3 (and current git) on cygwin

2016-09-22 Thread Timo Rothenpieler
Am 22.09.2016 um 13:37 schrieb Michael Fritscher: > Hi, > > ok, I rephrase it: I have the issue that HAVE_SETDLLDIRECTORY is > defined, but _WIN32 is not if compiling under cygwin (fresh install, no > mingw). > > SetDllDirectory() is called whenever HAVE_SETDLLDIRECTORY is defined, > there is no

Re: [FFmpeg-devel] [RFC FIX] Build error with ffmpeg 3.1.3 (and current git) on cygwin

2016-09-22 Thread Timo Rothenpieler
> master uses _WIN32 checks in both places so if its not set, it will > never error, because it'll never even try to call it. But wasn't the HAVE_SETDLLDIRECTORY introduced because of Windows XP compatibility, as the function doesn't exist there, but _WIN32 is obviously set? Or does master just

Re: [FFmpeg-devel] [PATCH] avformat/utils: force native h264 decoder for probing

2016-09-22 Thread Timo Rothenpieler
Am 22.09.2016 um 12:36 schrieb Michael Niedermayer: > On Thu, Sep 22, 2016 at 11:09:08AM +0200, Timo Rothenpieler wrote: >> --- >> libavformat/utils.c | 18 +++--- >> 1 file changed, 15 insertions(+), 3 deletions(-) >

[FFmpeg-devel] [PATCH 2/3] avformat/utils: avoid using marked decoders for probing

2016-09-22 Thread Timo Rothenpieler
--- libavformat/utils.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index 05d2315..87a6dd7 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -188,6 +188,8 @@ FF_ENABLE_DEPRECATION_WARNINGS static const

[FFmpeg-devel] [PATCH 1/3] avcodec: add new AVOID_PROBING capability

2016-09-22 Thread Timo Rothenpieler
--- doc/APIchanges | 3 +++ libavcodec/avcodec.h | 10 ++ libavcodec/version.h | 4 ++-- 3 files changed, 15 insertions(+), 2 deletions(-) diff --git a/doc/APIchanges b/doc/APIchanges index 158a0b2..5d577e4 100644 --- a/doc/APIchanges +++ b/doc/APIchanges @@ -15,6 +15,9 @@

[FFmpeg-devel] [PATCH 3/3] avcodec/cuvid: mark as avoid for probing

2016-09-22 Thread Timo Rothenpieler
--- libavcodec/cuvid.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/cuvid.c b/libavcodec/cuvid.c index db96ac6..c040b09 100644 --- a/libavcodec/cuvid.c +++ b/libavcodec/cuvid.c @@ -911,7 +911,7 @@ static const AVOption options[] = { .send_packet=

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-07 Thread Timo Rothenpieler
applied ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] avformat/utils: avoid using marked decoders for probing

2016-09-22 Thread Timo Rothenpieler
--- libavformat/utils.c | 19 ++- 1 file changed, 18 insertions(+), 1 deletion(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index 05d2315..93ea6ff 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -188,6 +188,8 @@ FF_ENABLE_DEPRECATION_WARNINGS static

Re: [FFmpeg-devel] [PATCH] Nvidia NVENC 10-bit HEVC encoding and rate control lookahead support

2016-08-27 Thread Timo Rothenpieler
On 8/25/2016 7:56 PM, Oliver Collyer wrote: > Hi Timo > > Thankyou for the clarification. > > Attached are what should be the final versions of these patches then, with > the support for YUV420P10 (and related conversion code) now dropped. While testing these patches, I noticed that you now

Re: [FFmpeg-devel] [PATCH] avcodec/nvenc: Include nvEncodeAPI v7 SDK header

2016-08-27 Thread Timo Rothenpieler
On 8/27/2016 3:07 PM, Thomas Volkert wrote: > Hi, > > On 27.08.2016 14:58, Timo Rothenpieler wrote: >> As Nvidia has put the most recent Video Codec SDK behind a double >> registration wall, of which one needs manual approval of a lenghty >> application, bundling this h

Re: [FFmpeg-devel] [PATCH] Nvidia NVENC 10-bit HEVC encoding and rate control lookahead support

2016-08-29 Thread Timo Rothenpieler
> Hi all > > Attached is a patch for the above. > > 10-bit HEVC encoding is a new feature of the latest Pascal Nvidia GPUs, > released in the past few months; I’ve added support for the yuv420p10le and > yuv444p10le pixel formats. > > Rate control lookahead is available on pre-Pascal models

Re: [FFmpeg-devel] [PATCH] avcodec/nvenc: Include nvEncodeAPI v7 SDK header

2016-08-29 Thread Timo Rothenpieler
pushed ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] FFmpeg 3.2

2016-09-28 Thread Timo Rothenpieler
Am 27.09.2016 um 15:30 schrieb Michael Niedermayer: > Hi all > > Its long since FFmpeg 3.1, so its time to make 3.2 > ill branch release/3.2 off master and make 3.2 in maybe about a week or > 2 unless something delays it Should ffserver be deprecated for 3.2, or will it be resurrected now? I'm

Re: [FFmpeg-devel] [PATCH 1/3] avutil/hwcontext_cuda: align allocated frames

2016-10-03 Thread Timo Rothenpieler
applied ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/cuvid: set best_effort_timestamp instead of frame pts

2016-10-03 Thread Timo Rothenpieler
> Please just leave the pts stuff as-is, or we'll have to undo these > changes right in the next merge again, which deprecates pkt_pts and > favors using AVFrame.pts for decoding as well (less redundant fields). > Setting both for the time being should not have any negative effects. Ok, I'll

[FFmpeg-devel] [PATCH 1/3] avutil/hwcontext_cuda: align allocated frames

2016-10-02 Thread Timo Rothenpieler
--- libavutil/hwcontext_cuda.c | 35 --- 1 file changed, 20 insertions(+), 15 deletions(-) diff --git a/libavutil/hwcontext_cuda.c b/libavutil/hwcontext_cuda.c index 40d2971..e1dcab0 100644 --- a/libavutil/hwcontext_cuda.c +++ b/libavutil/hwcontext_cuda.c @@ -25,6

[FFmpeg-devel] [PATCH 2/2] MAINTAINERS: add myself for hwcontext_cuda

2016-10-02 Thread Timo Rothenpieler
Merritt hash Reimar Doeffinger + hwcontext_cuda* Timo Rothenpieler intfloat* Michael Niedermayer integer.c, integer.h Michael Niedermayer lzo Reimar

[FFmpeg-devel] [PATCH 1/2] avutil/hwcontext_cuda: align allocated frames

2016-10-02 Thread Timo Rothenpieler
--- libavutil/hwcontext_cuda.c | 43 --- 1 file changed, 28 insertions(+), 15 deletions(-) diff --git a/libavutil/hwcontext_cuda.c b/libavutil/hwcontext_cuda.c index 40d2971..706d195 100644 --- a/libavutil/hwcontext_cuda.c +++ b/libavutil/hwcontext_cuda.c

Re: [FFmpeg-devel] [PATCH] configure: define posix source on cygwin

2016-10-02 Thread Timo Rothenpieler
applied ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH 2/3] MAINTAINERS: add myself for hwcontext_cuda

2016-10-02 Thread Timo Rothenpieler
Merritt hash Reimar Doeffinger + hwcontext_cuda* Timo Rothenpieler intfloat* Michael Niedermayer integer.c, integer.h Michael Niedermayer lzo Reimar

[FFmpeg-devel] [PATCH 3/3] avcodec/cuvid: set best_effort_timestamp instead of frame pts

2016-10-02 Thread Timo Rothenpieler
--- libavcodec/cuvid.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/libavcodec/cuvid.c b/libavcodec/cuvid.c index e9a1dd7..5ceb94e 100644 --- a/libavcodec/cuvid.c +++ b/libavcodec/cuvid.c @@ -451,25 +451,25 @@ static int cuvid_output_frame(AVCodecContext

Re: [FFmpeg-devel] [PATCH] avcodec/mpegvideo_enc: fix memory leak

2016-09-26 Thread Timo Rothenpieler
> When the input frames contain side data, it will accumulate endlessly in > the coded frame, as av_frame_copy_props will append any new side data. > --- > libavcodec/mpegvideo_enc.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavcodec/mpegvideo_enc.c b/libavcodec/mpegvideo_enc.c

Re: [FFmpeg-devel] [PATCH] avcodec/mpegvideo_enc: fix memory leak

2016-09-25 Thread Timo Rothenpieler
On 9/25/2016 3:44 PM, Carl Eugen Hoyos wrote: > 2016-09-25 15:43 GMT+02:00 Timo Rothenpieler <t...@rothenpieler.org>: >> When the input frames contain side data, it will accumulate endlessly in >> the coded frame, as av_frame_copy_props will append any new side data. >

[FFmpeg-devel] [PATCH] avcodec/mpegvideo_enc: fix memory leak

2016-09-25 Thread Timo Rothenpieler
When the input frames contain side data, it will accumulate endlessly in the coded frame, as av_frame_copy_props will append any new side data. --- libavcodec/mpegvideo_enc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/mpegvideo_enc.c b/libavcodec/mpegvideo_enc.c index

Re: [FFmpeg-devel] [PATCH] avcodec/mpegvideo_enc: fix memory leak

2016-09-25 Thread Timo Rothenpieler
On 9/25/2016 3:55 PM, Carl Eugen Hoyos wrote: > 2016-09-25 15:47 GMT+02:00 Timo Rothenpieler <t...@rothenpieler.org>: >> On 9/25/2016 3:44 PM, Carl Eugen Hoyos wrote: >>> 2016-09-25 15:43 GMT+02:00 Timo Rothenpieler <t...@rothenpieler.org>: >>>&g

Re: [FFmpeg-devel] [PATCH] configure: define posix source on cygwin

2016-10-01 Thread Timo Rothenpieler
Will push tomorrow if nobody objects. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] configure: define posix source on cygwin

2016-10-01 Thread Timo Rothenpieler
Some function definitions are missing without it, for example sigaction. Fixes ticket #5868 --- configure | 1 + 1 file changed, 1 insertion(+) diff --git a/configure b/configure index 74db9b5..ff2cb62 100755 --- a/configure +++ b/configure @@ -4806,6 +4806,7 @@ case $target_os in

Re: [FFmpeg-devel] [PATCH] lavc: set best effort timestamp if unset when using new decode API

2016-10-03 Thread Timo Rothenpieler
On 10/2/2016 6:56 PM, wm4 wrote: > Some API users (in particular ffmpeg.c) check the best effort timestamp > only. > --- > Still undecided if this is the right approach. > --- > libavcodec/utils.c | 7 ++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/utils.c

Re: [FFmpeg-devel] [PATCH] avformat/utils: only call h264 decoder private function if h264 decoder is in use

2016-10-30 Thread Timo Rothenpieler
> has the issue been fixed for all branches an cases or is something > missing that needs this ? > (if so this needs a null pointer check i think) This was fixed in a diffrent way in 6d9a46e884d090a68069112a3b0436aa8b563967 It forces the h264 decoder to be used, so the assumption it is in use is

Re: [FFmpeg-devel] [PATCH] lavc/vaapi: Add VP8 decode hwaccel

2016-11-11 Thread Timo Rothenpieler
This patch seems incomplete. Did you forget to add the vaapi_vp8.c file before commiting? Also, adding a new field to the API which is deprecated from the start seems odd. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

[FFmpeg-devel] [PATCH 6/8] avfilter/vf_hwupload_cuda: use new hwdevice allocation API

2016-10-19 Thread Timo Rothenpieler
--- libavfilter/vf_hwupload_cuda.c | 57 -- 1 file changed, 5 insertions(+), 52 deletions(-) diff --git a/libavfilter/vf_hwupload_cuda.c b/libavfilter/vf_hwupload_cuda.c index c1c..dba496f 100644 --- a/libavfilter/vf_hwupload_cuda.c +++

[FFmpeg-devel] [PATCH 7/8] avfilter/vf_hwupload_cuda: check ff_formats_ref for errors

2016-10-19 Thread Timo Rothenpieler
--- libavfilter/vf_hwupload_cuda.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_hwupload_cuda.c b/libavfilter/vf_hwupload_cuda.c index dba496f..14688df 100644 --- a/libavfilter/vf_hwupload_cuda.c +++ b/libavfilter/vf_hwupload_cuda.c @@ -54,6 +54,8

[FFmpeg-devel] [PATCH 5/8] avcodec/nvenc: use dynamically loaded CUDA

2016-10-19 Thread Timo Rothenpieler
--- configure | 1 + libavcodec/nvenc.c | 129 + libavcodec/nvenc.h | 46 ++- 3 files changed, 26 insertions(+), 150 deletions(-) diff --git a/configure b/configure index 8c5742f..78113e2 100755 --- a/configure +++

[FFmpeg-devel] [PATCH 8/8] configure: cuda is no longer nonfree, enable and autodetect by default

2016-10-19 Thread Timo Rothenpieler
--- configure | 24 ++-- 1 file changed, 6 insertions(+), 18 deletions(-) diff --git a/configure b/configure index 78113e2..b6a0276 100755 --- a/configure +++ b/configure @@ -299,8 +299,8 @@ External library support: The following libraries provide various hardware

[FFmpeg-devel] [PATCH 4/8] avfilter/vf_scale_npp: use dynamically loaded CUDA

2016-10-19 Thread Timo Rothenpieler
--- libavfilter/vf_scale_npp.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavfilter/vf_scale_npp.c b/libavfilter/vf_scale_npp.c index 35abe39..c84282f 100644 --- a/libavfilter/vf_scale_npp.c +++ b/libavfilter/vf_scale_npp.c @@ -29,7 +29,7 @@ #include

[FFmpeg-devel] [PATCH 2/8] avutil/hwcontext_cuda: use dynamically loaded CUDA

2016-10-19 Thread Timo Rothenpieler
--- configure | 2 + libavutil/hwcontext_cuda.c | 99 - libavutil/hwcontext_cuda.h | 5 ++ libavutil/hwcontext_cuda_internal.h | 37 ++ libavutil/version.h | 2 +- 5 files changed, 120

[FFmpeg-devel] [PATCH 3/8] avcodec/cuvid: use dynamically loaded CUDA/CUVID

2016-10-19 Thread Timo Rothenpieler
And remove the now obsolete compat headers. --- compat/cuda/cuviddec.h | 827 - compat/cuda/nvcuvid.h | 321 --- libavcodec/cuvid.c | 78 +++-- 3 files changed, 47 insertions(+), 1179 deletions(-) delete mode 100644

[FFmpeg-devel] [PATCH 1/8] compat/cuda: add dynamic loader

2016-10-19 Thread Timo Rothenpieler
--- compat/cuda/dynlink_cuda.h | 88 + compat/cuda/dynlink_cuviddec.h | 808 + compat/cuda/dynlink_loader.h | 254 + compat/cuda/dynlink_nvcuvid.h | 316 4 files changed, 1466 insertions(+) create mode 100644

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Timo Rothenpieler
I'm not exactly maintaining libnpp, I think it was merged from libav, but the patch looks fine regardless. No idea why I never got these mails, must have mass-marked them as read and then deleted all unread mails. Will apply the old patches as this one does not show up on patchwork.

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Timo Rothenpieler
Am 19.10.2016 um 11:15 schrieb Sven C. Dack: > On 19/10/16 09:34, Yogender Kumar Gupta wrote: >> Added capability check for Temporal AQ >> >> Thanks, >> Yogender > I've send in 3 patches regarding Nvidia stuff (1. check for temp aq, 2. > option for temp aq in hevc, 3. limitation work-around for

Re: [FFmpeg-devel] [PATCH 1/8] compat/cuda: add dynamic loader

2016-10-19 Thread Timo Rothenpieler
>> +/* >> + * This file is part of FFmpeg. >> + * >> + * FFmpeg is free software; you can redistribute it and/or >> + * modify it under the terms of the GNU Lesser General Public >> + * License as published by the Free Software Foundation; either >> + * version 2.1 of the License, or (at your

Re: [FFmpeg-devel] nvenc/cuda force IDR

2016-10-12 Thread Timo Rothenpieler
> I'm trying to put together a patch, but I'm struggling a bit setting up to > build. I just pushed the neccesary changes. Works slightly diffrent than on x264, as the option it has seems a bit broken, seeing that both 0 and 1 force IDR frames: x4->forced_idr >= 0 ? X264_TYPE_IDR :

[FFmpeg-devel] [PATCH] avcodec/libx264: fix forced_idr logic

2016-10-12 Thread Timo Rothenpieler
Currently, it forces IDR frames for both true and false. Not entirely sure what the original idea behind the tri-state bool option is. --- libavcodec/libx264.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c index

Re: [FFmpeg-devel] [PATCH] scale_npp: fix passthrough mode

2016-10-25 Thread Timo Rothenpieler
LGTM ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec/libx264: fix forced_idr logic

2016-11-11 Thread Timo Rothenpieler
> Actually, looking closer, the default is still -1. > > Thus either this patch us incorrect, or the default should be changed to be 0. -1 and 0 both mean false now, and I left in the option to pass -1 to stay compatible with possible 3rd parties who pass it. So changing to default to 0 doesn't

Re: [FFmpeg-devel] [PATCH] avcodec/nvenc: Remove aspect-ratio decompensation logic

2016-11-25 Thread Timo Rothenpieler
This LGTM, the compensation is indeed gone on all current Nvidia Drivers I tested. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] coverity testing of FFmpeg

2016-11-27 Thread Timo Rothenpieler
> 3. Proprietary dependencies, which may or may not currently be an issue > anymore. Philip and Timo, how easy is it to get cuda/nvenc/cuvid/npp to > compile. > cuda/cuvid/nvenc don't need any external dependencies to compile, only to run. libnpp needs proprietary and non-redistributable headers

Re: [FFmpeg-devel] [PATCH] configure: use check_lib2 for cuda and cuvid

2016-11-12 Thread Timo Rothenpieler
Seems like I never tested on any 32bit platform. lgtm ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [Patch] hwaccel transocode broken

2016-11-13 Thread Timo Rothenpieler
> I created a simpler patch that avoids modifying the external signature > of the function, and it still fixes it for me. Can you test and > confirm? Then we can apply this. Just tested this patch, and I can confirm that at least a cuvid hwaccel transcode works again.

Re: [FFmpeg-devel] [PATCH] NVENC: Maximum usable surfaces is limited to maximum registered frames

2016-11-21 Thread Timo Rothenpieler
Patch LGTM, applied locally, will push most likely tomorrow. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] NVENC: Better surface allocation alghoritm, fix rc_lookahead

2016-11-21 Thread Timo Rothenpieler
> Please split the patch into two (or three) patches to make > the review and possible regression tests easier. The bug was implicitly fixed by the new code, it doesn't seem necessary to me to fix it independently, specially as so far nobody seems to have run into it. Patch LGTM, applied

Re: [FFmpeg-devel] [PATCH] CUVID: Allow to set number of used surfaces for decoding (resend)

2016-11-21 Thread Timo Rothenpieler
Does not compile: libavcodec/cuvid.c:861:19: error: 'CuvidContext' has no member named 'surfaces' #define OFFSET(x) offsetof(CuvidContext, x) ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/8] compat/cuda: add dynamic loader

2016-11-20 Thread Timo Rothenpieler
ping Will push in 2 days if nobody objects. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec/cuvid: Add support for P010 as an output surface format

2016-11-20 Thread Timo Rothenpieler
I don't really like outputting P016 as P010. I'd prefer to add support for P016 to ffmpeg and swscale, which shouldn't be too hard as most P010 code can be reused. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

<    1   2   3   4   5   6   7   >