1. 22 Nov, 2016 2 commits
    • Miroslav Slugeň's avatar
      avcodec/nvenc: better surface allocation alghoritm, fix rc_lookahead · de2faec2
      Miroslav Slugeň authored
      User selectable surfaces are not working correctly, if you set number of
      surfaces on cmdline, it will always use minimum 32 or 48 depends on
      selected resolution, but in nvenc it is not necessary to use so many
      surfaces.
      
      So from now you can define as low as 1 surface and nvenc will still
      work, it will ofcourse lower GPU memory usage by 95% and async_delay to zero
      
      That was the easy part, now littlebit more...
      
      Next part of this patch is to always prefer rc_lookahead to be more
      important for number of surfaces, than user defined surfaces value.
      Maximum rc_lookahead from nvidia documentation is 32, but could increase
      in future generations so there is no limit for this yet. Value
      async_depth is still accepted and prefered over rc_lookahead.
      
      There were also bug when you request more than rc_lookahead > 31, it
      will always set maximum 31, because surface numbers recalculation was
      after setting lookahead, which is now fixed.
      
      Results:
      If you set -rc_lookahead 32 and -bf 3 it will now use only 40 surfaces
      and lower GPU memory usage by 20%, also it will now increase PSNR by 0.012dB
      
      Two more comments:
      
      1. from my internal test, i don't understand addition of 4 more surfaces
      when lookahead is calculated, i didn't used this and everything works as
      with those 4 more extra surfaces, does anybody know what is going on
      there? I looks like it was used for B frames which are calculated
      separately, because B frames maximum is 4.
      
      2. rc_lookahead is defined default to -1, but in test condition if
      (ctx->rc_lookahead) which sets lookahead it will be always true, i don't
      know if this is intended behavior, so in default behavior is lookahead
      always on!
      
      This is default condition when rc_lokkahead is -1 (not defined on
      cmdline), whis is maybe something that is not intended:
      ctx->encode_config.rcParams.enableLookahead = 1;
      ctx->encode_config.rcParams.lookaheadDepth  = 0;
      ctx->encode_config.rcParams.disableIadapt   = 0;
      ctx->encode_config.rcParams.disableBadapt   = 0;
      Signed-off-by: 's avatarTimo Rothenpieler <timo@rothenpieler.org>
      de2faec2
    • Timo Rothenpieler's avatar
      a66835bc
  2. 05 Nov, 2016 1 commit
  3. 19 Oct, 2016 1 commit
  4. 12 Oct, 2016 1 commit
  5. 28 Sep, 2016 3 commits
  6. 24 Sep, 2016 1 commit
  7. 08 Sep, 2016 1 commit
  8. 07 Sep, 2016 3 commits
  9. 28 Aug, 2016 7 commits
  10. 06 Jun, 2016 1 commit
  11. 04 Jun, 2016 1 commit
  12. 31 May, 2016 17 commits
  13. 25 May, 2016 1 commit