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

Fix linking of tool on Linux AArch64 #222

Merged
merged 1 commit into from
Oct 2, 2024
Merged

Conversation

Ivan-Velickovic
Copy link
Collaborator

I specified the linker to aarch64-none-elf-ld because I would've thought that even though we are targeting Linux, the linker shouldn't matter but on Linux AArch64 we get a segfault when running the Microkit tool.

Changing the linker to aarch64-linux-gnu-ld fixes that.

I'm not really happy with this solution as we now have to install a new C toolchain just to compile a Rust program, from searching on the internet the solution seems to be to use the cross-rs package, which just uses Docker containers so isn't actually cross compiling anything.

If anyone has any better ideas please let me know.

I specified the linker to `aarch64-none-elf-ld` because I would've
thought that even though we are targeting Linux, the linker shouldn't
matter but on Linux AArch64 we get a segfault when running the Microkit
tool.

Changing the linker to `aarch64-linux-gnu-ld` fixes that.

I'm not really happy with this solution as we now have to install a new
C toolchain just to compile a Rust program, from searching on the
internet the solution seems to be to use the cross-rs package, which
just uses Docker containers so isn't actually cross compiling anything.

If anyone has any better ideas please let me know.

Signed-off-by: Ivan Velickovic <[email protected]>
@Ivan-Velickovic Ivan-Velickovic merged commit 7e72f0b into main Oct 2, 2024
11 checks passed
@Ivan-Velickovic Ivan-Velickovic deleted the linux_aarch64_fix branch October 2, 2024 03:37
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.

1 participant