From: Christophe Leroy <christophe.leroy@csgroup.eu> To: Oleg Nesterov <oleg@redhat.com>, Benjamin Herrenschmidt <benh@kernel.crashing.org>, Madhavan Srinivasan <maddy@linux.vnet.ibm.com>, Michael Ellerman <mpe@ellerman.id.au>, Paul Mackerras <paulus@samba.org> Cc: Al Viro <viro@zeniv.linux.org.uk>, "Aneesh Kumar K.V" <aneesh.kumar@linux.ibm.com>, Christophe Leroy <christophe.leroy@c-s.fr>, Jan Kratochvil <jan.kratochvil@redhat.com>, Nicholas Piggin <npiggin@gmail.com>, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 2/2] powerpc/ptrace: Hard wire PT_SOFTE value to 1 in gpr_get() too Date: Thu, 19 Nov 2020 18:18:23 +0100 [thread overview] Message-ID: <4e132957-c53c-4b06-9546-26d1de0a672f@csgroup.eu> (raw) In-Reply-To: <20201119160247.GB5188@redhat.com> Le 19/11/2020 à 17:02, Oleg Nesterov a écrit : > The commit a8a4b03ab95f ("powerpc: Hard wire PT_SOFTE value to 1 in > ptrace & signals") changed ptrace_get_reg(PT_SOFTE) to report 0x1, > but PTRACE_GETREGS still copies pt_regs->softe as is. > > This is not consistent and this breaks the user-regs-peekpoke test > from https://sourceware.org/systemtap/wiki/utrace/tests/ > > Reported-by: Jan Kratochvil <jan.kratochvil@redhat.com> > Signed-off-by: Oleg Nesterov <oleg@redhat.com> > --- > arch/powerpc/kernel/ptrace/ptrace-tm.c | 8 +++++++- > arch/powerpc/kernel/ptrace/ptrace-view.c | 8 +++++++- > 2 files changed, 14 insertions(+), 2 deletions(-) > > diff --git a/arch/powerpc/kernel/ptrace/ptrace-tm.c b/arch/powerpc/kernel/ptrace/ptrace-tm.c > index f8fcbd85d4cb..d0d339f86e61 100644 > --- a/arch/powerpc/kernel/ptrace/ptrace-tm.c > +++ b/arch/powerpc/kernel/ptrace/ptrace-tm.c > @@ -87,6 +87,10 @@ int tm_cgpr_get(struct task_struct *target, const struct user_regset *regset, > struct membuf to) > { > struct membuf to_msr = membuf_at(&to, offsetof(struct pt_regs, msr)); > +#ifdef CONFIG_PPC64 > + struct membuf to_softe = membuf_at(&to, > + offsetof(struct pt_regs, softe)); Should fit on a single line I think. > +#endif > > if (!cpu_has_feature(CPU_FTR_TM)) > return -ENODEV; > @@ -102,7 +106,9 @@ int tm_cgpr_get(struct task_struct *target, const struct user_regset *regset, > sizeof(struct user_pt_regs)); > > membuf_store(&to_msr, get_user_ckpt_msr(target)); > - > +#ifdef CONFIG_PPC64 > + membuf_store(&to_softe, 0x1ul); > +#endif > return membuf_zero(&to, ELF_NGREG * sizeof(unsigned long) - > sizeof(struct user_pt_regs)); > } > diff --git a/arch/powerpc/kernel/ptrace/ptrace-view.c b/arch/powerpc/kernel/ptrace/ptrace-view.c > index 39686ede40b3..f554ccfcbfae 100644 > --- a/arch/powerpc/kernel/ptrace/ptrace-view.c > +++ b/arch/powerpc/kernel/ptrace/ptrace-view.c > @@ -218,6 +218,10 @@ static int gpr_get(struct task_struct *target, const struct user_regset *regset, > struct membuf to) > { > struct membuf to_msr = membuf_at(&to, offsetof(struct pt_regs, msr)); > +#ifdef CONFIG_PPC64 > + struct membuf to_softe = membuf_at(&to, > + offsetof(struct pt_regs, softe)); Should fit on a single line I think. > +#endif > int i; > > if (target->thread.regs == NULL) > @@ -233,7 +237,9 @@ static int gpr_get(struct task_struct *target, const struct user_regset *regset, > sizeof(struct user_pt_regs)); > > membuf_store(&to_msr, get_user_msr(target)); > - > +#ifdef CONFIG_PPC64 > + membuf_store(&to_softe, 0x1ul); > +#endif > return membuf_zero(&to, ELF_NGREG * sizeof(unsigned long) - > sizeof(struct user_pt_regs)); > } > Christophe
WARNING: multiple messages have this Message-ID (diff)
From: Christophe Leroy <christophe.leroy@csgroup.eu> To: Oleg Nesterov <oleg@redhat.com>, Benjamin Herrenschmidt <benh@kernel.crashing.org>, Madhavan Srinivasan <maddy@linux.vnet.ibm.com>, Michael Ellerman <mpe@ellerman.id.au>, Paul Mackerras <paulus@samba.org> Cc: Christophe Leroy <christophe.leroy@c-s.fr>, "Aneesh Kumar K.V" <aneesh.kumar@linux.ibm.com>, linux-kernel@vger.kernel.org, Nicholas Piggin <npiggin@gmail.com>, Jan Kratochvil <jan.kratochvil@redhat.com>, Al Viro <viro@zeniv.linux.org.uk>, linuxppc-dev@lists.ozlabs.org Subject: Re: [PATCH v3 2/2] powerpc/ptrace: Hard wire PT_SOFTE value to 1 in gpr_get() too Date: Thu, 19 Nov 2020 18:18:23 +0100 [thread overview] Message-ID: <4e132957-c53c-4b06-9546-26d1de0a672f@csgroup.eu> (raw) In-Reply-To: <20201119160247.GB5188@redhat.com> Le 19/11/2020 à 17:02, Oleg Nesterov a écrit : > The commit a8a4b03ab95f ("powerpc: Hard wire PT_SOFTE value to 1 in > ptrace & signals") changed ptrace_get_reg(PT_SOFTE) to report 0x1, > but PTRACE_GETREGS still copies pt_regs->softe as is. > > This is not consistent and this breaks the user-regs-peekpoke test > from https://sourceware.org/systemtap/wiki/utrace/tests/ > > Reported-by: Jan Kratochvil <jan.kratochvil@redhat.com> > Signed-off-by: Oleg Nesterov <oleg@redhat.com> > --- > arch/powerpc/kernel/ptrace/ptrace-tm.c | 8 +++++++- > arch/powerpc/kernel/ptrace/ptrace-view.c | 8 +++++++- > 2 files changed, 14 insertions(+), 2 deletions(-) > > diff --git a/arch/powerpc/kernel/ptrace/ptrace-tm.c b/arch/powerpc/kernel/ptrace/ptrace-tm.c > index f8fcbd85d4cb..d0d339f86e61 100644 > --- a/arch/powerpc/kernel/ptrace/ptrace-tm.c > +++ b/arch/powerpc/kernel/ptrace/ptrace-tm.c > @@ -87,6 +87,10 @@ int tm_cgpr_get(struct task_struct *target, const struct user_regset *regset, > struct membuf to) > { > struct membuf to_msr = membuf_at(&to, offsetof(struct pt_regs, msr)); > +#ifdef CONFIG_PPC64 > + struct membuf to_softe = membuf_at(&to, > + offsetof(struct pt_regs, softe)); Should fit on a single line I think. > +#endif > > if (!cpu_has_feature(CPU_FTR_TM)) > return -ENODEV; > @@ -102,7 +106,9 @@ int tm_cgpr_get(struct task_struct *target, const struct user_regset *regset, > sizeof(struct user_pt_regs)); > > membuf_store(&to_msr, get_user_ckpt_msr(target)); > - > +#ifdef CONFIG_PPC64 > + membuf_store(&to_softe, 0x1ul); > +#endif > return membuf_zero(&to, ELF_NGREG * sizeof(unsigned long) - > sizeof(struct user_pt_regs)); > } > diff --git a/arch/powerpc/kernel/ptrace/ptrace-view.c b/arch/powerpc/kernel/ptrace/ptrace-view.c > index 39686ede40b3..f554ccfcbfae 100644 > --- a/arch/powerpc/kernel/ptrace/ptrace-view.c > +++ b/arch/powerpc/kernel/ptrace/ptrace-view.c > @@ -218,6 +218,10 @@ static int gpr_get(struct task_struct *target, const struct user_regset *regset, > struct membuf to) > { > struct membuf to_msr = membuf_at(&to, offsetof(struct pt_regs, msr)); > +#ifdef CONFIG_PPC64 > + struct membuf to_softe = membuf_at(&to, > + offsetof(struct pt_regs, softe)); Should fit on a single line I think. > +#endif > int i; > > if (target->thread.regs == NULL) > @@ -233,7 +237,9 @@ static int gpr_get(struct task_struct *target, const struct user_regset *regset, > sizeof(struct user_pt_regs)); > > membuf_store(&to_msr, get_user_msr(target)); > - > +#ifdef CONFIG_PPC64 > + membuf_store(&to_softe, 0x1ul); > +#endif > return membuf_zero(&to, ELF_NGREG * sizeof(unsigned long) - > sizeof(struct user_pt_regs)); > } > Christophe
next prev parent reply other threads:[~2020-11-19 17:19 UTC|newest] Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-11-19 16:01 [PATCH v3 0/2] powerpc/ptrace: Hard wire PT_SOFTE value to 1 in gpr_get() too Oleg Nesterov 2020-11-19 16:01 ` Oleg Nesterov 2020-11-19 16:02 ` [PATCH v3 1/2] powerpc/ptrace: simplify gpr_get/tm_cgpr_get Oleg Nesterov 2020-11-19 16:02 ` Oleg Nesterov 2020-11-19 17:16 ` Christophe Leroy 2020-11-19 17:16 ` Christophe Leroy 2020-11-19 18:18 ` Oleg Nesterov 2020-11-19 18:18 ` Oleg Nesterov 2020-11-19 16:02 ` [PATCH v3 2/2] powerpc/ptrace: Hard wire PT_SOFTE value to 1 in gpr_get() too Oleg Nesterov 2020-11-19 16:02 ` Oleg Nesterov 2020-11-19 16:05 ` Oleg Nesterov 2020-11-19 16:05 ` Oleg Nesterov 2020-11-19 17:18 ` Christophe Leroy [this message] 2020-11-19 17:18 ` Christophe Leroy 2020-11-19 21:10 ` Christophe Leroy 2020-11-19 21:10 ` Christophe Leroy 2020-11-19 22:43 ` Oleg Nesterov 2020-11-19 22:43 ` Oleg Nesterov 2020-11-23 18:01 ` Oleg Nesterov 2020-11-23 18:01 ` Oleg Nesterov 2020-11-24 0:53 ` Michael Ellerman 2020-11-24 0:53 ` Michael Ellerman 2020-11-19 17:19 ` [PATCH v3 0/2] " Christophe Leroy 2020-11-19 17:19 ` Christophe Leroy 2020-11-19 18:22 ` Oleg Nesterov 2020-11-19 18:22 ` Oleg Nesterov 2020-12-10 11:30 ` Michael Ellerman 2020-12-10 11:30 ` Michael Ellerman
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=4e132957-c53c-4b06-9546-26d1de0a672f@csgroup.eu \ --to=christophe.leroy@csgroup.eu \ --cc=aneesh.kumar@linux.ibm.com \ --cc=benh@kernel.crashing.org \ --cc=christophe.leroy@c-s.fr \ --cc=jan.kratochvil@redhat.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linuxppc-dev@lists.ozlabs.org \ --cc=maddy@linux.vnet.ibm.com \ --cc=mpe@ellerman.id.au \ --cc=npiggin@gmail.com \ --cc=oleg@redhat.com \ --cc=paulus@samba.org \ --cc=viro@zeniv.linux.org.uk \ /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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.