From: Donet Tom <donettom@linux.ibm.com>
To: Mike Rapoport <rppt@kernel.org>,
Andrew Morton <akpm@linux-foundation.org>,
David Hildenbrand <david@kernel.org>
Cc: Baolin Wang <baolin.wang@linux.alibaba.com>,
Barry Song <baohua@kernel.org>, Dev Jain <dev.jain@arm.com>,
Jason Gunthorpe <jgg@ziepe.ca>,
John Hubbard <jhubbard@nvidia.com>,
"Liam R. Howlett" <Liam.Howlett@oracle.com>,
Lance Yang <lance.yang@linux.dev>,
Leon Romanovsky <leon@kernel.org>,
Lorenzo Stoakes <ljs@kernel.org>, Mark Brown <broonie@kernel.org>,
Michal Hocko <mhocko@suse.com>, Nico Pache <npache@redhat.com>,
Peter Xu <peterx@redhat.com>, Ryan Roberts <ryan.roberts@arm.com>,
Shuah Khan <shuah@kernel.org>,
Suren Baghdasaryan <surenb@google.com>,
Vlastimil Babka <vbabka@kernel.org>, Zi Yan <ziy@nvidia.com>,
linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org,
linux-mm@kvack.org
Subject: Re: [PATCH 14/53] selftests/mm: ksm_tests: use kselftest framework
Date: Tue, 14 Apr 2026 15:11:44 +0530 [thread overview]
Message-ID: <b49e883b-d666-49cb-bf4d-c83a8d92a0fe@linux.ibm.com> (raw)
In-Reply-To: <20260406141735.2179309-15-rppt@kernel.org>
Hi Mike
On 4/6/26 7:46 PM, Mike Rapoport wrote:
> From: "Mike Rapoport (Microsoft)" <rppt@kernel.org>
>
> Convert ksm_tests to use kselftest framework for reporting and tracking
> successful and failing runs.
>
> Signed-off-by: Mike Rapoport (Microsoft) <rppt@kernel.org>
> ---
> tools/testing/selftests/mm/ksm_tests.c | 180 +++++++++++--------------
> 1 file changed, 81 insertions(+), 99 deletions(-)
>
>
I think we missed one change
diff --git a/tools/testing/selftests/mm/ksm_tests.c
b/tools/testing/selftests/mm/ksm_tests.c
index 752a2c0d06ab..5219ee54577d 100644
--- a/tools/testing/selftests/mm/ksm_tests.c
+++ b/tools/testing/selftests/mm/ksm_tests.c
@@ -104,7 +104,7 @@ static void ksm_print_procfs(void)
if (!f) {
fprintf(stderr, "f %s\n", file_name);
- perror("fopen");
+ ksft_perror("fopen");
return;
}
Otherwise it looks good to me
Reviewed by: Donet Tom <donettom@linux.ibm.com>
-Donet
> diff --git a/tools/testing/selftests/mm/ksm_tests.c b/tools/testing/selftests/mm/ksm_tests.c
> index a0b48b839d54..752a2c0d06ab 100644
> --- a/tools/testing/selftests/mm/ksm_tests.c
> +++ b/tools/testing/selftests/mm/ksm_tests.c
> @@ -175,12 +175,12 @@ static void *allocate_memory(void *ptr, int prot, int mapping, char data, size_
> void *map_ptr = mmap(ptr, map_size, PROT_WRITE, mapping, -1, 0);
>
> if (!map_ptr) {
> - perror("mmap");
> + ksft_perror("mmap");
> return NULL;
> }
> memset(map_ptr, data, map_size);
> if (mprotect(map_ptr, map_size, prot)) {
> - perror("mprotect");
> + ksft_perror("mprotect");
> munmap(map_ptr, map_size);
> return NULL;
> }
> @@ -201,11 +201,11 @@ static int ksm_do_scan(int scan_count, struct timespec start_time, int timeout)
> if (ksm_read_sysfs(KSM_FP("full_scans"), &cur_scan))
> return 1;
> if (clock_gettime(CLOCK_MONOTONIC_RAW, &cur_time)) {
> - perror("clock_gettime");
> + ksft_perror("clock_gettime");
> return 1;
> }
> if ((cur_time.tv_sec - start_time.tv_sec) > timeout) {
> - printf("Scan time limit exceeded\n");
> + ksft_print_msg("Scan time limit exceeded\n");
> return 1;
> }
> }
> @@ -218,12 +218,12 @@ static int ksm_merge_pages(int merge_type, void *addr, size_t size,
> {
> if (merge_type == KSM_MERGE_MADVISE) {
> if (madvise(addr, size, MADV_MERGEABLE)) {
> - perror("madvise");
> + ksft_perror("madvise");
> return 1;
> }
> } else if (merge_type == KSM_MERGE_PRCTL) {
> if (prctl(PR_SET_MEMORY_MERGE, 1, 0, 0, 0)) {
> - perror("prctl");
> + ksft_perror("prctl");
> return 1;
> }
> }
> @@ -242,7 +242,7 @@ static int ksm_unmerge_pages(void *addr, size_t size,
> struct timespec start_time, int timeout)
> {
> if (madvise(addr, size, MADV_UNMERGEABLE)) {
> - perror("madvise");
> + ksft_perror("madvise");
> return 1;
> }
> return 0;
> @@ -324,7 +324,7 @@ static int check_ksm_merge(int merge_type, int mapping, int prot,
> struct timespec start_time;
>
> if (clock_gettime(CLOCK_MONOTONIC_RAW, &start_time)) {
> - perror("clock_gettime");
> + ksft_perror("clock_gettime");
> return KSFT_FAIL;
> }
>
> @@ -338,7 +338,6 @@ static int check_ksm_merge(int merge_type, int mapping, int prot,
>
> /* verify that the right number of pages are merged */
> if (assert_ksm_pages_count(page_count)) {
> - printf("OK\n");
> munmap(map_ptr, page_size * page_count);
> if (merge_type == KSM_MERGE_PRCTL)
> prctl(PR_SET_MEMORY_MERGE, 0, 0, 0, 0);
> @@ -346,7 +345,6 @@ static int check_ksm_merge(int merge_type, int mapping, int prot,
> }
>
> err_out:
> - printf("Not OK\n");
> munmap(map_ptr, page_size * page_count);
> return KSFT_FAIL;
> }
> @@ -358,7 +356,7 @@ static int check_ksm_unmerge(int merge_type, int mapping, int prot, int timeout,
> int page_count = 2;
>
> if (clock_gettime(CLOCK_MONOTONIC_RAW, &start_time)) {
> - perror("clock_gettime");
> + ksft_perror("clock_gettime");
> return KSFT_FAIL;
> }
>
> @@ -380,13 +378,11 @@ static int check_ksm_unmerge(int merge_type, int mapping, int prot, int timeout,
>
> /* check that unmerging was successful and 0 pages are currently merged */
> if (assert_ksm_pages_count(0)) {
> - printf("OK\n");
> munmap(map_ptr, page_size * page_count);
> return KSFT_PASS;
> }
>
> err_out:
> - printf("Not OK\n");
> munmap(map_ptr, page_size * page_count);
> return KSFT_FAIL;
> }
> @@ -398,7 +394,7 @@ static int check_ksm_zero_page_merge(int merge_type, int mapping, int prot, long
> struct timespec start_time;
>
> if (clock_gettime(CLOCK_MONOTONIC_RAW, &start_time)) {
> - perror("clock_gettime");
> + ksft_perror("clock_gettime");
> return KSFT_FAIL;
> }
>
> @@ -425,12 +421,10 @@ static int check_ksm_zero_page_merge(int merge_type, int mapping, int prot, long
> else if (!use_zero_pages && !assert_ksm_pages_count(page_count))
> goto err_out;
>
> - printf("OK\n");
> munmap(map_ptr, page_size * page_count);
> return KSFT_PASS;
>
> err_out:
> - printf("Not OK\n");
> munmap(map_ptr, page_size * page_count);
> return KSFT_FAIL;
> }
> @@ -465,16 +459,16 @@ static int check_ksm_numa_merge(int merge_type, int mapping, int prot, int timeo
> int first_node;
>
> if (clock_gettime(CLOCK_MONOTONIC_RAW, &start_time)) {
> - perror("clock_gettime");
> + ksft_perror("clock_gettime");
> return KSFT_FAIL;
> }
>
> if (numa_available() < 0) {
> - perror("NUMA support not enabled");
> + ksft_perror("NUMA support not enabled");
> return KSFT_SKIP;
> }
> if (numa_num_configured_nodes() <= 1) {
> - printf("At least 2 NUMA nodes must be available\n");
> + ksft_print_msg("At least 2 NUMA nodes must be available\n");
> return KSFT_SKIP;
> }
> if (ksm_write_sysfs(KSM_FP("merge_across_nodes"), merge_across_nodes))
> @@ -485,7 +479,7 @@ static int check_ksm_numa_merge(int merge_type, int mapping, int prot, int timeo
> numa1_map_ptr = numa_alloc_onnode(page_size, first_node);
> numa2_map_ptr = numa_alloc_onnode(page_size, get_next_mem_node(first_node));
> if (!numa1_map_ptr || !numa2_map_ptr) {
> - perror("numa_alloc_onnode");
> + ksft_perror("numa_alloc_onnode");
> return KSFT_FAIL;
> }
>
> @@ -510,13 +504,11 @@ static int check_ksm_numa_merge(int merge_type, int mapping, int prot, int timeo
>
> numa_free(numa1_map_ptr, page_size);
> numa_free(numa2_map_ptr, page_size);
> - printf("OK\n");
> return KSFT_PASS;
>
> err_out:
> numa_free(numa1_map_ptr, page_size);
> numa_free(numa2_map_ptr, page_size);
> - printf("Not OK\n");
> return KSFT_FAIL;
> }
>
> @@ -529,7 +521,7 @@ static int ksm_merge_hugepages_time(int merge_type, int mapping, int prot,
> int pagemap_fd, n_normal_pages, n_huge_pages;
>
> if (!thp_is_enabled()) {
> - printf("Transparent Hugepages not available\n");
> + ksft_print_msg("Transparent Hugepages not available\n");
> return KSFT_SKIP;
> }
>
> @@ -559,36 +551,35 @@ static int ksm_merge_hugepages_time(int merge_type, int mapping, int prot,
> else
> n_huge_pages++;
> }
> - printf("Number of normal pages: %d\n", n_normal_pages);
> - printf("Number of huge pages: %d\n", n_huge_pages);
> + ksft_print_msg("Number of normal pages: %d\n", n_normal_pages);
> + ksft_print_msg("Number of huge pages: %d\n", n_huge_pages);
>
> memset(map_ptr, '*', len);
>
> if (clock_gettime(CLOCK_MONOTONIC_RAW, &start_time)) {
> - perror("clock_gettime");
> + ksft_perror("clock_gettime");
> goto err_out;
> }
> if (ksm_merge_pages(merge_type, map_ptr, map_size, start_time, timeout))
> goto err_out;
> if (clock_gettime(CLOCK_MONOTONIC_RAW, &end_time)) {
> - perror("clock_gettime");
> + ksft_perror("clock_gettime");
> goto err_out;
> }
>
> scan_time_ns = (end_time.tv_sec - start_time.tv_sec) * NSEC_PER_SEC +
> (end_time.tv_nsec - start_time.tv_nsec);
>
> - printf("Total size: %lu MiB\n", map_size / MB);
> - printf("Total time: %ld.%09ld s\n", scan_time_ns / NSEC_PER_SEC,
> + ksft_print_msg("Total size: %lu MiB\n", map_size / MB);
> + ksft_print_msg("Total time: %ld.%09ld s\n", scan_time_ns / NSEC_PER_SEC,
> scan_time_ns % NSEC_PER_SEC);
> - printf("Average speed: %.3f MiB/s\n", (map_size / MB) /
> + ksft_print_msg("Average speed: %.3f MiB/s\n", (map_size / MB) /
> ((double)scan_time_ns / NSEC_PER_SEC));
>
> munmap(map_ptr_orig, len + HPAGE_SIZE);
> return KSFT_PASS;
>
> err_out:
> - printf("Not OK\n");
> munmap(map_ptr_orig, len + HPAGE_SIZE);
> return KSFT_FAIL;
> }
> @@ -606,30 +597,29 @@ static int ksm_merge_time(int merge_type, int mapping, int prot, int timeout, si
> return KSFT_FAIL;
>
> if (clock_gettime(CLOCK_MONOTONIC_RAW, &start_time)) {
> - perror("clock_gettime");
> + ksft_perror("clock_gettime");
> goto err_out;
> }
> if (ksm_merge_pages(merge_type, map_ptr, map_size, start_time, timeout))
> goto err_out;
> if (clock_gettime(CLOCK_MONOTONIC_RAW, &end_time)) {
> - perror("clock_gettime");
> + ksft_perror("clock_gettime");
> goto err_out;
> }
>
> scan_time_ns = (end_time.tv_sec - start_time.tv_sec) * NSEC_PER_SEC +
> (end_time.tv_nsec - start_time.tv_nsec);
>
> - printf("Total size: %lu MiB\n", map_size / MB);
> - printf("Total time: %ld.%09ld s\n", scan_time_ns / NSEC_PER_SEC,
> + ksft_print_msg("Total size: %lu MiB\n", map_size / MB);
> + ksft_print_msg("Total time: %ld.%09ld s\n", scan_time_ns / NSEC_PER_SEC,
> scan_time_ns % NSEC_PER_SEC);
> - printf("Average speed: %.3f MiB/s\n", (map_size / MB) /
> + ksft_print_msg("Average speed: %.3f MiB/s\n", (map_size / MB) /
> ((double)scan_time_ns / NSEC_PER_SEC));
>
> munmap(map_ptr, map_size);
> return KSFT_PASS;
>
> err_out:
> - printf("Not OK\n");
> munmap(map_ptr, map_size);
> return KSFT_FAIL;
> }
> @@ -646,37 +636,36 @@ static int ksm_unmerge_time(int merge_type, int mapping, int prot, int timeout,
> if (!map_ptr)
> return KSFT_FAIL;
> if (clock_gettime(CLOCK_MONOTONIC_RAW, &start_time)) {
> - perror("clock_gettime");
> + ksft_perror("clock_gettime");
> goto err_out;
> }
> if (ksm_merge_pages(merge_type, map_ptr, map_size, start_time, timeout))
> goto err_out;
>
> if (clock_gettime(CLOCK_MONOTONIC_RAW, &start_time)) {
> - perror("clock_gettime");
> + ksft_perror("clock_gettime");
> goto err_out;
> }
> if (ksm_unmerge_pages(map_ptr, map_size, start_time, timeout))
> goto err_out;
> if (clock_gettime(CLOCK_MONOTONIC_RAW, &end_time)) {
> - perror("clock_gettime");
> + ksft_perror("clock_gettime");
> goto err_out;
> }
>
> scan_time_ns = (end_time.tv_sec - start_time.tv_sec) * NSEC_PER_SEC +
> (end_time.tv_nsec - start_time.tv_nsec);
>
> - printf("Total size: %lu MiB\n", map_size / MB);
> - printf("Total time: %ld.%09ld s\n", scan_time_ns / NSEC_PER_SEC,
> + ksft_print_msg("Total size: %lu MiB\n", map_size / MB);
> + ksft_print_msg("Total time: %ld.%09ld s\n", scan_time_ns / NSEC_PER_SEC,
> scan_time_ns % NSEC_PER_SEC);
> - printf("Average speed: %.3f MiB/s\n", (map_size / MB) /
> + ksft_print_msg("Average speed: %.3f MiB/s\n", (map_size / MB) /
> ((double)scan_time_ns / NSEC_PER_SEC));
>
> munmap(map_ptr, map_size);
> return KSFT_PASS;
>
> err_out:
> - printf("Not OK\n");
> munmap(map_ptr, map_size);
> return KSFT_FAIL;
> }
> @@ -695,24 +684,24 @@ static int ksm_cow_time(int merge_type, int mapping, int prot, int timeout, size
> return KSFT_FAIL;
>
> if (clock_gettime(CLOCK_MONOTONIC_RAW, &start_time)) {
> - perror("clock_gettime");
> + ksft_perror("clock_gettime");
> return KSFT_FAIL;
> }
> for (size_t i = 0; i < page_count - 1; i = i + 2)
> memset(map_ptr + page_size * i, '-', 1);
> if (clock_gettime(CLOCK_MONOTONIC_RAW, &end_time)) {
> - perror("clock_gettime");
> + ksft_perror("clock_gettime");
> return KSFT_FAIL;
> }
>
> cow_time_ns = (end_time.tv_sec - start_time.tv_sec) * NSEC_PER_SEC +
> (end_time.tv_nsec - start_time.tv_nsec);
>
> - printf("Total size: %lu MiB\n\n", (page_size * page_count) / MB);
> - printf("Not merged pages:\n");
> - printf("Total time: %ld.%09ld s\n", cow_time_ns / NSEC_PER_SEC,
> + ksft_print_msg("Total size: %lu MiB\n\n", (page_size * page_count) / MB);
> + ksft_print_msg("Not merged pages:\n");
> + ksft_print_msg("Total time: %ld.%09ld s\n", cow_time_ns / NSEC_PER_SEC,
> cow_time_ns % NSEC_PER_SEC);
> - printf("Average speed: %.3f MiB/s\n\n", ((page_size * (page_count / 2)) / MB) /
> + ksft_print_msg("Average speed: %.3f MiB/s\n\n", ((page_size * (page_count / 2)) / MB) /
> ((double)cow_time_ns / NSEC_PER_SEC));
>
> /* Create 2000 pairs of duplicate pages */
> @@ -724,30 +713,29 @@ static int ksm_cow_time(int merge_type, int mapping, int prot, int timeout, size
> goto err_out;
>
> if (clock_gettime(CLOCK_MONOTONIC_RAW, &start_time)) {
> - perror("clock_gettime");
> + ksft_perror("clock_gettime");
> goto err_out;
> }
> for (size_t i = 0; i < page_count - 1; i = i + 2)
> memset(map_ptr + page_size * i, '-', 1);
> if (clock_gettime(CLOCK_MONOTONIC_RAW, &end_time)) {
> - perror("clock_gettime");
> + ksft_perror("clock_gettime");
> goto err_out;
> }
>
> cow_time_ns = (end_time.tv_sec - start_time.tv_sec) * NSEC_PER_SEC +
> (end_time.tv_nsec - start_time.tv_nsec);
>
> - printf("Merged pages:\n");
> - printf("Total time: %ld.%09ld s\n", cow_time_ns / NSEC_PER_SEC,
> + ksft_print_msg("Merged pages:\n");
> + ksft_print_msg("Total time: %ld.%09ld s\n", cow_time_ns / NSEC_PER_SEC,
> cow_time_ns % NSEC_PER_SEC);
> - printf("Average speed: %.3f MiB/s\n", ((page_size * (page_count / 2)) / MB) /
> + ksft_print_msg("Average speed: %.3f MiB/s\n", ((page_size * (page_count / 2)) / MB) /
> ((double)cow_time_ns / NSEC_PER_SEC));
>
> munmap(map_ptr, page_size * page_count);
> return KSFT_PASS;
>
> err_out:
> - printf("Not OK\n");
> munmap(map_ptr, page_size * page_count);
> return KSFT_FAIL;
> }
> @@ -765,6 +753,10 @@ int main(int argc, char *argv[])
> bool use_zero_pages = KSM_USE_ZERO_PAGES_DEFAULT;
> bool merge_across_nodes = KSM_MERGE_ACROSS_NODES_DEFAULT;
> long size_MB = 0;
> + const char *test_descr = "KSM merging";
> +
> + ksft_print_header();
> + ksft_set_plan(1);
>
> while ((opt = getopt(argc, argv, "dha:p:l:z:m:s:t:MUZNPCHD")) != -1) {
> switch (opt) {
> @@ -773,17 +765,13 @@ int main(int argc, char *argv[])
> break;
> case 'p':
> page_count = atol(optarg);
> - if (page_count <= 0) {
> - printf("The number of pages must be greater than 0\n");
> - return KSFT_FAIL;
> - }
> + if (page_count <= 0)
> + ksft_exit_fail_msg("The number of pages must be greater than 0\n");
> break;
> case 'l':
> ksm_scan_limit_sec = atoi(optarg);
> - if (ksm_scan_limit_sec <= 0) {
> - printf("Timeout value must be greater than 0\n");
> - return KSFT_FAIL;
> - }
> + if (ksm_scan_limit_sec <= 0)
> + ksft_exit_fail_msg("Timeout value must be greater than 0\n");
> break;
> case 'h':
> print_help();
> @@ -805,19 +793,15 @@ int main(int argc, char *argv[])
> break;
> case 's':
> size_MB = atoi(optarg);
> - if (size_MB <= 0) {
> - printf("Size must be greater than 0\n");
> - return KSFT_FAIL;
> - }
> + if (size_MB <= 0)
> + ksft_exit_fail_msg("Size must be greater than 0\n");
> break;
> case 't':
> {
> int tmp = atoi(optarg);
>
> - if (tmp < 0 || tmp > KSM_MERGE_LAST) {
> - printf("Invalid merge type\n");
> - return KSFT_FAIL;
> - }
> + if (tmp < 0 || tmp > KSM_MERGE_LAST)
> + ksft_exit_fail_msg("Invalid merge type\n");
> merge_type = tmp;
> }
> break;
> @@ -845,82 +829,80 @@ int main(int argc, char *argv[])
> test_name = KSM_COW_TIME;
> break;
> default:
> - return KSFT_FAIL;
> + ksft_exit_fail_msg("Unknown option\n");
> }
> }
>
> if (prot == 0)
> prot = str_to_prot(KSM_PROT_STR_DEFAULT);
>
> - if (access(KSM_SYSFS_PATH, F_OK)) {
> - printf("Config KSM not enabled\n");
> - return KSFT_SKIP;
> - }
> + if (access(KSM_SYSFS_PATH, F_OK))
> + ksft_exit_skip("Config KSM not enabled\n");
>
> - if (ksm_save_def(&ksm_sysfs_old)) {
> - printf("Cannot save default tunables\n");
> - return KSFT_FAIL;
> - }
> + if (ksm_save_def(&ksm_sysfs_old))
> + ksft_exit_fail_msg("Cannot save default tunables\n");
>
> if (ksm_write_sysfs(KSM_FP("run"), 2) ||
> ksm_write_sysfs(KSM_FP("sleep_millisecs"), 0) ||
> numa_available() ? 0 :
> ksm_write_sysfs(KSM_FP("merge_across_nodes"), 1) ||
> ksm_write_sysfs(KSM_FP("pages_to_scan"), page_count))
> - return KSFT_FAIL;
> + ksft_exit_fail_msg("Cannot set up KSM tunables\n");
>
> switch (test_name) {
> case CHECK_KSM_MERGE:
> + test_descr = "KSM merging";
> ret = check_ksm_merge(merge_type, MAP_PRIVATE | MAP_ANONYMOUS, prot, page_count,
> ksm_scan_limit_sec, page_size);
> break;
> case CHECK_KSM_UNMERGE:
> + test_descr = "KSM unmerging";
> ret = check_ksm_unmerge(merge_type, MAP_PRIVATE | MAP_ANONYMOUS, prot,
> ksm_scan_limit_sec, page_size);
> break;
> case CHECK_KSM_ZERO_PAGE_MERGE:
> + test_descr = "KSM zero page merging";
> ret = check_ksm_zero_page_merge(merge_type, MAP_PRIVATE | MAP_ANONYMOUS, prot,
> page_count, ksm_scan_limit_sec, use_zero_pages,
> page_size);
> break;
> case CHECK_KSM_NUMA_MERGE:
> + test_descr = "KSM NUMA merging";
> ret = check_ksm_numa_merge(merge_type, MAP_PRIVATE | MAP_ANONYMOUS, prot,
> ksm_scan_limit_sec, merge_across_nodes, page_size);
> break;
> case KSM_MERGE_TIME:
> - if (size_MB == 0) {
> - printf("Option '-s' is required.\n");
> - return KSFT_FAIL;
> - }
> + if (size_MB == 0)
> + ksft_exit_fail_msg("Option '-s' is required\n");
> + test_descr = "KSM merge time";
> ret = ksm_merge_time(merge_type, MAP_PRIVATE | MAP_ANONYMOUS, prot,
> ksm_scan_limit_sec, size_MB);
> break;
> case KSM_MERGE_TIME_HUGE_PAGES:
> - if (size_MB == 0) {
> - printf("Option '-s' is required.\n");
> - return KSFT_FAIL;
> - }
> + if (size_MB == 0)
> + ksft_exit_fail_msg("Option '-s' is required\n");
> + test_descr = "KSM merge time with huge pages";
> ret = ksm_merge_hugepages_time(merge_type, MAP_PRIVATE | MAP_ANONYMOUS, prot,
> ksm_scan_limit_sec, size_MB);
> break;
> case KSM_UNMERGE_TIME:
> - if (size_MB == 0) {
> - printf("Option '-s' is required.\n");
> - return KSFT_FAIL;
> - }
> + if (size_MB == 0)
> + ksft_exit_fail_msg("Option '-s' is required\n");
> + test_descr = "KSM unmerge time";
> ret = ksm_unmerge_time(merge_type, MAP_PRIVATE | MAP_ANONYMOUS, prot,
> ksm_scan_limit_sec, size_MB);
> break;
> case KSM_COW_TIME:
> + test_descr = "KSM COW time";
> ret = ksm_cow_time(merge_type, MAP_PRIVATE | MAP_ANONYMOUS, prot,
> ksm_scan_limit_sec, page_size);
> break;
> }
>
> - if (ksm_restore(&ksm_sysfs_old)) {
> - printf("Cannot restore default tunables\n");
> - return KSFT_FAIL;
> - }
> + if (ksm_restore(&ksm_sysfs_old))
> + ksft_print_msg("Cannot restore default tunables\n");
> +
> + ksft_test_result_report(ret, "%s\n", test_descr);
>
> - return ret;
> + ksft_finished();
> }
next prev parent reply other threads:[~2026-04-14 9:42 UTC|newest]
Thread overview: 83+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-04-06 14:16 [PATCH 00/53] selftests/mm: make MM selftests more CI friendly Mike Rapoport
2026-04-06 14:16 ` [PATCH 01/53] selftests/mm: hugetlb-read-hwpoison: add SIGBUS handler Mike Rapoport
2026-04-06 14:16 ` [PATCH 02/53] selftests/mm: khugepaged: enable collapse_single_pte_entry_compound for shmem Mike Rapoport
2026-04-06 14:16 ` [PATCH 03/53] selftests/mm: migration: don't assume hupe page is TWOMEG Mike Rapoport
2026-04-09 9:03 ` Donet Tom
2026-04-06 14:16 ` [PATCH 04/53] selftests/mm: run_vmtests.sh: don't gate THP and KSM tests on HAVE_HUGEPAGES Mike Rapoport
2026-04-06 14:16 ` [PATCH 05/53] selftests/mm: merge map_hugetlb into hugepage-mmap Mike Rapoport
2026-04-09 16:05 ` Donet Tom
2026-04-13 10:21 ` Mike Rapoport
2026-04-06 14:16 ` [PATCH 06/53] selftests/mm: rename hugepage-* tests to hugetlb-* Mike Rapoport
2026-04-09 9:33 ` Donet Tom
2026-04-13 10:15 ` Mike Rapoport
2026-04-06 14:16 ` [PATCH 07/53] selftests/mm: hugetlb-shm: use kselftest framework Mike Rapoport
2026-04-06 14:27 ` Mark Brown
2026-04-09 9:49 ` Donet Tom
2026-04-13 10:18 ` Mike Rapoport
2026-04-06 14:16 ` [PATCH 08/53] selftests/mm: hugetlb-vmemmap: " Mike Rapoport
2026-04-06 14:40 ` Mark Brown
2026-04-09 11:01 ` Donet Tom
2026-04-06 14:16 ` [PATCH 09/53] selftests/mm: hugetlb-madvise: " Mike Rapoport
2026-04-09 16:55 ` Donet Tom
2026-04-06 14:16 ` [PATCH 10/53] selftests/mm: hugetlb_madv_vs_map: " Mike Rapoport
2026-04-06 15:06 ` Mark Brown
2026-04-09 17:14 ` Donet Tom
2026-04-13 10:25 ` Mike Rapoport
2026-04-06 14:16 ` [PATCH 11/53] selftests/mm: hugetlb-read-hwpoison: " Mike Rapoport
2026-04-06 16:19 ` Mark Brown
2026-04-14 6:02 ` Donet Tom
2026-04-06 14:16 ` [PATCH 12/53] selftests/mm: khugepaged: group tests in an array Mike Rapoport
2026-04-06 16:49 ` Mark Brown
2026-04-14 7:02 ` Donet Tom
2026-04-14 15:59 ` Mike Rapoport
2026-04-06 14:16 ` [PATCH 13/53] selftests/mm: khugepaged: use ksefltest framework Mike Rapoport
2026-04-06 14:16 ` [PATCH 14/53] selftests/mm: ksm_tests: use kselftest framework Mike Rapoport
2026-04-14 9:41 ` Donet Tom [this message]
2026-04-14 16:01 ` Mike Rapoport
2026-04-06 14:16 ` [PATCH 15/53] selftests/mm: protection_keys: use descriptive test names in TAP output Mike Rapoport
2026-04-06 16:51 ` Mark Brown
2026-04-14 10:22 ` Donet Tom
2026-04-06 14:16 ` [PATCH 16/53] selftests/mm: protection_keys: use kselftest framework Mike Rapoport
2026-04-06 17:02 ` Mark Brown
2026-04-14 10:35 ` Donet Tom
2026-04-06 14:16 ` [PATCH 17/53] selftests/mm: uffd-stress: " Mike Rapoport
2026-04-07 12:31 ` Mark Brown
2026-04-06 14:17 ` [PATCH 18/53] selftests/mm: uffd-unit-tests: " Mike Rapoport
2026-04-07 12:58 ` Mark Brown
2026-04-06 14:17 ` [PATCH 19/53] selftests/mm: va_high_addr_switch: " Mike Rapoport
2026-04-07 14:34 ` Mark Brown
2026-04-06 14:17 ` [PATCH 20/53] selftests/mm: add atexit() and signal handlers to thp_settings Mike Rapoport
2026-04-06 14:17 ` [PATCH 21/53] selftests/mm: rename thp_settings.[ch] to hugepage_settings.[ch] Mike Rapoport
2026-04-06 14:17 ` [PATCH 22/53] selftests/mm: move HugeTLB helpers to hugepage_settings Mike Rapoport
2026-04-06 14:17 ` [PATCH 23/53] selftests/mm: hugepage_settings: use unsigned long in detect_hugetlb_page_size Mike Rapoport
2026-04-06 14:17 ` [PATCH 24/53] selftests/mm: hugepage_settings: add APIs to get and set nr_hugepages Mike Rapoport
2026-04-06 14:17 ` [PATCH 25/53] selftests/mm: hugepage_settings: rename get_free_hugepages() Mike Rapoport
2026-04-06 14:17 ` [PATCH 26/53] selftests/mm: hugepage_settings: add APIs for HugeTLB setup and teardown Mike Rapoport
2026-04-06 14:17 ` [PATCH 27/53] selftests/mm: move read_file(), read_num() and write_num() to vm_util Mike Rapoport
2026-04-06 14:17 ` [PATCH 28/53] selftests/mm: vm_util: add helpers to set and restore shm limits Mike Rapoport
2026-04-06 14:17 ` [PATCH 29/53] selftests/mm: compaction_test: use HugeTLB helpers Mike Rapoport
2026-04-06 14:17 ` [PATCH 30/53] selftests/mm: cow: add setup of HugeTLB pages Mike Rapoport
2026-04-06 14:17 ` [PATCH 31/53] selftests/mm: gup_longterm: " Mike Rapoport
2026-04-06 14:17 ` [PATCH 32/53] selftests/mm: gup_test: " Mike Rapoport
2026-04-06 14:17 ` [PATCH 33/53] selftests/mm: hmm-tests: " Mike Rapoport
2026-04-06 14:17 ` [PATCH 34/53] selftests/mm: hugepage_dio: " Mike Rapoport
2026-04-06 14:17 ` [PATCH 35/53] selftests/mm: hugetlb_fault_after_madv: " Mike Rapoport
2026-04-06 14:17 ` [PATCH 36/53] selftests/mm: hugetlb-madvise: " Mike Rapoport
2026-04-06 14:17 ` [PATCH 37/53] selftests/mm: hugetlb_madv_vs_map: " Mike Rapoport
2026-04-06 14:17 ` [PATCH 38/53] selftests/mm: hugetlb-mmap: " Mike Rapoport
2026-04-06 14:17 ` [PATCH 39/53] selftests/mm: hugetlb-mremap: " Mike Rapoport
2026-04-06 14:17 ` [PATCH 40/53] selftests/mm: hugetlb-shm: " Mike Rapoport
2026-04-06 14:17 ` [PATCH 41/53] selftests/mm: hugetlb-soft-online: " Mike Rapoport
2026-04-06 14:17 ` [PATCH 42/53] selftests/mm: hugetlb-vmemmap: " Mike Rapoport
2026-04-06 14:17 ` [PATCH 43/53] selftests/mm: migration: " Mike Rapoport
2026-04-06 14:17 ` [PATCH 44/53] selftests/mm: pagemap_ioctl: " Mike Rapoport
2026-04-06 14:17 ` [PATCH 45/53] selftests/mm: protection_keys: use library code for HugeTLB setup Mike Rapoport
2026-04-06 14:17 ` [PATCH 46/53] selftests/mm: thuge-gen: add setup of HugeTLB pages Mike Rapoport
2026-04-06 14:17 ` [PATCH 47/53] selftests/mm: uffd-stress: use hugetlb_save and alloc huge pages Mike Rapoport
2026-04-06 14:17 ` [PATCH 48/53] selftests/mm: uffd-unit-tests: add setup of HugeTLB pages Mike Rapoport
2026-04-06 14:17 ` [PATCH 49/53] selftests/mm: uffd-wp-mremap: " Mike Rapoport
2026-04-06 14:17 ` [PATCH 50/53] selftests/mm: va_high_addr_switch: " Mike Rapoport
2026-04-06 14:17 ` [PATCH 51/53] selftests/mm: va_high_addr_switch.sh: drop huge pages setup Mike Rapoport
2026-04-06 14:17 ` [PATCH 52/53] selftests/mm: run_vmtests.sh: free memory if available memory is low Mike Rapoport
2026-04-06 14:17 ` [PATCH 53/53] selftests/mm: run_vmtests.sh: drop detection and setup of HugeTLB Mike Rapoport
2026-04-07 12:20 ` [PATCH 00/53] selftests/mm: make MM selftests more CI friendly Mark Brown
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=b49e883b-d666-49cb-bf4d-c83a8d92a0fe@linux.ibm.com \
--to=donettom@linux.ibm.com \
--cc=Liam.Howlett@oracle.com \
--cc=akpm@linux-foundation.org \
--cc=baohua@kernel.org \
--cc=baolin.wang@linux.alibaba.com \
--cc=broonie@kernel.org \
--cc=david@kernel.org \
--cc=dev.jain@arm.com \
--cc=jgg@ziepe.ca \
--cc=jhubbard@nvidia.com \
--cc=lance.yang@linux.dev \
--cc=leon@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-kselftest@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=ljs@kernel.org \
--cc=mhocko@suse.com \
--cc=npache@redhat.com \
--cc=peterx@redhat.com \
--cc=rppt@kernel.org \
--cc=ryan.roberts@arm.com \
--cc=shuah@kernel.org \
--cc=surenb@google.com \
--cc=vbabka@kernel.org \
--cc=ziy@nvidia.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox