| Message ID | 20250221093331.2184245-2-catalin.marinas@arm.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | kselftest/arm64: mte: Minor fixes to the MTE hugetlb test | expand |
On 2/21/25 1:33 AM, Catalin Marinas wrote: > The architecture doesn't define precise/imprecise MTE tag check modes, > only synchronous and asynchronous. Use the correct naming and also > ensure they match the MTE_{ASYNC,SYNC}_ERR type. > > Fixes: 27879e8cb6b0 ("selftests: arm64: add hugetlb mte tests") > Cc: Yang Shi <yang@os.amperecomputing.com> > Signed-off-by: Catalin Marinas <catalin.marinas@arm.com> > --- > tools/testing/selftests/arm64/mte/check_hugetlb_options.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) Reviewed-by: Yang Shi <yang@os.amperecomputing.com> > > diff --git a/tools/testing/selftests/arm64/mte/check_hugetlb_options.c b/tools/testing/selftests/arm64/mte/check_hugetlb_options.c > index 303260a6dc65..11f812635b51 100644 > --- a/tools/testing/selftests/arm64/mte/check_hugetlb_options.c > +++ b/tools/testing/selftests/arm64/mte/check_hugetlb_options.c > @@ -270,13 +270,13 @@ int main(int argc, char *argv[]) > "Check clear PROT_MTE flags with private mapping and sync error mode and mmap/mprotect memory\n"); > > evaluate_test(check_child_hugetlb_memory_mapping(USE_MMAP, MTE_SYNC_ERR, MAP_PRIVATE | MAP_HUGETLB), > - "Check child hugetlb memory with private mapping, precise mode and mmap memory\n"); > + "Check child hugetlb memory with private mapping, sync error mode and mmap memory\n"); > evaluate_test(check_child_hugetlb_memory_mapping(USE_MMAP, MTE_ASYNC_ERR, MAP_PRIVATE | MAP_HUGETLB), > - "Check child hugetlb memory with private mapping, precise mode and mmap memory\n"); > + "Check child hugetlb memory with private mapping, async error mode and mmap memory\n"); > evaluate_test(check_child_hugetlb_memory_mapping(USE_MPROTECT, MTE_SYNC_ERR, MAP_PRIVATE | MAP_HUGETLB), > - "Check child hugetlb memory with private mapping, precise mode and mmap/mprotect memory\n"); > + "Check child hugetlb memory with private mapping, sync error mode and mmap/mprotect memory\n"); > evaluate_test(check_child_hugetlb_memory_mapping(USE_MPROTECT, MTE_ASYNC_ERR, MAP_PRIVATE | MAP_HUGETLB), > - "Check child hugetlb memory with private mapping, precise mode and mmap/mprotect memory\n"); > + "Check child hugetlb memory with private mapping, async error mode and mmap/mprotect memory\n"); > > mte_restore_setup(); > free_hugetlb();
diff --git a/tools/testing/selftests/arm64/mte/check_hugetlb_options.c b/tools/testing/selftests/arm64/mte/check_hugetlb_options.c index 303260a6dc65..11f812635b51 100644 --- a/tools/testing/selftests/arm64/mte/check_hugetlb_options.c +++ b/tools/testing/selftests/arm64/mte/check_hugetlb_options.c @@ -270,13 +270,13 @@ int main(int argc, char *argv[]) "Check clear PROT_MTE flags with private mapping and sync error mode and mmap/mprotect memory\n"); evaluate_test(check_child_hugetlb_memory_mapping(USE_MMAP, MTE_SYNC_ERR, MAP_PRIVATE | MAP_HUGETLB), - "Check child hugetlb memory with private mapping, precise mode and mmap memory\n"); + "Check child hugetlb memory with private mapping, sync error mode and mmap memory\n"); evaluate_test(check_child_hugetlb_memory_mapping(USE_MMAP, MTE_ASYNC_ERR, MAP_PRIVATE | MAP_HUGETLB), - "Check child hugetlb memory with private mapping, precise mode and mmap memory\n"); + "Check child hugetlb memory with private mapping, async error mode and mmap memory\n"); evaluate_test(check_child_hugetlb_memory_mapping(USE_MPROTECT, MTE_SYNC_ERR, MAP_PRIVATE | MAP_HUGETLB), - "Check child hugetlb memory with private mapping, precise mode and mmap/mprotect memory\n"); + "Check child hugetlb memory with private mapping, sync error mode and mmap/mprotect memory\n"); evaluate_test(check_child_hugetlb_memory_mapping(USE_MPROTECT, MTE_ASYNC_ERR, MAP_PRIVATE | MAP_HUGETLB), - "Check child hugetlb memory with private mapping, precise mode and mmap/mprotect memory\n"); + "Check child hugetlb memory with private mapping, async error mode and mmap/mprotect memory\n"); mte_restore_setup(); free_hugetlb();
The architecture doesn't define precise/imprecise MTE tag check modes, only synchronous and asynchronous. Use the correct naming and also ensure they match the MTE_{ASYNC,SYNC}_ERR type. Fixes: 27879e8cb6b0 ("selftests: arm64: add hugetlb mte tests") Cc: Yang Shi <yang@os.amperecomputing.com> Signed-off-by: Catalin Marinas <catalin.marinas@arm.com> --- tools/testing/selftests/arm64/mte/check_hugetlb_options.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)