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

powerpc64-execute-only (Failed) on ppc64 #989

Closed
marxin opened this issue Feb 14, 2023 · 7 comments
Closed

powerpc64-execute-only (Failed) on ppc64 #989

marxin opened this issue Feb 14, 2023 · 7 comments

Comments

@marxin
Copy link
Contributor

marxin commented Feb 14, 2023

Must be a recent regression (last successful build was from ~3 days ago):

[  284s] ++ code=1
[  284s] ++ echo 'command failed: 15: grep -q '\''Hello world'\'''
[  284s] command failed: 15: grep -q 'Hello world'
[  284s] ++ trap - EXIT
@rui314
Copy link
Owner

rui314 commented Feb 14, 2023

Do you mind if I ask you to bisect? The test doesn't fail in my ppc64 qemu environment.

@marxin
Copy link
Contributor Author

marxin commented Feb 14, 2023

It's been there since 45bece4 and I get a Segmentation fault when I run $QEMU $t/exe in the mentioned test-case.

@marxin
Copy link
Contributor Author

marxin commented Feb 14, 2023

[  500s] + gdb -batch -ex run out/test/elf/ppc64/execute-only/exe
[  500s] [Thread debugging using libthread_db enabled]
[  500s] Using host libthread_db library "/lib64/libthread_db.so.1".
[  500s] 
[  500s] Program received signal SIGSEGV, Segmentation fault.
[  500s] 0x00000001000106bc in __glink_PLTresolve ()

@rui314
Copy link
Owner

rui314 commented Feb 14, 2023

Maybe that test has been broken since the beginning but hadn't been executed until the recent change to the cmakefile. Can you share the executable file with me?

Maybe we can just disable the test on ppc64 because no one probably want to use the modern -execute-only option on ppc64v1 though.

@marxin
Copy link
Contributor Author

marxin commented Feb 14, 2023

Maybe that test has been broken since the beginning but hadn't been executed until the recent change to the cmakefile. Can you share the executable file with me?

Sure.
ppc64.out.gz

@rui314 rui314 closed this as completed in a42626b Feb 15, 2023
@rui314
Copy link
Owner

rui314 commented Feb 15, 2023

Thanks. PPC64 .text segment contains not only code but also data, so it wasn't compatible with -execute-only.

@marxin
Copy link
Contributor Author

marxin commented Feb 15, 2023

Good, thanks for the investigation.

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

2 participants