From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id CB221F99C7C for ; Sat, 18 Apr 2026 10:56:59 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 3E92D6B020E; Sat, 18 Apr 2026 06:56:59 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 39EB56B0213; Sat, 18 Apr 2026 06:56:59 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 288E46B0246; Sat, 18 Apr 2026 06:56:59 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id 140F06B020E for ; Sat, 18 Apr 2026 06:56:59 -0400 (EDT) Received: from smtpin20.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id D569113B006 for ; Sat, 18 Apr 2026 10:56:58 +0000 (UTC) X-FDA: 84671374116.20.70020F0 Received: from sea.source.kernel.org (sea.source.kernel.org [172.234.252.31]) by imf21.hostedemail.com (Postfix) with ESMTP id 26A071C0002 for ; Sat, 18 Apr 2026 10:56:56 +0000 (UTC) Authentication-Results: imf21.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=njfDby3R; dmarc=pass (policy=quarantine) header.from=kernel.org; spf=pass (imf21.hostedemail.com: domain of rppt@kernel.org designates 172.234.252.31 as permitted sender) smtp.mailfrom=rppt@kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1776509817; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=wq7KOyoGG5rSzdZE/i533JG3hRgpPVpE76XvvG6Y/gU=; b=ZAg6YOu+LKXnUVnGkAMVppc8SYR3bt5r//9leKWq4PSZaNYkgrwrD7VQld54dnWiWbHXsy AUH+wUNPfb4HNo9pSoGEzRc7ZoWGV0q47JJRKxs7kLWYA86wdMrEHCDxK2cnWG6vB8Zk9f 8ZTaadZmeghT7HpFRBtdk5Zam4BLQzA= ARC-Authentication-Results: i=1; imf21.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=njfDby3R; dmarc=pass (policy=quarantine) header.from=kernel.org; spf=pass (imf21.hostedemail.com: domain of rppt@kernel.org designates 172.234.252.31 as permitted sender) smtp.mailfrom=rppt@kernel.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1776509817; a=rsa-sha256; cv=none; b=wSLDl2Np0uLzVYkEqdp7PLNkuJPjdbC0Hr9q0dmClcwfai8WLd7LUcI/4C2qgq4cXgBvAB bzJ3shC301bzGUrU+EITBrNRkKGRcMfEpYBiVRF8wDOtYFmTGPWkOwOQL+PW8NUNoKRBnv ysN6Of1IIqejulm781Roi1TEaSnVuOw= Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sea.source.kernel.org (Postfix) with ESMTP id 2DBD9435C8; Sat, 18 Apr 2026 10:56:56 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 65D7CC19424; Sat, 18 Apr 2026 10:56:50 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1776509816; bh=oZxt8zBBbGTvdYs1lP1r+GpZXATmjus45JMc8292lqk=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=njfDby3RQ4O+Kd7He1QTqhtFGrrrVM6NmRYHnt7tjUqXmGtdSNyLDpoWfRLREWrRS OKHxVLXUosEkc6DjnM6p92gsyjJYNH3fegI7DUk0+IkpJ1nDjehJxC4PsHhyKBV76E IFt9t/XyI84K4dG296e0hNMCV+DXuw1wN8siXSl1BBu7IgAdHmyue3BKMrHMEmVDM6 yMubT1gNT99Vg2Y8Okmz35kX4gmZRTH8URhhLUZfSQkpJ79ySR+zudwoSy9XnhpNNp cyD4qub05ar2MaGznUtQq9znGgVMT8dwnJ/eOHObXIXz1KPRsuH2prtQhgepsYBYTM ZxOw0QpjZ1ynw== From: Mike Rapoport To: Andrew Morton , David Hildenbrand Cc: Baolin Wang , Barry Song , Dev Jain , Donet Tom , Jason Gunthorpe , John Hubbard , "Liam R. Howlett" , Lance Yang , Leon Romanovsky , Lorenzo Stoakes , Mark Brown , Michal Hocko , Mike Rapoport , Nico Pache , Peter Xu , Ryan Roberts , Sarthak Sharma , Shuah Khan , Suren Baghdasaryan , Vlastimil Babka , Zi Yan , linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org, linux-mm@kvack.org Subject: [PATCH v2 11/53] selftests/mm: hugetlb-read-hwpoison: use kselftest framework Date: Sat, 18 Apr 2026 13:54:56 +0300 Message-ID: <20260418105539.1261536-12-rppt@kernel.org> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260418105539.1261536-1-rppt@kernel.org> References: <20260418105539.1261536-1-rppt@kernel.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Rspam-User: X-Rspamd-Server: rspam01 X-Rspamd-Queue-Id: 26A071C0002 X-Stat-Signature: 875dui8gt5wpo5jn7z31fw8y8hg1844o X-HE-Tag: 1776509816-417117 X-HE-Meta: U2FsdGVkX1+XCypliZn7KIKADaLw40R5vCynIVxc+A/jw6tIpe+Ss6qC2Y1F0bwvR8tjCLFo77IpsTh+topXNgLhWeYeUdYRwNcLgbGIG5ByFF1fFX53Wjxmq/RIB7Y/73d+zAeRrbBDe8mD3nnd05YlP2IoepdiinXxIBWkwvKkL/GifSbl2BQsr6NJEWGh25fe0sHheYfHt7Mjiv2ARLOtjqRn55G8BGZSTc7TqJPckPOLIWqeqJ20ttSRW3e+xqSk2mSUb+TAFPm22Q5gDE/pG+Vxs+Dh66OsCYpuQSKj3jqXxIx+EoSnzTnhcWb386jW2nlo8OTwd8CO5Q3Vxug+gLAPt3ZhMp/0GwrhvfwKW9gnVtPouNcqinsPSTEhB4grBTVyXHGOM6SMOGCyDi4x4PW5RkHH4KgDD6khLDEcQCYapRkyW+5fUgWuGfD/7gNcvD4Kn797+Z4vC3Jk1ZjW4UxoPixt8a+J9P+FRv5FZNEfUIX85iifmAHPoNOCgVkZPYu6HYZghnqMrB2b2TUlFqkRQbv9iC2G6YshXd1FlmspjkZb/sFEuoqwqrM+4BBm7l9SDkg79Lespv4JClDh89W2+Cgy3Sv7cUG48cN4QQpVCYAthBzz1JZt0Wumsuqbd156uatJEMsKDa5MK63sqUn88duidL345O4PWObQ+rrYSnDEAIUCieVEOYAhEKFSs9CFJDpmRrIw+2ucfi7wi/OFhOai3bf0TCGi6LZ3mLVf48BRDQeJz1ALx5uM1Eovc4v5Y8/bmZEtwaM1nnj/D9gzC/Kagi2JHUgk0q79BIy5TlW4jwFsYpJhsdVZV2QeCs61op1a5cwNvCcJZowiOpbexmTcXNA+ugYC5/a3TWEOyCanq42O1w932Ajaw0V1htj69wTzyLWHNzuhsKo6AYLlXNl1k9AV+6NC+yYqM9zI+E7FTF4cE7phEgHXCyFyXpVMVYmYM4dWGkb MNvXjSBZ W5R10EuPtcHd00Haj94VFsGe3NwT4/jLPyIe0iBbQKSYfHYUj/Pm2FY6xK1+92ifeXbt0WKvniMmTrxHnwJsv/KbyIkfFSS9jvUHOGcAUaV/wvd6Ec0IRMkAFVBd8Uxz2i/V9heieDfpNFeYwIGHoxDjnQ9v4BKS5nZ9xk65K5Mx/EkuAJiqu199AtmrIa7nT4KVPkPQo/AnlOxVqXzUVc5HpHkgUNWlXUE+V4IfH4xFbpNgnneoAGgR6/goHz+XEMf2f Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: From: "Mike Rapoport (Microsoft)" Convert hugetlb-read-hwpoison test to use kselftest framework for reporting and tracking successful and failing runs. Reviewed-by: Mark Brown Reviewed by: Donet Tom Signed-off-by: Mike Rapoport (Microsoft) --- .../selftests/mm/hugetlb-read-hwpoison.c | 118 +++++++++--------- 1 file changed, 57 insertions(+), 61 deletions(-) diff --git a/tools/testing/selftests/mm/hugetlb-read-hwpoison.c b/tools/testing/selftests/mm/hugetlb-read-hwpoison.c index 6bbf15f78061..01cd5465e0f8 100644 --- a/tools/testing/selftests/mm/hugetlb-read-hwpoison.c +++ b/tools/testing/selftests/mm/hugetlb-read-hwpoison.c @@ -13,9 +13,7 @@ #include #include "kselftest.h" - -#define PREFIX " ... " -#define ERROR_PREFIX " !!! " +#include "vm_util.h" #define MAX_WRITE_READ_CHUNK_SIZE (getpagesize() * 16) #define MAX(a, b) (((a) > (b)) ? (a) : (b)) @@ -26,17 +24,22 @@ enum test_status { TEST_SKIPPED = 2, }; -static char *status_to_str(enum test_status status) +static void report_status(enum test_status status, const char *test_name, + size_t chunk_size) { switch (status) { case TEST_PASSED: - return "TEST_PASSED"; + ksft_test_result_pass("%s chunk_size=0x%lx\n", + test_name, chunk_size); + break; case TEST_FAILED: - return "TEST_FAILED"; + ksft_test_result_fail("%s chunk_size=0x%lx\n", + test_name, chunk_size); + break; case TEST_SKIPPED: - return "TEST_SKIPPED"; - default: - return "TEST_???"; + ksft_test_result_skip("%s chunk_size=0x%lx\n", + test_name, chunk_size); + break; } } @@ -59,8 +62,8 @@ static bool verify_chunk(char *buf, size_t len, char val) for (i = 0; i < len; ++i) { if (buf[i] != val) { - printf(PREFIX ERROR_PREFIX "check fail: buf[%lu] = %u != %u\n", - i, buf[i], val); + ksft_print_msg("check fail: buf[%lu] = %u != %u\n", + i, buf[i], val); return false; } } @@ -76,21 +79,21 @@ static bool seek_read_hugepage_filemap(int fd, size_t len, size_t wr_chunk_size, ssize_t total_ret_count = 0; char val = offset / wr_chunk_size + offset % wr_chunk_size; - printf(PREFIX PREFIX "init val=%u with offset=0x%lx\n", val, offset); - printf(PREFIX PREFIX "expect to read 0x%lx bytes of data in total\n", - expected); + ksft_print_msg("init val=%u with offset=0x%lx\n", val, offset); + ksft_print_msg("expect to read 0x%lx bytes of data in total\n", + expected); if (lseek(fd, offset, SEEK_SET) < 0) { - perror(PREFIX ERROR_PREFIX "seek failed"); + ksft_perror("seek failed"); return false; } while (offset + total_ret_count < len) { ret_count = read(fd, buf, wr_chunk_size); if (ret_count == 0) { - printf(PREFIX PREFIX "read reach end of the file\n"); + ksft_print_msg("read reach end of the file\n"); break; } else if (ret_count < 0) { - perror(PREFIX ERROR_PREFIX "read failed"); + ksft_perror("read failed"); break; } ++val; @@ -99,8 +102,8 @@ static bool seek_read_hugepage_filemap(int fd, size_t len, size_t wr_chunk_size, total_ret_count += ret_count; } - printf(PREFIX PREFIX "actually read 0x%lx bytes of data in total\n", - total_ret_count); + ksft_print_msg("actually read 0x%lx bytes of data in total\n", + total_ret_count); return total_ret_count == expected; } @@ -113,15 +116,15 @@ static bool read_hugepage_filemap(int fd, size_t len, ssize_t total_ret_count = 0; char val = 0; - printf(PREFIX PREFIX "expect to read 0x%lx bytes of data in total\n", - expected); + ksft_print_msg("expect to read 0x%lx bytes of data in total\n", + expected); while (total_ret_count < len) { ret_count = read(fd, buf, wr_chunk_size); if (ret_count == 0) { - printf(PREFIX PREFIX "read reach end of the file\n"); + ksft_print_msg("read reach end of the file\n"); break; } else if (ret_count < 0) { - perror(PREFIX ERROR_PREFIX "read failed"); + ksft_perror("read failed"); break; } ++val; @@ -130,8 +133,8 @@ static bool read_hugepage_filemap(int fd, size_t len, total_ret_count += ret_count; } - printf(PREFIX PREFIX "actually read 0x%lx bytes of data in total\n", - total_ret_count); + ksft_print_msg("actually read 0x%lx bytes of data in total\n", + total_ret_count); return total_ret_count == expected; } @@ -143,14 +146,14 @@ test_hugetlb_read(int fd, size_t len, size_t wr_chunk_size) char *filemap = NULL; if (ftruncate(fd, len) < 0) { - perror(PREFIX ERROR_PREFIX "ftruncate failed"); + ksft_perror("ftruncate failed"); return status; } filemap = mmap(NULL, len, PROT_READ | PROT_WRITE, MAP_SHARED | MAP_POPULATE, fd, 0); if (filemap == MAP_FAILED) { - perror(PREFIX ERROR_PREFIX "mmap for primary mapping failed"); + ksft_perror("mmap for primary mapping failed"); goto done; } @@ -163,7 +166,7 @@ test_hugetlb_read(int fd, size_t len, size_t wr_chunk_size) munmap(filemap, len); done: if (ftruncate(fd, 0) < 0) { - perror(PREFIX ERROR_PREFIX "ftruncate back to 0 failed"); + ksft_perror("ftruncate back to 0 failed"); status = TEST_FAILED; } @@ -180,14 +183,14 @@ test_hugetlb_read_hwpoison(int fd, size_t len, size_t wr_chunk_size, const unsigned long pagesize = getpagesize(); if (ftruncate(fd, len) < 0) { - perror(PREFIX ERROR_PREFIX "ftruncate failed"); + ksft_perror("ftruncate failed"); return status; } filemap = mmap(NULL, len, PROT_READ | PROT_WRITE, MAP_SHARED | MAP_POPULATE, fd, 0); if (filemap == MAP_FAILED) { - perror(PREFIX ERROR_PREFIX "mmap for primary mapping failed"); + ksft_perror("mmap for primary mapping failed"); goto done; } @@ -202,7 +205,7 @@ test_hugetlb_read_hwpoison(int fd, size_t len, size_t wr_chunk_size, */ hwp_addr = filemap + len / 2 + pagesize; if (madvise(hwp_addr, pagesize, MADV_HWPOISON) < 0) { - perror(PREFIX ERROR_PREFIX "MADV_HWPOISON failed"); + ksft_perror("MADV_HWPOISON failed"); goto unmap; } @@ -229,7 +232,7 @@ test_hugetlb_read_hwpoison(int fd, size_t len, size_t wr_chunk_size, munmap(filemap, len); done: if (ftruncate(fd, 0) < 0) { - perror(PREFIX ERROR_PREFIX "ftruncate back to 0 failed"); + ksft_perror("ftruncate back to 0 failed"); status = TEST_FAILED; } @@ -242,17 +245,17 @@ static int create_hugetlbfs_file(struct statfs *file_stat) fd = memfd_create("hugetlb_tmp", MFD_HUGETLB); if (fd < 0) { - perror(PREFIX ERROR_PREFIX "could not open hugetlbfs file"); + ksft_perror("could not open hugetlbfs file"); return -1; } memset(file_stat, 0, sizeof(*file_stat)); if (fstatfs(fd, file_stat)) { - perror(PREFIX ERROR_PREFIX "fstatfs failed"); + ksft_perror("fstatfs failed"); goto close; } if (file_stat->f_type != HUGETLBFS_MAGIC) { - printf(PREFIX ERROR_PREFIX "not hugetlbfs file\n"); + ksft_print_msg("not hugetlbfs file\n"); goto close; } @@ -264,7 +267,7 @@ static int create_hugetlbfs_file(struct statfs *file_stat) static void sigbus_handler(int sig) { - printf(PREFIX "received SIGBUS\n"); + ksft_print_msg("received SIGBUS\n"); } int main(void) @@ -279,51 +282,44 @@ int main(void) }; size_t i; + ksft_print_header(); + ksft_set_plan(ARRAY_SIZE(wr_chunk_sizes) * 3); + signal(SIGBUS, sigbus_handler); for (i = 0; i < ARRAY_SIZE(wr_chunk_sizes); ++i) { - printf("Write/read chunk size=0x%lx\n", - wr_chunk_sizes[i]); + ksft_print_msg("Write/read chunk size=0x%lx\n", + wr_chunk_sizes[i]); fd = create_hugetlbfs_file(&file_stat); if (fd < 0) - goto create_failure; - printf(PREFIX "HugeTLB read regression test...\n"); + ksft_exit_fail_msg("Failed to create hugetlbfs file\n"); + status = test_hugetlb_read(fd, file_stat.f_bsize, wr_chunk_sizes[i]); - printf(PREFIX "HugeTLB read regression test...%s\n", - status_to_str(status)); close(fd); - if (status == TEST_FAILED) - return -1; + report_status(status, "HugeTLB read regression", + wr_chunk_sizes[i]); fd = create_hugetlbfs_file(&file_stat); if (fd < 0) - goto create_failure; - printf(PREFIX "HugeTLB read HWPOISON test...\n"); + ksft_exit_fail_msg("Failed to create hugetlbfs file\n"); + status = test_hugetlb_read_hwpoison(fd, file_stat.f_bsize, wr_chunk_sizes[i], false); - printf(PREFIX "HugeTLB read HWPOISON test...%s\n", - status_to_str(status)); close(fd); - if (status == TEST_FAILED) - return -1; + report_status(status, "HugeTLB read HWPOISON", + wr_chunk_sizes[i]); fd = create_hugetlbfs_file(&file_stat); if (fd < 0) - goto create_failure; - printf(PREFIX "HugeTLB seek then read HWPOISON test...\n"); + ksft_exit_fail_msg("Failed to create hugetlbfs file\n"); + status = test_hugetlb_read_hwpoison(fd, file_stat.f_bsize, wr_chunk_sizes[i], true); - printf(PREFIX "HugeTLB seek then read HWPOISON test...%s\n", - status_to_str(status)); close(fd); - if (status == TEST_FAILED) - return -1; + report_status(status, "HugeTLB seek then read HWPOISON", + wr_chunk_sizes[i]); } - return 0; - -create_failure: - printf(ERROR_PREFIX "Abort test: failed to create hugetlbfs file\n"); - return -1; + ksft_finished(); } -- 2.53.0