On 27/03/2023 17:05, Teo Couprie Diaz wrote:
Add a set of skip files for running the syscalls set of tests in compat or purecap:
- syscalls_morello_musl_skip skips all Musl related failures,
- syscalls_morello_skip is for skipped and not built tests in compat,
Should be updated too I suppose.
- syscalls_morello_purecap_skip is for tests skipped in purecap.
Update the README to reflect those additions and explain how to use the skip files.
This is preparation for CI testing in Morello release 1.6 and later.
Signed-off-by: Teo Couprie Diaz teo.coupriediaz@arm.com
v5:
- Clarified the wording around syscalls_morello_skip thanks to Kevin's comments.
- Added command line example for purecap skip files.
v4:
- Some README rephrasing to address Beata's comments
- Changed kernel version dependency comment in the musl skip file
- Renamed all skip files to accomodate for a purecap one
- Added purecap skip file
- Updated README and commit message for purecap skip file
v3:
- Added *_16 tests to the skip list
- Added tests from kernel/security as we don't build them yet
- Mentioned Morello 1.6 release
v2:
- Removed mentions of COMPAT
- Reworded comment around Alpine
- Added bpf_prog02 to the musl skips after further testing
README.rst | 30 +++++ runtest/syscalls_morello_musl_skip | 18 +++ runtest/syscalls_morello_purecap_skip | 8 ++ runtest/syscalls_morello_skip | 172 ++++++++++++++++++++++++++ 4 files changed, 228 insertions(+) create mode 100644 runtest/syscalls_morello_musl_skip create mode 100644 runtest/syscalls_morello_purecap_skip create mode 100644 runtest/syscalls_morello_skip
diff --git a/README.rst b/README.rst index 8b6e3a7769d0..73edda443cc4 100644 --- a/README.rst +++ b/README.rst @@ -136,6 +136,36 @@ Running tests: runltp -f morello_transitional,morello_transitional_extended +Skipping tests: +---------------
+In certain cases, you might want to skip tests that are known to be failing or are yet to be supported.
+This is done by passing a file containing the names of the tests to be skip with the option ``-S`` to +``runltp``.
+We currently provide the following skip files under ``runtest/``:
- ``syscalls_morello_musl_skip``: Skips all tests failing on a regular AArch64 Musl based system.
- ``syscalls_morello_skip``: Skips all tests not considered due to the Morello build, or the run environment.
"Morello build" on its own sounds like there are defects in the build output, when really the issue is simply that we are not building some tests at all. "Morello build setup" maybe?
- ``syscalls_morello_purecap_skip``: Skips all tests not considered while testing with purecap binaries.
+``runltp`` does not support passing multiple skip files, however this can be circumvented by using +process substitution.
+.. code-block::
- ./runltp -f syscalls -S <( cat runtest/syscalls_morello{,_musl}_skip )
+To use all relevant skip files relevant for plain AArch64.
+.. code-block::
- ./runltp -f syscalls -S <( cat runtest/syscalls_morello{,_musl,_purecap}_skip )
+To use all relevant skip files relevant for plain pure-capability.
"plain pure-capability" is new terminology :) Just "pure-capability"?
+In order for the skip files to successfully filter-out expected/known failures in AArch64, +morello-release-1.6.0 release, or newer is required.
I think that one might be confusing because it's unclear which component that requirement applies to, especially since the files we're adding require release 1.5 for the kernel and Musl, not 1.6.
Developers playground
diff --git a/runtest/syscalls_morello_musl_skip b/runtest/syscalls_morello_musl_skip new file mode 100644 index 000000000000..aa3dff6c9449 --- /dev/null +++ b/runtest/syscalls_morello_musl_skip @@ -0,0 +1,18 @@ +# All the following tests from the syscalls test list are failing in Aarch64 +# Musl-based systems (Alpine, Debian...). Thus they depend on Musl or LTP, +# rather than the kernel, to be fixed. +# MUSL - Depends on Musl release > morello-release-1.5.0 +bpf_prog02 +gethostbyname_r01 +mq_open01 +nftw01 +nftw6401 +open13 +perf_event_open02 +recvmmsg01 +sbrk01 +sendmsg01 +setgroups03 +sigrelse01 +sigtimedwait01 +sigwaitinfo01 diff --git a/runtest/syscalls_morello_purecap_skip b/runtest/syscalls_morello_purecap_skip new file mode 100644 index 000000000000..e4d66ee1b27f --- /dev/null +++ b/runtest/syscalls_morello_purecap_skip @@ -0,0 +1,8 @@ +# This skip file contains all the tests skipped from the syscalls command file +# for testing the PCuABI kernel in purecap.
+# These tests share capabilities between proccesses through shared memory. +# This is explicitly prevented by the PCuABI, so there's no point running them. +process_vm_readv02 +process_vm_readv03 +process_vm_writev02 diff --git a/runtest/syscalls_morello_skip b/runtest/syscalls_morello_skip new file mode 100644 index 000000000000..b9c0a2479ca9 --- /dev/null +++ b/runtest/syscalls_morello_skip @@ -0,0 +1,172 @@ +# This skip file contains all the tests skipped from the syscalls command file +# because of either the build for Morello or the system the tests are run with.
Ditto here, would say "build setup".
Other than these superficial comments, the patch looks good and I confirmed locally that all the expected tests are passing.
Kevin