Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update for CUDA 12.8.0 #33

Merged
merged 7 commits into from
Jan 28, 2025
Merged

Conversation

adibbley
Copy link
Contributor

@conda-forge-admin
Copy link
Contributor

conda-forge-admin commented Jan 23, 2025

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

I do have some suggestions for making it better though...

For recipe/meta.yaml:

  • ℹ️ The recipe is not parsable by parser conda-souschef (grayskull). This parser is not currently used by conda-forge, but may be in the future. We are collecting information to see which recipes are compatible with grayskull.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/13016619362. Examine the logs at this URL for more detail.

@adibbley
Copy link
Contributor Author

@conda-forge-admin, please rerender

@adibbley
Copy link
Contributor Author

@conda-forge-admin, please restart ci

@adibbley adibbley marked this pull request as ready for review January 23, 2025 22:47
@adibbley adibbley requested a review from a team as a code owner January 23, 2025 22:47
@adibbley adibbley added the automerge Merge the PR when CI passes label Jan 23, 2025
@conda-forge-admin
Copy link
Contributor

conda-forge-admin commented Jan 23, 2025

Hi! This is the friendly conda-forge automerge bot!

I considered the following status checks when analyzing this PR:

  • linter: passed
  • azure: passed

Thus the PR was passing and merged! Have a great day!

@adibbley adibbley removed the automerge Merge the PR when CI passes label Jan 23, 2025
@adibbley adibbley marked this pull request as draft January 23, 2025 23:12
@adibbley
Copy link
Contributor Author

@conda-forge-admin, please rerender

@adibbley adibbley marked this pull request as ready for review January 24, 2025 22:13
@adibbley adibbley added the automerge Merge the PR when CI passes label Jan 24, 2025
@jakirkham
Copy link
Member

Seeing the following error on CI (also attached log for posterity)

+ /home/conda/feedstock_root/build_artifacts/libcublas_1737758744897/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placeh/bin/x86_64-conda-linux-gnu-gcc -I/home/conda/feedstock_root/build_artifacts/libcublas_1737758744897/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placeh/targets/x86_64-linux/include -L/home/conda/feedstock_root/build_artifacts/libcublas_1737758744897/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placeh/targets/x86_64-linux/lib -lcublas /home/conda/recipe_root/test.c
Compiler: $PREFIX/bin/x86_64-conda-linux-gnu-gcc
/home/conda/feedstock_root/build_artifacts/libcublas_1737758744897/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placeh/bin/../lib/gcc/x86_64-conda-linux-gnu/13.3.0/../../../../x86_64-conda-linux-gnu/bin/ld: /home/conda/feedstock_root/build_artifacts/libcublas_1737758744897/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placeh/targets/x86_64-linux/lib/libcublasLt.so.12: undefined reference to `log2f@GLIBC_2.27'
/home/conda/feedstock_root/build_artifacts/libcublas_1737758744897/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placeh/bin/../lib/gcc/x86_64-conda-linux-gnu/13.3.0/../../../../x86_64-conda-linux-gnu/bin/ld: /home/conda/feedstock_root/build_artifacts/libcublas_1737758744897/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placeh/targets/x86_64-linux/lib/libcublasLt.so.12: undefined reference to `__cxa_thread_atexit_impl@GLIBC_2.18'

Looks like Linux x86_64 needs GLIBC 2.28

@jakirkham
Copy link
Member

Was wondering why we don't see the same error on Linux aarch64. However it looks like we are using this workaround

- sysroot_linux-aarch64 2.28 # [aarch64] # needed to compile test.c for aarch64 as some symbols are introduced in glibc 2.27

@vyasr
Copy link
Contributor

vyasr commented Jan 25, 2025

Should we be setting the c stdlib version in the CBC then? The file is pretty empty right now.

@jakirkham
Copy link
Member

We need to make several changes here. Unfortunately need to call it for the day. We will need to pick this up Monday

@jakirkham
Copy link
Member

jakirkham commented Jan 27, 2025

Submitted fixes in PR: #34

These are against CUDA 12.6. Once those go in, we can make one line change to conda_build_config.yaml and re-render

@conda-forge-curator conda-forge-curator bot removed the automerge Merge the PR when CI passes label Jan 28, 2025
@conda-forge-admin
Copy link
Contributor

Hi! This is the friendly conda-forge automerge bot!

Commits were made to this PR after the automerge label was added. For security reasons, I have disabled automerge by removing the automerge label. Please add the automerge label again (or ask a maintainer to do so) if you'd like to enable automerge again!

@jakirkham
Copy link
Member

@conda-forge-admin , please re-render

Copy link
Member

@jakirkham jakirkham left a comment

Choose a reason for hiding this comment

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

Merged the aforementioned GLIBC fixes in PR: #34

Have merged in the latest from main into this PR and resolved conflicts

Bumped Linux x86_64's GLIBC version to 2.28 based on our observations above (please see comments below for details). Also re-rendered to propagate this info through

This should fix the test failure we saw before unblocking this PR

@@ -2,4 +2,4 @@ arm_variant_type: # [aarch64]
- sbsa # [aarch64]
c_stdlib_version: # [linux]
- 2.28 # [linux and aarch64]
- 2.17 # [linux and x86_64]
- 2.28 # [linux and x86_64]
Copy link
Member

Choose a reason for hiding this comment

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

Have set Linux x86_64 to use GLIBC 2.28 based on our observations above: #33 (comment)

@@ -5,7 +5,7 @@ c_compiler_version:
c_stdlib:
- sysroot
c_stdlib_version:
- '2.17'
- '2.28'
Copy link
Member

Choose a reason for hiding this comment

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

Can now see GLIBC 2.28 is used after a re-render


build:
number: 1
number: 0
Copy link
Member

Choose a reason for hiding this comment

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

Reset the build/number as it was bumped in the previous PR

@adibbley adibbley added the automerge Merge the PR when CI passes label Jan 28, 2025
@conda-forge-admin conda-forge-admin merged commit 7f3ccae into conda-forge:main Jan 28, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge the PR when CI passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants