The patch titled Subject: selftests/mm: uffd-*: fix all type mismatch warnings has been added to the -mm mm-unstable branch. Its filename is selftests-mm-uffd-fix-all-type-mismatch-warnings.patch This patch will shortly appear at https://git.kernel.org/pub/scm/linux/kernel/git/akpm/25-new.git/tree/patches/selftests-mm-uffd-fix-all-type-mismatch-warnings.patch This patch will later appear in the mm-unstable branch at git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm Before you just go and hit "reply", please: a) Consider who else should be cc'ed b) Prefer to cc a suitable mailing list as well c) Ideally: find the original patch on the mailing list and do a reply-to-all to that, adding suitable additional cc's *** Remember to use Documentation/process/submit-checklist.rst when testing your code *** The -mm tree is included into linux-next via the mm-everything branch at git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm and is updated there every 2-3 working days ------------------------------------------------------ From: Muhammad Usama Anjum <usama.anjum@xxxxxxxxxxxxx> Subject: selftests/mm: uffd-*: fix all type mismatch warnings Date: Thu, 9 Jan 2025 22:38:41 +0500 Fix all type mismatch warnings in all uffd-* files. Link: https://lkml.kernel.org/r/20250109173842.1142376-16-usama.anjum@xxxxxxxxxxxxx Signed-off-by: Muhammad Usama Anjum <usama.anjum@xxxxxxxxxxxxx> Cc: Andy Lutomirski <luto@xxxxxxxxxxxxxx> Cc: Jérôme Glisse <jglisse@xxxxxxxxxx> Cc: Kees Cook <kees@xxxxxxxxxx> Cc: Shuah Khan <shuah@xxxxxxxxxx> Cc: Will Drewry <wad@xxxxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- tools/testing/selftests/mm/uffd-common.c | 6 +++--- tools/testing/selftests/mm/uffd-stress.c | 2 +- tools/testing/selftests/mm/uffd-unit-tests.c | 6 +++--- 3 files changed, 7 insertions(+), 7 deletions(-) --- a/tools/testing/selftests/mm/uffd-common.c~selftests-mm-uffd-fix-all-type-mismatch-warnings +++ a/tools/testing/selftests/mm/uffd-common.c @@ -450,7 +450,7 @@ void uffd_handle_page_fault(struct uffd_ args->wp_faults++; } else if (msg->arg.pagefault.flags & UFFD_PAGEFAULT_FLAG_MINOR) { uint8_t *area; - int b; + unsigned int b; /* * Minor page faults @@ -621,7 +621,7 @@ int __copy_page(int ufd, unsigned long o err("UFFDIO_COPY error: %"PRId64, (int64_t)uffdio_copy.copy); wake_range(ufd, uffdio_copy.dst, page_size); - } else if (uffdio_copy.copy != page_size) { + } else if (uffdio_copy.copy != (signed long)page_size) { err("UFFDIO_COPY error: %"PRId64, (int64_t)uffdio_copy.copy); } else { if (test_uffdio_copy_eexist && retry) { @@ -655,7 +655,7 @@ int move_page(int ufd, unsigned long off err("UFFDIO_MOVE error: %"PRId64, (int64_t)uffdio_move.move); wake_range(ufd, uffdio_move.dst, len); - } else if (uffdio_move.move != len) { + } else if (uffdio_move.move != (signed long)len) { err("UFFDIO_MOVE error: %"PRId64, (int64_t)uffdio_move.move); } else return 1; --- a/tools/testing/selftests/mm/uffd-stress.c~selftests-mm-uffd-fix-all-type-mismatch-warnings +++ a/tools/testing/selftests/mm/uffd-stress.c @@ -77,7 +77,7 @@ static void usage(void) static void uffd_stats_reset(struct uffd_args *args, unsigned long n_cpus) { - int i; + unsigned int i; for (i = 0; i < n_cpus; i++) { args[i].cpu = i; --- a/tools/testing/selftests/mm/uffd-unit-tests.c~selftests-mm-uffd-fix-all-type-mismatch-warnings +++ a/tools/testing/selftests/mm/uffd-unit-tests.c @@ -917,7 +917,7 @@ static bool do_uffdio_zeropage(int ufd, else if (res != -EINVAL) err("UFFDIO_ZEROPAGE not -EINVAL"); } else if (has_zeropage) { - if (res != page_size) + if (res != (signed long)page_size) err("UFFDIO_ZEROPAGE unexpected size"); else retry_uffdio_zeropage(ufd, &uffdio_zeropage); @@ -949,7 +949,7 @@ uffd_register_detect_zeropage(int uffd, static void uffd_zeropage_test(uffd_test_args_t __attribute__((unused)) *args) { bool has_zeropage; - int i; + unsigned int i; has_zeropage = uffd_register_detect_zeropage(uffd, area_dst, page_size); if (area_dst_alias) @@ -997,7 +997,7 @@ static void do_uffdio_poison(int uffd, u if (ret) err("UFFDIO_POISON error: %"PRId64, (int64_t)res); - else if (res != page_size) + else if (res != (signed long)page_size) err("UFFDIO_POISON unexpected size: %"PRId64, (int64_t)res); } _ Patches currently in -mm which might be from usama.anjum@xxxxxxxxxxxxx are selftests-mm-thp_settings-remove-const-from-return-type.patch selftests-mm-pagemap_ioctl-fix-types-mismatches-shown-by-compiler-options.patch selftests-mm-mseal_test-remove-unused-variables.patch selftests-mm-mremap_test-remove-unused-variable-and-type-mismatches.patch selftests-mm-remove-argc-and-argv-unused-parameters.patch selftests-mm-fix-unused-parameter-warnings.patch selftests-mm-fix-type-mismatch-warnings.patch selftests-mm-kselftest_harness-fix-warnings.patch selftests-mm-cow-remove-unused-variables-and-fix-type-mismatch-errors.patch selftests-mm-hmm-tests-remove-always-false-expressions.patch selftests-mm-guard-pages-fix-type-mismatch-warnings.patch selftests-mm-hugetlb-madvise-fix-type-mismatch-issues.patch selftests-mm-hugepage-vmemmap-fix-type-mismatch-warnings.patch selftests-mm-hugetlb-read-hwpoison-fix-type-mismatch-warnings.patch selftests-mm-khugepaged-fix-type-mismatch-warnings.patch selftests-mm-protection_keys-fix-variables-types-mismatch-warnings.patch selftests-mm-thuge-gen-fix-type-mismatch-warnings.patch selftests-mm-uffd-fix-all-type-mismatch-warnings.patch selftests-mm-makefile-add-the-compiler-flags.patch