Enhance Cycles docs on building CUDA kernel
ClosedPublic
Actions

Authored by FeRD (Frank Dana) (ferdnyc) on Aug 15 2018, 10:50 PM.

Details

Reviewers

Sergey Sharybin (sergey)
Brecht Van Lommel (brecht)

Group Reviewers

Documentation
Cycles

Commits

rBM4424: Cycles: suggest better solution for unsupported GNU version errors.

Summary

The Cycles docs on GPU Rendering contain a section on common error messages encountered when attempting to build the CUDA kernel, including the dreaded "Unsupported GNU version!" when an incompatible GCC is used.

Currently the docs recommend editing the CUDA header files to remove CUDA's version protection, which honestly is a bit shocking as a "solution". It's also unnecessary, since Blender supports the CYCLES_CUDA_EXTRA_CFLAGS environment variable which can be used to customize the kernel compilation process.

This patch expands that section with instructions on how to specify an alternate compiler for CUDA kernel compilation, only recommending that the header files be edited as a last resort if that's unsuccessful.

Diff Detail

Repository: rBM Blender Manual

Event Timeline

FeRD (Frank Dana) (ferdnyc) created this revision.Aug 15 2018, 10:50 PM

This is my first Phabricator submission and I may have mis-filed this diff, apologies if so.

Reuploading against Blender Manual repository.

Inês Almeida (brita_) added reviewers: Sergey Sharybin (sergey), Brecht Van Lommel (brecht).Aug 22 2018, 10:10 PM

Inês Almeida (brita_) added a reviewer: Cycles.

I'll commit this with some tweaks.

Note that for Cycles it's quite safe to remove this test, as we don't compile any host code with nvcc and so version incompatibilities with GCC are not a problem.

This revision is now accepted and ready to land.Aug 23 2018, 12:41 AM

Closed by commit rBM4424: Cycles: suggest better solution for unsupported GNU version errors. (authored by Brecht Van Lommel (brecht)). · Explain WhyAug 23 2018, 12:42 AM

This revision was automatically updated to reflect the committed changes.

In D3614#83377, @Brecht Van Lommel (brecht) wrote:

Note that for Cycles it's quite safe to remove this test, as we don't compile any host code with nvcc and so version incompatibilities with GCC are not a problem.

Maybe so, but editing /usr/local/cuda/include/host_config.h doesn't only remove it for Cycles, it disables CUDA's version checks completely.

Revision Contents
Changeset List

Path

Size

manual/

render/

cycles/

gpu_rendering.rst

27 lines

Diff 11656

View Options

manual/render/cycles/gpu_rendering.rst

Enhance Cycles docs on building CUDA kernelClosedPublicActions