From patchwork Fri Apr 5 13:17:47 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Muhammad Usama Anjum X-Patchwork-Id: 13619062 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]) by smtp.lore.kernel.org (Postfix) with ESMTP id 97584C67861 for ; Fri, 5 Apr 2024 13:18:16 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 1EC8A6B017B; Fri, 5 Apr 2024 09:18:16 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 19DB36B017D; Fri, 5 Apr 2024 09:18:16 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 03D056B017E; Fri, 5 Apr 2024 09:18:15 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id DB6B26B017B for ; Fri, 5 Apr 2024 09:18:15 -0400 (EDT) Received: from smtpin06.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay05.hostedemail.com (Postfix) with ESMTP id 7551B411D5 for ; Fri, 5 Apr 2024 13:18:15 +0000 (UTC) X-FDA: 81975531750.06.DCAE005 Received: from madrid.collaboradmins.com (madrid.collaboradmins.com [46.235.227.194]) by imf04.hostedemail.com (Postfix) with ESMTP id B8F5A40003 for ; Fri, 5 Apr 2024 13:18:13 +0000 (UTC) Authentication-Results: imf04.hostedemail.com; dkim=pass header.d=collabora.com header.s=mail header.b=4kK09E5y; spf=pass (imf04.hostedemail.com: domain of usama.anjum@collabora.com designates 46.235.227.194 as permitted sender) smtp.mailfrom=usama.anjum@collabora.com; dmarc=pass (policy=quarantine) header.from=collabora.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1712323093; 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=0rpP1b9ywzeBVaIjeTitxCbUcmIJSm+jrrHMh0zIUvo=; b=C7Xl0u5OfHZmht5i2JaqHBUjtsaSumnNhr+Aaw/uhMnT4nTOvtMiH/ID19YGqIKtS9F8JY OuE46Wj5XSjQaZ3dy1PVvdXWDvCHGT0KxpKqx3kpC+48d9GZVN9ieXOzFYPX64ALMBmBub 7xUIsF5rowSvkOGHVYGGRBLS4E8MAPI= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1712323093; a=rsa-sha256; cv=none; b=OOaBcjBgpU56qB8AUi2FvmQ/y1tcSOlAZQ15D5M0KnqPgCnWJRQ5ytT88KipWN6Nh17rcB zONMAX6oIhsjq5XauRlDKnT0HnaqSEOJ7Vy1+2XuEv1WxM62Z3scDZXBDR6xqQuYoNN6Nj XmEKNF2ehgJwUE2hV6e2LwL7aNgaEWM= ARC-Authentication-Results: i=1; imf04.hostedemail.com; dkim=pass header.d=collabora.com header.s=mail header.b=4kK09E5y; spf=pass (imf04.hostedemail.com: domain of usama.anjum@collabora.com designates 46.235.227.194 as permitted sender) smtp.mailfrom=usama.anjum@collabora.com; dmarc=pass (policy=quarantine) header.from=collabora.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1712323092; bh=BK5hFtNFiolnEtNNhwHZpqgolDF4VaR+KUumWAMdM1A=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=4kK09E5yM/N5Y46oRwtZvG0xaYdNBBoY3Ta/kCD2Dd8fITbqzSB5MwLHdpIW7s4AH JzH0nrhZbvvcMIrGOvKMvQnlWM61Nd5OFA/gzFVjQz3Jnyvtyd19UsH/kpWJksWxM4 +sPw0Z0FyE7PMG0VFYXfkpdg1Sw8rEQNABIfAAJ7io5IPY5NmRJJfK2nZq/wxO0l+x lbcrt+IIdt6A5qXttc/8wQVlTDxGijbu/zXutjxLKHYU4aOn1TLPfX5jK6LVD32K7l PzdCEJHiehqOONBXTYyclHJS2hzL4CPES0e+64R/YKbXqGL883xWem24jsls29s614 h/cQQDqJcwqUg== Received: from localhost.localdomain (broslavsky.collaboradmins.com [68.183.210.73]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: usama.anjum) by madrid.collaboradmins.com (Postfix) with ESMTPSA id EDDEA3782131; Fri, 5 Apr 2024 13:18:08 +0000 (UTC) From: Muhammad Usama Anjum To: Eric Biederman , Kees Cook , Shuah Khan , Andrew Morton , Muhammad Usama Anjum , linux-mm@kvack.org, linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org Cc: kernel@collabora.com Subject: [PATCH 1/2] selftests: Replace "Bail out" with "Error" Date: Fri, 5 Apr 2024 18:17:47 +0500 Message-Id: <20240405131748.1582646-2-usama.anjum@collabora.com> X-Mailer: git-send-email 2.39.2 In-Reply-To: <20240405131748.1582646-1-usama.anjum@collabora.com> References: <20240405131748.1582646-1-usama.anjum@collabora.com> MIME-Version: 1.0 X-Rspamd-Queue-Id: B8F5A40003 X-Rspam-User: X-Stat-Signature: iuqbrayjy63xow1spq531j9kqixtcsd9 X-Rspamd-Server: rspam03 X-HE-Tag: 1712323093-954911 X-HE-Meta: U2FsdGVkX19ZqImXgVSKGqnnB2cj+PNhpdcLX3DZhreqKLIjyKX6VCM6Sl8tJTHZ3xnkxtTBYUyUFo87aAiKWdkv4phbhxLD2NfRCXTs5emgcT2LWuoixpqqeInhWCtaElhlLHtUNPDJlL3N5li9xoQRJmJ26fEC46hgoOIgxSXB87AX5Epf7lyodFxb5NrK6qANFVHTMtafS+2kgxmiABKFzOaCloEXy5Ccu4mWERKbljfByQqMDUX1mLNoGRjWJDFTLz3mX72s5JSAJFjZEgU1UvMocfLyhS0uxeiKklHoAblRSkU5sBAbunuUQCQIUhkRA57RZFRtofzGMO9x2jJSdpExYVBqwV4drNuarzkpc/tKan+5vAm26JONvREQIccVumscl4si3Fdw/VcUVm/BeeLVevcOWQZsfMyGk/OopYgSSKfgqcj422CV25jkfeeLA6IP/b7gXhDgunyXBjOnPeI+A/9suBAJJn/qHXJa3TnSq3ZH6WcrQwMz4KiwrjGQy7/9VH44VAmIO0WIFsihofla0/SymRotjB4C8uMRFrZRCGWkNDJf1mQq567fCI/WPUGD+BgxAHzZrTDOYvw+GwEpQvJ7Jm1jCNwkOY+8wpfrfHjKBpT9TsQZ906lzirkvFGThPcF6OkcT9QwBT6w8+KgN5HSHmLPtPBqse/a1UJAd71M+Hef05fD6jQnGBarDDzguAYpNLXgkItct5saFdeLbgWMaYdfZK/CWYAzg8DZ9XQSQcP6JBtHrzSYiidP253Hebm4wnJlrsILSZZlu0809Yrx+xBF4FDLjBqJHMJT2Bp/APjdOEpRz6NV15jLn8XpX6J77N3Lpgth63oLV+6n7rm/EmWzKO6NSLTKsulUA08Qhdu4ZctJ0OhJ0RIepwuUKv5BD4w3pHZ/eMfY8YDcGv89WHvUUrWC0GdK2YZHHlvIaeRoZYg24enn1a6LrLj8ZUULM6s7BAG SDcuASxg sf8aJl4pmjHiKWL+YiDxWURWil76/ae6DEEvzaddbKSRbX7vyZpT1+VOzg60Mz9BiUzxGjDtTkYvpJ6Q+nNwZJGF603X5x+7b4ypbE/wViMic0atnaIhPiZpAX/RYC0moPt464Nd4NlLgMxTEny9KUnKtITMotzp9W95LZdlIGJSSiFOLCeHsg2Q3hvRrsBfRvTUafNqYMhuWJ2e7hcgwWldJ40HdhArjtateyzcaxzxgdd3G4RF3JhdWoU1PUgVOiForSRsEMoXHAhKUSDGnbo3h6v8nxNMvNjZnrpx9fXyhjRKLFwsklt/Ruw== X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: "Bail out! " is not descriptive. It rather should be: "Failed: " and then that added prefix doesn't need to be added everywhere. Usually in the logs, we are searching for "Failed" or "Error" instead of "Bail out". Suggested-by: Kees Cook Signed-off-by: Muhammad Usama Anjum Reviewed-by: Kees Cook --- tools/testing/selftests/kselftest.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/testing/selftests/kselftest.h b/tools/testing/selftests/kselftest.h index 2cd93d220f434..9d1d8c5a73ab2 100644 --- a/tools/testing/selftests/kselftest.h +++ b/tools/testing/selftests/kselftest.h @@ -362,7 +362,7 @@ static inline __printf(1, 2) int ksft_exit_fail_msg(const char *msg, ...) va_list args; va_start(args, msg); - printf("Bail out! "); + printf("Error: "); errno = saved_errno; vprintf(msg, args); va_end(args); From patchwork Fri Apr 5 13:17:48 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Muhammad Usama Anjum X-Patchwork-Id: 13619063 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]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1C940CD11C2 for ; Fri, 5 Apr 2024 13:18:33 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id A026A6B017D; Fri, 5 Apr 2024 09:18:32 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 9B2996B017F; Fri, 5 Apr 2024 09:18:32 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 8056C6B0180; Fri, 5 Apr 2024 09:18:32 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id 579846B017D for ; Fri, 5 Apr 2024 09:18:32 -0400 (EDT) Received: from smtpin10.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id DA0608118F for ; Fri, 5 Apr 2024 13:18:31 +0000 (UTC) X-FDA: 81975532422.10.D596F16 Received: from madrid.collaboradmins.com (madrid.collaboradmins.com [46.235.227.194]) by imf20.hostedemail.com (Postfix) with ESMTP id C54391C0012 for ; Fri, 5 Apr 2024 13:18:29 +0000 (UTC) Authentication-Results: imf20.hostedemail.com; dkim=pass header.d=collabora.com header.s=mail header.b=gUfX2mXb; dmarc=pass (policy=quarantine) header.from=collabora.com; spf=pass (imf20.hostedemail.com: domain of usama.anjum@collabora.com designates 46.235.227.194 as permitted sender) smtp.mailfrom=usama.anjum@collabora.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1712323110; 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=tbhfBBQFbfjx8EvH9t7wwV64gRc28oYamdT7C8xNCX4=; b=Slv7EgN4cdya0Q0ZLUZBk0r8CJT3oeFx19VmKcG2ifO7T7RTuizYeC0TWPoZAeKx5cdww5 sJkMwaJcaKgNLJAPvDGEXb5UlK/bmkAauy76puduxPUK5wQPZEgvfjJ0AlzOgr23QTDT5x qqBTGnq9ehsQ7g2XMDT6qpSnm2ioUQY= ARC-Authentication-Results: i=1; imf20.hostedemail.com; dkim=pass header.d=collabora.com header.s=mail header.b=gUfX2mXb; dmarc=pass (policy=quarantine) header.from=collabora.com; spf=pass (imf20.hostedemail.com: domain of usama.anjum@collabora.com designates 46.235.227.194 as permitted sender) smtp.mailfrom=usama.anjum@collabora.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1712323110; a=rsa-sha256; cv=none; b=s3xrto+o/3i6cTDDGMP/69qTtpcm1WOu/zWDqrDu26/Cs0AFGRuYcqLnqqrGW1UJXBU/bn 7VAS1jNNFYopgAA5GNtIHgZ0uNdFYf5Bbw6Sdjze17VY4rhAArqW22KmTlEla/cj5DozfK RkNL63wPJo3tH+gxLB63PwLY+VCz7fs= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1712323108; bh=2NJTbB3KaJRZg2EfipZnI/yOoGaBufin+vZb188D0aw=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=gUfX2mXbHlaeYra2Prqyfh+icckGOEpq9cvuiO8/FveKqOaWIw4Ujpu/3siAnID0I QU3oGklkyUNZIstg+EujOVnN8JgvMKIJNmuqircjMQG3IQkWqWZjvysI+mVY+4d1sh ZSSnqCoQUEo/AFkSZkUjJNdsb/wzhPImmRu9qm4aNGGbG86ZXKYt2uTFXhoGRoRVaj XHWIw8MqAZQQm16TbNRzKr9CTHWx9f6FB54eFBshsHLWZTJ4m0PMteIUo4szwXkOSR 4MtOxCxo6ZKbS0temHeWzoZNIBVMs4Q2d9JErFhYwiwtHktIBqL8Kqdhmb8gA9Kayy LbnTriNqjuX3Q== Received: from localhost.localdomain (broslavsky.collaboradmins.com [68.183.210.73]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: usama.anjum) by madrid.collaboradmins.com (Postfix) with ESMTPSA id 57C653782128; Fri, 5 Apr 2024 13:18:25 +0000 (UTC) From: Muhammad Usama Anjum To: Eric Biederman , Kees Cook , Shuah Khan , Andrew Morton , Muhammad Usama Anjum , linux-mm@kvack.org, linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org Cc: kernel@collabora.com Subject: [PATCH 2/2] selftests: Remove Error/Failed prefix from ksft_exit_fail*() usages Date: Fri, 5 Apr 2024 18:17:48 +0500 Message-Id: <20240405131748.1582646-3-usama.anjum@collabora.com> X-Mailer: git-send-email 2.39.2 In-Reply-To: <20240405131748.1582646-1-usama.anjum@collabora.com> References: <20240405131748.1582646-1-usama.anjum@collabora.com> MIME-Version: 1.0 X-Rspam-User: X-Rspamd-Server: rspam12 X-Rspamd-Queue-Id: C54391C0012 X-Stat-Signature: nda5hw1qkkxm4un5cdrksz4ku64ohn33 X-HE-Tag: 1712323109-644278 X-HE-Meta: U2FsdGVkX1/TCQ0r+7alNEkNI7CNLgxUBGM/rq4ZRFZoWcdz/ARy5w47kL6lzZxAoYIFWcREMxXxDizN1RgChi1JA983v2omWIPdfNSKBvAC0IqFb/eVlSkyhvTeSTbFlUi5ZqfetlZ74NK/c5q5s0zfDcmbsivt7zbL7O4+8Q+Yj9G7hpJRdY4UKkEDuTrel5ow8g06K33TH5ewkg630PBEsSRgkHZgst6L8qRFQLIlMBlZw45soAb4Stm9EfjpNVusRLPZFzwhznPBETY+3fpmbo4R6AbzZSxDle0p3DCvTzEFOCdhNf4f8GThXCaInbju/fSkJbssA3JNqOY4PEXMcRc9skoRH7Z9ARJZN5ytaw0F5pr7BZynqVoIM2GfBacAwvAu8KyBYPrSYXhpO1UQmBO7Qo3bIEWjGxpLu8NrhzUcUBNymNkKBdyAvc1tfh57JZyQ/ifM3yyi05xceRCKhjB3ototT8bWMHSvbZuVl7nO95o0gCzeeVsJr40xooQ1ObObXfE1yMVIn8CAR1KJYy3FllEh9tzdsvXSE+D1W7ih7Lnr0UslQl5Onzu65lCwEySsfUKH5nwkmzgcL3O362dBwi4cFRRYt6AWf5IlVGCq8Gw5IUfol0Oeb/u52xnFN/wxXfW9O45QB2AIfepjXOvXEEHYSR6AwadPykpHtUdQWk9fbNS5o86TdLzIzt6aVhaGW0k9p+9aIXU+hRP6Fcbm7Q3YrRNQB6FPF5iFuBbQRlSrBZ+ZaKyOIFVX1jXwgHzXA97KhESj1OeOROZYSbPw4/FUPbpmmsEXlGhPgkieyetQbXuMZu7G3TzorrKB+XtI0KrTVdxCe3OHIuTfh/8jHBBgNq/S5KtxUnIslleKUxyV9nr+BobDTKFFcQfCCqnhkr5bIpx8OOwZ3UylaziHy3uXS37QatH9EuEUYEu9OlS8XKS0nSFTGFA7iLZ6La2QqtoickERbht Yvy0RB5q rJecJxBC1QdfigOIVg2T+Fts4CgFbSAL85JVZrckXP+/6UhOpvJoH8Ym3LZR4N4iXa0YGBMbFk3J66ThuxPnzOip88U0j5FTi/K5UJkgqqPHAuDLzRax7LMl+6N/LmHX2AIotsjcMnrQPZCX/SHfGNZT7HLZsKitbPRTdcTx2uLr8ThI2mcAICowMMIMJKZZ6G7Of3RGT/TixEF/We3vT3VbdGsp8PIJw6WydwdFXz6uzqO2vcD7wNY/6lDnUIEKLtJWF9zeNxup0EGFV0w1CMW7Hz+vd6Xmq7Q1Nr2kXgbUwvgY= X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: Remove Error/Failed prefix from ksft_exit_fail*() usages as prefix is already present in the ksft_exit_fail*(). Signed-off-by: Muhammad Usama Anjum Reviewed-by: Kees Cook --- tools/testing/selftests/exec/load_address.c | 8 +- .../testing/selftests/exec/recursion-depth.c | 10 +- .../selftests/mm/map_fixed_noreplace.c | 24 +-- tools/testing/selftests/mm/map_populate.c | 2 +- tools/testing/selftests/mm/mremap_dontunmap.c | 2 +- tools/testing/selftests/mm/pagemap_ioctl.c | 166 +++++++++--------- .../selftests/mm/split_huge_page_test.c | 2 +- 7 files changed, 107 insertions(+), 107 deletions(-) diff --git a/tools/testing/selftests/exec/load_address.c b/tools/testing/selftests/exec/load_address.c index 17e3207d34ae7..438856b07260c 100644 --- a/tools/testing/selftests/exec/load_address.c +++ b/tools/testing/selftests/exec/load_address.c @@ -47,16 +47,16 @@ int main(int argc, char **argv) ret = dl_iterate_phdr(ExtractStatistics, &extracted); if (ret != 1) - ksft_exit_fail_msg("FAILED: dl_iterate_phdr\n"); + ksft_exit_fail_msg("dl_iterate_phdr\n"); if (extracted.alignment == 0) - ksft_exit_fail_msg("FAILED: No alignment found\n"); + ksft_exit_fail_msg("No alignment found\n"); else if (extracted.alignment & (extracted.alignment - 1)) - ksft_exit_fail_msg("FAILED: Alignment is not a power of 2\n"); + ksft_exit_fail_msg("Alignment is not a power of 2\n"); misalign = extracted.load_address & (extracted.alignment - 1); if (misalign) - ksft_exit_fail_msg("FAILED: alignment = %llu, load_address = %llu\n", + ksft_exit_fail_msg("alignment = %llu, load_address = %llu\n", extracted.alignment, extracted.load_address); ksft_test_result_pass("Completed\n"); diff --git a/tools/testing/selftests/exec/recursion-depth.c b/tools/testing/selftests/exec/recursion-depth.c index 438c8ff2fd260..9dd20251dc7eb 100644 --- a/tools/testing/selftests/exec/recursion-depth.c +++ b/tools/testing/selftests/exec/recursion-depth.c @@ -37,25 +37,25 @@ int main(void) ksft_test_result_skip("error: unshare, errno %d\n", errno); ksft_finished(); } - ksft_exit_fail_perror("error: unshare"); + ksft_exit_fail_perror("unshare"); } if (mount(NULL, "/", NULL, MS_PRIVATE | MS_REC, NULL) == -1) - ksft_exit_fail_perror("error: mount '/'"); + ksft_exit_fail_perror("mount '/'"); /* Require "exec" filesystem. */ if (mount(NULL, "/tmp", "ramfs", 0, NULL) == -1) - ksft_exit_fail_perror("error: mount ramfs"); + ksft_exit_fail_perror("mount ramfs"); #define FILENAME "/tmp/1" fd = creat(FILENAME, 0700); if (fd == -1) - ksft_exit_fail_perror("error: creat"); + ksft_exit_fail_perror("creat"); #define S "#!" FILENAME "\n" if (write(fd, S, strlen(S)) != strlen(S)) - ksft_exit_fail_perror("error: write"); + ksft_exit_fail_perror("write"); close(fd); diff --git a/tools/testing/selftests/mm/map_fixed_noreplace.c b/tools/testing/selftests/mm/map_fixed_noreplace.c index b74813fdc9514..58fb562efe25b 100644 --- a/tools/testing/selftests/mm/map_fixed_noreplace.c +++ b/tools/testing/selftests/mm/map_fixed_noreplace.c @@ -30,10 +30,10 @@ static unsigned long find_base_addr(unsigned long size) flags = MAP_PRIVATE | MAP_ANONYMOUS; addr = mmap(NULL, size, PROT_NONE, flags, -1, 0); if (addr == MAP_FAILED) - ksft_exit_fail_msg("Error: couldn't map the space we need for the test\n"); + ksft_exit_fail_msg("couldn't map the space we need for the test\n"); if (munmap(addr, size) != 0) - ksft_exit_fail_msg("Error: munmap failed\n"); + ksft_exit_fail_msg("munmap failed\n"); return (unsigned long)addr; } @@ -61,11 +61,11 @@ int main(void) p = mmap((void *)addr, size, PROT_NONE, flags, -1, 0); if (p == MAP_FAILED) { dump_maps(); - ksft_exit_fail_msg("Error: couldn't map the space we need for the test\n"); + ksft_exit_fail_msg("couldn't map the space we need for the test\n"); } if (munmap((void *)addr, 5 * page_size) != 0) { dump_maps(); - ksft_exit_fail_msg("Error: munmap failed!?\n"); + ksft_exit_fail_msg("munmap failed!?\n"); } ksft_test_result_pass("mmap() @ 0x%lx-0x%lx p=%p result=%m\n", addr, addr + size, p); @@ -74,7 +74,7 @@ int main(void) p = mmap((void *)addr, size, PROT_NONE, flags, -1, 0); if (p == MAP_FAILED) { dump_maps(); - ksft_exit_fail_msg("Error: first mmap() failed unexpectedly\n"); + ksft_exit_fail_msg("first mmap() failed unexpectedly\n"); } ksft_test_result_pass("mmap() @ 0x%lx-0x%lx p=%p result=%m\n", addr, addr + size, p); @@ -91,7 +91,7 @@ int main(void) p = mmap((void *)addr, size, PROT_NONE, flags, -1, 0); if (p != MAP_FAILED) { dump_maps(); - ksft_exit_fail_msg("Error:1: mmap() succeeded when it shouldn't have\n"); + ksft_exit_fail_msg("1: mmap() succeeded when it shouldn't have\n"); } ksft_test_result_pass("mmap() @ 0x%lx-0x%lx p=%p result=%m\n", addr, addr + size, p); @@ -109,7 +109,7 @@ int main(void) p = mmap((void *)addr, size, PROT_NONE, flags, -1, 0); if (p != MAP_FAILED) { dump_maps(); - ksft_exit_fail_msg("Error:2: mmap() succeeded when it shouldn't have\n"); + ksft_exit_fail_msg("2: mmap() succeeded when it shouldn't have\n"); } ksft_test_result_pass("mmap() @ 0x%lx-0x%lx p=%p result=%m\n", addr, addr + size, p); @@ -126,7 +126,7 @@ int main(void) p = mmap((void *)addr, size, PROT_NONE, flags, -1, 0); if (p != MAP_FAILED) { dump_maps(); - ksft_exit_fail_msg("Error:3: mmap() succeeded when it shouldn't have\n"); + ksft_exit_fail_msg("3: mmap() succeeded when it shouldn't have\n"); } ksft_test_result_pass("mmap() @ 0x%lx-0x%lx p=%p result=%m\n", addr, addr + size, p); @@ -143,7 +143,7 @@ int main(void) p = mmap((void *)addr, size, PROT_NONE, flags, -1, 0); if (p != MAP_FAILED) { dump_maps(); - ksft_exit_fail_msg("Error:4: mmap() succeeded when it shouldn't have\n"); + ksft_exit_fail_msg("4: mmap() succeeded when it shouldn't have\n"); } ksft_test_result_pass("mmap() @ 0x%lx-0x%lx p=%p result=%m\n", addr, addr + size, p); @@ -160,7 +160,7 @@ int main(void) p = mmap((void *)addr, size, PROT_NONE, flags, -1, 0); if (p == MAP_FAILED) { dump_maps(); - ksft_exit_fail_msg("Error:5: mmap() failed when it shouldn't have\n"); + ksft_exit_fail_msg("5: mmap() failed when it shouldn't have\n"); } ksft_test_result_pass("mmap() @ 0x%lx-0x%lx p=%p result=%m\n", addr, addr + size, p); @@ -177,7 +177,7 @@ int main(void) p = mmap((void *)addr, size, PROT_NONE, flags, -1, 0); if (p == MAP_FAILED) { dump_maps(); - ksft_exit_fail_msg("Error:6: mmap() failed when it shouldn't have\n"); + ksft_exit_fail_msg("6: mmap() failed when it shouldn't have\n"); } ksft_test_result_pass("mmap() @ 0x%lx-0x%lx p=%p result=%m\n", addr, addr + size, p); @@ -185,7 +185,7 @@ int main(void) size = 5 * page_size; if (munmap((void *)addr, size) != 0) { dump_maps(); - ksft_exit_fail_msg("Error: munmap failed!?\n"); + ksft_exit_fail_msg("munmap failed!?\n"); } ksft_test_result_pass("Base Address unmap() successful\n"); diff --git a/tools/testing/selftests/mm/map_populate.c b/tools/testing/selftests/mm/map_populate.c index 5c8a53869b1bd..8254f80da376b 100644 --- a/tools/testing/selftests/mm/map_populate.c +++ b/tools/testing/selftests/mm/map_populate.c @@ -23,7 +23,7 @@ #define BUG_ON(condition, description) \ do { \ if (condition) \ - ksft_exit_fail_msg("[FAIL]\t%s:%d\t%s:%s\n", \ + ksft_exit_fail_msg("%s:%d\t%s:%s\n", \ __func__, __LINE__, (description), \ strerror(errno)); \ } while (0) diff --git a/tools/testing/selftests/mm/mremap_dontunmap.c b/tools/testing/selftests/mm/mremap_dontunmap.c index 1d75084b9ca56..bb5c878059fa0 100644 --- a/tools/testing/selftests/mm/mremap_dontunmap.c +++ b/tools/testing/selftests/mm/mremap_dontunmap.c @@ -31,7 +31,7 @@ static void dump_maps(void) do { \ if (condition) { \ dump_maps(); \ - ksft_exit_fail_msg("[FAIL]\t%s:%d\t%s:%s\n", \ + ksft_exit_fail_msg("%s:%d\t%s:%s\n", \ __func__, __LINE__, (description), \ strerror(errno)); \ } \ diff --git a/tools/testing/selftests/mm/pagemap_ioctl.c b/tools/testing/selftests/mm/pagemap_ioctl.c index d59517ed3d48b..da90948f5dcc5 100644 --- a/tools/testing/selftests/mm/pagemap_ioctl.c +++ b/tools/testing/selftests/mm/pagemap_ioctl.c @@ -153,7 +153,7 @@ int wp_addr_range(void *lpBaseAddress, int dwRegionSize) if (pagemap_ioctl(lpBaseAddress, dwRegionSize, NULL, 0, PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN) < 0) - ksft_exit_fail_msg("error %d %d %s\n", 1, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", 1, errno, strerror(errno)); return 0; } @@ -190,7 +190,7 @@ int userfaultfd_tests(void) mem_size = num_pages * page_size; mem = mmap(NULL, mem_size, PROT_NONE, MAP_PRIVATE | MAP_ANON, -1, 0); if (mem == MAP_FAILED) - ksft_exit_fail_msg("error nomem\n"); + ksft_exit_fail_msg("nomem\n"); wp_init(mem, mem_size); @@ -213,7 +213,7 @@ int userfaultfd_tests(void) written = pagemap_ioctl(mem, mem_size, vec, 1, PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, vec_size - 2, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (written < 0) - ksft_exit_fail_msg("error %d %d %s\n", written, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", written, errno, strerror(errno)); ksft_test_result(written == 0, "%s all new pages must not be written (dirty)\n", __func__); @@ -246,15 +246,15 @@ int sanity_tests_sd(void) vec = malloc(sizeof(struct page_region) * vec_size); if (!vec) - ksft_exit_fail_msg("error nomem\n"); + ksft_exit_fail_msg("nomem\n"); vec2 = malloc(sizeof(struct page_region) * vec_size); if (!vec2) - ksft_exit_fail_msg("error nomem\n"); + ksft_exit_fail_msg("nomem\n"); mem = mmap(NULL, mem_size, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANON, -1, 0); if (mem == MAP_FAILED) - ksft_exit_fail_msg("error nomem\n"); + ksft_exit_fail_msg("nomem\n"); wp_init(mem, mem_size); wp_addr_range(mem, mem_size); @@ -321,7 +321,7 @@ int sanity_tests_sd(void) ret = pagemap_ioctl(mem, mem_size, vec, vec_size, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == mem_size/(page_size * 2), "%s Repeated pattern of written and non-written pages\n", __func__); @@ -331,18 +331,18 @@ int sanity_tests_sd(void) PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, num_pages/2 - 2, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ret2 = pagemap_ioctl(mem, mem_size, vec, 2, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret2 < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret2, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret2, errno, strerror(errno)); ret3 = pagemap_ioctl(mem, mem_size, vec, vec_size, PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret3 < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret3, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret3, errno, strerror(errno)); ksft_test_result((ret + ret3) == num_pages/2 && ret2 == 2, "%s Repeated pattern of written and non-written pages in parts %d %d %d\n", @@ -357,13 +357,13 @@ int sanity_tests_sd(void) PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, num_pages/2, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ret2 = pagemap_ioctl(mem, mem_size, vec, vec_size, PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret2 < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret2, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret2, errno, strerror(errno)); ksft_test_result(ret == num_pages/2 && ret2 == 1, "%s Repeated pattern of written and non-written pages max_pages\n", @@ -378,12 +378,12 @@ int sanity_tests_sd(void) PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, 2, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ret2 = pagemap_ioctl(mem, mem_size, vec2, vec_size, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret2 < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret2, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret2, errno, strerror(errno)); ksft_test_result(ret == 1 && LEN(vec[0]) == 2 && vec[0].start == (uintptr_t)(mem + page_size) && @@ -398,10 +398,10 @@ int sanity_tests_sd(void) /* 7. Two regions */ m[0] = mmap(NULL, mem_size, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANON, -1, 0); if (m[0] == MAP_FAILED) - ksft_exit_fail_msg("error nomem\n"); + ksft_exit_fail_msg("nomem\n"); m[1] = mmap(NULL, mem_size, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANON, -1, 0); if (m[1] == MAP_FAILED) - ksft_exit_fail_msg("error nomem\n"); + ksft_exit_fail_msg("nomem\n"); wp_init(m[0], mem_size); wp_init(m[1], mem_size); @@ -416,7 +416,7 @@ int sanity_tests_sd(void) ret = pagemap_ioctl(m[1], mem_size, vec, 1, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 1 && LEN(vec[0]) == mem_size/page_size, "%s Two regions\n", __func__); @@ -435,11 +435,11 @@ int sanity_tests_sd(void) vec = malloc(sizeof(struct page_region) * vec_size); if (!vec) - ksft_exit_fail_msg("error nomem\n"); + ksft_exit_fail_msg("nomem\n"); mem = mmap(NULL, mem_size, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANON, -1, 0); if (mem == MAP_FAILED) - ksft_exit_fail_msg("error nomem\n"); + ksft_exit_fail_msg("nomem\n"); wp_init(mem, mem_size); wp_addr_range(mem, mem_size); @@ -448,7 +448,7 @@ int sanity_tests_sd(void) PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); for (i = 0; i < mem_size/page_size; i += 2) mem[i * page_size]++; @@ -457,7 +457,7 @@ int sanity_tests_sd(void) PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, mem_size/(page_size*5), PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); total_pages += ret; @@ -465,7 +465,7 @@ int sanity_tests_sd(void) PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, mem_size/(page_size*5), PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); total_pages += ret; @@ -473,7 +473,7 @@ int sanity_tests_sd(void) PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, mem_size/(page_size*5), PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); total_pages += ret; @@ -490,11 +490,11 @@ int sanity_tests_sd(void) vec = malloc(sizeof(struct page_region) * vec_size); if (!vec) - ksft_exit_fail_msg("error nomem\n"); + ksft_exit_fail_msg("nomem\n"); mem = mmap(NULL, mem_size, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANON, -1, 0); if (mem == MAP_FAILED) - ksft_exit_fail_msg("error nomem\n"); + ksft_exit_fail_msg("nomem\n"); wp_init(mem, mem_size); wp_addr_range(mem, mem_size); @@ -515,7 +515,7 @@ int sanity_tests_sd(void) vec_size, PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN, &walk_end); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); if (ret > vec_size) break; @@ -540,11 +540,11 @@ int sanity_tests_sd(void) vec = malloc(sizeof(struct page_region) * vec_size); if (!vec) - ksft_exit_fail_msg("error nomem\n"); + ksft_exit_fail_msg("nomem\n"); mem = mmap(NULL, mem_size, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANON, -1, 0); if (mem == MAP_FAILED) - ksft_exit_fail_msg("error nomem\n"); + ksft_exit_fail_msg("nomem\n"); wp_init(mem, mem_size); wp_addr_range(mem, mem_size); @@ -554,63 +554,63 @@ int sanity_tests_sd(void) ret = pagemap_ioc(mem, 0, vec, vec_size, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN, &walk_end); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 0 && walk_end == (long)mem, "Walk_end: Same start and end address\n"); ret = pagemap_ioc(mem, 0, vec, vec_size, PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN, &walk_end); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 0 && walk_end == (long)mem, "Walk_end: Same start and end with WP\n"); ret = pagemap_ioc(mem, 0, vec, 0, PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN, &walk_end); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 0 && walk_end == (long)mem, "Walk_end: Same start and end with 0 output buffer\n"); ret = pagemap_ioc(mem, mem_size, vec, vec_size, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN, &walk_end); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 1 && walk_end == (long)(mem + mem_size), "Walk_end: Big vec\n"); ret = pagemap_ioc(mem, mem_size, vec, 1, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN, &walk_end); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 1 && walk_end == (long)(mem + mem_size), "Walk_end: vec of minimum length\n"); ret = pagemap_ioc(mem, mem_size, vec, 1, 0, vec_size, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN, &walk_end); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 1 && walk_end == (long)(mem + mem_size), "Walk_end: Max pages specified\n"); ret = pagemap_ioc(mem, mem_size, vec, vec_size, 0, vec_size/2, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN, &walk_end); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 1 && walk_end == (long)(mem + mem_size/2), "Walk_end: Half max pages\n"); ret = pagemap_ioc(mem, mem_size, vec, vec_size, 0, 1, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN, &walk_end); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 1 && walk_end == (long)(mem + page_size), "Walk_end: 1 max page\n"); ret = pagemap_ioc(mem, mem_size, vec, vec_size, 0, -1, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN, &walk_end); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 1 && walk_end == (long)(mem + mem_size), "Walk_end: max pages\n"); @@ -621,49 +621,49 @@ int sanity_tests_sd(void) ret = pagemap_ioc(mem, mem_size, vec, vec_size, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN, &walk_end); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == vec_size/2 && walk_end == (long)(mem + mem_size), "Walk_end sparse: Big vec\n"); ret = pagemap_ioc(mem, mem_size, vec, 1, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN, &walk_end); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 1 && walk_end == (long)(mem + page_size * 2), "Walk_end sparse: vec of minimum length\n"); ret = pagemap_ioc(mem, mem_size, vec, 1, 0, vec_size, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN, &walk_end); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 1 && walk_end == (long)(mem + page_size * 2), "Walk_end sparse: Max pages specified\n"); ret = pagemap_ioc(mem, mem_size, vec, vec_size/2, 0, vec_size, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN, &walk_end); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == vec_size/2 && walk_end == (long)(mem + mem_size), "Walk_end sparse: Max pages specified\n"); ret = pagemap_ioc(mem, mem_size, vec, vec_size, 0, vec_size, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN, &walk_end); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == vec_size/2 && walk_end == (long)(mem + mem_size), "Walk_end sparse: Max pages specified\n"); ret = pagemap_ioc(mem, mem_size, vec, vec_size, 0, vec_size/2, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN, &walk_end); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == vec_size/2 && walk_end == (long)(mem + mem_size), "Walk_endsparse : Half max pages\n"); ret = pagemap_ioc(mem, mem_size, vec, vec_size, 0, 1, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN, &walk_end); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 1 && walk_end == (long)(mem + page_size * 2), "Walk_end: 1 max page\n"); @@ -698,7 +698,7 @@ int base_tests(char *prefix, char *mem, int mem_size, int skip) written = pagemap_ioctl(mem, mem_size, vec, 1, PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, vec_size - 2, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (written < 0) - ksft_exit_fail_msg("error %d %d %s\n", written, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", written, errno, strerror(errno)); ksft_test_result(written == 0, "%s all new pages must not be written (dirty)\n", prefix); @@ -708,7 +708,7 @@ int base_tests(char *prefix, char *mem, int mem_size, int skip) written = pagemap_ioctl(mem, mem_size, vec, 1, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (written < 0) - ksft_exit_fail_msg("error %d %d %s\n", written, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", written, errno, strerror(errno)); ksft_test_result(written == 1 && LEN(vec[0]) == mem_size/page_size, "%s all pages must be written (dirty)\n", prefix); @@ -717,14 +717,14 @@ int base_tests(char *prefix, char *mem, int mem_size, int skip) written = pagemap_ioctl(mem, mem_size, vec, 1, PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (written < 0) - ksft_exit_fail_msg("error %d %d %s\n", written, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", written, errno, strerror(errno)); memset(mem + page_size, 0, mem_size - (2 * page_size)); written = pagemap_ioctl(mem, mem_size, vec, 1, PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (written < 0) - ksft_exit_fail_msg("error %d %d %s\n", written, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", written, errno, strerror(errno)); ksft_test_result(written == 1 && LEN(vec[0]) >= vec_size - 2 && LEN(vec[0]) <= vec_size, "%s all pages dirty other than first and the last one\n", prefix); @@ -732,7 +732,7 @@ int base_tests(char *prefix, char *mem, int mem_size, int skip) written = pagemap_ioctl(mem, mem_size, vec, 1, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (written < 0) - ksft_exit_fail_msg("error %d %d %s\n", written, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", written, errno, strerror(errno)); ksft_test_result(written == 0, "%s PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC\n", prefix); @@ -741,14 +741,14 @@ int base_tests(char *prefix, char *mem, int mem_size, int skip) written = pagemap_ioctl(mem, mem_size, vec, 1, PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (written < 0) - ksft_exit_fail_msg("error %d %d %s\n", written, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", written, errno, strerror(errno)); mem[vec_size/2 * page_size]++; written = pagemap_ioctl(mem, mem_size, vec, vec_size, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (written < 0) - ksft_exit_fail_msg("error %d %d %s\n", written, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", written, errno, strerror(errno)); ksft_test_result(written == 1 && LEN(vec[0]) >= 1, "%s only middle page dirty\n", prefix); @@ -757,7 +757,7 @@ int base_tests(char *prefix, char *mem, int mem_size, int skip) written = pagemap_ioctl(mem, mem_size, vec, 1, PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN | PAGE_IS_HUGE); if (written < 0) - ksft_exit_fail_msg("error %d %d %s\n", written, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", written, errno, strerror(errno)); mem[vec_size/2 * page_size]++; mem[(vec_size/2 + 1) * page_size]++; @@ -765,7 +765,7 @@ int base_tests(char *prefix, char *mem, int mem_size, int skip) written = pagemap_ioctl(&mem[vec_size/2 * page_size], 2 * page_size, vec, 1, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN | PAGE_IS_HUGE); if (written < 0) - ksft_exit_fail_msg("error %d %d %s\n", written, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", written, errno, strerror(errno)); ksft_test_result(written == 1 && vec[0].start == (uintptr_t)(&mem[vec_size/2 * page_size]) && LEN(vec[0]) == 2, @@ -818,7 +818,7 @@ int hpage_unit_tests(void) PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 0, "%s all new huge page must not be written (dirty)\n", __func__); @@ -827,7 +827,7 @@ int hpage_unit_tests(void) ret = pagemap_ioctl(map, map_size, vec, vec_size, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 0, "%s all the huge page must not be written\n", __func__); @@ -837,7 +837,7 @@ int hpage_unit_tests(void) PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 1 && vec[0].start == (uintptr_t)map && LEN(vec[0]) == vec_size && vec[0].categories == PAGE_IS_WRITTEN, @@ -854,7 +854,7 @@ int hpage_unit_tests(void) ret = pagemap_ioctl(map, map_size, vec, vec_size, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 1 && LEN(vec[0]) > 0, "%s only middle page written\n", __func__); @@ -881,7 +881,7 @@ int hpage_unit_tests(void) ret = pagemap_ioctl(map, map_size, vec, vec_size, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 1 && LEN(vec[0]) == vec_size/2 && vec[0].start == (uintptr_t)(map + map_size/2), @@ -904,12 +904,12 @@ int hpage_unit_tests(void) PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, vec_size/2, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ret = pagemap_ioctl(map, map_size, vec, vec_size, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 1 && LEN(vec[0]) == vec_size/2 && vec[0].start == (uintptr_t)(map + map_size/2), @@ -934,12 +934,12 @@ int hpage_unit_tests(void) PM_SCAN_WP_MATCHING | PM_SCAN_CHECK_WPASYNC, vec_size/2, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ret = pagemap_ioctl(map, map_size, vec, vec_size, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 1 && LEN(vec[0]) == vec_size/2, "%s clear second half huge page\n", __func__); @@ -963,7 +963,7 @@ int hpage_unit_tests(void) hpage_size/(2*page_size), PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 1 && LEN(vec[0]) == hpage_size/(2*page_size), "%s get half huge page\n", __func__); @@ -971,7 +971,7 @@ int hpage_unit_tests(void) ret2 = pagemap_ioctl(map, map_size, vec, vec_size, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret2 < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret2, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret2, errno, strerror(errno)); ksft_test_result(ret2 == 1 && LEN(vec[0]) == (map_size - hpage_size/2)/page_size, "%s get half huge page\n", __func__); @@ -999,7 +999,7 @@ int unmapped_region_tests(void) written = pagemap_ioctl(start, len, vec, vec_size, 0, 0, PAGEMAP_NON_WRITTEN_BITS, 0, 0, PAGEMAP_NON_WRITTEN_BITS); if (written < 0) - ksft_exit_fail_msg("error %d %d %s\n", written, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", written, errno, strerror(errno)); ksft_test_result(written >= 0, "%s Get status of pages\n", __func__); @@ -1060,7 +1060,7 @@ int sanity_tests(void) vec = malloc(sizeof(struct page_region) * vec_size); mem = mmap(NULL, mem_size, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANON, -1, 0); if (mem == MAP_FAILED || vec == MAP_FAILED) - ksft_exit_fail_msg("error nomem\n"); + ksft_exit_fail_msg("nomem\n"); wp_init(mem, mem_size); wp_addr_range(mem, mem_size); @@ -1088,7 +1088,7 @@ int sanity_tests(void) /* 2. Get sd and present pages with anyof_mask */ mem = mmap(NULL, mem_size, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANON, -1, 0); if (mem == MAP_FAILED) - ksft_exit_fail_msg("error nomem\n"); + ksft_exit_fail_msg("nomem\n"); wp_init(mem, mem_size); wp_addr_range(mem, mem_size); @@ -1134,7 +1134,7 @@ int sanity_tests(void) /* 8. Find written present pages with return mask */ mem = mmap(NULL, mem_size, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANON, -1, 0); if (mem == MAP_FAILED) - ksft_exit_fail_msg("error nomem\n"); + ksft_exit_fail_msg("nomem\n"); wp_init(mem, mem_size); wp_addr_range(mem, mem_size); @@ -1156,11 +1156,11 @@ int sanity_tests(void) ret = stat(progname, &sbuf); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); fmem = mmap(NULL, sbuf.st_size, PROT_READ, MAP_PRIVATE, fd, 0); if (fmem == MAP_FAILED) - ksft_exit_fail_msg("error nomem %d %s\n", errno, strerror(errno)); + ksft_exit_fail_msg("nomem %d %s\n", errno, strerror(errno)); tmp_buf = malloc(sbuf.st_size); memcpy(tmp_buf, fmem, sbuf.st_size); @@ -1190,7 +1190,7 @@ int sanity_tests(void) fmem = mmap(NULL, buf_size, PROT_READ | PROT_WRITE, MAP_PRIVATE, fd, 0); if (fmem == MAP_FAILED) - ksft_exit_fail_msg("error nomem %d %s\n", errno, strerror(errno)); + ksft_exit_fail_msg("nomem %d %s\n", errno, strerror(errno)); wp_init(fmem, buf_size); wp_addr_range(fmem, buf_size); @@ -1232,7 +1232,7 @@ int mprotect_tests(void) /* 1. Map two pages */ mem = mmap(0, 2 * page_size, PROT_READ|PROT_WRITE, MAP_PRIVATE | MAP_ANON, -1, 0); if (mem == MAP_FAILED) - ksft_exit_fail_msg("error nomem\n"); + ksft_exit_fail_msg("nomem\n"); wp_init(mem, 2 * page_size); wp_addr_range(mem, 2 * page_size); @@ -1242,7 +1242,7 @@ int mprotect_tests(void) ret = pagemap_ioctl(mem, 2 * page_size, &vec, 1, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 1 && LEN(vec) == 2, "%s Both pages written\n", __func__); @@ -1257,7 +1257,7 @@ int mprotect_tests(void) mem2 = mmap(mem + page_size, page_size, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANON|MAP_FIXED, -1, 0); if (mem2 == MAP_FAILED) - ksft_exit_fail_msg("error nomem\n"); + ksft_exit_fail_msg("nomem\n"); wp_init(mem2, page_size); wp_addr_range(mem2, page_size); @@ -1277,7 +1277,7 @@ int mprotect_tests(void) ret = pagemap_ioctl(mem, 2 * page_size, &vec, 1, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 1 && LEN(vec) == 2, "%s Both pages written after remap and mprotect\n", __func__); @@ -1290,7 +1290,7 @@ int mprotect_tests(void) ret = pagemap_ioctl(mem, 2 * page_size, &vec, 1, 0, 0, PAGE_IS_WRITTEN, 0, 0, PAGE_IS_WRITTEN); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); ksft_test_result(ret == 1 && LEN(vec) == 2, "%s Clear and make the pages written\n", __func__); @@ -1397,7 +1397,7 @@ static void transact_test(int page_size) mem = mmap(NULL, 0x1000 * nthreads * pages_per_thread, PROT_READ | PROT_WRITE, MAP_ANONYMOUS | MAP_PRIVATE, -1, 0); if (mem == MAP_FAILED) - ksft_exit_fail_msg("Error mmap %s.\n", strerror(errno)); + ksft_exit_fail_msg("mmap %s.\n", strerror(errno)); wp_init(mem, 0x1000 * nthreads * pages_per_thread); wp_addr_range(mem, 0x1000 * nthreads * pages_per_thread); @@ -1502,7 +1502,7 @@ int main(int argc, char *argv[]) mem_size = 10 * page_size; mem = mmap(NULL, mem_size, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANON, -1, 0); if (mem == MAP_FAILED) - ksft_exit_fail_msg("error nomem\n"); + ksft_exit_fail_msg("nomem\n"); wp_init(mem, mem_size); wp_addr_range(mem, mem_size); @@ -1515,7 +1515,7 @@ int main(int argc, char *argv[]) mem_size = 512 * 10 * page_size; mem = mmap(NULL, mem_size, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANON, -1, 0); if (mem == MAP_FAILED) - ksft_exit_fail_msg("error nomem\n"); + ksft_exit_fail_msg("nomem\n"); wp_init(mem, mem_size); wp_addr_range(mem, mem_size); @@ -1595,11 +1595,11 @@ int main(int argc, char *argv[]) ret = stat(__FILE__".tmp0", &sbuf); if (ret < 0) - ksft_exit_fail_msg("error %d %d %s\n", ret, errno, strerror(errno)); + ksft_exit_fail_msg("%d %d %s\n", ret, errno, strerror(errno)); fmem = mmap(NULL, sbuf.st_size, PROT_READ | PROT_WRITE, MAP_PRIVATE, fd, 0); if (fmem == MAP_FAILED) - ksft_exit_fail_msg("error nomem %d %s\n", errno, strerror(errno)); + ksft_exit_fail_msg("nomem %d %s\n", errno, strerror(errno)); wp_init(fmem, sbuf.st_size); wp_addr_range(fmem, sbuf.st_size); @@ -1619,7 +1619,7 @@ int main(int argc, char *argv[]) strerror(errno)); if (ftruncate(fd, buf_size)) - ksft_exit_fail_msg("Error ftruncate\n"); + ksft_exit_fail_msg("ftruncate\n"); for (i = 0; i < buf_size; i++) if (write(fd, "c", 1) < 0) @@ -1627,7 +1627,7 @@ int main(int argc, char *argv[]) fmem = mmap(NULL, buf_size, PROT_READ | PROT_WRITE, MAP_PRIVATE, fd, 0); if (fmem == MAP_FAILED) - ksft_exit_fail_msg("error nomem %d %s\n", errno, strerror(errno)); + ksft_exit_fail_msg("nomem %d %s\n", errno, strerror(errno)); wp_init(fmem, buf_size); wp_addr_range(fmem, buf_size); diff --git a/tools/testing/selftests/mm/split_huge_page_test.c b/tools/testing/selftests/mm/split_huge_page_test.c index 6c988bd2f3356..f0a400f196956 100644 --- a/tools/testing/selftests/mm/split_huge_page_test.c +++ b/tools/testing/selftests/mm/split_huge_page_test.c @@ -264,7 +264,7 @@ void split_file_backed_thp(void) cleanup: umount(tmpfs_loc); rmdir(tmpfs_loc); - ksft_exit_fail_msg("Error occurred\n"); + ksft_exit_fail_msg("occurred\n"); } bool prepare_thp_fs(const char *xfs_path, char *thp_fs_template,