From: zhouchengming <zhouchengming1@huawei.com>
To: Yury Norov <ynorov@caviumnetworks.com>
Cc: <arnd@arndb.de>, <catalin.marinas@arm.com>,
<linux-arm-kernel@lists.infradead.org>,
<linux-kernel@vger.kernel.org>, <linux-doc@vger.kernel.org>,
<linux-arch@vger.kernel.org>, <libc-alpha@sourceware.org>,
<schwidefsky@de.ibm.com>, <heiko.carstens@de.ibm.com>,
<pinskia@gmail.com>, <broonie@kernel.org>,
<joseph@codesourcery.com>,
<christoph.muellner@theobroma-systems.com>,
<bamvor.zhangjian@huawei.com>, <szabolcs.nagy@arm.com>,
<klimov.linux@gmail.com>, <Nathan_Lynch@mentor.com>,
<agraf@suse.de>, <Prasun.Kapoor@caviumnetworks.com>,
<kilobyte@angband.pl>, <geert@linux-m68k.org>,
<philipp.tomsich@theobroma-systems.com>,
<manuel.montezelo@gmail.com>, <linyongting@huawei.com>,
<maxim.kuvyrkov@linaro.org>, <davem@davemloft.net>
Subject: Re: [PATCH] ilp32: fix {GET,SET}SIGMASK request for ptrace
Date: Mon, 27 Jun 2016 18:30:51 +0800 [thread overview]
Message-ID: <5771005B.3060605@huawei.com> (raw)
In-Reply-To: <20160627053916.GA305@yury-N73SV>
[-- Attachment #1: Type: text/plain, Size: 5700 bytes --]
On 2016/6/27 13:39, Yury Norov wrote:
> Hi Zhou,
>
> Thank you for the patch. The idea is ok, but patch format got broken
> for some reason. Could you re-send it?
>
> Yury.
Sorry for the broken patch, maybe my mail guest has some problems. So
I send the patch as an attachment.
Thanks!
>
> On Mon, Jun 27, 2016 at 12:49:05PM +0800, zhouchengming wrote:
>> atus: RO
>> Content-Length: 4732
>> Lines: 181
>>
>> The function compat_ptrace_request(used by ilp32) don't handle
>> {GET,SET}SIGMASK request, so it will be handled by ptrace_request.
>> But it's wrong because the compat_sigset_t of ilp32 differs from
>> the sigset_t of aarch64. The patch fixes it.
>>
>> Signed-off-by: Zhou Chengming<zhouchengming1@huawei.com>
>> ---
>> arch/arm64/include/asm/signal_ilp32.h | 22 ++++++++++++
>> arch/arm64/kernel/ptrace.c | 62
>> +++++++++++++++++++++++++++++++++
>
> Here - unneeded line break
>
>> arch/arm64/kernel/signal_ilp32.c | 23 +------------
>> 3 files changed, 85 insertions(+), 22 deletions(-)
>>
>> diff --git a/arch/arm64/include/asm/signal_ilp32.h
>> b/arch/arm64/include/asm/signal_ilp32.h
>
> and here
>
>> index 30eff23..ed52607 100644
>> --- a/arch/arm64/include/asm/signal_ilp32.h
>> +++ b/arch/arm64/include/asm/signal_ilp32.h
>> @@ -21,6 +21,28 @@
>> int ilp32_setup_rt_frame(int usig, struct ksignal *ksig, sigset_t *set,
>> struct pt_regs *regs);
>>
>> +static inline int put_sigset_t(compat_sigset_t __user *uset, sigset_t *set)
>> +{
>> + compat_sigset_t cset;
>> +
>> + cset.sig[0] = set->sig[0]& 0xffffffffull;
>> + cset.sig[1] = set->sig[0]>> 32;
>> +
>> + return copy_to_user(uset,&cset, sizeof(*uset));
>> +}
>> +
>> +static inline int get_sigset_t(sigset_t *set,
>> + const compat_sigset_t __user *uset)
>> +{
>> + compat_sigset_t s32;
>> +
>> + if (copy_from_user(&s32, uset, sizeof(*uset)))
>> + return -EFAULT;
>> +
>> + set->sig[0] = s32.sig[0] | (((long)s32.sig[1])<< 32);
>> + return 0;
>> +}
>> +
>> #else
>>
>> static inline int ilp32_setup_rt_frame(int usig, struct ksignal *ksig,
>> sigset_t *set,
>
> and here
>
>> diff --git a/arch/arm64/kernel/ptrace.c b/arch/arm64/kernel/ptrace.c
>> index a861105..8d4cad1 100644
>> --- a/arch/arm64/kernel/ptrace.c
>> +++ b/arch/arm64/kernel/ptrace.c
>> @@ -44,6 +44,7 @@
>> #include<asm/syscall.h>
>> #include<asm/traps.h>
>> #include<asm/system_misc.h>
>> +#include<asm/signal_ilp32.h>
>>
>> #define CREATE_TRACE_POINTS
>> #include<trace/events/syscalls.h>
>> @@ -1231,12 +1232,73 @@ COMPAT_SYSCALL_DEFINE4(aarch32_ptrace,
>> compat_long_t, request, compat_long_t, pi
>
> and later on the patch
>
>>
>> #endif /* CONFIG_AARCH32_EL0 */
>>
>> +#ifdef CONFIG_ARM64_ILP32
>> +
>> +static int compat_ilp32_ptrace(struct task_struct *child, compat_long_t
>> request,
>> + compat_ulong_t addr, compat_ulong_t data)
>> +{
>> + compat_ulong_t __user *datap = compat_ptr(data);
>> + int ret;
>> +
>> + switch (request) {
>> + case PTRACE_GETSIGMASK:
>> + if (addr != sizeof(compat_sigset_t)) {
>> + ret = -EINVAL;
>> + break;
>> + }
>> +
>> + if (put_sigset_t((compat_sigset_t __user *)datap,&child->blocked))
>> + ret = -EFAULT;
>> + else
>> + ret = 0;
>> + break;
>> +
>> + case PTRACE_SETSIGMASK: {
>> + sigset_t new_set;
>> + if (addr != sizeof(compat_sigset_t)) {
>> + ret = -EINVAL;
>> + break;
>> + }
>> +
>> + if (get_sigset_t(&new_set, (compat_sigset_t __user *)datap)) {
>> + ret = -EFAULT;
>> + break;
>> + }
>> +
>> + sigdelsetmask(&new_set, sigmask(SIGKILL)|sigmask(SIGSTOP));
>> +
>> + /*
>> + * Every thread does recalc_sigpending() after resume, so
>> + * retarget_shared_pending() and recalc_sigpending() are not
>> + * called here.
>> + */
>> + spin_lock_irq(&child->sighand->siglock);
>> + child->blocked = new_set;
>> + spin_unlock_irq(&child->sighand->siglock);
>> +
>> + ret = 0;
>> + break;
>> + }
>> +
>> + default:
>> + ret = compat_ptrace_request(child, request, addr, data);
>> + }
>> +
>> + return ret;
>> +}
>> +
>> +#endif /* CONFIG_ARM64_ILP32 */
>> +
>> #ifdef CONFIG_COMPAT
>>
>> long compat_arch_ptrace(struct task_struct *child, compat_long_t request,
>> compat_ulong_t caddr, compat_ulong_t cdata)
>> {
>> +#ifdef CONFIG_ARM64_ILP32
>> + return compat_ilp32_ptrace(child, request, caddr, cdata);
>> +#else
>> return compat_ptrace_request(child, request, caddr, cdata);
>> +#endif
>> }
>>
>> #endif /* CONFIG_COMPAT */
>> diff --git a/arch/arm64/kernel/signal_ilp32.c
>> b/arch/arm64/kernel/signal_ilp32.c
>> index 8ca64b9..3ef2749 100644
>> --- a/arch/arm64/kernel/signal_ilp32.c
>> +++ b/arch/arm64/kernel/signal_ilp32.c
>> @@ -28,6 +28,7 @@
>> #include<asm/fpsimd.h>
>> #include<asm/signal32_common.h>
>> #include<asm/signal_common.h>
>> +#include<asm/signal_ilp32.h>
>> #include<asm/uaccess.h>
>> #include<asm/unistd.h>
>> #include<asm/ucontext.h>
>> @@ -58,28 +59,6 @@ struct ilp32_rt_sigframe {
>> struct ilp32_sigframe sig;
>> };
>>
>> -static inline int put_sigset_t(compat_sigset_t __user *uset, sigset_t *set)
>> -{
>> - compat_sigset_t cset;
>> -
>> - cset.sig[0] = set->sig[0]& 0xffffffffull;
>> - cset.sig[1] = set->sig[0]>> 32;
>> -
>> - return copy_to_user(uset,&cset, sizeof(*uset));
>> -}
>> -
>> -static inline int get_sigset_t(sigset_t *set,
>> - const compat_sigset_t __user *uset)
>> -{
>> - compat_sigset_t s32;
>> -
>> - if (copy_from_user(&s32, uset, sizeof(*uset)))
>> - return -EFAULT;
>> -
>> - set->sig[0] = s32.sig[0] | (((long)s32.sig[1])<< 32);
>> - return 0;
>> -}
>> -
>> static int restore_ilp32_sigframe(struct pt_regs *regs,
>> struct
>
> .
>
[-- Attachment #2: 0001-ilp32-fix-GET-SET-SIGMASK-request-for-ptrace.patch --]
[-- Type: text/plain, Size: 4926 bytes --]
>From 06f5abeca59069cb41b0b4fdfa06240d3f78910d Mon Sep 17 00:00:00 2001
From: Zhou Chengming <zhouchengming1@huawei.com>
Date: Mon, 27 Jun 2016 12:57:02 +0800
Subject: [PATCH] ilp32: fix {GET,SET}SIGMASK request for ptrace
The function compat_ptrace_request(used by ilp32) don't handle
{GET,SET}SIGMASK request, so it will be handled by ptrace_request.
But it's wrong because the compat_sigset_t of ilp32 differs from
the sigset_t of aarch64. The patch fixes it.
Signed-off-by: Zhou Chengming <zhouchengming1@huawei.com>
---
arch/arm64/include/asm/signal_ilp32.h | 22 ++++++++++++
arch/arm64/kernel/ptrace.c | 62 +++++++++++++++++++++++++++++++++
arch/arm64/kernel/signal_ilp32.c | 23 +------------
3 files changed, 85 insertions(+), 22 deletions(-)
diff --git a/arch/arm64/include/asm/signal_ilp32.h b/arch/arm64/include/asm/signal_ilp32.h
index 30eff23..ed52607 100644
--- a/arch/arm64/include/asm/signal_ilp32.h
+++ b/arch/arm64/include/asm/signal_ilp32.h
@@ -21,6 +21,28 @@
int ilp32_setup_rt_frame(int usig, struct ksignal *ksig, sigset_t *set,
struct pt_regs *regs);
+static inline int put_sigset_t(compat_sigset_t __user *uset, sigset_t *set)
+{
+ compat_sigset_t cset;
+
+ cset.sig[0] = set->sig[0] & 0xffffffffull;
+ cset.sig[1] = set->sig[0] >> 32;
+
+ return copy_to_user(uset, &cset, sizeof(*uset));
+}
+
+static inline int get_sigset_t(sigset_t *set,
+ const compat_sigset_t __user *uset)
+{
+ compat_sigset_t s32;
+
+ if (copy_from_user(&s32, uset, sizeof(*uset)))
+ return -EFAULT;
+
+ set->sig[0] = s32.sig[0] | (((long)s32.sig[1]) << 32);
+ return 0;
+}
+
#else
static inline int ilp32_setup_rt_frame(int usig, struct ksignal *ksig, sigset_t *set,
diff --git a/arch/arm64/kernel/ptrace.c b/arch/arm64/kernel/ptrace.c
index a861105..8d4cad1 100644
--- a/arch/arm64/kernel/ptrace.c
+++ b/arch/arm64/kernel/ptrace.c
@@ -44,6 +44,7 @@
#include <asm/syscall.h>
#include <asm/traps.h>
#include <asm/system_misc.h>
+#include <asm/signal_ilp32.h>
#define CREATE_TRACE_POINTS
#include <trace/events/syscalls.h>
@@ -1231,12 +1232,73 @@ COMPAT_SYSCALL_DEFINE4(aarch32_ptrace, compat_long_t, request, compat_long_t, pi
#endif /* CONFIG_AARCH32_EL0 */
+#ifdef CONFIG_ARM64_ILP32
+
+static int compat_ilp32_ptrace(struct task_struct *child, compat_long_t request,
+ compat_ulong_t addr, compat_ulong_t data)
+{
+ compat_ulong_t __user *datap = compat_ptr(data);
+ int ret;
+
+ switch (request) {
+ case PTRACE_GETSIGMASK:
+ if (addr != sizeof(compat_sigset_t)) {
+ ret = -EINVAL;
+ break;
+ }
+
+ if (put_sigset_t((compat_sigset_t __user *)datap, &child->blocked))
+ ret = -EFAULT;
+ else
+ ret = 0;
+ break;
+
+ case PTRACE_SETSIGMASK: {
+ sigset_t new_set;
+ if (addr != sizeof(compat_sigset_t)) {
+ ret = -EINVAL;
+ break;
+ }
+
+ if (get_sigset_t(&new_set, (compat_sigset_t __user *)datap)) {
+ ret = -EFAULT;
+ break;
+ }
+
+ sigdelsetmask(&new_set, sigmask(SIGKILL)|sigmask(SIGSTOP));
+
+ /*
+ * Every thread does recalc_sigpending() after resume, so
+ * retarget_shared_pending() and recalc_sigpending() are not
+ * called here.
+ */
+ spin_lock_irq(&child->sighand->siglock);
+ child->blocked = new_set;
+ spin_unlock_irq(&child->sighand->siglock);
+
+ ret = 0;
+ break;
+ }
+
+ default:
+ ret = compat_ptrace_request(child, request, addr, data);
+ }
+
+ return ret;
+}
+
+#endif /* CONFIG_ARM64_ILP32 */
+
#ifdef CONFIG_COMPAT
long compat_arch_ptrace(struct task_struct *child, compat_long_t request,
compat_ulong_t caddr, compat_ulong_t cdata)
{
+#ifdef CONFIG_ARM64_ILP32
+ return compat_ilp32_ptrace(child, request, caddr, cdata);
+#else
return compat_ptrace_request(child, request, caddr, cdata);
+#endif
}
#endif /* CONFIG_COMPAT */
diff --git a/arch/arm64/kernel/signal_ilp32.c b/arch/arm64/kernel/signal_ilp32.c
index 8ca64b9..3ef2749 100644
--- a/arch/arm64/kernel/signal_ilp32.c
+++ b/arch/arm64/kernel/signal_ilp32.c
@@ -28,6 +28,7 @@
#include <asm/fpsimd.h>
#include <asm/signal32_common.h>
#include <asm/signal_common.h>
+#include <asm/signal_ilp32.h>
#include <asm/uaccess.h>
#include <asm/unistd.h>
#include <asm/ucontext.h>
@@ -58,28 +59,6 @@ struct ilp32_rt_sigframe {
struct ilp32_sigframe sig;
};
-static inline int put_sigset_t(compat_sigset_t __user *uset, sigset_t *set)
-{
- compat_sigset_t cset;
-
- cset.sig[0] = set->sig[0] & 0xffffffffull;
- cset.sig[1] = set->sig[0] >> 32;
-
- return copy_to_user(uset, &cset, sizeof(*uset));
-}
-
-static inline int get_sigset_t(sigset_t *set,
- const compat_sigset_t __user *uset)
-{
- compat_sigset_t s32;
-
- if (copy_from_user(&s32, uset, sizeof(*uset)))
- return -EFAULT;
-
- set->sig[0] = s32.sig[0] | (((long)s32.sig[1]) << 32);
- return 0;
-}
-
static int restore_ilp32_sigframe(struct pt_regs *regs,
struct ilp32_sigframe __user *sf)
{
--
1.7.7
next prev parent reply other threads:[~2016-06-27 10:33 UTC|newest]
Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-17 23:54 [RFC nowrap: PATCH v7 00/19] ILP32 for ARM64 Yury Norov
2016-06-17 23:54 ` [PATCH 01/19] compat ABI: use non-compat openat and open_by_handle_at variants Yury Norov
2016-06-27 7:47 ` Andreas Schwab
2016-08-15 14:30 ` Yury Norov
2016-08-25 15:52 ` Arnd Bergmann
2016-08-29 8:43 ` Yury Norov
2016-06-17 23:54 ` [PATCH 02/19] 32-bit ABI: introduce ARCH_32BIT_OFF_T config option Yury Norov
2016-06-17 23:54 ` [PATCH 03/19] arm64: ilp32: add documentation on the ILP32 ABI for ARM64 Yury Norov
2016-06-17 23:54 ` [PATCH 04/19] arm64: ensure the kernel is compiled for LP64 Yury Norov
2016-06-17 23:54 ` [PATCH 05/19] arm64: rename COMPAT to AARCH32_EL0 in Kconfig Yury Norov
2016-08-11 7:35 ` Zhangjian (Bamvor)
2016-08-11 8:53 ` Arnd Bergmann
2016-08-11 14:50 ` Catalin Marinas
2016-08-11 15:16 ` Arnd Bergmann
2016-08-11 16:30 ` Catalin Marinas
2016-08-11 20:29 ` Arnd Bergmann
2016-08-12 14:36 ` Catalin Marinas
2016-08-13 15:17 ` Yury Norov
2016-08-15 9:38 ` Catalin Marinas
2016-06-17 23:54 ` [PATCH 06/19] arm64:uapi: set __BITS_PER_LONG correctly for ILP32 and LP64 Yury Norov
2016-06-17 23:54 ` [PATCH 07/19] thread: move thread bits accessors to separated file Yury Norov
2016-06-17 23:54 ` [PATCH 08/19] arm64: introduce is_a32_task and is_a32_thread (for AArch32 compat) Yury Norov
2016-06-21 10:23 ` Zhangjian (Bamvor)
2016-06-21 11:34 ` Andreas Schwab
2016-06-21 15:47 ` Arnd Bergmann
2016-06-17 23:54 ` [PATCH 09/19] arm64: ilp32: add is_ilp32_compat_{task,thread} and TIF_32BIT_AARCH64 Yury Norov
2016-06-17 23:54 ` [PATCH 10/19] arm64: introduce binfmt_elf32.c Yury Norov
2016-06-17 23:54 ` [PATCH 11/19] arm64: ilp32: introduce binfmt_ilp32.c Yury Norov
2016-06-17 23:54 ` [PATCH 12/19] arm64: ptrace: handle ptrace_request differently for aarch32 and ilp32 Yury Norov
2016-06-27 4:47 ` zhouchengming
2016-06-27 4:49 ` [PATCH] ilp32: fix {GET,SET}SIGMASK request for ptrace zhouchengming
2016-06-27 5:39 ` Yury Norov
2016-06-27 10:30 ` zhouchengming [this message]
2016-08-13 14:52 ` Yury Norov
2016-06-17 23:54 ` [PATCH 13/19] arm64: ilp32: share aarch32 syscall handlers Yury Norov
2016-06-17 23:54 ` [PATCH 14/19] arm64: ilp32: add sys_ilp32.c and a separate table (in entry.S) to use it Yury Norov
2016-06-20 12:48 ` Yury Norov
2016-06-17 23:54 ` [PATCH 15/19] arm64: signal: share lp64 signal routines to ilp32 Yury Norov
2016-06-17 23:54 ` [PATCH 16/19] arm64: signal32: move ilp32 and aarch32 common code to separated file Yury Norov
2016-06-17 23:54 ` [PATCH 17/19] arm64: ilp32: introduce ilp32-specific handlers for sigframe and ucontext Yury Norov
2016-06-17 23:54 ` [PATCH 18/19] arm64:ilp32: add vdso-ilp32 and use for signal return Yury Norov
2016-06-20 14:54 ` Andreas Schwab
2016-08-13 14:54 ` Yury Norov
2016-06-17 23:54 ` [PATCH 19/19] arm64:ilp32: add ARM64_ILP32 to Kconfig Yury Norov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5771005B.3060605@huawei.com \
--to=zhouchengming1@huawei.com \
--cc=Nathan_Lynch@mentor.com \
--cc=Prasun.Kapoor@caviumnetworks.com \
--cc=agraf@suse.de \
--cc=arnd@arndb.de \
--cc=bamvor.zhangjian@huawei.com \
--cc=broonie@kernel.org \
--cc=catalin.marinas@arm.com \
--cc=christoph.muellner@theobroma-systems.com \
--cc=davem@davemloft.net \
--cc=geert@linux-m68k.org \
--cc=heiko.carstens@de.ibm.com \
--cc=joseph@codesourcery.com \
--cc=kilobyte@angband.pl \
--cc=klimov.linux@gmail.com \
--cc=libc-alpha@sourceware.org \
--cc=linux-arch@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linyongting@huawei.com \
--cc=manuel.montezelo@gmail.com \
--cc=maxim.kuvyrkov@linaro.org \
--cc=philipp.tomsich@theobroma-systems.com \
--cc=pinskia@gmail.com \
--cc=schwidefsky@de.ibm.com \
--cc=szabolcs.nagy@arm.com \
--cc=ynorov@caviumnetworks.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).