| Message ID | 20201026121248.2340-5-vincenzo.frascino@arm.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 7419390a466e20d4a00cecdb4a68af47d0b21ca5 |
| Headers | show |
| Series | kselftest/arm64: MTE fixes | expand |
On 10/26/20 5:42 PM, Vincenzo Frascino wrote: > The check_mmap_options test reports the error below because the test > plan is not declared correctly: > > # Planned tests != run tests (0 != 22) > > 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: 53ec81d23213 ("kselftest/arm64: Verify all different mmap MTE options") > 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_mmap_options.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/tools/testing/selftests/arm64/mte/check_mmap_options.c b/tools/testing/selftests/arm64/mte/check_mmap_options.c > index 33b13b86199b..a04b12c21ac9 100644 > --- a/tools/testing/selftests/arm64/mte/check_mmap_options.c > +++ b/tools/testing/selftests/arm64/mte/check_mmap_options.c > @@ -205,7 +205,11 @@ int main(int argc, char *argv[]) > mte_register_signal(SIGBUS, mte_default_handler); > mte_register_signal(SIGSEGV, mte_default_handler); > > + /* Set test plan */ > + ksft_set_plan(22); > + > mte_enable_pstate_tco(); > + > evaluate_test(check_anonymous_memory_mapping(USE_MMAP, MTE_SYNC_ERR, MAP_PRIVATE, TAG_CHECK_OFF), > "Check anonymous memory with private mapping, sync error mode, mmap memory and tag check off\n"); > evaluate_test(check_file_memory_mapping(USE_MPROTECT, MTE_SYNC_ERR, MAP_PRIVATE, TAG_CHECK_OFF), >
diff --git a/tools/testing/selftests/arm64/mte/check_mmap_options.c b/tools/testing/selftests/arm64/mte/check_mmap_options.c index 33b13b86199b..a04b12c21ac9 100644 --- a/tools/testing/selftests/arm64/mte/check_mmap_options.c +++ b/tools/testing/selftests/arm64/mte/check_mmap_options.c @@ -205,7 +205,11 @@ int main(int argc, char *argv[]) mte_register_signal(SIGBUS, mte_default_handler); mte_register_signal(SIGSEGV, mte_default_handler); + /* Set test plan */ + ksft_set_plan(22); + mte_enable_pstate_tco(); + evaluate_test(check_anonymous_memory_mapping(USE_MMAP, MTE_SYNC_ERR, MAP_PRIVATE, TAG_CHECK_OFF), "Check anonymous memory with private mapping, sync error mode, mmap memory and tag check off\n"); evaluate_test(check_file_memory_mapping(USE_MPROTECT, MTE_SYNC_ERR, MAP_PRIVATE, TAG_CHECK_OFF),
The check_mmap_options test reports the error below because the test plan is not declared correctly: # Planned tests != run tests (0 != 22) Fix the test adding the correct test plan declaration. Fixes: 53ec81d23213 ("kselftest/arm64: Verify all different mmap MTE options") 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_mmap_options.c | 4 ++++ 1 file changed, 4 insertions(+)