Re: [FFmpeg-devel] [PATCH] avcodec/nvdec: Fix capability check with old drivers.

2017-12-11 Thread Timo Rothenpieler
This patch is already applied for a while now. I'm not always near a my development system to test and push patches... smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] Lagarith codec and FFmpeg

2017-12-10 Thread Timo Rothenpieler
Not a question for ffmpeg-devel. Also, lagarith decoding is supported natively. smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] NVENC - HEVC + WP

2017-10-25 Thread Timo Rothenpieler
Am 25.10.2017 um 06:06 schrieb Yogender Gupta: One of the users had recently complained about encoding problems with HEVC encoding with WP (Weighted Prediction). This is a driver issue that has been identified, and shows up when using HEVC + WP + (PictureTimingSEI messages and or buffering

Re: [FFmpeg-devel] order T-shirts

2017-10-20 Thread Timo Rothenpieler
Am 20.10.2017 um 17:16 schrieb Thilo Borgmann: Hi, Some guys at the VDD asked for FFmpeg T-shirts. I'd like to do a new T-shirt order. The shirts could be given to multimedia devs who stop at one of our next booths. My idea is to order 25 shirts: 1*S 5*M 10*L 7*XL 2*XXL Last time we ordered

Re: [FFmpeg-devel] On in-tree external headers

2017-10-31 Thread Timo Rothenpieler
Removing the nvenc/cuvid headers from the tree would imply the following procedure for anyone wanting to build ffmpeg with nvenc/cuvid: Register with nvidia to get access to their Video SDK Downloads. Extract the headers from their massive SDK. Patch them so that ffmpeg can make use of them.

Re: [FFmpeg-devel] [PATCH] libavcodec/cuviddec A53CC closed captions support added to cuviddec & nvenc

2018-05-04 Thread Timo Rothenpieler
applied smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH v4]avocdec/nvenc: Reconfigure bitrate on the fly

2018-05-04 Thread Timo Rothenpieler
applied smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH v4]avocdec/nvenc: Reconfigure bitrate on the fly

2018-05-05 Thread Timo Rothenpieler
Am 05.05.2018 um 02:00 schrieb Carl Eugen Hoyos: 2018-05-04 10:32 GMT+02:00, Timo Rothenpieler <t...@rothenpieler.org>: I'm not sure if actual resolution changes are even supported by avcodec. There are better (read: DVB) samples but I can't find them atm: https://trac.ffmpeg.o

Re: [FFmpeg-devel] [PATCH] Added the possibility to pass an externally created CUDA context to libavutil/hwcontext.c/av_hwdevice_ctx_create() for decoding with NVDEC

2018-05-07 Thread Timo Rothenpieler
On 26.04.2018 18:03, Oscar Amoros Huguet wrote: > Thanks Mark, > > You are right, we can implement in our code a sort of "av_hwdevice_ctx_set" > (which does not exist), by using av_hwdevice_ctx_alloc() + > av_hwdevice_ctx_init(). We actually use av_hwdevice_ctx_alloc in our code to > use the

Re: [FFmpeg-devel] [PATCH] Added the possibility to pass an externally created CUDA context to libavutil/hwcontext.c/av_hwdevice_ctx_create() for decoding with NVDEC

2018-05-07 Thread Timo Rothenpieler
>> Additionally, could you give your opinion on the feature we also may want to add in the future, that we mentioned in the previous email? Basically, we may want to add one more CUDA function, specifically cuMemcpy2DAsync, and the possibility to set a CUStream in AVCUDADeviceContext, so it is

Re: [FFmpeg-devel] [PATCH v4]avocdec/nvenc: Reconfigure bitrate on the fly

2018-05-04 Thread Timo Rothenpieler
> for dynamic resolution, maybe add a check ? > > ctx->support_dyn_res = nvenc_check_cap(avctx, > NV_ENC_CAPS_SUPPORT_DYN_RES_CHANGE); > > and: > if (dw != ctx->init_encode_params.darWidth || dh != > ctx->init_encode_params.darHeight) > > ==> > if ( (dw != ctx->init_encode_params.darWidth || dh

Re: [FFmpeg-devel] [PATCH] Added the possibility to pass an externally created CUDA context to libavutil/hwcontext.c/av_hwdevice_ctx_create() for decoding with NVDEC

2018-05-07 Thread Timo Rothenpieler
Am 07.05.2018 um 17:05 schrieb Oscar Amoros Huguet: To clarify a bit what I was saying in the last email. When I said CUDA non-blocking streams, I meant non-default streams. All non-blocking streams are non-default streams, but non-default streams can be blocking or non-bloking with respect

Re: [FFmpeg-devel] [PATCH] avutil/hwcontext_cuda: add AVCUDAFramesContext and AVCUDAFramesContext.flags

2018-05-07 Thread Timo Rothenpieler
Am 07.05.2018 um 23:22 schrieb Mark Thompson: On 07/05/18 22:10, Timo Rothenpieler wrote: Frames can be mapped from nvdec/cuvid, not needing any actual memory allocation, but all other features of the hw_frames_ctx. Hence the dummy-mode, which does not allocate any (notable amounts of) memory

Re: [FFmpeg-devel] [PATCH] Added the possibility to pass an externally created CUDA context to libavutil/hwcontext.c/av_hwdevice_ctx_create() for decoding with NVDEC

2018-05-07 Thread Timo Rothenpieler
Am 07.05.2018 um 19:37 schrieb Oscar Amoros Huguet: I was looking at the NVIDIA Video codec sdk samples (https://developer.nvidia.com/nvidia-video-codec-sdk#Download), where you can find the header NvDecoder.h next to cuviddec.h where CUVIDPROCPARAMS is defined. Anyway, I should have looked

[FFmpeg-devel] [PATCH] avutil/hwcontext_cuda: add AVCUDAFramesContext and AVCUDAFramesContext.flags

2018-05-07 Thread Timo Rothenpieler
Frames can be mapped from nvdec/cuvid, not needing any actual memory allocation, but all other features of the hw_frames_ctx. Hence the dummy-mode, which does not allocate any (notable amounts of) memory but otherwise behaves the exact same. --- doc/APIchanges | 3 +++

[FFmpeg-devel] [PATCH] avutil/hwcontext_cuda: add AVCUDAFramesContext and AVCUDAFramesContext.flags

2018-05-07 Thread Timo Rothenpieler
Frames can be mapped from nvdec/cuvid, not needing any actual memory allocation, but all other features of the hw_frames_ctx. Hence the dummy-mode, which does not allocate any (notable amounts of) memory but otherwise behaves the exact same. --- doc/APIchanges | 3 +++

Re: [FFmpeg-devel] [PATCH] avutil/hwcontext: add flags field to AVHWFramesContext

2018-05-07 Thread Timo Rothenpieler
Nack. Implementation-specific details go in the implementation-specific structure (AVHWFramesContext.hwctx). What are you actually thining of using this for? If you want to add flags which are in common between multiple different implementations then maybe it would be suitable to put it

Re: [FFmpeg-devel] [PATCH 2/5] avutil/hwcontext_cuda: add CUstream in cuda hwctx

2018-05-09 Thread Timo Rothenpieler
On 08.05.2018 23:00, James Almer wrote: > On 5/8/2018 3:36 PM, Timo Rothenpieler wrote: >> --- >> configure | 6 -- >> doc/APIchanges | 3 +++ >> libavutil/hwcontext_cuda.c | 3 +++ >> libavutil/hwcontext_cuda.h | 1 + >>

Re: [FFmpeg-devel] [PATCH] Added the possibility to pass an externally created CUDA context to libavutil/hwcontext.c/av_hwdevice_ctx_create() for decoding with NVDEC

2018-05-08 Thread Timo Rothenpieler
On 08.05.2018 10:11, Oscar Amoros Huguet wrote: > Thank you so much! > > We will test this hopefully today, and verify the expected behavior with > NSIGHT. > > By the way, I'm new to ffmpeg, so... I don't know if you use your fork to > test things first, before adding the changes to the main

Re: [FFmpeg-devel] [PATCH 2/5] avutil/hwcontext_cuda: add CUstream in cuda hwctx

2018-05-09 Thread Timo Rothenpieler
Am 09.05.2018 um 10:49 schrieb Timo Rothenpieler: On 08.05.2018 23:00, James Almer wrote: On 5/8/2018 3:36 PM, Timo Rothenpieler wrote: --- configure | 6 -- doc/APIchanges | 3 +++ libavutil/hwcontext_cuda.c | 3 +++ libavutil/hwcontext_cuda.h | 1

Re: [FFmpeg-devel] [PATCH 1/1] fix cuCtxPushCurrent signature

2018-04-27 Thread Timo Rothenpieler
applied signature.asc Description: OpenPGP digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/6] avcodec/nvdec: avoid needless copy of output frame

2018-05-09 Thread Timo Rothenpieler
applied follow-up CUstream patches also applied smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/6] avutil/hwcontext_cuda: add AVCUDAFramesContext and AVCUDAFramesContext.flags

2018-05-09 Thread Timo Rothenpieler
dropped this one. Missed I could just override the pool. smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/6] avcodec/nvdec: avoid needless copy of output frame

2018-05-10 Thread Timo Rothenpieler
Am 10.05.2018 um 04:51 schrieb Wang Bin: -offset += cpy.Height; +frame->buf[1] = av_buffer_create((uint8_t *)unmap_data, sizeof(*unmap_data), + nvdec_unmap_mapped_frame, (void*)devptr, +

Re: [FFmpeg-devel] [PATCH] avcodec/nvdec_hevc: Fix scaling lists

2018-05-10 Thread Timo Rothenpieler
Am 10.05.2018 um 03:48 schrieb Philip Langdale: The main issue here was the use of [i] instead of [i * 3] for the 32x32 matrix. As part of fixing this, I changed the code to match that used in vdpau_hevc, which I spent a lot of time verifying. I also changed to calculating NumPocTotalCurr

[FFmpeg-devel] [PATCH] avcodec/hevcdec: make ff_hevc_frame_nb_refs take a const pointer

2018-05-10 Thread Timo Rothenpieler
--- libavcodec/hevc_refs.c | 4 ++-- libavcodec/hevcdec.h | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/hevc_refs.c b/libavcodec/hevc_refs.c index 68c730edcc..7cf3a55725 100644 --- a/libavcodec/hevc_refs.c +++ b/libavcodec/hevc_refs.c @@ -508,12 +508,12 @@

Re: [FFmpeg-devel] [PATCH] avcodec/hevcdec: make ff_hevc_frame_nb_refs take a const pointer

2018-05-10 Thread Timo Rothenpieler
applied and backported to 4.0 smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec/nvdec_hevc: Fix scaling lists

2018-05-10 Thread Timo Rothenpieler
applied and backported to 4.0 smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] avutil/hwcontext: add flags field to AVHWFramesContext

2018-05-07 Thread Timo Rothenpieler
--- doc/APIchanges| 3 +++ libavutil/hwcontext.h | 7 +++ libavutil/version.h | 2 +- 3 files changed, 11 insertions(+), 1 deletion(-) diff --git a/doc/APIchanges b/doc/APIchanges index ede5b186ae..307c7a51ee 100644 --- a/doc/APIchanges +++ b/doc/APIchanges @@ -15,6 +15,9 @@

[FFmpeg-devel] [PATCH 1/6] avutil/hwcontext_cuda: add AVCUDAFramesContext and AVCUDAFramesContext.flags

2018-05-08 Thread Timo Rothenpieler
Frames can be mapped from nvdec/cuvid, not needing any actual memory allocation, but all other features of the hw_frames_ctx. Hence the dummy-mode, which does not allocate any (notable amounts of) memory but otherwise behaves the exact same. --- doc/APIchanges | 3 +++

[FFmpeg-devel] [PATCH 6/6] avcodec/cuviddec: explicitly synchronize cuMemcpy calls

2018-05-08 Thread Timo Rothenpieler
--- libavcodec/cuviddec.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/libavcodec/cuviddec.c b/libavcodec/cuviddec.c index 122c28f6e8..4d3caf924e 100644 --- a/libavcodec/cuviddec.c +++ b/libavcodec/cuviddec.c @@ -550,12 +550,16 @@ static int

[FFmpeg-devel] [PATCH 2/6] avcodec/nvdec: avoid needless copy of output frame

2018-05-08 Thread Timo Rothenpieler
Replaces the data pointers with the mapped cuvid ones. Adds buffer_refs to the frame to ensure the needed contexts stay alive and the cuvid idx stays allocated. Adds another buffer_ref to unmap the frame when it's unreferenced itself. --- libavcodec/nvdec.c | 83

[FFmpeg-devel] [PATCH 3/6] avutil/hwcontext_cuda: add CUstream in cuda hwctx

2018-05-08 Thread Timo Rothenpieler
--- configure | 6 -- doc/APIchanges | 3 ++- libavutil/hwcontext_cuda.c | 3 +++ libavutil/hwcontext_cuda.h | 1 + libavutil/version.h| 2 +- 5 files changed, 11 insertions(+), 4 deletions(-) diff --git a/configure b/configure index

Re: [FFmpeg-devel] [PATCH 2/6] avcodec/nvdec: avoid needless copy of output frame

2018-05-08 Thread Timo Rothenpieler
Am 08.05.2018 um 17:25 schrieb wm4: On Tue, 8 May 2018 15:31:28 +0200 Timo Rothenpieler <t...@rothenpieler.org> wrote: Replaces the data pointers with the mapped cuvid ones. Adds buffer_refs to the frame to ensure the needed contexts stay alive and the cuvid idx stays allocated. Adds a

Re: [FFmpeg-devel] [PATCH] avutil/hwcontext_cuda: add AVCUDAFramesContext and AVCUDAFramesContext.flags

2018-05-08 Thread Timo Rothenpieler
-frame->buf[0] = av_buffer_pool_get(ctx->pool); +if (frctx->flags & AV_CUDA_HWFRAMES_DUMMY_MODE) +frame->buf[0] = av_buffer_create(NULL, 0, NULL, NULL, 0); +else +frame->buf[0] = av_buffer_pool_get(ctx->pool); + Is this really needed? Because at least videotoolbox

Re: [FFmpeg-devel] [PATCH 3/6] avutil/hwcontext_cuda: add CUstream in cuda hwctx

2018-05-08 Thread Timo Rothenpieler
Am 08.05.2018 um 17:26 schrieb wm4: On Tue, 8 May 2018 15:31:29 +0200 Timo Rothenpieler <t...@rothenpieler.org> wrote: --- configure | 6 -- doc/APIchanges | 3 ++- libavutil/hwcontext_cuda.c | 3 +++ libavutil/hwcontext_cuda.h | 1 + lib

Re: [FFmpeg-devel] [PATCH] make install prefix configurable

2018-05-05 Thread Timo Rothenpieler
Am 05.05.2018 um 17:28 schrieb Daniel Oberhoff: --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index a51c2c9..c3a9209 100644 --- a/Makefile +++ b/Makefile @@ -1,4 +1,4 @@ -PREFIX = /usr/local +PREFIX ?= /usr/local This shouldn't be

[FFmpeg-devel] [PATCH 5/6] avutil/hwcontext_cuda: explicitly synchronize cuMemcpy calls

2018-05-08 Thread Timo Rothenpieler
--- libavutil/hwcontext_cuda.c | 18 +++--- 1 file changed, 15 insertions(+), 3 deletions(-) diff --git a/libavutil/hwcontext_cuda.c b/libavutil/hwcontext_cuda.c index 8024eec79d..7f55881ba5 100644 --- a/libavutil/hwcontext_cuda.c +++ b/libavutil/hwcontext_cuda.c @@ -267,13 +267,19

[FFmpeg-devel] [PATCH 4/6] avcodec/nvdec: pass CUstream in vpp parameters

2018-05-08 Thread Timo Rothenpieler
--- libavcodec/nvdec.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/libavcodec/nvdec.c b/libavcodec/nvdec.c index d98f9dd95e..c5c5c83dc7 100644 --- a/libavcodec/nvdec.c +++ b/libavcodec/nvdec.c @@ -39,6 +39,7 @@ typedef struct NVDECDecoder { AVBufferRef

Re: [FFmpeg-devel] [PATCH] Added the possibility to pass an externally created CUDA context to libavutil/hwcontext.c/av_hwdevice_ctx_create() for decoding with NVDEC

2018-05-07 Thread Timo Rothenpieler
Am 07.05.2018 um 18:25 schrieb Oscar Amoros Huguet: Have a look at this, looks pretty interesting: /** * @brief This function decodes a frame and returns the locked frame buffers * This makes the buffers available for use by the application without the buffers * getting

Re: [FFmpeg-devel] [PATCH] avutil/hwcontext_cuda: add AVCUDAFramesContext and AVCUDAFramesContext.flags

2018-05-08 Thread Timo Rothenpieler
Am 08.05.2018 um 17:49 schrieb wm4: On Tue, 8 May 2018 17:43:49 +0200 Timo Rothenpieler <t...@rothenpieler.org> wrote: -frame->buf[0] = av_buffer_pool_get(ctx->pool); +if (frctx->flags & AV_CUDA_HWFRAMES_DUMMY_MODE) +frame->buf[0] = av_buffer_create(

Re: [FFmpeg-devel] [PATCH 3/6] avutil/hwcontext_cuda: add CUstream in cuda hwctx

2018-05-08 Thread Timo Rothenpieler
What is this? https://docs.nvidia.com/cuda/cuda-driver-api/stream-sync-behavior.html It allows asynchronous processing of CUDA workloads. The next couple patches make use of it. There's no change in behaviour if it remains unset/NULL, but if you set one, the workload won't block the main CUDA

[FFmpeg-devel] [PATCH 1/5] avcodec/nvdec: avoid needless copy of output frame

2018-05-08 Thread Timo Rothenpieler
Replaces the data pointers with the mapped cuvid ones. Adds buffer_refs to the frame to ensure the needed contexts stay alive and the cuvid idx stays allocated. Adds another buffer_ref to unmap the frame when it's unreferenced itself. --- libavcodec/nvdec.c | 96

[FFmpeg-devel] [PATCH 4/5] avutil/hwcontext_cuda: explicitly synchronize cuMemcpy calls

2018-05-08 Thread Timo Rothenpieler
--- libavutil/hwcontext_cuda.c | 18 +++--- 1 file changed, 15 insertions(+), 3 deletions(-) diff --git a/libavutil/hwcontext_cuda.c b/libavutil/hwcontext_cuda.c index f3e81680e9..668293bffc 100644 --- a/libavutil/hwcontext_cuda.c +++ b/libavutil/hwcontext_cuda.c @@ -258,13 +258,19

[FFmpeg-devel] [PATCH 2/5] avutil/hwcontext_cuda: add CUstream in cuda hwctx

2018-05-08 Thread Timo Rothenpieler
--- configure | 6 -- doc/APIchanges | 3 +++ libavutil/hwcontext_cuda.c | 3 +++ libavutil/hwcontext_cuda.h | 1 + libavutil/version.h| 2 +- 5 files changed, 12 insertions(+), 3 deletions(-) diff --git a/configure b/configure index

[FFmpeg-devel] [PATCH 3/5] avcodec/nvdec: pass CUstream in vpp parameters

2018-05-08 Thread Timo Rothenpieler
--- libavcodec/nvdec.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/libavcodec/nvdec.c b/libavcodec/nvdec.c index 502747adfd..e779be3a45 100644 --- a/libavcodec/nvdec.c +++ b/libavcodec/nvdec.c @@ -39,6 +39,7 @@ typedef struct NVDECDecoder { AVBufferRef

[FFmpeg-devel] [PATCH 5/5] avcodec/cuviddec: explicitly synchronize cuMemcpy calls

2018-05-08 Thread Timo Rothenpieler
--- libavcodec/cuviddec.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/libavcodec/cuviddec.c b/libavcodec/cuviddec.c index 122c28f6e8..4d3caf924e 100644 --- a/libavcodec/cuviddec.c +++ b/libavcodec/cuviddec.c @@ -550,12 +550,16 @@ static int

Re: [FFmpeg-devel] [PATCH] avutil/hwcontext_cuda: fix YUV420P cuda_get_buffer

2018-05-15 Thread Timo Rothenpieler
On 14.05.2018 23:29, Marton Balint wrote: > Regression since ece068a771ac3f725e854c681ecbef08e792addc. > > Signed-off-by: Marton Balint > --- > libavutil/hwcontext_cuda.c | 9 ++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > > diff --git

Re: [FFmpeg-devel] [PATCH] avutil/error: hidden macro av_err2str for C++

2018-05-17 Thread Timo Rothenpieler
On 17.05.2018 06:46, Zhao Zhili wrote: > --- > libavutil/error.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavutil/error.h b/libavutil/error.h > index 71df4da..8a35fef 100644 > --- a/libavutil/error.h > +++ b/libavutil/error.h > @@ -116,8 +116,10 @@ static inline char

Re: [FFmpeg-devel] [PATCH] avformat/hls: tag as AVFMT_TS_DISCONT

2018-05-17 Thread Timo Rothenpieler
> I think I'm against this. HLS streams do not typically contain > timestamp resets (even if they could). Otherwise you might as well add > this flag to the Matroska demuxer. Besides, it would break some of my > code, which uses this flag as a heuristic to detect mpeg-ts style > non-container

Re: [FFmpeg-devel] [PATCH] avutil/hwcontext_cuda: fix YUV420P cuda_get_buffer

2018-05-15 Thread Timo Rothenpieler
On 15.05.2018 13:37, Marton Balint wrote: >> Are you sure it isn't already that way? I tested specially YUV420P and >> could not make out any issues. > > For 4K content linesize[0] is 3840, but linesize[1] becomes 2048 instead > of 1920. That's odd, 2048 seems more correct since it's a multiple

[FFmpeg-devel] [PATCH] avfilter/dnn_backend_native: fix leaks in error paths

2018-06-08 Thread Timo Rothenpieler
Fixes CID #1435888 --- libavfilter/dnn_backend_native.c | 4 1 file changed, 4 insertions(+) diff --git a/libavfilter/dnn_backend_native.c b/libavfilter/dnn_backend_native.c index 6e80dd3663..e0a9ec5003 100644 --- a/libavfilter/dnn_backend_native.c +++ b/libavfilter/dnn_backend_native.c @@

Re: [FFmpeg-devel] [PATCH] lavc/nvenc: enable nvenc encoder instance reuse (v2)

2018-06-13 Thread Timo Rothenpieler
-if (ctx->encoder_flushing) -return AVERROR_EOF; +if (ctx->encoder_flushing) { +if (avctx->internal->draining) +return AVERROR_EOF; + +ctx->encoder_flushing = 0; +ctx->first_packet_output = 0; +ctx->initial_pts[0] = AV_NOPTS_VALUE; +

Re: [FFmpeg-devel] [PATCH] lavc/nvenc: enable nvenc encoder instance reuse (v2)

2018-06-15 Thread Timo Rothenpieler
applied smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] avcodec/qsvenc: fix version detection on cygwin

2018-06-15 Thread Timo Rothenpieler
--- libavcodec/qsvenc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/qsvenc.h b/libavcodec/qsvenc.h index d48272224c..bb175c5df8 100644 --- a/libavcodec/qsvenc.h +++ b/libavcodec/qsvenc.h @@ -45,7 +45,7 @@ #define QSV_HAVE_LA_DS QSV_VERSION_ATLEAST(1, 8)

Re: [FFmpeg-devel] [PATCH] avcodec/qsvenc: fix version detection on cygwin

2018-06-17 Thread Timo Rothenpieler
Am 17.06.2018 um 17:48 schrieb Mark Thompson: On 15/06/18 15:52, Timo Rothenpieler wrote: --- libavcodec/qsvenc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/qsvenc.h b/libavcodec/qsvenc.h index d48272224c..bb175c5df8 100644 --- a/libavcodec/qsvenc.h

Re: [FFmpeg-devel] [PATCH 2/5] avutil/hwcontext_cuda: add CUstream in cuda hwctx

2018-06-09 Thread Timo Rothenpieler
Am 09.06.2018 um 03:25 schrieb James Almer: On 5/9/2018 5:49 AM, Timo Rothenpieler wrote: On 08.05.2018 23:00, James Almer wrote: On 5/8/2018 3:36 PM, Timo Rothenpieler wrote: --- configure | 6 -- doc/APIchanges | 3 +++ libavutil/hwcontext_cuda.c | 3

Re: [FFmpeg-devel] NVIDIA CUVID decoding issue.

2018-06-05 Thread Timo Rothenpieler
On 04.06.2018 12:27, Nánási Ferenc wrote: > Hi! > I'm trying to transcode H.264 Stream to HEVC. Everything just working > fine when I'm using the in-built CPU decoder, but when I turn on the > Hardware-accelerated CUVID module, it throws some error with most of my > streams. > * > **FFMPEG

Re: [FFmpeg-devel] [PATCH] lavc/nvenc: enable nvenc encoder instance reuse after draining

2018-06-07 Thread Timo Rothenpieler
On 07.06.2018 06:38, Pavel Koshevoy wrote: > --- > libavcodec/nvenc.c | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c > index b4186c0bec..8928eacc70 100644 > --- a/libavcodec/nvenc.c > +++ b/libavcodec/nvenc.c > @@ -2181,6 +2181,12 @@ int

Re: [FFmpeg-devel] [PATCH] avcodec/qsvenc: fix version detection on cygwin

2018-06-27 Thread Timo Rothenpieler
applied smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH]avocdec/nvenc: Reconfigure bitrate on the fly

2018-05-02 Thread Timo Rothenpieler
Am 02.05.2018 um 23:18 schrieb pkv.stream: Hello, The patch enables dynamic bitrate through ReconfigureEncoder method from nvenc API. This is useful for live streaming in case of network congestion. A similar patch for changing aspect ratio dynamically was previously submitted by Miroslav

Re: [FFmpeg-devel] [PATCH]avocdec/nvenc: Reconfigure bitrate on the fly

2018-05-02 Thread Timo Rothenpieler
Am 02.05.2018 um 23:58 schrieb Timo Rothenpieler: Am 02.05.2018 um 23:18 schrieb pkv.stream: Hello, The patch enables dynamic bitrate through ReconfigureEncoder method from nvenc API. This is useful for live streaming in case of network congestion. A similar patch for changing aspect ratio

Re: [FFmpeg-devel] [PATCH]avocdec/nvenc: Reconfigure bitrate on the fly

2018-05-02 Thread Timo Rothenpieler
Hi thanks for your feedback. I was not sure of whether to put the capability check with the others in nvenc_check_capabilities because if the cap is missing this would remove the encoder altogether: ex: ln. 453 :     if ((ret = nvenc_check_capabilities(avctx)) < 0)                    

Re: [FFmpeg-devel] [PATCH] libavcodec/cuviddec A53CC closed captions support added to cuviddec & nvenc

2018-05-03 Thread Timo Rothenpieler
The nvenc side looks ok except for one or two minor nits which I will correct locally and push it with my next batch of patches. I'm not so sure on the cuvid side of things. A53 support is already provided via the new nvdec hwaccel, which supports it natively through the ffmpeg codec parsers.

Re: [FFmpeg-devel] [PATCH] libavcodec/cuviddec A53CC closed captions support added to cuviddec & nvenc

2018-05-03 Thread Timo Rothenpieler
Slightly refactored nvenc part can be found here: https://github.com/BtbN/FFmpeg/commit/e5d85ac3382cef35dc5ba799f026d6aa0a792a39 Will push tomorrow if no issue with it comes up. smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel

Re: [FFmpeg-devel] [PATCH v4]avocdec/nvenc: Reconfigure bitrate on the fly

2018-05-03 Thread Timo Rothenpieler
Am 03.05.2018 um 09:51 schrieb pkv.stream: Hi, in previous versions ReconfigureEncoder was called at each frame once bitrate was changed. Fixed that. The patch was rebased against https://github.com/BtbN/FFmpeg/commit/4e8265e3a63a71a40730e8eb575d8aa760bdcddc Slightly modified the approach

Re: [FFmpeg-devel] [PATCH 1/1] add CUgraphicsRegisterFlags enum

2018-04-29 Thread Timo Rothenpieler
pushed equivalent changes as part of a different patch Also started an sdk/8.0 branch and backported all non-video-sdk changes there. smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] http: block while waiting for reconnecting

2017-12-30 Thread Timo Rothenpieler
Am 30.12.2017 um 18:07 schrieb wm4: It makes no sense to return an error after the first reconnect, and then somehow resume the next time it's called. Also make the wait reasonably interruptible. Since there is no mechanism for this in the API, polling is the best we can do. (Some effort could

Re: [FFmpeg-devel] [PATCH] cuvid : add support to force intra frames as in input source

2018-01-20 Thread Timo Rothenpieler
Shouldn't we generally recommend people use the new NVDEC implementation instead? It already has all these features and more. :) There are still features missing from nvdec which will take a while to implement, if it's possible at all. Like deinterlacing/cropping. Also, cuviddec is still

Re: [FFmpeg-devel] [PATCH] cuvid : add support to force intra frames as in input source

2018-01-19 Thread Timo Rothenpieler
Am 19.01.2018 um 19:47 schrieb Michael Niedermayer: On Fri, Jan 19, 2018 at 11:09:51AM +0100, Timo Rothenpieler wrote: Am 18.01.2018 um 07:52 schrieb Yogender Gupta: Improved the patch by dynamic allocation. Thanks, Yogender @@ -1072,6 +1083,8 @@ static void cuvid_flush(AVCodecContext

Re: [FFmpeg-devel] [PATCH] cuvid : add support to force intra frames as in input source

2018-01-19 Thread Timo Rothenpieler
Am 18.01.2018 um 07:52 schrieb Yogender Gupta: Improved the patch by dynamic allocation. Thanks, Yogender > @@ -1072,6 +1083,8 @@ static void cuvid_flush(AVCodecContext *avctx) > if (ret < 0) > goto error; > > +av_free(ctx->key_frame); > + > ctx->prev_pts = INT64_MIN;

Re: [FFmpeg-devel] [PATCH]: Change Stack Frame Limit in Cuda Context

2018-01-26 Thread Timo Rothenpieler
Am 26.01.2018 um 10:06 schrieb Ben Chang: Thanks for the review Mark. On Thu, Jan 25, 2018 at 4:13 PM, Mark Thompson wrote: diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c index 4a91d99..2da251b 100644 --- a/libavcodec/nvenc.c +++ b/libavcodec/nvenc.c @@ -420,6 +420,12

[FFmpeg-devel] [PATCH] avcodec/nvenc: refcount input frame mappings

2018-01-26 Thread Timo Rothenpieler
If some logic like vsync in ffmpeg.c duplicates frames, it might pass the same frame twice, which will result in a crash due it being effectively mapped and unmapped twice. Signed-off-by: Timo Rothenpieler <t...@rothenpieler.org> --- libavcodec/nvenc.

Re: [FFmpeg-devel] [PATCH] cuvid : add support to force intra frames as in input source

2018-01-17 Thread Timo Rothenpieler
Am 17.01.2018 um 06:12 schrieb Yogender Gupta: Please find attached a patch for setting key frames. "-force_key_frames" can then use this option. Thanks, Yogender > diff --git a/libavcodec/cuviddec.c b/libavcodec/cuviddec.c > index c23033c..54f6cb7 100644 > --- a/libavcodec/cuviddec.c > +++

Re: [FFmpeg-devel] [PATCH]: Change Stack Frame Limit in Cuda Context

2018-01-25 Thread Timo Rothenpieler
Am 26.01.2018 um 00:17 schrieb Ben Chang: Just use another provider like gmail. Done. Patch looks ok on first glance, will do a proper review tomorrow. smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH] avcodec/nvenc: refcount input frame mappings

2018-01-28 Thread Timo Rothenpieler
applied together with some other minor fixes. Also backported to 3.4 and 3.3, as it fixes a crash. smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] libavfilter : cuda linkage changed for vf_scale_cuda & vf_thumbnail_cuda

2018-02-13 Thread Timo Rothenpieler
On 13.02.2018 12:38, Hendrik Leppkes wrote: > On Tue, Feb 13, 2018 at 12:31 PM, Roman Arzumanyan > wrote: >> Please find attached the patch that introduces structure to store CUDA >> function pointers to be used within scale & thumbnail filters. >> > > The entire point

Re: [FFmpeg-devel] [PATCH] compat: remove in-tree NVidia headers

2018-02-25 Thread Timo Rothenpieler
The reason I haven't pushed this yet is because it breaks the build on MSVC, and I haven't gotten around to figuring out pkg-config for it. smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] avformat/librtmp: fix returning EOF from Read/Write

2018-07-27 Thread Timo Rothenpieler
applied smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avformat/librtmp: fix returning EOF from Read/Write

2018-07-27 Thread Timo Rothenpieler
I did a 24h test with this, and it seems to fix the issue. Will apply and backport to 4.0 in 24h if nobody objects. smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

[FFmpeg-devel] [PATCH] avformat/librtmp: fix returning EOF from Read/Write

2018-07-26 Thread Timo Rothenpieler
--- libavformat/librtmp.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/libavformat/librtmp.c b/libavformat/librtmp.c index f3cfa9a8e2..43013e46e0 100644 --- a/libavformat/librtmp.c +++ b/libavformat/librtmp.c @@ -261,7 +261,10 @@ static int rtmp_write(URLContext

Re: [FFmpeg-devel] [PATCH] avformat/librtmp: fix returning EOF from Read/Write

2018-07-31 Thread Timo Rothenpieler
Am 28.07.2018 um 11:48 schrieb Jan Ekström: On Thu, Jul 26, 2018 at 1:20 PM, Timo Rothenpieler wrote: --- libavformat/librtmp.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/libavformat/librtmp.c b/libavformat/librtmp.c index f3cfa9a8e2..43013e46e0 100644

Re: [FFmpeg-devel] [PATCH] libavcodec/cuviddec A53CC closed captions support added to cuviddec & nvenc

2018-08-20 Thread Timo Rothenpieler
On 8/17/2018 2:38 PM, Carl Eugen Hoyos wrote: 2018-05-03 19:42 GMT+02:00, Timo Rothenpieler : Slightly refactored nvenc part can be found here: https://github.com/BtbN/FFmpeg/commit/e5d85ac3 Will push tomorrow if no issue with it comes up. Why is there no option to disable Closed Captions

[FFmpeg-devel] [PATCH] avcodec/utils: document allocation requirement of extradata

2018-08-24 Thread Timo Rothenpieler
--- libavcodec/avcodec.h | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index 31e50d5a94..2c5e78c88d 100644 --- a/libavcodec/avcodec.h +++ b/libavcodec/avcodec.h @@ -1616,6 +1616,7 @@ typedef struct AVCodecContext { * The allocated memory

[FFmpeg-devel] [PATCH] avcodec/utils: document allocation requirement of extradata

2018-08-24 Thread Timo Rothenpieler
--- libavcodec/avcodec.h | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index 31e50d5a94..5ec7a2430b 100644 --- a/libavcodec/avcodec.h +++ b/libavcodec/avcodec.h @@ -1616,6 +1616,7 @@ typedef struct AVCodecContext { * The allocated memory

Re: [FFmpeg-devel] [PATCH] avcodec/utils: document allocation requirement of extradata

2018-08-25 Thread Timo Rothenpieler
On 8/25/2018 5:15 PM, Michael Niedermayer wrote: On Fri, Aug 24, 2018 at 08:53:29PM +0200, Timo Rothenpieler wrote: --- libavcodec/avcodec.h | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index 31e50d5a94..2c5e78c88d 100644 --- a/libavcodec

Re: [FFmpeg-devel] [PATCH] configure: speedup x2-x8

2018-08-25 Thread Timo Rothenpieler
Please use git send-email to send your patches, or at least send each patch, created by git format-patch, as individual attachment. Your files seem to contain multiple patches one after another, which makes them very hard to follow. But nice work! Let's hope this does not cause any

Re: [FFmpeg-devel] [PATCH] avfilter: add nvidia NPP based transpose filter

2018-09-08 Thread Timo Rothenpieler
On 9/8/2018 3:49 PM, Timo Rothenpieler wrote: From: Roman Arzumanyan Signed-off-by: Timo Rothenpieler --- I'm not overly a fan of a rotate filter that only support 90° angles either. So here's my modified version of the original transpose filter, which now behaves the exact same

Re: [FFmpeg-devel] [PATCH] avfilter: add nvidia NPP based transpose filter

2018-09-08 Thread Timo Rothenpieler
On 9/8/2018 5:38 PM, Paul B Mahol wrote: I'll probably remove the interp_algo from this before committing, hard coding it to nearest neighbor. I'm unable to see any difference between them for perfect 90DEG angles except that NN is easily 10 times faster than the current default Cubic. Perhaps

Re: [FFmpeg-devel] [PATCH] avfilter: add nvidia NPP based transpose filter

2018-09-09 Thread Timo Rothenpieler
On 9/9/2018 2:12 AM, Michael Niedermayer wrote: On Sat, Sep 08, 2018 at 03:49:29PM +0200, Timo Rothenpieler wrote: From: Roman Arzumanyan Signed-off-by: Timo Rothenpieler --- I'm not overly a fan of a rotate filter that only support 90° angles either. So here's my modified version

[FFmpeg-devel] [PATCH] avfilter: add nvidia NPP based transpose filter

2018-09-09 Thread Timo Rothenpieler
From: Roman Arzumanyan Signed-off-by: Timo Rothenpieler --- configure | 5 +- doc/filters.texi | 42 +++ libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/version.h | 2 +- libavfilter/vf_transpose_npp.c

Re: [FFmpeg-devel] [PATCH] avfilter/vf_transpose adding NPP transpose filter

2018-09-04 Thread Timo Rothenpieler
On 9/4/2018 3:54 PM, Moritz Barsnick wrote: On Tue, Sep 04, 2018 at 07:43:10 +, Roman Arzumanyan wrote: Hi Timo, Refactored the patch according to your review: +{ "interp_algo","Interpolation algorithm used for transposing", OFFSET(interp_algo), AV_OPT_TYPE_INT,{ .i64 =

Re: [FFmpeg-devel] [PATCH] avfilter: add nvidia NPP based transpose filter

2018-09-10 Thread Timo Rothenpieler
applied smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] avfilter: add nvidia NPP based transpose filter

2018-09-08 Thread Timo Rothenpieler
From: Roman Arzumanyan Signed-off-by: Timo Rothenpieler --- I'm not overly a fan of a rotate filter that only support 90° angles either. So here's my modified version of the original transpose filter, which now behaves the exact same as the software transpose filter. Additionally, I removed

Re: [FFmpeg-devel] [PATCH] avfilter/vf_transpose adding NPP transpose filter

2018-08-30 Thread Timo Rothenpieler
Thanks! I'm a bit short on time at the moment, but I'll try to have a look during the weekend. If someone else also wants to give a review, please go ahead! smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] mistake in patch "Add lensfun filter"

2018-07-12 Thread Timo Rothenpieler
On 12/07/2018 06:40, Stephen Seo wrote: Hi, It appears I used the wrong license in libavfilter/vf_lensfun.c It derives from the Lensfun library that uses (L)GPL3, but I used GPL2 by mistake. Should I 'git send-email' the whole patch again with this mistake resolved, or should I email another

Re: [FFmpeg-devel] RTMPS to FFM

2018-01-18 Thread Timo Rothenpieler
Am 18.01.2018 um 03:38 schrieb Gonzalo Martinez: Hi I have a client written in C++ that use a Hardware Encoder called Cedar and Stream the Video and Audio to my RTMPS server but now I need to change that to Stream locally to a ffserver. The problem that I have when modify the URL from rtmps to

Re: [FFmpeg-devel] [PATCH] compat: remove in-tree NVidia headers

2018-02-27 Thread Timo Rothenpieler
+cuda_deps="ffnvcodec"> cuda_deps_any="libdl LoadLibrary" cuvid_deps="cuda" d3d11va_deps="dxva_h ID3D11VideoDecoder ID3D11VideoContext" @@ -5811,6 +5812,9 @@ check_type "va/va.h va/va_enc_vp9.h" "VAEncPictureParameterBufferVP9" check_type "vdpau/vdpau.h" "VdpPictureInfoHEVC"

Re: [FFmpeg-devel] [FFmpeg-cvslog] compat: remove in-tree NVidia headers

2018-02-27 Thread Timo Rothenpieler
Am 27.02.2018 um 16:24 schrieb Nicolas George: Timo Rothenpieler (2018-02-27): compat: remove in-tree NVidia headers Unless I was mistaken, there was no consensus for that yet. From how I understood the discussion the only issue people had was that it was still not pushed, with MSVC

Re: [FFmpeg-devel] [PATCH] Fix iterating of input and output devices

2018-03-14 Thread Timo Rothenpieler
>-if (!(prev = ((AVOutputFormat *)prev)->next)) >+if (!(prev = prev ? ((AVInputFormat *)prev)->next : (void*)outdev_list[0])) AVOutputFormat smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list

<    1   2   3   4   5   6   7   >