From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f44.google.com (mail-wm1-f44.google.com [209.85.128.44]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B07853033E1 for ; Thu, 19 Mar 2026 14:11:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.44 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773929487; cv=none; b=HZXBJ5sbOjRM3kzs1zNtjwwWSQ/Dcm3zStJ1iawURqquEKwCfvLytfgeL0bB37c/MlUOtE78UhI57oL8fjnAlO5s6IisxA/bXRsfM0u73dZNFdG9vlUs+g7We6faoYO9F8ta9czd9ypGnfPGQsK0AwuqdW0gCA0XRgiEdayK9jU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773929487; c=relaxed/simple; bh=ZoKqfLZmL/m8aDQmzpCMQIqxQutaiEBwAABS+Lwg80w=; h=Message-ID:Subject:From:To:Cc:Date:In-Reply-To:References: Content-Type:MIME-Version; b=Fv7sK4lzqeRbiL/Eg68YvFkRPPoNnE9F8w7SJnFM5c8lRy+iGFnWdhhkynz2dJx3xLuXQJ7vaPPJTmwb4mWyZLKBWlzjZQjmpETlJaM2iw5WceeKLSou0dDdJ1x7eLLrHI1Rko/F82Pe82P0tO7qkfRuF6Xy/O6zYlVjBI1SoVk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com; spf=pass smtp.mailfrom=suse.com; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b=S5m57HWr; arc=none smtp.client-ip=209.85.128.44 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=suse.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b="S5m57HWr" Received: by mail-wm1-f44.google.com with SMTP id 5b1f17b1804b1-486fe2024a9so268115e9.0 for ; Thu, 19 Mar 2026 07:11:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=google; t=1773929484; x=1774534284; darn=vger.kernel.org; h=mime-version:user-agent:content-transfer-encoding:autocrypt :references:in-reply-to:date:cc:to:from:subject:message-id:from:to :cc:subject:date:message-id:reply-to; bh=ZoKqfLZmL/m8aDQmzpCMQIqxQutaiEBwAABS+Lwg80w=; b=S5m57HWr87aghFW/RbHgO8/EdA9Lzep/dbn8C1EUNPLwIBRMG1eCLTK0nAUdT+Ip0O szn8za9rWVDady2QlwylZk9CDol9jSUHJeWAuYDQDB8/45qCk+XsyOeSc+Y7SK5pvDcH +cv4iNnUZ1Pd1G5fcYzz74fsBfKuZ58X3sGQUu2BP/83MhcjC2Rq3SVExf+/B61jlt+x SGyEXX7orHWY5wRIoa7VbMkYng7Rh/l2F+Hte7qu/h56SYKdF9klcSXoaPHGHeRn3CV/ BZ+SP4ahzZul7MliJeIa8HwngjD3zhQdwiSuT20Hz4atrP9s1Ix/ffVsxXHzqihCr9nc O+IQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1773929484; x=1774534284; h=mime-version:user-agent:content-transfer-encoding:autocrypt :references:in-reply-to:date:cc:to:from:subject:message-id:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=ZoKqfLZmL/m8aDQmzpCMQIqxQutaiEBwAABS+Lwg80w=; b=QbvI89DsMADPQCcsr6Z6VKk89AqpI3gQhJD1xw0g9I8g1++Q0Or8QECxuDUob8NJU0 6cX0gvEUojYvcId/+cbSzV+AjFwS/JsE2YIyVaho09mP9CoQ5i1rCIFnaED7bhCla68a CnFdiNVNQG3+kI87ZywlpqQdW8p0FCpexgXsiEd1KdeOwdE5+QI2iU7kUirPe1uCnpty JSjmBkh8UMPPnumYjYXsYB0fiCRF1CwTuH3LZQl8glbaE+HVoeoH/yChCOmjdPdggZfD 3JL9at++i6KSRBHTJ9rNLU1BGhbT7fam+f2ExX+dR6Dk0XTP0LGYZdbb+RIRljaTV6P8 5tMQ== X-Forwarded-Encrypted: i=1; AJvYcCVOMBX0s/I450uZ2Bfm4HCsOfTAufoLrAi+GherZOMlKacq39HIbQ7roRNjaOAi0NmFvKMqQNfCL09Dw5k=@vger.kernel.org X-Gm-Message-State: AOJu0Yw5UCbV7J+seZ42xqa5OLNrsdEyQOOaKTebeaK+rCF3HA1uJ+nM WmgltxlYttcmpYv87/NC+Htuyx3Twxmd6vyqEKMtXAaI14x9YSJ13DKoijYPelysh30= X-Gm-Gg: ATEYQzx/NCYBYBB1vWICRE9Xehw+fgvTcPURfmIb+5hAjazNmpgWGYQOVYGEnkjFjLq O6wiBoiZgV8WP4/VYXlB2zDcc5XP+4bvc+Ot7StTOgyWO9kSqEmC2dUKuPSc9ghgOH8d6W76V1y nnZkcR74EV4IPrG5RKcBzOycSvyYhN+JzPEcJDf66v/mnGfu/kscyhYenksLnD5mIsfIbnQXytM lFqCUUtvuH7NeTdJoRAEXC+R7iwbeXa7c9jljY5VqkwQg4YDxAkZCGZFbHVJegCumN8ANjqq9Yo hDjCBEYQNKmglCOoT30GMW1j6rMq4gt2c80xGHmIHUtCE3qy4OmppG7qo6RQCo8IeXeeG0MA/s6 zcmaM76zzAQukPV7l5/LxS4IZ8VTsqQgmx3Xu3FJbpSOMmDFw1uGC6uve9bd4T2qboEaNbT0gfy CcnJ8R/mbtesiQv6YvqkjFO2lIXiFhCwO/rNFiLDAXKh+69wnrbhRBjSWXUqi08A== X-Received: by 2002:a05:600c:3516:b0:485:3b34:2f51 with SMTP id 5b1f17b1804b1-486f44220f2mr121294065e9.4.1773929483990; Thu, 19 Mar 2026 07:11:23 -0700 (PDT) Received: from ?IPv6:2804:5078:834:1300:58f2:fc97:371f:3? ([2804:5078:834:1300:58f2:fc97:371f:3]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-486f4bc320dsm77929105e9.4.2026.03.19.07.11.21 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 19 Mar 2026 07:11:23 -0700 (PDT) Message-ID: <0d85d8d7533a7a78d1f8fcc1fff8ffc73b1cf225.camel@suse.com> Subject: Re: [PATCH 1/8] selftests: livepatch: test-syscall: Check for ARCH_HAS_SYSCALL_WRAPPER From: Marcos Paulo de Souza To: Miroslav Benes , Joe Lawrence Cc: Josh Poimboeuf , Jiri Kosina , Petr Mladek , Shuah Khan , live-patching@vger.kernel.org, linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org Date: Thu, 19 Mar 2026 11:11:19 -0300 In-Reply-To: References: <20260313-lp-tests-old-fixes-v1-0-71ac6dfb3253@suse.com> <20260313-lp-tests-old-fixes-v1-1-71ac6dfb3253@suse.com> Autocrypt: addr=mpdesouza@suse.com; prefer-encrypt=mutual; keydata=mDMEZ/0YqhYJKwYBBAHaRw8BAQdA4JZz0FED+JD5eKlhkNyjDrp6lAGmgR3LPTduPYGPT Km0Kk1hcmNvcyBQYXVsbyBkZSBTb3V6YSA8bXBkZXNvdXphQHN1c2UuY29tPoiTBBMWCgA7FiEE2g gC66iLbhUsCBoBemssEuRpLLUFAmf9GKoCGwMFCwkIBwICIgIGFQoJCAsCBBYCAwECHgcCF4AACgk QemssEuRpLLWGxwD/S1I0bjp462FlKb81DikrOfWbeJ0FOJP44eRzmn20HmEBALBZIMrfIH2dJ5eM GO8seNG8sYiP6JfRjl7Hyqca6YsE Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.58.3 (by Flathub.org) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 On Thu, 2026-03-19 at 13:54 +0100, Miroslav Benes wrote: > On Mon, 16 Mar 2026, Joe Lawrence wrote: >=20 > > On Fri, Mar 13, 2026 at 05:58:32PM -0300, Marcos Paulo de Souza > > wrote: > > > Instead of checking if the architecture running the test was > > > powerpc, > > > check if CONF_ARCH_HAS_SYSCALL_WRAPPER is defined or not. >=20 > There is a typo...=20 > s/CONF_ARCH_HAS_SYSCALL_WRAPPER/CONFIG_ARCH_HAS_SYSCALL_WRAPPER/ Thanks, I'll fix it in my next version. >=20 > > >=20 > > > No functional changes. > > >=20 > > > Signed-off-by: Marcos Paulo de Souza > > > --- > > > =C2=A0tools/testing/selftests/livepatch/test_modules/test_klp_syscall= . > > > c | 7 +++---- > > > =C2=A01 file changed, 3 insertions(+), 4 deletions(-) > > >=20 > > > diff --git > > > a/tools/testing/selftests/livepatch/test_modules/test_klp_syscall > > > .c > > > b/tools/testing/selftests/livepatch/test_modules/test_klp_syscall > > > .c > > > index dd802783ea849..c01a586866304 100644 > > > --- > > > a/tools/testing/selftests/livepatch/test_modules/test_klp_syscall > > > .c > > > +++ > > > b/tools/testing/selftests/livepatch/test_modules/test_klp_syscall > > > .c > > > @@ -12,15 +12,14 @@ > > > =C2=A0#include > > > =C2=A0#include > > > =C2=A0 > > > -#if defined(__x86_64__) > > > +#if !defined(CONFIG_ARCH_HAS_SYSCALL_WRAPPER) > > > +#define FN_PREFIX > > > +#elif defined(__x86_64__) > > > =C2=A0#define FN_PREFIX __x64_ > > > =C2=A0#elif defined(__s390x__) > > > =C2=A0#define FN_PREFIX __s390x_ > > > =C2=A0#elif defined(__aarch64__) > > > =C2=A0#define FN_PREFIX __arm64_ > > > -#else > > > -/* powerpc does not select ARCH_HAS_SYSCALL_WRAPPER */ > > > -#define FN_PREFIX > >=20 > > The patch does maintain the previous behavior, but I'm wondering if > > the > > original assertion about ARCH_HAS_SYSCALL_WRAPPER on Power was > > correct: > >=20 > > =C2=A0 $ grep ARCH_HAS_SYSCALL_WRAPPER arch/powerpc/Kconfig > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 select ARCH_HAS_= SYSCALL_WRAPPER=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 if !SPU_BAS= E && > > !COMPAT > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 depends on PPC64= && ARCH_HAS_SYSCALL_WRAPPER > >=20 > > Perhaps I just forgot what that additional piece of information > > that > > explains the comment (highly probable these days), and if so, might > > be > > nice to add to this commit since I don't see it in 6a71770442b5 > > ("selftests: livepatch: Test livepatching a heavily called > > syscall"). >=20 > I would take a bit further. We would rely on=20 > CONFIG_ARCH_HAS_SYSCALL_WRAPPER being set/unset per listed > architectures=20 > "correctly" for us. If it changes somehow (though I cannot imagine > reasons=20 > for that but let's say we add new architecture. LoongArch also > supports=20 > live patching.), the above might evaluate to something broken. >=20 I agree. Given that nobody even complained about it, I would say that people testing on ppc64le has this defined correctly. Whenever new archs start supporting livepatching, we can always revisit. > So I would perhaps prefer to stay with the logic that defines > FN_PREFIX=20 > per architecture and has also #else branch for the rest. And more > comments=20 > never hurt. Agreed. >=20 > Btw, see also=20 > https://sashiko.dev/#/patchset/20260313-lp-tests-old-fixes-v1-0-71ac6dfb3= 253%40suse.com > =C2=A0 > for the Sashiko AI review. It also commented on this patch. Marcos, I > guess that you will look there and I will just omit what Sashiko > found in=20 > my review if I spot the same thing. I already checked there. Maybe adding more context to the patch and code will avoid further confusion about it. Let me add it in the v2. Thanks for the reviews Miroslav and Joe! >=20 > Miroslav