Skip to content
This repository was archived by the owner on May 9, 2024. It is now read-only.

Move to ubuntu 22.04 in docker files and update compute-runtime version #622

Merged
merged 4 commits into from
Aug 10, 2023

Conversation

kurapov-peter
Copy link
Contributor

p.s. why do we have Dockerfile.clang that is identical to Dockerfile.cuda?

@kurapov-peter
Copy link
Contributor Author

The cause of the installation failure is that cuda-drivers-525 fails to build the module (similar to https://forums.developer.nvidia.com/t/510-68-02-driver-breaks-installation-for-newer-linux-kernel-5-17-8-on-ubuntu-20-04/214531):

gcc: error: unrecognized command line option '-mharden-sls=all'
make[2]: *** [scripts/Makefile.build:297: /var/lib/dkms/nvidia/525.125.06/build/nvidia/nv.o] Error 1

Autoconf reports a compiler version mismatch:

warning: the compiler differs from the one used to build the kernel
  The kernel was built by: gcc (Ubuntu 11.3.0-1ubuntu1~22.04.1) 11.3.0
  You are using:           gcc (Ubuntu 9.5.0-1ubuntu1~22.04) 9.5.0

This is caused by our docker file to update-alternative and set gcc default version to 9.5. Setting the default version back to 11 (even though the installed one is 11.4) solved the problem locally.

@kurapov-peter
Copy link
Contributor Author

Now it is fatal error: 'cstdint' file not found. We already saw this some time ago when we tried to update gcc. The stupid solution was to install g++-12. I tried installing libstdc++ separately and clang finds it after installation, but cstdint is still missing.

@kurapov-peter
Copy link
Contributor Author

The build issue is resolved. Fun fact: cuda docker build now needs ~25 mins less to complete (2/3 of the usual time).

Copy link
Contributor

@leshikus leshikus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@leshikus
Copy link
Contributor

@kurapov-peter

p.s. why do we have Dockerfile.clang that is identical to Dockerfile.cuda?

This reflects that the configurations match

@kurapov-peter kurapov-peter merged commit 92e5eb1 into main Aug 10, 2023
@kurapov-peter kurapov-peter deleted the pakurapo/l0-driver-update branch August 10, 2023 14:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants