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 3A9E2C001E0 for ; Wed, 16 Aug 2023 09:36:34 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id B596F94003D; Wed, 16 Aug 2023 05:36:33 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id B08C58D0001; Wed, 16 Aug 2023 05:36:33 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 9F89A94003D; Wed, 16 Aug 2023 05:36:33 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id 91D0C8D0001 for ; Wed, 16 Aug 2023 05:36:33 -0400 (EDT) Received: from smtpin26.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id 5ADF412065E for ; Wed, 16 Aug 2023 09:36:33 +0000 (UTC) X-FDA: 81129462666.26.7BFDEAD Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by imf21.hostedemail.com (Postfix) with ESMTP id 255AC1C0023 for ; Wed, 16 Aug 2023 09:36:30 +0000 (UTC) Authentication-Results: imf21.hostedemail.com; dkim=none; dmarc=pass (policy=none) header.from=arm.com; spf=pass (imf21.hostedemail.com: domain of andre.przywara@arm.com designates 217.140.110.172 as permitted sender) smtp.mailfrom=andre.przywara@arm.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1692178591; a=rsa-sha256; cv=none; b=c2xYDYo6t7vUb0xqlxC/aFh73iY9KWyHXG26cbfzEAzlsF+VXZ6mOTn/ttAZH1emBIRYzD NAm3ke8Yrm48lHZkH6zE2YUgZS43ETSrq/zvtMU7+65KKUcob8Q3QQmbMWJ4xLMD/X6wGX 0T2AIKxMJPQc3TCzK0vBbxlJdLPZXPg= ARC-Authentication-Results: i=1; imf21.hostedemail.com; dkim=none; dmarc=pass (policy=none) header.from=arm.com; spf=pass (imf21.hostedemail.com: domain of andre.przywara@arm.com designates 217.140.110.172 as permitted sender) smtp.mailfrom=andre.przywara@arm.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1692178591; 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; bh=h+D+fl08y8y5VMXTmUjaYCwOrXpMs20KGDW1GOY4DJs=; b=KqsBLTdsJ2h6undsgY1opL9N6Cc/ha1RNvNuVXVT1t8eD5Nu11Dzy0UvCFOOsa9+1FdBuX 2NY5TW4mcEUONWrbEr8uL9Lt41Joil8vjickXpChW8Fwt2bF+EPsdx16ybi980pooYU7IA cD87NZh7n8/xMMD2+qSxvkZLLicjV/g= Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id DD2D71063; Wed, 16 Aug 2023 02:37:10 -0700 (PDT) Received: from donnerap.manchester.arm.com (usa-sjc-imap-foss1.foss.arm.com [10.121.207.14]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id B1F993F64C; Wed, 16 Aug 2023 02:36:28 -0700 (PDT) Date: Wed, 16 Aug 2023 10:36:26 +0100 From: Andre Przywara To: Nhat Pham Cc: Shuah Khan , Johannes Weiner , linux-kselftest@vger.kernel.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 3/3] selftests: cachestat: test for cachestat availability Message-ID: <20230816103626.0f679788@donnerap.manchester.arm.com> In-Reply-To: References: <20230815155612.2535947-1-andre.przywara@arm.com> <20230815155612.2535947-4-andre.przywara@arm.com> Organization: ARM X-Mailer: Claws Mail 3.18.0 (GTK+ 2.24.32; aarch64-unknown-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Rspamd-Server: rspam08 X-Rspamd-Queue-Id: 255AC1C0023 X-Stat-Signature: fupgifaauqdeo6ur7mrs39gwadq8br5n X-Rspam-User: X-HE-Tag: 1692178590-942611 X-HE-Meta: U2FsdGVkX1+XFTJ6XHlFrHau7OYmADDfFPyHW7D32DfatKsTzc85P7LF6xfcskjypuhGPA05HEyReSsZWpo7NnsCf63Lg/dHZ7cvxxmAoNXySz9jDsmf0B0IwsO0mNUhSH3xk5S6WXvzsDdqOpThnIK5lg2Up+ATqhYxUGAgwuPFG15RFCUoPjN/dA/ZWWBGjYxg1uEUmZJff4fmV+L63/MfwiwOPVuRZmvceLyQa4DMGBw7jVX4OiYxDxE//6rAiC9Vyf31PDMh/NC9k4ql5hC/6riFeBwx5wAGm4pdN/H27ROo/pRVDK1P++63ipbnI60gcnciqQtPC6z4ezSFA0wVoLaRzZgeotHLNxwQzRoxPM7cTgpHpXrA3l7jAovLNRWnvpJE5XnDmh8p7w7YptUmDiHuJBObltZjwR2lw9ar0JAvYKFENMTBBNBuUAP5lgQs9/m2un8wN8XabKKKDhtDyL3PIr2JDyp56V03wKPbKeJg+IhuBC3JvQAxRJFVPfeL7UI4/uWwGrY0cTa2NPqGmBEiAinw2StIXcfqASOcOaFlONByQEmUopvgaa56nqfVBh0x5CMBk8B9y72/7YhWiu2Xlbq0ZZo3hI7H9WRTYTJ8NOE/IO560g2PbmGuuzSEzOKnuSW1gkrYC8IexywuboKkrpC19Wav2vpatuiRaL6+Uoi6rXpSCOGOnXmMS4M0AZa/qr143TFAuEa6b49IJVH4LJDXqu7hYRuqe1IUZpsKyvcFlVHNQaouyya3O/WrFw1n/N0nSbNJitFWKkFj550SidymP25ombz+uBlTBeaJKyTgasXpPo/ZsjwyqIDBydaM2GN42E/qW4p18VV1bJWdYmzci4YFl5KikWGH4l2MJc7ut+IBVU18NUJ+XXEpM2kRG4GreakIedZbXp8czTW4LnUz118HH9McuMd9PwZYzXNGhC3tFGfYdTgQafo3Ku7VzuqP52PNmYC NNNU0JxT zuj0BuPtbyhK6iD3cAkyO4i+nTSMLt4K0r0NB5An6fWvRdkbyxkh2JYirzeJAlAbeDRCm2mvmq2xFEYHRVLUkxi+m/zO0pHgeFUf490tIjFX7NQxDBE9JSqDNZPdSRstg4UJQCcBs0W8LBTcVAcmXnsoTFkVuH4RYaiyBIRlsHJad7xHvhpqkTdwa4WpGMP4RRlx8+W4NYVNMhRRKuXgwTfbkRQfVhxnzXKvCPu1eI/OPslp5pegd5WmzQEsENNBnXcH91Xu5ti8b4LrQk4oJNY8t0jFgacuOJAjF 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: On Tue, 15 Aug 2023 16:25:54 -0700 Nhat Pham wrote: Hi Nhat, many thanks for having a look! > On Tue, Aug 15, 2023 at 8:56=E2=80=AFAM Andre Przywara wrote: > > > > As cachestat is a new syscall, it won't be available on older kernels, > > for instance those running on a build machine. In this case, a run > > reports all tests as "not ok" at the moment. =20 > Interesting - I was under the assumption that if you backported the > selftests for cachestat, you would also backport the syscall's implementa= tion > and wiring. Well, just running the tests on the kernel you just built is only one use case. I build the tests from latest git HEAD, then copy them to a target system with some kernel running. Or I just build the tests and run them for regression testing on my build system with a distro kernel, which is Ubuntu's 5.15 flavour, in my case. The documentation explicitly mentions that selftests should work on older kernels (copying the normal userland compatibility requirements), check the second paragraph of Documentation/dev-tools/kselftest.rst. > But yeah, I guess if you build with !CONFIG_CACHESTAT_SYSCALL, > these tests would fail. > > > > Test for the cachestat syscall availability first, before doing further > > tests, and bail out early with a TAP SKIP comment. > > > > This also uses the opportunity to add the proper TAP headers, and add > > one check for the syscall error handling (illegal file descriptor). =20 > Thanks for the addition! > > > > Signed-off-by: Andre Przywara > > --- > > .../selftests/cachestat/test_cachestat.c | 22 ++++++++++++++++++- > > 1 file changed, 21 insertions(+), 1 deletion(-) > > > > diff --git a/tools/testing/selftests/cachestat/test_cachestat.c b/tools= /testing/selftests/cachestat/test_cachestat.c > > index a5a4ac8dcb76c..77620e7ecf562 100644 > > --- a/tools/testing/selftests/cachestat/test_cachestat.c > > +++ b/tools/testing/selftests/cachestat/test_cachestat.c > > @@ -15,6 +15,8 @@ > > > > #include "../kselftest.h" > > > > +#define NR_TESTS 8 > > + > > static const char * const dev_files[] =3D { > > "/dev/zero", "/dev/null", "/dev/urandom", > > "/proc/version", "/proc" > > @@ -235,7 +237,25 @@ bool test_cachestat_shmem(void) > > > > int main(void) > > { > > - int ret =3D 0; > > + int ret; > > + > > + ksft_print_header(); > > + > > + ret =3D syscall(__NR_cachestat, -1, NULL, NULL, 0); > > + if (ret =3D=3D -1 && errno =3D=3D ENOSYS) { =20 > nit: if (ret && errno =3D=3D ENOSYS) sounds cleaner, but up to you. Do you ever return a positive value other than 0? I think technically errno is only valid when the return value is -1, so in the error case, which I wanted to test here explicitly. Some syscall selftests (I checked landlock the other day) do very elaborate testing of the error path, trying to carefully cover all corner cases: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/too= ls/testing/selftests/landlock/base_test.c#n24 So this test was inspired by that, but I didn't want to go that far here ;-) > > + printf("1..0 # Skipped: cachestat syscall not available= \n"); =20 > nit: perhaps ksft_print_msg()? Ah, yes, of course! > > + return KSFT_SKIP; > > + } > > + > > + ksft_set_plan(NR_TESTS); > > + > > + if (ret =3D=3D -1 && errno =3D=3D EBADF) { > > + ksft_test_result_pass("bad file descriptor recognized\n= "); > > + ret =3D 0; > > + } else { > > + ksft_test_result_fail("bad file descriptor ignored\n"); > > + ret =3D 1; > > + } =20 > Nice! > > > > for (int i =3D 0; i < 5; i++) { > > const char *dev_filename =3D dev_files[i]; > > -- > > 2.25.1 > > =20 > Nitpicking aside: > Acked-by: Nhat Pham Thanks, I will send a v2 later, using ksft_print_msg(). But first I will try if I can detect a tmpfs instance without boiling the ocean. Cheers, Andre