Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B5A403ACEF4; Thu, 17 Sep 2026 18:55:19 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789671321; cv=none; b=XjK15TcCZDlJ3ASiGPl+5E6OcvqxxoHNw6CAgEh9UXStJetTKgQE21vm3YA8tGitCPc9fRIrmCkeL9CWCvg3++g2ICLCX19SP2x9l666nn+Dvz6pq1LN0uguLPGqPB9S1O9xZAgXK2k9OF2kHFknGfEkDTAybC5LlnPga0+F/AU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789671321; c=relaxed/simple; bh=I98x1jSUExzRWRAY2oXwCrpKmBWvwOjO4l9N18yoXBM=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=ubdUBJ6icJkLDpVkNKbbyHs0Nv7sKlDncTSX/0MjrBsQEdfdAa8+P1vf9glNn8tgpushZUEdf3WT8yEugXfgbwh5vSOtLWHH5TMBRCKOeW0pyL3Sr30rkbkqgMlbFraXsxMlrT7yVVEEumpwPR73dJ/XyNoAf5mHcXH50+lMmms= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b=10Pj0Uv/; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="10Pj0Uv/" Received: by smtp.kernel.org (Postfix) with ESMTPSA id ADF591F000FF; Thu, 17 Sep 2026 18:55:18 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linuxfoundation.org; s=korg; t=1789671319; bh=etXpJcSBHlnENg/BqqAomqhHz27j5VxyBu2qWzlAPcE=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=10Pj0Uv/3hJ1afdMg35GhFSZtKYUdmd9BoUpfhjjol2mncmRAW29cmSiR/BN3+Nnj RlBbvQp0+kmyWVqZ1tlblUHEg98K74kGjAO/VwzHApQ8EZdTMAlBRgKxfRyzZpYm+Z bx72Uj7BCU3uXh1WBizKBVepQmpC4OZ3HT/vT78Y= From: Greg Kroah-Hartman To: stable@vger.kernel.org Cc: Greg Kroah-Hartman , patches@lists.linux.dev, Sasha Levin Subject: [PATCH 6.12 0938/1102] Revert "selftests/mm: report unique test names for each cow test" Date: Thu, 17 Sep 2026 16:14:41 +0100 Message-ID: <20260917151604.191984516@linuxfoundation.org> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20260917151539.408551884@linuxfoundation.org> References: <20260917151539.408551884@linuxfoundation.org> User-Agent: quilt/0.69 X-stable: review X-Patchwork-Hint: ignore Precedence: bulk X-Mailing-List: patches@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit 6.12-stable review patch. If anyone has any objections, please let me know. ------------------ This reverts commit 0531bd6ef3670d5c84b9bd86add623798ce216cc. Signed-off-by: Sasha Levin --- tools/testing/selftests/mm/cow.c | 333 +++++++++++-------------------- 1 file changed, 116 insertions(+), 217 deletions(-) diff --git a/tools/testing/selftests/mm/cow.c b/tools/testing/selftests/mm/cow.c index d0e11a40269ea..f0cb14ea86084 100644 --- a/tools/testing/selftests/mm/cow.c +++ b/tools/testing/selftests/mm/cow.c @@ -112,12 +112,9 @@ struct comm_pipes { static int setup_comm_pipes(struct comm_pipes *comm_pipes) { - if (pipe(comm_pipes->child_ready) < 0) { - ksft_perror("pipe()"); + if (pipe(comm_pipes->child_ready) < 0) return -errno; - } if (pipe(comm_pipes->parent_ready) < 0) { - ksft_perror("pipe()"); close(comm_pipes->child_ready[0]); close(comm_pipes->child_ready[1]); return -errno; @@ -210,14 +207,13 @@ static void do_test_cow_in_parent(char *mem, size_t size, bool do_mprotect, ret = setup_comm_pipes(&comm_pipes); if (ret) { - log_test_result(KSFT_FAIL); + ksft_test_result_fail("pipe() failed\n"); return; } ret = fork(); if (ret < 0) { - ksft_perror("fork() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("fork() failed\n"); goto close_comm_pipes; } else if (!ret) { exit(fn(mem, size, &comm_pipes)); @@ -232,18 +228,9 @@ static void do_test_cow_in_parent(char *mem, size_t size, bool do_mprotect, * write-faults by directly mapping pages writable. */ ret = mprotect(mem, size, PROT_READ); + ret |= mprotect(mem, size, PROT_READ|PROT_WRITE); if (ret) { - ksft_perror("mprotect() failed"); - log_test_result(KSFT_FAIL); - write(comm_pipes.parent_ready[1], "0", 1); - wait(&ret); - goto close_comm_pipes; - } - - ret = mprotect(mem, size, PROT_READ|PROT_WRITE); - if (ret) { - ksft_perror("mprotect() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("mprotect() failed\n"); write(comm_pipes.parent_ready[1], "0", 1); wait(&ret); goto close_comm_pipes; @@ -261,16 +248,16 @@ static void do_test_cow_in_parent(char *mem, size_t size, bool do_mprotect, ret = -EINVAL; if (!ret) { - log_test_result(KSFT_PASS); + ksft_test_result_pass("No leak from parent into child\n"); } else if (xfail) { /* * With hugetlb, some vmsplice() tests are currently expected to * fail because (a) harder to fix and (b) nobody really cares. * Flag them as expected failure for now. */ - log_test_result(KSFT_XFAIL); + ksft_test_result_xfail("Leak from parent into child\n"); } else { - log_test_result(KSFT_FAIL); + ksft_test_result_fail("Leak from parent into child\n"); } close_comm_pipes: close_comm_pipes(&comm_pipes); @@ -319,29 +306,26 @@ static void do_test_vmsplice_in_parent(char *mem, size_t size, ret = setup_comm_pipes(&comm_pipes); if (ret) { - log_test_result(KSFT_FAIL); + ksft_test_result_fail("pipe() failed\n"); goto free; } if (pipe(fds) < 0) { - ksft_perror("pipe() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("pipe() failed\n"); goto close_comm_pipes; } if (before_fork) { transferred = vmsplice(fds[1], &iov, 1, 0); if (transferred <= 0) { - ksft_print_msg("vmsplice() failed\n"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("vmsplice() failed\n"); goto close_pipe; } } ret = fork(); if (ret < 0) { - ksft_perror("fork() failed\n"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("fork() failed\n"); goto close_pipe; } else if (!ret) { write(comm_pipes.child_ready[1], "0", 1); @@ -355,8 +339,7 @@ static void do_test_vmsplice_in_parent(char *mem, size_t size, if (!before_fork) { transferred = vmsplice(fds[1], &iov, 1, 0); if (transferred <= 0) { - ksft_perror("vmsplice() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("vmsplice() failed\n"); wait(&ret); goto close_pipe; } @@ -365,8 +348,7 @@ static void do_test_vmsplice_in_parent(char *mem, size_t size, while (read(comm_pipes.child_ready[0], &buf, 1) != 1) ; if (munmap(mem, size) < 0) { - ksft_perror("munmap() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("munmap() failed\n"); goto close_pipe; } write(comm_pipes.parent_ready[1], "0", 1); @@ -374,8 +356,7 @@ static void do_test_vmsplice_in_parent(char *mem, size_t size, /* Wait until the child is done writing. */ wait(&ret); if (!WIFEXITED(ret)) { - ksft_perror("wait() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("wait() failed\n"); goto close_pipe; } @@ -383,23 +364,22 @@ static void do_test_vmsplice_in_parent(char *mem, size_t size, for (total = 0; total < transferred; total += cur) { cur = read(fds[0], new + total, transferred - total); if (cur < 0) { - ksft_perror("read() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("read() failed\n"); goto close_pipe; } } if (!memcmp(old, new, transferred)) { - log_test_result(KSFT_PASS); + ksft_test_result_pass("No leak from child into parent\n"); } else if (xfail) { /* * With hugetlb, some vmsplice() tests are currently expected to * fail because (a) harder to fix and (b) nobody really cares. * Flag them as expected failure for now. */ - log_test_result(KSFT_XFAIL); + ksft_test_result_xfail("Leak from child into parent\n"); } else { - log_test_result(KSFT_FAIL); + ksft_test_result_fail("Leak from child into parent\n"); } close_pipe: close(fds[0]); @@ -436,14 +416,13 @@ static void do_test_iouring(char *mem, size_t size, bool use_fork) ret = setup_comm_pipes(&comm_pipes); if (ret) { - log_test_result(KSFT_FAIL); + ksft_test_result_fail("pipe() failed\n"); return; } file = tmpfile(); if (!file) { - ksft_perror("tmpfile() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("tmpfile() failed\n"); goto close_comm_pipes; } fd = fileno(file); @@ -451,16 +430,14 @@ static void do_test_iouring(char *mem, size_t size, bool use_fork) tmp = malloc(size); if (!tmp) { - ksft_print_msg("malloc() failed\n"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("malloc() failed\n"); goto close_file; } /* Skip on errors, as we might just lack kernel support. */ ret = io_uring_queue_init(1, &ring, 0); if (ret < 0) { - ksft_print_msg("io_uring_queue_init() failed\n"); - log_test_result(KSFT_SKIP); + ksft_test_result_skip("io_uring_queue_init() failed\n"); goto free_tmp; } @@ -475,8 +452,7 @@ static void do_test_iouring(char *mem, size_t size, bool use_fork) iov.iov_len = size; ret = io_uring_register_buffers(&ring, &iov, 1); if (ret) { - ksft_print_msg("io_uring_register_buffers() failed\n"); - log_test_result(KSFT_SKIP); + ksft_test_result_skip("io_uring_register_buffers() failed\n"); goto queue_exit; } @@ -487,8 +463,7 @@ static void do_test_iouring(char *mem, size_t size, bool use_fork) */ ret = fork(); if (ret < 0) { - ksft_perror("fork() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("fork() failed\n"); goto unregister_buffers; } else if (!ret) { write(comm_pipes.child_ready[1], "0", 1); @@ -508,17 +483,10 @@ static void do_test_iouring(char *mem, size_t size, bool use_fork) * if the page is mapped R/O vs. R/W). */ ret = mprotect(mem, size, PROT_READ); - if (ret) { - ksft_perror("mprotect() failed"); - log_test_result(KSFT_FAIL); - goto unregister_buffers; - } - clear_softdirty(); - ret = mprotect(mem, size, PROT_READ | PROT_WRITE); + ret |= mprotect(mem, size, PROT_READ | PROT_WRITE); if (ret) { - ksft_perror("mprotect() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("mprotect() failed\n"); goto unregister_buffers; } } @@ -530,29 +498,25 @@ static void do_test_iouring(char *mem, size_t size, bool use_fork) memset(mem, 0xff, size); sqe = io_uring_get_sqe(&ring); if (!sqe) { - ksft_print_msg("io_uring_get_sqe() failed\n"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("io_uring_get_sqe() failed\n"); goto quit_child; } io_uring_prep_write_fixed(sqe, fd, mem, size, 0, 0); ret = io_uring_submit(&ring); if (ret < 0) { - ksft_print_msg("io_uring_submit() failed\n"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("io_uring_submit() failed\n"); goto quit_child; } ret = io_uring_wait_cqe(&ring, &cqe); if (ret < 0) { - ksft_print_msg("io_uring_wait_cqe() failed\n"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("io_uring_wait_cqe() failed\n"); goto quit_child; } if (cqe->res != size) { - ksft_print_msg("write_fixed failed\n"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("write_fixed failed\n"); goto quit_child; } io_uring_cqe_seen(&ring, cqe); @@ -562,18 +526,15 @@ static void do_test_iouring(char *mem, size_t size, bool use_fork) while (total < size) { cur = pread(fd, tmp + total, size - total, total); if (cur < 0) { - ksft_print_msg("pread() failed\n"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("pread() failed\n"); goto quit_child; } total += cur; } /* Finally, check if we read what we expected. */ - if (!memcmp(mem, tmp, size)) - log_test_result(KSFT_PASS); - else - log_test_result(KSFT_FAIL); + ksft_test_result(!memcmp(mem, tmp, size), + "Longterm R/W pin is reliable\n"); quit_child: if (use_fork) { @@ -621,21 +582,19 @@ static void do_test_ro_pin(char *mem, size_t size, enum ro_pin_test test, int ret; if (gup_fd < 0) { - ksft_print_msg("gup_test not available\n"); - log_test_result(KSFT_SKIP); + ksft_test_result_skip("gup_test not available\n"); return; } tmp = malloc(size); if (!tmp) { - ksft_print_msg("malloc() failed\n"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("malloc() failed\n"); return; } ret = setup_comm_pipes(&comm_pipes); if (ret) { - log_test_result(KSFT_FAIL); + ksft_test_result_fail("pipe() failed\n"); goto free_tmp; } @@ -650,8 +609,7 @@ static void do_test_ro_pin(char *mem, size_t size, enum ro_pin_test test, */ ret = fork(); if (ret < 0) { - ksft_perror("fork() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("fork() failed\n"); goto close_comm_pipes; } else if (!ret) { write(comm_pipes.child_ready[1], "0", 1); @@ -688,8 +646,7 @@ static void do_test_ro_pin(char *mem, size_t size, enum ro_pin_test test, clear_softdirty(); ret |= mprotect(mem, size, PROT_READ | PROT_WRITE); if (ret) { - ksft_perror("mprotect() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("mprotect() failed\n"); goto close_comm_pipes; } break; @@ -704,11 +661,9 @@ static void do_test_ro_pin(char *mem, size_t size, enum ro_pin_test test, ret = ioctl(gup_fd, PIN_LONGTERM_TEST_START, &args); if (ret) { if (errno == EINVAL) - ret = KSFT_SKIP; + ksft_test_result_skip("PIN_LONGTERM_TEST_START failed\n"); else - ret = KSFT_FAIL; - ksft_perror("PIN_LONGTERM_TEST_START failed"); - log_test_result(ret); + ksft_test_result_fail("PIN_LONGTERM_TEST_START failed\n"); goto wait; } @@ -721,26 +676,22 @@ static void do_test_ro_pin(char *mem, size_t size, enum ro_pin_test test, */ tmp_val = (__u64)(uintptr_t)tmp; ret = ioctl(gup_fd, PIN_LONGTERM_TEST_READ, &tmp_val); - if (ret) { - ksft_perror("PIN_LONGTERM_TEST_READ failed"); - log_test_result(KSFT_FAIL); - } else { - if (!memcmp(mem, tmp, size)) - log_test_result(KSFT_PASS); - else - log_test_result(KSFT_FAIL); - } + if (ret) + ksft_test_result_fail("PIN_LONGTERM_TEST_READ failed\n"); + else + ksft_test_result(!memcmp(mem, tmp, size), + "Longterm R/O pin is reliable\n"); ret = ioctl(gup_fd, PIN_LONGTERM_TEST_STOP); if (ret) - ksft_perror("PIN_LONGTERM_TEST_STOP failed"); + ksft_print_msg("[INFO] PIN_LONGTERM_TEST_STOP failed\n"); wait: switch (test) { case RO_PIN_TEST_SHARED: write(comm_pipes.parent_ready[1], "0", 1); wait(&ret); if (!WIFEXITED(ret)) - ksft_perror("wait() failed"); + ksft_print_msg("[INFO] wait() failed\n"); break; default: break; @@ -795,16 +746,14 @@ static void do_run_with_base_page(test_fn fn, bool swapout) mem = mmap(NULL, pagesize, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANONYMOUS, -1, 0); if (mem == MAP_FAILED) { - ksft_perror("mmap() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("mmap() failed\n"); return; } ret = madvise(mem, pagesize, MADV_NOHUGEPAGE); /* Ignore if not around on a kernel. */ if (ret && errno != EINVAL) { - ksft_perror("MADV_NOHUGEPAGE failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("MADV_NOHUGEPAGE failed\n"); goto munmap; } @@ -814,8 +763,7 @@ static void do_run_with_base_page(test_fn fn, bool swapout) if (swapout) { madvise(mem, pagesize, MADV_PAGEOUT); if (!pagemap_is_swapped(pagemap_fd, mem)) { - ksft_print_msg("MADV_PAGEOUT did not work, is swap enabled?\n"); - log_test_result(KSFT_SKIP); + ksft_test_result_skip("MADV_PAGEOUT did not work, is swap enabled?\n"); goto munmap; } } @@ -827,13 +775,13 @@ static void do_run_with_base_page(test_fn fn, bool swapout) static void run_with_base_page(test_fn fn, const char *desc) { - log_test_start("%s ... with base page", desc); + ksft_print_msg("[RUN] %s ... with base page\n", desc); do_run_with_base_page(fn, false); } static void run_with_base_page_swap(test_fn fn, const char *desc) { - log_test_start("%s ... with swapped out base page", desc); + ksft_print_msg("[RUN] %s ... with swapped out base page\n", desc); do_run_with_base_page(fn, true); } @@ -859,8 +807,7 @@ static void do_run_with_thp(test_fn fn, enum thp_run thp_run, size_t thpsize) mmap_mem = mmap(NULL, mmap_size, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANONYMOUS, -1, 0); if (mmap_mem == MAP_FAILED) { - ksft_perror("mmap() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("mmap() failed\n"); return; } @@ -869,8 +816,7 @@ static void do_run_with_thp(test_fn fn, enum thp_run thp_run, size_t thpsize) ret = madvise(mem, thpsize, MADV_HUGEPAGE); if (ret) { - ksft_perror("MADV_HUGEPAGE failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("MADV_HUGEPAGE failed\n"); goto munmap; } @@ -880,8 +826,7 @@ static void do_run_with_thp(test_fn fn, enum thp_run thp_run, size_t thpsize) */ mem[0] = 1; if (!pagemap_is_populated(pagemap_fd, mem + thpsize - pagesize)) { - ksft_print_msg("Did not get a THP populated\n"); - log_test_result(KSFT_SKIP); + ksft_test_result_skip("Did not get a THP populated\n"); goto munmap; } memset(mem, 1, thpsize); @@ -901,14 +846,12 @@ static void do_run_with_thp(test_fn fn, enum thp_run thp_run, size_t thpsize) */ ret = mprotect(mem + pagesize, pagesize, PROT_READ); if (ret) { - ksft_perror("mprotect() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("mprotect() failed\n"); goto munmap; } ret = mprotect(mem + pagesize, pagesize, PROT_READ | PROT_WRITE); if (ret) { - ksft_perror("mprotect() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("mprotect() failed\n"); goto munmap; } break; @@ -920,8 +863,7 @@ static void do_run_with_thp(test_fn fn, enum thp_run thp_run, size_t thpsize) */ ret = madvise(mem + pagesize, thpsize - pagesize, MADV_DONTNEED); if (ret) { - ksft_perror("MADV_DONTNEED failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("MADV_DONTNEED failed\n"); goto munmap; } size = pagesize; @@ -935,15 +877,13 @@ static void do_run_with_thp(test_fn fn, enum thp_run thp_run, size_t thpsize) mremap_mem = mmap(NULL, mremap_size, PROT_NONE, MAP_PRIVATE | MAP_ANONYMOUS, -1, 0); if (mremap_mem == MAP_FAILED) { - ksft_perror("mmap() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("mmap() failed\n"); goto munmap; } tmp = mremap(mem + mremap_size, mremap_size, mremap_size, MREMAP_MAYMOVE | MREMAP_FIXED, mremap_mem); if (tmp != mremap_mem) { - ksft_perror("mremap() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("mremap() failed\n"); goto munmap; } size = mremap_size; @@ -956,14 +896,12 @@ static void do_run_with_thp(test_fn fn, enum thp_run thp_run, size_t thpsize) */ ret = madvise(mem + pagesize, thpsize - pagesize, MADV_DONTFORK); if (ret) { - ksft_perror("MADV_DONTFORK failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("MADV_DONTFORK failed\n"); goto munmap; } ret = fork(); if (ret < 0) { - ksft_perror("fork() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("fork() failed\n"); goto munmap; } else if (!ret) { exit(0); @@ -972,8 +910,7 @@ static void do_run_with_thp(test_fn fn, enum thp_run thp_run, size_t thpsize) /* Allow for sharing all pages again. */ ret = madvise(mem + pagesize, thpsize - pagesize, MADV_DOFORK); if (ret) { - ksft_perror("MADV_DOFORK failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("MADV_DOFORK failed\n"); goto munmap; } break; @@ -987,8 +924,7 @@ static void do_run_with_thp(test_fn fn, enum thp_run thp_run, size_t thpsize) case THP_RUN_SINGLE_PTE_SWAPOUT: madvise(mem, size, MADV_PAGEOUT); if (!range_is_swapped(mem, size)) { - ksft_print_msg("MADV_PAGEOUT did not work, is swap enabled?\n"); - log_test_result(KSFT_SKIP); + ksft_test_result_skip("MADV_PAGEOUT did not work, is swap enabled?\n"); goto munmap; } break; @@ -1005,56 +941,56 @@ static void do_run_with_thp(test_fn fn, enum thp_run thp_run, size_t thpsize) static void run_with_thp(test_fn fn, const char *desc, size_t size) { - log_test_start("%s ... with THP (%zu kB)", + ksft_print_msg("[RUN] %s ... with THP (%zu kB)\n", desc, size / 1024); do_run_with_thp(fn, THP_RUN_PMD, size); } static void run_with_thp_swap(test_fn fn, const char *desc, size_t size) { - log_test_start("%s ... with swapped-out THP (%zu kB)", + ksft_print_msg("[RUN] %s ... with swapped-out THP (%zu kB)\n", desc, size / 1024); do_run_with_thp(fn, THP_RUN_PMD_SWAPOUT, size); } static void run_with_pte_mapped_thp(test_fn fn, const char *desc, size_t size) { - log_test_start("%s ... with PTE-mapped THP (%zu kB)", + ksft_print_msg("[RUN] %s ... with PTE-mapped THP (%zu kB)\n", desc, size / 1024); do_run_with_thp(fn, THP_RUN_PTE, size); } static void run_with_pte_mapped_thp_swap(test_fn fn, const char *desc, size_t size) { - log_test_start("%s ... with swapped-out, PTE-mapped THP (%zu kB)", + ksft_print_msg("[RUN] %s ... with swapped-out, PTE-mapped THP (%zu kB)\n", desc, size / 1024); do_run_with_thp(fn, THP_RUN_PTE_SWAPOUT, size); } static void run_with_single_pte_of_thp(test_fn fn, const char *desc, size_t size) { - log_test_start("%s ... with single PTE of THP (%zu kB)", + ksft_print_msg("[RUN] %s ... with single PTE of THP (%zu kB)\n", desc, size / 1024); do_run_with_thp(fn, THP_RUN_SINGLE_PTE, size); } static void run_with_single_pte_of_thp_swap(test_fn fn, const char *desc, size_t size) { - log_test_start("%s ... with single PTE of swapped-out THP (%zu kB)", + ksft_print_msg("[RUN] %s ... with single PTE of swapped-out THP (%zu kB)\n", desc, size / 1024); do_run_with_thp(fn, THP_RUN_SINGLE_PTE_SWAPOUT, size); } static void run_with_partial_mremap_thp(test_fn fn, const char *desc, size_t size) { - log_test_start("%s ... with partially mremap()'ed THP (%zu kB)", + ksft_print_msg("[RUN] %s ... with partially mremap()'ed THP (%zu kB)\n", desc, size / 1024); do_run_with_thp(fn, THP_RUN_PARTIAL_MREMAP, size); } static void run_with_partial_shared_thp(test_fn fn, const char *desc, size_t size) { - log_test_start("%s ... with partially shared THP (%zu kB)", + ksft_print_msg("[RUN] %s ... with partially shared THP (%zu kB)\n", desc, size / 1024); do_run_with_thp(fn, THP_RUN_PARTIAL_SHARED, size); } @@ -1064,15 +1000,14 @@ static void run_with_hugetlb(test_fn fn, const char *desc, size_t hugetlbsize) int flags = MAP_PRIVATE | MAP_ANONYMOUS | MAP_HUGETLB; char *mem, *dummy; - log_test_start("%s ... with hugetlb (%zu kB)", desc, + ksft_print_msg("[RUN] %s ... with hugetlb (%zu kB)\n", desc, hugetlbsize / 1024); flags |= __builtin_ctzll(hugetlbsize) << MAP_HUGE_SHIFT; mem = mmap(NULL, hugetlbsize, PROT_READ | PROT_WRITE, flags, -1, 0); if (mem == MAP_FAILED) { - ksft_perror("need more free huge pages"); - log_test_result(KSFT_SKIP); + ksft_test_result_skip("need more free huge pages\n"); return; } @@ -1085,8 +1020,7 @@ static void run_with_hugetlb(test_fn fn, const char *desc, size_t hugetlbsize) */ dummy = mmap(NULL, hugetlbsize, PROT_READ | PROT_WRITE, flags, -1, 0); if (dummy == MAP_FAILED) { - ksft_perror("need more free huge pages"); - log_test_result(KSFT_SKIP); + ksft_test_result_skip("need more free huge pages\n"); goto munmap; } munmap(dummy, hugetlbsize); @@ -1292,7 +1226,7 @@ static void do_test_anon_thp_collapse(char *mem, size_t size, ret = setup_comm_pipes(&comm_pipes); if (ret) { - log_test_result(KSFT_FAIL); + ksft_test_result_fail("pipe() failed\n"); return; } @@ -1302,14 +1236,12 @@ static void do_test_anon_thp_collapse(char *mem, size_t size, */ ret = mprotect(mem + pagesize, pagesize, PROT_READ); if (ret) { - ksft_perror("mprotect() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("mprotect() failed\n"); goto close_comm_pipes; } ret = mprotect(mem + pagesize, pagesize, PROT_READ | PROT_WRITE); if (ret) { - ksft_perror("mprotect() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("mprotect() failed\n"); goto close_comm_pipes; } @@ -1318,8 +1250,8 @@ static void do_test_anon_thp_collapse(char *mem, size_t size, /* Collapse before actually COW-sharing the page. */ ret = madvise(mem, size, MADV_COLLAPSE); if (ret) { - ksft_perror("MADV_COLLAPSE failed"); - log_test_result(KSFT_SKIP); + ksft_test_result_skip("MADV_COLLAPSE failed: %s\n", + strerror(errno)); goto close_comm_pipes; } break; @@ -1330,8 +1262,7 @@ static void do_test_anon_thp_collapse(char *mem, size_t size, /* Don't COW-share the upper part of the THP. */ ret = madvise(mem + size / 2, size / 2, MADV_DONTFORK); if (ret) { - ksft_perror("MADV_DONTFORK failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("MADV_DONTFORK failed\n"); goto close_comm_pipes; } break; @@ -1339,8 +1270,7 @@ static void do_test_anon_thp_collapse(char *mem, size_t size, /* Don't COW-share the lower part of the THP. */ ret = madvise(mem, size / 2, MADV_DONTFORK); if (ret) { - ksft_perror("MADV_DONTFORK failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("MADV_DONTFORK failed\n"); goto close_comm_pipes; } break; @@ -1350,8 +1280,7 @@ static void do_test_anon_thp_collapse(char *mem, size_t size, ret = fork(); if (ret < 0) { - ksft_perror("fork() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("fork() failed\n"); goto close_comm_pipes; } else if (!ret) { switch (test) { @@ -1385,8 +1314,7 @@ static void do_test_anon_thp_collapse(char *mem, size_t size, */ ret = madvise(mem, size, MADV_DOFORK); if (ret) { - ksft_perror("MADV_DOFORK failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("MADV_DOFORK failed\n"); write(comm_pipes.parent_ready[1], "0", 1); wait(&ret); goto close_comm_pipes; @@ -1396,8 +1324,8 @@ static void do_test_anon_thp_collapse(char *mem, size_t size, /* Collapse before anyone modified the COW-shared page. */ ret = madvise(mem, size, MADV_COLLAPSE); if (ret) { - ksft_perror("MADV_COLLAPSE failed"); - log_test_result(KSFT_SKIP); + ksft_test_result_skip("MADV_COLLAPSE failed: %s\n", + strerror(errno)); write(comm_pipes.parent_ready[1], "0", 1); wait(&ret); goto close_comm_pipes; @@ -1417,10 +1345,7 @@ static void do_test_anon_thp_collapse(char *mem, size_t size, else ret = -EINVAL; - if (!ret) - log_test_result(KSFT_PASS); - else - log_test_result(KSFT_FAIL); + ksft_test_result(!ret, "No leak from parent into child\n"); close_comm_pipes: close_comm_pipes(&comm_pipes); } @@ -1505,7 +1430,7 @@ static void run_anon_thp_test_cases(void) for (i = 0; i < ARRAY_SIZE(anon_thp_test_cases); i++) { struct test_case const *test_case = &anon_thp_test_cases[i]; - log_test_start("%s", test_case->desc); + ksft_print_msg("[RUN] %s\n", test_case->desc); do_run_with_thp(test_case->fn, THP_RUN_PMD, pmdsize); } } @@ -1528,10 +1453,8 @@ static void test_cow(char *mem, const char *smem, size_t size) memset(mem, 0xff, size); /* See if we still read the old values via the other mapping. */ - if (!memcmp(smem, old, size)) - log_test_result(KSFT_PASS); - else - log_test_result(KSFT_FAIL); + ksft_test_result(!memcmp(smem, old, size), + "Other mapping not modified\n"); free(old); } @@ -1549,20 +1472,18 @@ static void run_with_zeropage(non_anon_test_fn fn, const char *desc) { char *mem, *smem, tmp; - log_test_start("%s ... with shared zeropage", desc); + ksft_print_msg("[RUN] %s ... with shared zeropage\n", desc); mem = mmap(NULL, pagesize, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANON, -1, 0); if (mem == MAP_FAILED) { - ksft_perror("mmap() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("mmap() failed\n"); return; } smem = mmap(NULL, pagesize, PROT_READ, MAP_PRIVATE | MAP_ANON, -1, 0); if (smem == MAP_FAILED) { - ksft_perror("mmap() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("mmap() failed\n"); goto munmap; } @@ -1583,11 +1504,10 @@ static void run_with_huge_zeropage(non_anon_test_fn fn, const char *desc) size_t mmap_size; int ret; - log_test_start("%s ... with huge zeropage", desc); + ksft_print_msg("[RUN] %s ... with huge zeropage\n", desc); if (!has_huge_zeropage) { - ksft_print_msg("Huge zeropage not enabled\n"); - log_test_result(KSFT_SKIP); + ksft_test_result_skip("Huge zeropage not enabled\n"); return; } @@ -1596,15 +1516,13 @@ static void run_with_huge_zeropage(non_anon_test_fn fn, const char *desc) mmap_mem = mmap(NULL, mmap_size, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANONYMOUS, -1, 0); if (mmap_mem == MAP_FAILED) { - ksft_perror("mmap() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("mmap() failed\n"); return; } mmap_smem = mmap(NULL, mmap_size, PROT_READ, MAP_PRIVATE | MAP_ANONYMOUS, -1, 0); if (mmap_smem == MAP_FAILED) { - ksft_perror("mmap() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("mmap() failed\n"); goto munmap; } @@ -1613,15 +1531,9 @@ static void run_with_huge_zeropage(non_anon_test_fn fn, const char *desc) smem = (char *)(((uintptr_t)mmap_smem + pmdsize) & ~(pmdsize - 1)); ret = madvise(mem, pmdsize, MADV_HUGEPAGE); - if (ret != 0) { - ksft_perror("madvise()"); - log_test_result(KSFT_FAIL); - goto munmap; - } ret |= madvise(smem, pmdsize, MADV_HUGEPAGE); - if (ret != 0) { - ksft_perror("madvise()"); - log_test_result(KSFT_FAIL); + if (ret) { + ksft_test_result_fail("MADV_HUGEPAGE failed\n"); goto munmap; } @@ -1650,33 +1562,29 @@ static void run_with_memfd(non_anon_test_fn fn, const char *desc) char *mem, *smem, tmp; int fd; - log_test_start("%s ... with memfd", desc); + ksft_print_msg("[RUN] %s ... with memfd\n", desc); fd = memfd_create("test", 0); if (fd < 0) { - ksft_perror("memfd_create() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("memfd_create() failed\n"); return; } /* File consists of a single page filled with zeroes. */ if (fallocate(fd, 0, 0, pagesize)) { - ksft_perror("fallocate() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("fallocate() failed\n"); goto close; } /* Create a private mapping of the memfd. */ mem = mmap(NULL, pagesize, PROT_READ | PROT_WRITE, MAP_PRIVATE, fd, 0); if (mem == MAP_FAILED) { - ksft_perror("mmap() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("mmap() failed\n"); goto close; } smem = mmap(NULL, pagesize, PROT_READ, MAP_SHARED, fd, 0); if (smem == MAP_FAILED) { - ksft_perror("mmap() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("mmap() failed\n"); goto munmap; } @@ -1699,40 +1607,35 @@ static void run_with_tmpfile(non_anon_test_fn fn, const char *desc) FILE *file; int fd; - log_test_start("%s ... with tmpfile", desc); + ksft_print_msg("[RUN] %s ... with tmpfile\n", desc); file = tmpfile(); if (!file) { - ksft_perror("tmpfile() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("tmpfile() failed\n"); return; } fd = fileno(file); if (fd < 0) { - ksft_perror("fileno() failed"); - log_test_result(KSFT_SKIP); + ksft_test_result_skip("fileno() failed\n"); return; } /* File consists of a single page filled with zeroes. */ if (fallocate(fd, 0, 0, pagesize)) { - ksft_perror("fallocate() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("fallocate() failed\n"); goto close; } /* Create a private mapping of the memfd. */ mem = mmap(NULL, pagesize, PROT_READ | PROT_WRITE, MAP_PRIVATE, fd, 0); if (mem == MAP_FAILED) { - ksft_perror("mmap() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("mmap() failed\n"); goto close; } smem = mmap(NULL, pagesize, PROT_READ, MAP_SHARED, fd, 0); if (smem == MAP_FAILED) { - ksft_perror("mmap() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("mmap() failed\n"); goto munmap; } @@ -1756,22 +1659,20 @@ static void run_with_memfd_hugetlb(non_anon_test_fn fn, const char *desc, char *mem, *smem, tmp; int fd; - log_test_start("%s ... with memfd hugetlb (%zu kB)", desc, + ksft_print_msg("[RUN] %s ... with memfd hugetlb (%zu kB)\n", desc, hugetlbsize / 1024); flags |= __builtin_ctzll(hugetlbsize) << MFD_HUGE_SHIFT; fd = memfd_create("test", flags); if (fd < 0) { - ksft_perror("memfd_create() failed"); - log_test_result(KSFT_SKIP); + ksft_test_result_skip("memfd_create() failed\n"); return; } /* File consists of a single page filled with zeroes. */ if (fallocate(fd, 0, 0, hugetlbsize)) { - ksft_perror("need more free huge pages"); - log_test_result(KSFT_SKIP); + ksft_test_result_skip("need more free huge pages\n"); goto close; } @@ -1779,14 +1680,12 @@ static void run_with_memfd_hugetlb(non_anon_test_fn fn, const char *desc, mem = mmap(NULL, hugetlbsize, PROT_READ | PROT_WRITE, MAP_PRIVATE, fd, 0); if (mem == MAP_FAILED) { - ksft_perror("need more free huge pages"); - log_test_result(KSFT_SKIP); + ksft_test_result_skip("need more free huge pages\n"); goto close; } smem = mmap(NULL, hugetlbsize, PROT_READ, MAP_SHARED, fd, 0); if (smem == MAP_FAILED) { - ksft_perror("mmap() failed"); - log_test_result(KSFT_FAIL); + ksft_test_result_fail("mmap() failed\n"); goto munmap; } -- 2.53.0