Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
TestKprobeOffset used to hardcode specific offsets to test against, which meant we had to make it amd64 specific. It turns out that even on a single arch the offsets aren't stable due to changes in the compiler. Hence the test was changed to brute force a valid offset, but we forgot to run it on all arches. Trying to execute the test on Linux 5.13 on arm64 triggers EINVAL instead of the expected EILSEQ however. This is because arm64 returns EINVAL when trying to use an offset that is not a multiple of four. Relying on EILSEQ is therefore not portable, and probably shouldn't be encouraged. Drop the check for EILSEQ from TestKprobeOffset and execute it on all arches.
- Loading branch information
Showing
5 changed files
with
23 additions
and
35 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters