Re: [FFmpeg-devel] [PATCH] nvenc: Make AUD optional for h264_nvenc and hevc_nvenc
Yes, please try the latest driver version. Thanks Yogender > On 13-Jun-2017, at 6:47 PM, Ali KIZIL wrote: > > 2017-01-05 13:29 GMT+03:00 Yogender Gupta : > There is BUG in Nvidia NVENC when you use AUD for H264 with B-frames, >> it will return corrupted stream, because NVIDIA is inserting AUD type 0 >> (I-frame) before B-frames instead of AUD type 7 (any-frame). >> >> Thanks for bringing this to notice. We have added a fix in the Nvidia >> driver and the newer driver releases will insert the correct AUD type. >> >> Thanks, >> Yogender >> >> >> --- >> This email message is for the sole use of the intended recipient(s) and >> may contain >> confidential information. Any unauthorized review, use, disclosure or >> distribution >> is prohibited. If you are not the intended recipient, please contact the >> sender by >> reply email and destroy all copies of the original message. >> >> --- >> ___ >> ffmpeg-devel mailing list >> ffmpeg-devel@ffmpeg.org >> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel >> > > Yogender, is new driver released for correct AUD type insertion ? > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] h264_nvenc encoding issue
>> I already post my mail to ffmpeg-users group regarding this issue but no one could help me out and advised to ask more experienced users. So i hope you can tell me something and sorry posting here. Here is my re-encoding command's output (Windows 8.1 with Bash): *$ ffmpeg -i perspective.mp4 -c:v h264_nvenc -profile:v high -preset:v fast perspective_out.mp4* Please share the i/p file. Thanks, Yogender ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ
>> Temporal AQ is implemented in CUDA, not in silicon. So far does the API (Video SDK 7.0.1) accept the flag without giving any errors. The output however remains identical currently (tested on a GTX 960). You'll have to ask Nvidia why they haven't implemented it yet (i'ts probably too soon). I am expecting it to become available with a future driver (or SDK or CUDA). It's already available for NVENC when encoding H.264/AVC and it's not only a method used in H.264 encoding, but one which is used in H.265 encoding, too (for example does x265 call auto variance '--aq-mode 2'). I am from Nvidia Video Team, so can give a good answer. Currently it has been implemented for H264 only. HEVC TAQ may be released in a future SDK release. All queries for NVENC/CUVID can also be directed to me. Thanks, Yogender ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ
>> This patch LGTM, minus the one spurious space at the end of the if. What is the procedure to submit, do I do a git push Thanks, Yogender ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
[FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ
>> 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 scale npp), but there seems to be no maintainer present at the moment to pick them up... Tempoal AQ HEVC not supported yet. Thanks, Yogender ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
[FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ
Added capability check for Temporal AQ Thanks, Yogender 0001-nvenc-Add-capability-check-for-Temporal-AQ.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH]Fix NVENC HEVC Profiles
Updated patch with requested changes. Thanks, Yogender On Wed, Sep 14, 2016 at 11:00 PM, Carl Eugen Hoyos wrote: > 2016-09-14 19:22 GMT+02:00 Yogender Kumar Gupta >: > > This patch fixes HEVC profiles for 444 and 444-10 that should be rext > > instead of main10. > > > +} > > +else if (ctx->data_pix_fmt == AV_PIX_FMT_P010) { > > Please merge these lines. > > Carl Eugen > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > 0001-Fix-NVENC-HEVC-Profiles.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
[FFmpeg-devel] [PATCH]Fix NVENC HEVC Profiles
This patch fixes HEVC profiles for 444 and 444-10 that should be rext instead of main10. Thanks, Yogender 0001-Fix-NVENC-HEVC-Profiles.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] Patch for fixing of nvenc.c compilation using msvc tools
I agree the change was not good, but since it was not compiling on my system using VS2013 I submitted this, but I hear that rather it may be an issue with my system as the code compiles well using VS2013 tools. I will retry at my end. On Wed, Sep 14, 2016 at 10:08 PM, Timo Rothenpieler wrote: > 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-arranged to look a bit more sane, I don't see the > point of changing this. > Any sane C-Compiler should not complain about this, and never did in all > my tests on various platforms and toolchains. > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
[FFmpeg-devel] [PATCH] Patch for Improved Log for NVENC
The existing logs have been improved to give a better idea of how what steps have been performed. Thanks, Yogender 0001-Improve-Log-for-NVENC.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] Patch for SDK 7.0 for NVENC
I have made only changes specific to SDK_7_0 in the attached patch. Also generated the patch using format-patch. Please let me know if anything else needs to be taken care of. Thanks, Yogender On Wed, Sep 14, 2016 at 8:48 PM, Carl Eugen Hoyos wrote: > 2016-09-14 15:43 GMT+02:00 Yogender Kumar Gupta >: > > 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 > > The first two hunks do not match your description, please send a > separate patch for the log changes. > The positioning of braces should match the FFmpeg style. > > > as well as fixes an issue with HEVC profile > > I may misunderstand but this could also be separate patch. > > Patch made with git format-patch are preferred, Carl Eugen > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > SDK_7_0.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
[FFmpeg-devel] [PATCH] Patch for SDK 7.0 for NVENC
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 sdk_7_0.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] Patch for fixing of nvenc.c compilation using msvc tools
Microsoft (R) C/C++ Optimizing Compiler Version 18.00.30501 for x64 Copyright (C) Microsoft Corporation. All rights reserved. is what is reported when I run cl in my mingW environment. and the shell is launched from the VS2013 x64 Native Tools Command Prompt On Wed, Sep 14, 2016 at 4:15 PM, Matt Oliver wrote: > On 14 September 2016 at 20:22, Yogender Kumar Gupta < > yogender.gu...@gmail.com> wrote: > > > Currently libavcodec.c/nvenc.c is not compilable using MSVC tools. I am > > attaching a patch that fixes this issue. > > > > Thanks, > > Yogender > > > > What version of msvc are you using? 2015 and 2013 all work fine currently > so I assume this is with an older version. Since nvenc is currently enabled > by default then this should have been picked up by FATE (which currently > has not had this issue) and older versions of msvc rely on the c99 > converter which i would have thought should have properly handled that kind > of code. > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
[FFmpeg-devel] Patch for fixing of nvenc.c compilation using msvc tools
Currently libavcodec.c/nvenc.c is not compilable using MSVC tools. I am attaching a patch that fixes this issue. Thanks, Yogender msvc_compile_fix.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel