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

Memory tests fail in PIC mode #40

Open
hainest opened this issue Feb 12, 2019 · 3 comments
Open

Memory tests fail in PIC mode #40

hainest opened this issue Feb 12, 2019 · 3 comments
Assignees
Labels

Comments

@hainest
Copy link
Contributor

hainest commented Feb 12, 2019

Building commit e5ec233. The main error is

relocation R_X86_64_32 against `.text' can not be used when making a PIE object; recompile with -fPIC
/usr/bin/ld: final link failed: Nonrepresentable section on output

See gist for output details, ld, and gcc versions. This is the first time I've had the build completely fail. This my first time building on 'cayenne.cs.wisc.edu', though.

@mxz297
Copy link
Member

mxz297 commented Feb 12, 2019

I know what's going on, but I don't know how to fix this.

File test6LS-x86_64.s is a test written by in assembly, which apparently is not PIC. In newer linux systems, PIE is the default. So, when you compile the test suite, the linker will not be able to link our non-PIC hand written assembly with other PIC objects.

Two things to do to fix the problem: (1) when we compiler non-pic tests, we need to force the compiler to generate non-pic code. (2) disable pic tests for such human written disassembly or provide a pic version of test6LS-x86_64.s.

A third thing I just find: currently when we compile pic tests, the generated tests are still non-pic ...

@hainest
Copy link
Contributor Author

hainest commented Mar 6, 2019

mem_tests were removed in #41. We need to fix them and get them back into the test suite for 10.1.

@hainest hainest self-assigned this Mar 6, 2019
@hainest hainest added bug v10.2-release Required for the v10.2 release. labels Mar 7, 2019
@hainest
Copy link
Contributor Author

hainest commented Mar 12, 2019

This was seen previously in #23.

@hainest hainest changed the title Build fails for test_mem_7.dyn_gcc_64_none_none Memory tests fail in PIC mode Mar 12, 2019
@hainest hainest removed the v10.2-release Required for the v10.2 release. label Oct 16, 2019
@bigtrak bigtrak assigned bigtrak and unassigned hainest Jul 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants