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

ecs-gpu-init: clean up linker flags #3919

Closed
wants to merge 1 commit into from

Conversation

yeazelm
Copy link
Contributor

@yeazelm yeazelm commented Apr 26, 2024

This commit was cherry picked from #3887 since we took the SDK where the change was needed. This avoids building ecs-gpu-init with broken symbols which show up with these errors in the system log:

[   14.946273] ecs-gpu-init[1868]: /usr/bin/ecs-gpu-init: symbol lookup error: /usr/bin/ecs-gpu-init: undefined symbol: nvmlGpuInstanceGetComputeInstanceProfileInfoV
[FAILED] Failed to start Initialize ECS GPU config.
See 'systemctl status ecs-gpu-init.service' for details.
[DEPEND] Dependency failed for Bottlerocket initial configuration complete.
[DEPEND] Dependency failed for Isolates configured.target.

From the original commit:
--export-dynamic is a separate option, independent of -z, so pass it as its own -Wl flag.

For consistency, pass the same flags to the external linker.

Thanks for @bcressey for having this fix already under review!

Testing done:
Built an aws-ecs-2-nvidia image and it boots correctly and uses the GPU

Terms of contribution:

By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.

`--export-dynamic` is a separate option, independent of `-z`, so pass
it as its own `-Wl` flag.

For consistency, pass the same flags to the external linker.

Signed-off-by: Ben Cressey <bcressey@amazon.com>
(cherry picked from commit 182a188)
@yeazelm yeazelm changed the base branch from 1.19.x to develop April 26, 2024 21:23
@yeazelm yeazelm changed the base branch from develop to 1.19.x April 26, 2024 21:23
@yeazelm yeazelm closed this Apr 26, 2024
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.

None yet

2 participants