| Message ID | 20201026121248.2340-3-vincenzo.frascino@arm.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 041fa41f5422ee275a8f60a182e1b32d196a1a2a |
| Headers | show |
| Series | kselftest/arm64: MTE fixes | expand |
On 10/26/20 5:42 PM, Vincenzo Frascino wrote: > The check_tags_inclusion test reports the error below because the test > plan is not declared correctly: > > # Planned tests != run tests (0 != 4) > > 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: f3b2a26ca78d ("kselftest/arm64: Verify mte tag inclusion via prctl") > 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_tags_inclusion.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/tools/testing/selftests/arm64/mte/check_tags_inclusion.c b/tools/testing/selftests/arm64/mte/check_tags_inclusion.c > index 94d245a0ed56..deaef1f61076 100644 > --- a/tools/testing/selftests/arm64/mte/check_tags_inclusion.c > +++ b/tools/testing/selftests/arm64/mte/check_tags_inclusion.c > @@ -170,6 +170,9 @@ int main(int argc, char *argv[]) > /* Register SIGSEGV handler */ > mte_register_signal(SIGSEGV, mte_default_handler); > > + /* Set test plan */ > + ksft_set_plan(4); > + > evaluate_test(check_single_included_tags(USE_MMAP, MTE_SYNC_ERR), > "Check an included tag value with sync mode\n"); > evaluate_test(check_multiple_included_tags(USE_MMAP, MTE_SYNC_ERR), >
diff --git a/tools/testing/selftests/arm64/mte/check_tags_inclusion.c b/tools/testing/selftests/arm64/mte/check_tags_inclusion.c index 94d245a0ed56..deaef1f61076 100644 --- a/tools/testing/selftests/arm64/mte/check_tags_inclusion.c +++ b/tools/testing/selftests/arm64/mte/check_tags_inclusion.c @@ -170,6 +170,9 @@ int main(int argc, char *argv[]) /* Register SIGSEGV handler */ mte_register_signal(SIGSEGV, mte_default_handler); + /* Set test plan */ + ksft_set_plan(4); + evaluate_test(check_single_included_tags(USE_MMAP, MTE_SYNC_ERR), "Check an included tag value with sync mode\n"); evaluate_test(check_multiple_included_tags(USE_MMAP, MTE_SYNC_ERR),
The check_tags_inclusion test reports the error below because the test plan is not declared correctly: # Planned tests != run tests (0 != 4) Fix the test adding the correct test plan declaration. Fixes: f3b2a26ca78d ("kselftest/arm64: Verify mte tag inclusion via prctl") 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_tags_inclusion.c | 3 +++ 1 file changed, 3 insertions(+)