Skip to content

Conversation

AkashiSN
Copy link
Contributor

Build Status

Summary

  • A CUDA testing environment with Docker and native OSX testing have been added to Travis CI.
  • Changed Dockerfile to use a multi-stage build.
  • Added Dockerfile for building CUDA into ffmpeg with nvidia/cuda container. (cuda-ubuntu.dockerfile)
  • The indentation was unified to Tab and re-indent.
  • Changed the config option (--enable-*) to be added in each build section (CONFIGURE_OPTIONS).
  • Removed some config options because they were redundant.

About CUDA

Since the version of CUDA published by nvidia/cuda is 11.1 in ubuntu 20.04, and the target architecture is very old (SM30) in ffmpeg's default nvccflags and completely removed in CUDA11, the target architecture is SM52 added a flag to change to. (ref: https://arnon.dk/matching-sm-architectures-arch-and-gencode-for-various-nvidia-cards/)
--nvccflags=-gencode arch=compute_52,code=sm_52

Backwards compatibility exists (just not optimal), so you only need to change this flag when using older graphics cards or if you want to optimize for a particular graphics card.

Removed some config options

Add config options

  • --enable-libxvid : Adding this option resulted in dynamic linking, so the code to remove the dynamic library was added.

Sincerely,

@AkashiSN AkashiSN marked this pull request as ready for review October 10, 2020 15:55
@markus-perl markus-perl merged commit 9ea68ed into markus-perl:master Oct 11, 2020
@markus-perl
Copy link
Owner

Hi AkashiSN, Thank you for all this improvements and the high quality of the pull request. I mode some small improvements and merged the commit.

Best,
Markus

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants