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

AArch64 SVE: Add 128 bit vector length tests to CI #6777

Open
AssadHashmi opened this issue Apr 17, 2024 · 0 comments
Open

AArch64 SVE: Add 128 bit vector length tests to CI #6777

AssadHashmi opened this issue Apr 17, 2024 · 0 comments

Comments

@AssadHashmi
Copy link
Contributor

The current SVE tests (aarch64-precommit (ubuntu-20-arm64-sve)) run on hardware which has a 256 bit vector length (VL) as a thread default, the maximum on that hardware. The fix for #6760 exposed lack of test coverage based on different VLs.

SVE hardware and SVE support in the Linux kernel enables users to select VLs shorter than the maximum. We can use the same hardware and the prctl(2) syscall to set VL to 128 to defend changes in VL sensitive code paths. This would be in addition to the existing tests using VL of 256 bits.

joshua-warburton added a commit that referenced this issue May 9, 2024
This patch configures the github CI to spawn 3 AArch64 precommit jobs
without sve, with sve 256 and with sve 128. This is accomplished by
a small python program which makes a call to set the vector length
and an update to the job matrix to spawn the appropriate jobs.

issue: #6777
Change-Id: I238d883cd805e0a643e392fe481a0abf9359a058
joshua-warburton added a commit that referenced this issue May 10, 2024
Job names were changed to be in human readable format. This
patch changes them to be more consistent with the autogenerated
names used by all the other CI jobs.

issue: #6777

Change-Id: I3c8f7d9575d0def487a07022bbf116d9bfc75bc8
AssadHashmi pushed a commit that referenced this issue May 13, 2024
Job names were changed to be in human readable format. This patch
changes them to be more consistent with the autogenerated names used by
all the other CI jobs.

issue: #6777
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant