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

Test playTests fails on ppc64 (openSUSE Tumbleweed) #3125

Closed
jaimeMF opened this issue May 7, 2022 · 4 comments · Fixed by #3392
Closed

Test playTests fails on ppc64 (openSUSE Tumbleweed) #3125

jaimeMF opened this issue May 7, 2022 · 4 comments · Fixed by #3392
Assignees

Comments

@jaimeMF
Copy link

jaimeMF commented May 7, 2022

I was trying to use cmake for building the RPM for openSUSE and the test playTests fails on ppc64. Here is the log of the test and here is the full build log, from https://build.opensuse.org/package/live_build_log/home:jaimeMF:branches:Archiving/zstd/openSUSE_Factory_PowerPC/ppc64 . The test pass on ppc64le and ppc ( https://build.opensuse.org/package/show/home:jaimeMF:branches:Archiving/zstd ).

I don't have a ppc machine, but I algo get the error using qemu.

@Cyan4973
Copy link
Contributor

Cyan4973 commented May 10, 2022

The following log lines are suspicious :

[  374s] test: check if binary has executable stack (#2963)
[  374s] zstd binary has executable stack!

though why would this test fail on ppc64 only, and not ppc64le, is a mystery ...
edit : maybe the ppc64le version does not employ cmake ?

@jaimeMF
Copy link
Author

jaimeMF commented May 10, 2022

edit : maybe the ppc64le version does not employ cmake ?

In the modified RPM all are built with cmake and they all seem to run the same tests.

@terrelln terrelln self-assigned this Dec 21, 2022
@terrelln
Copy link
Contributor

Basically, I've found out that on ppc64 with ELFv1 gcc doesn't emit stack notes by default. But in ELFv2 it does. See

https://patches.linaro.org/project/binutils/patch/1506025575-1559-1-git-send-email-jim.wilson@linaro.org/

So this is likely a test issue, not a build issue.

However, it seems prudent to mark as noexecstack anways, so I've put up a PR that adds the noexecstack compiler flags that we use in make to cmake.

@jaimeMF
Copy link
Author

jaimeMF commented Dec 24, 2022

Thank you for the fix.

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 a pull request may close this issue.

3 participants