| Message ID | 20201026121248.2340-4-vincenzo.frascino@arm.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 386cf789fa6dc4e0f35d25cba9469d62321e7180 |
| Headers | show |
| Series | kselftest/arm64: MTE fixes | expand |
On 10/26/20 5:42 PM, Vincenzo Frascino wrote: > The check_child_memory test reports the error below because the test > plan is not declared correctly: > > # Planned tests != run tests (0 != 12) > > Fix the test adding the correct test plan declaration. This change is required and got missed earlier. Acked by: Amit Daniel Kachhap <amit.kachhap@arm.com> > > Fixes: dfe537cf4718 ("kselftest/arm64: Check forked child mte memory accessibility") > Cc: Shuah Khan <shuah@kernel.org> > Cc: Catalin Marinas <catalin.marinas@arm.com> > Cc: Will Deacon <will@kernel.org> > Cc: Gabor Kertesz <gabor.kertesz@arm.com> > Cc: Amit Daniel Kachhap <amit.kachhap@arm.com> > Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com> > --- > tools/testing/selftests/arm64/mte/check_child_memory.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/tools/testing/selftests/arm64/mte/check_child_memory.c b/tools/testing/selftests/arm64/mte/check_child_memory.c > index 97bebdecd29e..43bd94f853ba 100644 > --- a/tools/testing/selftests/arm64/mte/check_child_memory.c > +++ b/tools/testing/selftests/arm64/mte/check_child_memory.c > @@ -163,6 +163,9 @@ int main(int argc, char *argv[]) > mte_register_signal(SIGSEGV, mte_default_handler); > mte_register_signal(SIGBUS, mte_default_handler); > > + /* Set test plan */ > + ksft_set_plan(12); > + > evaluate_test(check_child_memory_mapping(USE_MMAP, MTE_SYNC_ERR, MAP_PRIVATE), > "Check child anonymous memory with private mapping, precise mode and mmap memory\n"); > evaluate_test(check_child_memory_mapping(USE_MMAP, MTE_SYNC_ERR, MAP_SHARED), >
diff --git a/tools/testing/selftests/arm64/mte/check_child_memory.c b/tools/testing/selftests/arm64/mte/check_child_memory.c index 97bebdecd29e..43bd94f853ba 100644 --- a/tools/testing/selftests/arm64/mte/check_child_memory.c +++ b/tools/testing/selftests/arm64/mte/check_child_memory.c @@ -163,6 +163,9 @@ int main(int argc, char *argv[]) mte_register_signal(SIGSEGV, mte_default_handler); mte_register_signal(SIGBUS, mte_default_handler); + /* Set test plan */ + ksft_set_plan(12); + evaluate_test(check_child_memory_mapping(USE_MMAP, MTE_SYNC_ERR, MAP_PRIVATE), "Check child anonymous memory with private mapping, precise mode and mmap memory\n"); evaluate_test(check_child_memory_mapping(USE_MMAP, MTE_SYNC_ERR, MAP_SHARED),
The check_child_memory test reports the error below because the test plan is not declared correctly: # Planned tests != run tests (0 != 12) Fix the test adding the correct test plan declaration. Fixes: dfe537cf4718 ("kselftest/arm64: Check forked child mte memory accessibility") Cc: Shuah Khan <shuah@kernel.org> Cc: Catalin Marinas <catalin.marinas@arm.com> Cc: Will Deacon <will@kernel.org> Cc: Gabor Kertesz <gabor.kertesz@arm.com> Cc: Amit Daniel Kachhap <amit.kachhap@arm.com> Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com> --- tools/testing/selftests/arm64/mte/check_child_memory.c | 3 +++ 1 file changed, 3 insertions(+)