Skip to content

Test all x86_64 configurations*...#212

Merged
fgvanzee merged 20 commits intodevfrom
travis-virtualization
May 29, 2018
Merged

Test all x86_64 configurations*...#212
fgvanzee merged 20 commits intodevfrom
travis-virtualization

Conversation

@devinamatthews
Copy link
Member

@devinamatthews devinamatthews commented May 26, 2018

...using Intel SDE. Some things to note:

  • gcc-6 and binutils-2.26 must be loaded (and the system ld and as replaced with newer versions) so that no configurations are blacklisted.
  • sudo: required is needed so that SDE will run.
  • Dropbox is used to host SDE with a secret API token (don't echo it!).
  • Because glibc/libm use cpuid to do their own runtime detection, care must be taken to disable this for the AMD configs, otherwise FMA4 instructions are executed which SDE doesn't do. A binary patch is applied to the system ld.so, libc.so, and libm.so.
  • except bulldozer

devinamatthews and others added 18 commits May 23, 2018 11:31
@devinamatthews
Add custom SDE cpuid files.
7866314
@devinamatthews
Set up testing of all x86_64 architectures (except bulldozer) using SDE.
aa8679c
@devinamatthews
Merge remote-tracking branch 'upstream/dev' into travis-virtualization
0bf4cec
@devinamatthews
Update .travis.yml
ae67b96 
[ci skip]
@devinamatthews
Update do_testsuite.sh
66cc4a4 
[ci skip]
@fgvanzee
Updated .travis.yml with my secret token.
e27e1dd 
Details:
- Replaced Devin's temporary secret token with my own, which is used by
  Travis when accessing the Intel SDE via Dropbox.
@devinamatthews
Work around CPUID dispatch in glibc/libm by patching ld.so.
bb1cd71
@devinamatthews
Detect path of loader at runtime.
3044508
@devinamatthews
Attempt to make SDE run on Travis
4c76a17
@devinamatthews
Allow unpatched ld.so if we don't know how to patch it.
85aed51 
I *think* this only happens for older glibc without the multi-arch stuff (e.g. Ubuntu 14.04 on Travis), but who knows?
@devinamatthews
Upgrade Travis to gcc-6 and binutils-2.26.
1e6be6b
@devinamatthews
Try to get Travis to use the right assembler.
25baafa
@devinamatthews
Apparently you need ld-2.26 too.
d90e298
@devinamatthews
Try to also patch ld.so from Ubuntu 14.04.
aecf386
@devinamatthews
Take the nuclear option.
f6ba59d
@devinamatthews
Account for non-absolute dependencies in ldd output.
a2d6d26
@devinamatthews
String manipulation fail.
679f796
@devinamatthews
Update patch-ld-so.py
ab70a87
@devinamatthews devinamatthews mentioned this pull request May 26, 2018
Closed
@devinamatthews
Copy link
Member Author

devinamatthews commented May 29, 2018

@fgvanzee if you'll take a look to familiarize yourself with the changes then I'll merge to dev.

@devinamatthews devinamatthews requested a review from fgvanzee May 29, 2018 15:17
devinamatthews and others added 2 commits May 29, 2018 12:11
@devinamatthews
Add Zen to SDE testing.
d7f531b
@fgvanzee
Removed dead variable from travis/do_testsuite.sh.
0a92881 
Details:
- Removed 'BLIS_ENABLE_TEST_OUTPUT=yes' from make invocations in
  travis/do_testsuite.sh. This variable is no longer present in the
  BLIS build system (if it ever was?), and therefore has no effect.
@fgvanzee fgvanzee merged commit 850a8a4 into dev May 29, 2018
@fgvanzee fgvanzee deleted the travis-virtualization branch May 29, 2018 18:51
@fgvanzee
Copy link
Member

fgvanzee commented May 29, 2018

Thanks @devinamatthews and @jeffhammond for all your work on this. (Travis CI is being ornery right now and putting out false test fails, but we're confident that it will pass once the service is working again.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fgvanzee fgvanzee fgvanzee approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@devinamatthews @fgvanzee