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

riscv64 test segfault with GCC 14 #1222

Closed
marxin opened this issue Mar 20, 2024 · 7 comments
Closed

riscv64 test segfault with GCC 14 #1222

marxin opened this issue Mar 20, 2024 · 7 comments

Comments

@marxin
Copy link
Contributor

marxin commented Mar 20, 2024

The following failures happen with the latest compiler which might be the culprit of the troubles. All binaries segfault:

[ 5403s] The following tests FAILED:
[ 5403s] 	115 - riscv64-ifunc-static-pie (Failed)
[ 5403s] 	202 - riscv64-section-order (Failed)
[ 5403s] 	214 - riscv64-static-pie (Failed)

Binaries:
riscv64.tar.gz

@rui314
Copy link
Owner

rui314 commented Mar 21, 2024

Could you share the entire test directory?

@marxin
Copy link
Contributor Author

marxin commented Mar 21, 2024

Note I don't have a direct access to the machine, but I need to use xxd to dump the file contents. Please list me files you are interested in?

@marxin
Copy link
Contributor Author

marxin commented Mar 24, 2024

@rui314 ping

rui314 added a commit that referenced this issue Mar 25, 2024
RISC-V psABI requires that symbol to be exported from an executable
if there's a GP-relative reference. For simplicity, we always export
it from executable as long as it has a .dynamic section.

#1222
@rui314
Copy link
Owner

rui314 commented Mar 25, 2024

The above change may fix the issue. At least it seems related.

@marxin
Copy link
Contributor Author

marxin commented Mar 26, 2024

Apparently, it hasn't helped with the mentioned commit: https://build.opensuse.org/package/live_build_log/devel:gcc:next/mold/openSUSE_Factory_RISCV/riscv64.
Maybe @andreas-schwab can help you with that?

@andreas-schwab
Copy link
Contributor

See dd1a251

@marxin
Copy link
Contributor Author

marxin commented Mar 26, 2024

Ok, so it's addressed with #1206, am I correct? Can you please address also the other tests mentioned in this issue?

@marxin marxin closed this as completed Mar 29, 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

No branches or pull requests

3 participants