From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from vmicros1.altlinux.org (vmicros1.altlinux.org [194.107.17.57]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 9201C3D6A; Wed, 26 Mar 2025 09:06:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=194.107.17.57 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742980009; cv=none; b=ddGr8ETQ1N+8OwRWe0qOCViQthONLAP/l1dOpgyIMOGUwCsHv0AQXh2SeX07uRUtwU2yrJtlChi0C4HbMvlMkurQszl6gWmDg7SjWT3shCAwI5ygXtBuIrDRlqtGowLUv7eiFHH/thPSs8JhEFb+9pB9Z1gyfvOCYgFKiAK8PAg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742980009; c=relaxed/simple; bh=ZiJPbInomXuz/9oF7rNjIFif43TGI37yHFbWqkKInm0=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=bR7Sac/wIR914ZvoTc/6ZBsPyxTF3Q5cJ3EwBTVuFYFjM12yNuI7TQsdsnLox0+nxoqLpKuqUX6/tjBORTKleUR9mY9ZjOcIxXj6ygnusYZO8lsy3UK00CLmzjJ+lE9vB+Yjrmwc72LLZNZ2Qmqxbdw0Lsvxu3wDvXeIaS5iRjs= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=strace.io; spf=pass smtp.mailfrom=altlinux.org; arc=none smtp.client-ip=194.107.17.57 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=strace.io Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=altlinux.org Received: from mua.local.altlinux.org (mua.local.altlinux.org [192.168.1.14]) by vmicros1.altlinux.org (Postfix) with ESMTP id DD97F72C8CC; Wed, 26 Mar 2025 12:06:43 +0300 (MSK) Received: by mua.local.altlinux.org (Postfix, from userid 508) id CDF3E7CCB3A; Wed, 26 Mar 2025 11:06:43 +0200 (IST) Date: Wed, 26 Mar 2025 11:06:43 +0200 From: "Dmitry V. Levin" To: Shuah Khan , Andrew Morton Cc: Oleg Nesterov , "Maciej W. Rozycki" , strace-devel@lists.strace.io, linux-kselftest@vger.kernel.org, linux-mips@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] selftests/ptrace/get_syscall_info: fix for MIPS n32 Message-ID: <20250326090643.GA30882@strace.io> References: <20250115233747.GA28541@strace.io> Precedence: bulk X-Mailing-List: linux-kselftest@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20250115233747.GA28541@strace.io> Could somebody pick up this patch, please? Nothing has changed since v2, so I have nothing new to add. v2: https://lore.kernel.org/all/20250115233747.GA28541@strace.io/ On Thu, Jan 16, 2025 at 01:37:47AM +0200, Dmitry V. Levin wrote: > MIPS n32 is one of two ILP32 architectures supported by the kernel > that have 64-bit syscall arguments (another one is x32). > > When this test passed 32-bit arguments to syscall(), they were > sign-extended in libc, PTRACE_GET_SYSCALL_INFO reported these > sign-extended 64-bit values, and the test complained about the mismatch. > > Fix this by passing arguments of the appropriate type to syscall(), > which is "unsigned long long" on MIPS n32, and __kernel_ulong_t on other > architectures. > > As a side effect, this also extends the test on all 64-bit architectures > by choosing constants that don't fit into 32-bit integers. > > Signed-off-by: Dmitry V. Levin > --- > > v2: Fixed MIPS #ifdef. > > .../selftests/ptrace/get_syscall_info.c | 53 +++++++++++-------- > 1 file changed, 32 insertions(+), 21 deletions(-) > > diff --git a/tools/testing/selftests/ptrace/get_syscall_info.c b/tools/testing/selftests/ptrace/get_syscall_info.c > index 5bcd1c7b5be6..2970f72d66d3 100644 > --- a/tools/testing/selftests/ptrace/get_syscall_info.c > +++ b/tools/testing/selftests/ptrace/get_syscall_info.c > @@ -11,8 +11,19 @@ > #include > #include > #include > +#include > #include "linux/ptrace.h" > > +#if defined(_MIPS_SIM) && _MIPS_SIM == _MIPS_SIM_NABI32 > +/* > + * MIPS N32 is the only architecture where __kernel_ulong_t > + * does not match the bitness of syscall arguments. > + */ > +typedef unsigned long long kernel_ulong_t; > +#else > +typedef __kernel_ulong_t kernel_ulong_t; > +#endif > + > static int > kill_tracee(pid_t pid) > { > @@ -42,37 +53,37 @@ sys_ptrace(int request, pid_t pid, unsigned long addr, unsigned long data) > > TEST(get_syscall_info) > { > - static const unsigned long args[][7] = { > + const kernel_ulong_t args[][7] = { > /* a sequence of architecture-agnostic syscalls */ > { > __NR_chdir, > - (unsigned long) "", > - 0xbad1fed1, > - 0xbad2fed2, > - 0xbad3fed3, > - 0xbad4fed4, > - 0xbad5fed5 > + (uintptr_t) "", > + (kernel_ulong_t) 0xdad1bef1bad1fed1ULL, > + (kernel_ulong_t) 0xdad2bef2bad2fed2ULL, > + (kernel_ulong_t) 0xdad3bef3bad3fed3ULL, > + (kernel_ulong_t) 0xdad4bef4bad4fed4ULL, > + (kernel_ulong_t) 0xdad5bef5bad5fed5ULL > }, > { > __NR_gettid, > - 0xcaf0bea0, > - 0xcaf1bea1, > - 0xcaf2bea2, > - 0xcaf3bea3, > - 0xcaf4bea4, > - 0xcaf5bea5 > + (kernel_ulong_t) 0xdad0bef0caf0bea0ULL, > + (kernel_ulong_t) 0xdad1bef1caf1bea1ULL, > + (kernel_ulong_t) 0xdad2bef2caf2bea2ULL, > + (kernel_ulong_t) 0xdad3bef3caf3bea3ULL, > + (kernel_ulong_t) 0xdad4bef4caf4bea4ULL, > + (kernel_ulong_t) 0xdad5bef5caf5bea5ULL > }, > { > __NR_exit_group, > 0, > - 0xfac1c0d1, > - 0xfac2c0d2, > - 0xfac3c0d3, > - 0xfac4c0d4, > - 0xfac5c0d5 > + (kernel_ulong_t) 0xdad1bef1fac1c0d1ULL, > + (kernel_ulong_t) 0xdad2bef2fac2c0d2ULL, > + (kernel_ulong_t) 0xdad3bef3fac3c0d3ULL, > + (kernel_ulong_t) 0xdad4bef4fac4c0d4ULL, > + (kernel_ulong_t) 0xdad5bef5fac5c0d5ULL > } > }; > - const unsigned long *exp_args; > + const kernel_ulong_t *exp_args; > > pid_t pid = fork(); > > @@ -154,7 +165,7 @@ TEST(get_syscall_info) > } > ASSERT_LT(0, (rc = sys_ptrace(PTRACE_GET_SYSCALL_INFO, > pid, size, > - (unsigned long) &info))) { > + (uintptr_t) &info))) { > LOG_KILL_TRACEE("PTRACE_GET_SYSCALL_INFO: %m"); > } > ASSERT_EQ(expected_none_size, rc) { > @@ -177,7 +188,7 @@ TEST(get_syscall_info) > case SIGTRAP | 0x80: > ASSERT_LT(0, (rc = sys_ptrace(PTRACE_GET_SYSCALL_INFO, > pid, size, > - (unsigned long) &info))) { > + (uintptr_t) &info))) { > LOG_KILL_TRACEE("PTRACE_GET_SYSCALL_INFO: %m"); > } > switch (ptrace_stop) { > -- > ldv -- ldv