[FFmpeg-devel] [PATCH 2/2] libavutil/hwcontect_cuda: add option to use current CUDA context

Timo Rothenpieler timo at rothenpieler.org
Thu Sep 28 20:27:32 EEST 2023


On 14.09.2023 18:52, Roman Arzumanyan wrote:
> Got it, thanks.
> Please find the patches in attachment.
> 
> Just for my own understanding - could you please explain how this magic
> works in the configure script? I mean this section:
> 
> if ! disabled ffnvcodec; then
> 
>      ffnv_hdr_list="ffnvcodec/nvEncodeAPI.h ffnvcodec/dynlink_cuda.h
> ffnvcodec/dynlink_cuviddec.h ffnvcodec/dynlink_nvcuvid.h"
> 
>      check_pkg_config ffnvcodec "ffnvcodec >= 12.1.14.0" "$ffnv_hdr_list" ""
> || \
> 
>        check_pkg_config ffnvcodec "ffnvcodec >= 12.0.16.0  ffnvcodec < 12.1"
> "$ffnv_hdr_list" "" || \
> 
>        check_pkg_config ffnvcodec "ffnvcodec >= 11.1.5.2 ffnvcodec < 12.0"
> "$ffnv_hdr_list" "" || \
> 
>        check_pkg_config ffnvcodec "ffnvcodec >= 11.0.10.2 ffnvcodec < 11.1"
> "$ffnv_hdr_list" "" || \
> 
>        check_pkg_config ffnvcodec "ffnvcodec >= 8.1.24.14 ffnvcodec < 8.2"
> "$ffnv_hdr_list" ""
> 
> fi

Applied with various amendments:

  - Added missing version bump.
  - Added missing APIchanges entry.
  - Made the "both options are set"-check actually check both options, 
and not the same one twice.
  - Dropped "if != 0" checks, FFmpeg generally just does "if (variable)" 
in those cases.
  - Updated version checks in configure.


More information about the ffmpeg-devel mailing list