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]) by smtp.lore.kernel.org (Postfix) with ESMTP id ABE54C5478C for ; Tue, 5 Mar 2024 06:26:32 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 3E3076B0085; Tue, 5 Mar 2024 01:26:32 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 31D8A6B0088; Tue, 5 Mar 2024 01:26:32 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 1BDBF6B0089; Tue, 5 Mar 2024 01:26:32 -0500 (EST) 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 055D36B0085 for ; Tue, 5 Mar 2024 01:26:32 -0500 (EST) Received: from smtpin05.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay05.hostedemail.com (Postfix) with ESMTP id D039A40187 for ; Tue, 5 Mar 2024 06:26:31 +0000 (UTC) X-FDA: 81862001382.05.7184CF0 Received: from madrid.collaboradmins.com (madrid.collaboradmins.com [46.235.227.194]) by imf04.hostedemail.com (Postfix) with ESMTP id DC80340005 for ; Tue, 5 Mar 2024 06:26:29 +0000 (UTC) Authentication-Results: imf04.hostedemail.com; dkim=pass header.d=collabora.com header.s=mail header.b=f0vPPKX0; 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=1709619990; 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-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=4CkX8OoN6+5/Qmk7Hmcu+oTfTUZa028OQ/hUewrNSn0=; b=QJL6Er77aXJRglqg9MJ34ko2Iru2so15O/EcaytpUmcspeCcxfISKoFXckS1ZKh1Z/vFIy LL/kDwOWBl5XCziBuQgOYzPIB8w+p7XDQQ2imrgconO58jIT01TBAkSwoi8+A96yUKdPtp VeP59lgqjyL4W7SCL4zSlbuJ+8AKwQE= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1709619990; a=rsa-sha256; cv=none; b=VJRR+Vo3MuXg/r1Sxrp4RfXJXg4dR7y1+cXq1DX3z5da4hZjxhbkbrSbeWz86iUOdejNe3 b9/9eGdM9RgfSbnKAnS7s8FP9KggK6uuX8mfl/oNltoPtU7PTD4Wb4c/hzlzaM3GlS/70T 7xCzx1RS5Q9om+M2n3LfQxIWRsPbBMA= ARC-Authentication-Results: i=1; imf04.hostedemail.com; dkim=pass header.d=collabora.com header.s=mail header.b=f0vPPKX0; 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=1709619988; bh=5a8HIqzrX82rzPacl8CmdfWgBErNrkuXsBd+I+i7i5o=; h=Date:Cc:Subject:To:References:From:In-Reply-To:From; b=f0vPPKX052JKjEAwY+97iWsrSxbgUxxKmcdpt7tOotAlwStsiTUwAH84ciL0tuyUa F+zpwzZ0PgKvhmbJGx7loz5Dg3TCIyRZaJcpLFcQQ4ILvrPBptHyojVA5AWs00TV5B Ex9OUCMQmS7vUrTC/qpScZjo1ambv9l87q494Xcw6PBbLyGwgglnBcVi1OpXX39Yug klyKXGk9QRjjfN41b4pFpgzOA2nMsQ74dRLcnOVY7vpLMUCUyGFoAQrxEzuG06HJXW vkQH0PWlBL3TZkxhNdnOLqh17D5WjdoyVkfapYdKq/86ApKfBTmIcV5MVSrTmsSqvR BXwl8D+uiWqYg== Received: from [10.193.1.1] (broslavsky.collaboradmins.com [68.183.210.73]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 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 E4B1F3782073; Tue, 5 Mar 2024 06:26:25 +0000 (UTC) Message-ID: <2fc8f4d5-0906-46dd-8b71-9425dc96b000@collabora.com> Date: Tue, 5 Mar 2024 11:26:52 +0500 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Cc: Muhammad Usama Anjum , Eric Biederman , kernel@collabora.com, kernel-janitors@vger.kernel.org, linux-mm@kvack.org, linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/3] selftest/exec: conform test to TAP format output Content-Language: en-US To: Kees Cook , Shuah Khan References: <20240304155928.1818928-1-usama.anjum@collabora.com> <20240304155928.1818928-2-usama.anjum@collabora.com> <202403041256.80EDE9BA2@keescook> From: Muhammad Usama Anjum In-Reply-To: <202403041256.80EDE9BA2@keescook> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Rspamd-Queue-Id: DC80340005 X-Rspam-User: X-Rspamd-Server: rspam11 X-Stat-Signature: dr3my93x4axpabopaxwyajxt9hao6au1 X-HE-Tag: 1709619989-672699 X-HE-Meta: U2FsdGVkX190y0Pk4Y3WEP1AZ5vW9QIJFaHjX15ivL60whd8y/RLmYoPUXyV+BhBrUUYvm6Kh3ulizqj4SuVRL4zrxPGz/Sh52id4t0wlMpk8bF6RLWW56eYzR/rj/p+7YnO8Ko54opR5Q+XZoq1Sxz9Nf3NV8W2+hIu5lYHHv4lfLBtryO/kjXZdQ3GuP1PaBeXzhPlrCPionUQN0PiqqBMOpxUkTrpKjc9Jp4k1h06ca2pBXjOArLFAxWdFsB+SuakMbOyAZujDSYrnU1k78SWXkF3df8BFKfTeoZAuRPZ7NZY+7buBNt/5x0kwrd0VdVwudzPzLV6LP59FdhmggzSjV8OYybESqEhNDReKa9VBhCdenLePM3Cgj0kZ8SrhxIUtn67Zj/YIA/1HYz+Vi+O4/xKUtpkvILkiUv+A4L42SA5NPcWYCXfXRc70zDZWfpLrLAiAwZpb/cRKmWlZBdcHEOKHtdMRXZroqXoFPc4JtEGz97/e0pvA3VyXc/2zpPLu38VXsY8gzgZMxGqzYpwkzGURrOGvnOLq1GL2LX1ylwuQMgN/KKRRAFbzQ7PZMzhaqchrc60Gf0J+unfVesDs6uRegnPr7hVNzgW4SbRfjUT01amMKzzotTSIioJ2Q3wQUxRLHWfKxME3wMYtJiO/+gEenlrtoCbdMqCG08JRc2dovcEACMEb6fKC10BFwxQ4EcOSffIPmuUFTtXsKuIN+xs8jbs0X/wkPRZ0EtjEAVVwmf1xsPBNgcPH/63goyONPezCPb2mSgUG0/lQ1576flnK0QqO1/acAZl/OKLTROHvIlGxUMIcrHLQXQmZ/xaLiODlGTrbgtYriz3Mv3u4KTgaAq5BmDneOm/z8AbIdTATeFM0VYzTtyAkOeTkj1xFKFS2FGQpeM3qSgfusGB1Cky3h3aO0HvgBvDh6pu30KLgB+r7w== 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: On 3/5/24 1:58 AM, Kees Cook wrote: > On Mon, Mar 04, 2024 at 08:59:24PM +0500, Muhammad Usama Anjum wrote: >> Conform the layout, informational and status messages to TAP. No >> functional change is intended other than the layout of output messages. >> >> Signed-off-by: Muhammad Usama Anjum >> --- >> tools/testing/selftests/exec/load_address.c | 34 +++++++++------------ >> 1 file changed, 15 insertions(+), 19 deletions(-) >> >> diff --git a/tools/testing/selftests/exec/load_address.c b/tools/testing/selftests/exec/load_address.c >> index d487c2f6a6150..17e3207d34ae7 100644 >> --- a/tools/testing/selftests/exec/load_address.c >> +++ b/tools/testing/selftests/exec/load_address.c >> @@ -5,6 +5,7 @@ >> #include >> #include >> #include >> +#include "../kselftest.h" >> >> struct Statistics { >> unsigned long long load_address; >> @@ -41,28 +42,23 @@ int main(int argc, char **argv) >> unsigned long long misalign; >> int ret; >> >> + ksft_print_header(); >> + ksft_set_plan(1); >> + >> ret = dl_iterate_phdr(ExtractStatistics, &extracted); >> - if (ret != 1) { >> - fprintf(stderr, "FAILED\n"); >> - return 1; >> - } >> + if (ret != 1) >> + ksft_exit_fail_msg("FAILED: dl_iterate_phdr\n"); > > I'm for this series, but I do note a weird glitch in the ksft API. > ksft_exit_fail_msg does: > > va_start(args, msg); > printf("Bail out! "); > errno = saved_errno; > vprintf(msg, args); > va_end(args); > > "Bail out!" is not very descriptive. I think I'd rather this should be: > > "FAILED: " > > and then that added prefix doesn't need to be added everywhere in this > patch, nor the "error: " prefix in the next patch. If we want to make this change, FAILED should be removed from all the tests. We should do it in separate patch. I've taken note and will do it separate from this series. -- BR, Muhammad Usama Anjum