From: Nicholas Piggin <npiggin@gmail.com> To: "Michal Suchánek" <msuchanek@suse.de> Cc: Arnaldo Carvalho de Melo <acme@kernel.org>, Alexander Shishkin <alexander.shishkin@linux.intel.com>, Allison Randal <allison@lohutok.net>, Andy Shevchenko <andriy.shevchenko@linux.intel.com>, Arnd Bergmann <arnd@arndb.de>, Thiago Jung Bauermann <bauerman@linux.ibm.com>, Benjamin Herrenschmidt <benh@kernel.crashing.org>, Sebastian Andrzej Siewior <bigeasy@linutronix.de>, Claudio Carvalho <cclaudio@linux.ibm.com>, Christophe Leroy <christophe.leroy@c-s.fr>, "David S. Miller" <davem@davemloft.net>, "Eric W. Biederman" <ebiederm@xmission.com>, Eric Richter <erichte@linux.ibm.com>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Gustavo Luiz Duarte <gustavold@linux.ibm.com>, Hari Bathini <hbathini@linux.ibm.com>, Jordan Niethe <jniethe5@gmail.com>, Jiri Olsa <jolsa@redhat.com>, Jonathan Cameron <Jonathan.Cameron@huawei.com>, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, Mark Rutland <mark.rutland@arm.com>, Masahiro Yamada <masahiroy@kernel.org>, Mauro Carvalho Chehab <mchehab+huawei@kernel.org>, Michael Neuling <mikey@neuling.org>, Ingo Molnar <mingo@redhat.com>, Michael Ellerman <mpe@ellerman.id.au>, Namhyung Kim <namhyung@kernel.org>, Nayna Jain <nayna@linux.ibm.com>, Paul Mackerras <paulus@samba.org>, Peter Zijlstra <peterz@infradead.org>, Rob Herring <robh@kernel.org>, Thomas Gleixner <tglx@linutronix.de>, Valentin Schneider <valentin.schneider@arm.com>, Alexander Viro <viro@zeniv.linux.org.uk> Subject: Re: [PATCH v11 3/8] powerpc/perf: consolidate read_user_stack_32 Date: Fri, 03 Apr 2020 21:26:27 +1000 [thread overview] Message-ID: <1585913065.zoacp2kzsv.astroid@bobo.none> (raw) In-Reply-To: <20200403105234.GX25468@kitsune.suse.cz> Michal Suchánek's on April 3, 2020 8:52 pm: > Hello, > > there are 3 variants of the function > > read_user_stack_64 > > 32bit read_user_stack_32 > 64bit read_user_Stack_32 Right. > On Fri, Apr 03, 2020 at 05:13:25PM +1000, Nicholas Piggin wrote: [...] >> #endif /* CONFIG_PPC64 */ >> >> +static int read_user_stack_32(unsigned int __user *ptr, unsigned int *ret) >> +{ >> + return __read_user_stack(ptr, ret, sizeof(*ret)); > Does not work for 64bit read_user_stack_32 ^ this should be 4. > > Other than that it should preserve the existing logic just fine. sizeof(int) == 4 on 64bit so it should work. Thanks, Nick
WARNING: multiple messages have this Message-ID (diff)
From: Nicholas Piggin <npiggin@gmail.com> To: "Michal Suchánek" <msuchanek@suse.de> Cc: Mark Rutland <mark.rutland@arm.com>, Gustavo Luiz Duarte <gustavold@linux.ibm.com>, Alexander Shishkin <alexander.shishkin@linux.intel.com>, Sebastian Andrzej Siewior <bigeasy@linutronix.de>, linux-kernel@vger.kernel.org, Paul Mackerras <paulus@samba.org>, Jiri Olsa <jolsa@redhat.com>, Rob Herring <robh@kernel.org>, Michael Neuling <mikey@neuling.org>, Eric Richter <erichte@linux.ibm.com>, Masahiro Yamada <masahiroy@kernel.org>, Nayna Jain <nayna@linux.ibm.com>, Peter Zijlstra <peterz@infradead.org>, Ingo Molnar <mingo@redhat.com>, Hari Bathini <hbathini@linux.ibm.com>, Jordan Niethe <jniethe5@gmail.com>, Valentin Schneider <valentin.schneider@arm.com>, Arnd Bergmann <arnd@arndb.de>, Arnaldo Carvalho de Melo <acme@kernel.org>, Alexander Viro <viro@zeniv.linux.org.uk>, Jonathan Cameron <Jonathan.Cameron@huawei.com>, Namhyung Kim <namhyung@kernel.org>, Thomas Gleixner <tglx@linutronix.de>, Andy Shevchenko <andriy.shevchenko@linux.intel.com>, Allison Randal <allison@lohutok.net>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Claudio Carvalho <cclaudio@linux.ibm.com>, Mauro Carvalho Chehab <mchehab+huawei@kernel.org>, "Eric W. Biederman" <ebiederm@xmission.com>, linux-fsdevel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, "David S. Miller" <davem@davemloft.net>, Thiago Jung Bauermann <bauerman@linux.ibm.com> Subject: Re: [PATCH v11 3/8] powerpc/perf: consolidate read_user_stack_32 Date: Fri, 03 Apr 2020 21:26:27 +1000 [thread overview] Message-ID: <1585913065.zoacp2kzsv.astroid@bobo.none> (raw) In-Reply-To: <20200403105234.GX25468@kitsune.suse.cz> Michal Suchánek's on April 3, 2020 8:52 pm: > Hello, > > there are 3 variants of the function > > read_user_stack_64 > > 32bit read_user_stack_32 > 64bit read_user_Stack_32 Right. > On Fri, Apr 03, 2020 at 05:13:25PM +1000, Nicholas Piggin wrote: [...] >> #endif /* CONFIG_PPC64 */ >> >> +static int read_user_stack_32(unsigned int __user *ptr, unsigned int *ret) >> +{ >> + return __read_user_stack(ptr, ret, sizeof(*ret)); > Does not work for 64bit read_user_stack_32 ^ this should be 4. > > Other than that it should preserve the existing logic just fine. sizeof(int) == 4 on 64bit so it should work. Thanks, Nick
next prev parent reply other threads:[~2020-04-03 11:26 UTC|newest] Thread overview: 163+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-02-25 17:35 [PATCH v3 00/32] powerpc/64: interrupts and syscalls series Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 01/32] powerpc/64s/exception: Introduce INT_DEFINE parameter block for code generation Nicholas Piggin 2020-04-01 12:53 ` Michael Ellerman 2020-02-25 17:35 ` [PATCH v3 02/32] powerpc/64s/exception: Add GEN_COMMON macro that uses INT_DEFINE parameters Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 03/32] powerpc/64s/exception: Add GEN_KVM " Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 04/32] powerpc/64s/exception: Expand EXC_COMMON and EXC_COMMON_ASYNC macros Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 05/32] powerpc/64s/exception: Move all interrupt handlers to new style code gen macros Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 06/32] powerpc/64s/exception: Remove old INT_ENTRY macro Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 07/32] powerpc/64s/exception: Remove old INT_COMMON macro Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 08/32] powerpc/64s/exception: Remove old INT_KVM_HANDLER Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 09/32] powerpc/64s/exception: Add ISIDE option Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 10/32] powerpc/64s/exception: move real->virt switch into the common handler Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 11/32] powerpc/64s/exception: move soft-mask test to common code Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 12/32] powerpc/64s/exception: move KVM " Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 13/32] powerpc/64s/exception: remove confusing IEARLY option Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 14/32] powerpc/64s/exception: remove the SPR saving patch code macros Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 15/32] powerpc/64s/exception: trim unused arguments from KVMTEST macro Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 16/32] powerpc/64s/exception: hdecrementer avoid touching the stack Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 17/32] powerpc/64s/exception: re-inline some handlers Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 18/32] powerpc/64s/exception: Clean up SRR specifiers Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 19/32] powerpc/64s/exception: add more comments for interrupt handlers Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 20/32] powerpc/64s/exception: only test KVM in SRR interrupts when PR KVM is supported Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 21/32] powerpc/64s/exception: sreset interrupts reconcile fix Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 22/32] powerpc/64s/exception: soft nmi interrupt should not use ret_from_except Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 23/32] powerpc/64: system call remove non-volatile GPR save optimisation Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 24/32] powerpc/64: sstep ifdef the deprecated fast endian switch syscall Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 25/32] powerpc/64: system call implement entry/exit logic in C Nicholas Piggin 2020-03-19 9:18 ` Christophe Leroy 2020-03-20 3:39 ` Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 26/32] powerpc/64: system call zero volatile registers when returning Nicholas Piggin 2020-02-25 21:20 ` Segher Boessenkool 2020-02-26 3:39 ` Nicholas Piggin 2020-03-07 0:54 ` [PATCH] Fix " Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 27/32] powerpc/64: implement soft interrupt replay in C Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 28/32] powerpc/64s: interrupt implement exit logic " Nicholas Piggin 2021-01-27 8:54 ` Christophe Leroy 2021-01-28 0:09 ` Nicholas Piggin 2021-02-03 16:25 ` Christophe Leroy 2021-02-04 3:27 ` Nicholas Piggin 2021-02-04 8:03 ` Christophe Leroy 2021-02-04 8:53 ` Nicholas Piggin 2021-02-05 0:22 ` Michael Ellerman 2021-02-05 2:16 ` Nicholas Piggin 2021-02-05 6:04 ` Christophe Leroy 2021-02-06 2:28 ` Nicholas Piggin 2021-02-27 10:07 ` Christophe Leroy 2021-03-01 0:47 ` Nicholas Piggin 2021-03-15 13:41 ` Christophe Leroy 2021-03-16 7:36 ` Nicholas Piggin 2021-03-19 11:44 ` Michael Ellerman 2020-02-25 17:35 ` [PATCH v3 29/32] powerpc/64s/exception: remove lite interrupt return Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 30/32] powerpc/64: system call reconcile interrupts Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 31/32] powerpc/64s/exception: treat NIA below __end_interrupts as soft-masked Nicholas Piggin 2020-02-25 17:35 ` [PATCH v3 32/32] powerpc/64s: system call support for scv/rfscv instructions Nicholas Piggin 2020-03-01 12:20 ` kbuild test robot 2020-03-01 12:20 ` kbuild test robot 2020-03-19 12:19 ` [PATCH v11 0/8] Disable compat cruft on ppc64le v11 Michal Suchanek 2020-03-19 12:19 ` Michal Suchanek 2020-03-19 12:19 ` [PATCH v11 1/8] powerpc: Add back __ARCH_WANT_SYS_LLSEEK macro Michal Suchanek 2020-03-19 12:19 ` Michal Suchanek 2020-03-19 12:19 ` [PATCH v11 2/8] powerpc: move common register copy functions from signal_32.c to signal.c Michal Suchanek 2020-03-19 12:19 ` Michal Suchanek 2020-03-19 12:19 ` [PATCH v11 3/8] powerpc/perf: consolidate read_user_stack_32 Michal Suchanek 2020-03-19 12:19 ` Michal Suchanek 2020-03-24 8:48 ` Nicholas Piggin 2020-03-24 8:48 ` Nicholas Piggin 2020-03-24 19:38 ` Michal Suchánek 2020-03-24 19:38 ` Michal Suchánek 2020-04-03 7:13 ` Nicholas Piggin 2020-04-03 7:13 ` Nicholas Piggin 2020-04-03 10:52 ` Michal Suchánek 2020-04-03 10:52 ` Michal Suchánek 2020-04-03 11:26 ` Nicholas Piggin [this message] 2020-04-03 11:26 ` Nicholas Piggin 2020-04-03 11:51 ` Michal Suchánek 2020-04-03 11:51 ` Michal Suchánek 2020-04-06 20:52 ` Michal Suchánek 2020-04-06 20:52 ` Michal Suchánek 2020-04-06 21:00 ` [PATCH] powerpcs: perf: consolidate perf_callchain_user_64 and perf_callchain_user_32 Michal Suchanek 2020-04-06 21:00 ` Michal Suchanek 2020-04-07 5:21 ` Christophe Leroy 2020-04-07 5:21 ` Christophe Leroy 2020-04-09 11:22 ` Michal Suchánek 2020-04-09 11:22 ` Michal Suchánek 2020-03-19 12:19 ` [PATCH v11 4/8] powerpc/perf: consolidate valid_user_sp Michal Suchanek 2020-03-19 12:19 ` Michal Suchanek 2020-03-19 12:19 ` [PATCH v11 5/8] powerpc/64: make buildable without CONFIG_COMPAT Michal Suchanek 2020-03-19 12:19 ` Michal Suchanek 2020-03-24 8:54 ` Nicholas Piggin 2020-03-24 8:54 ` Nicholas Piggin 2020-03-24 19:30 ` Michal Suchánek 2020-03-24 19:30 ` Michal Suchánek 2020-04-03 7:16 ` Nicholas Piggin 2020-04-03 7:16 ` Nicholas Piggin 2020-03-19 12:19 ` [PATCH v11 6/8] powerpc/64: Make COMPAT user-selectable disabled on littleendian by default Michal Suchanek 2020-03-19 12:19 ` Michal Suchanek 2020-03-19 12:19 ` [PATCH v11 7/8] powerpc/perf: split callchain.c by bitness Michal Suchanek 2020-03-19 12:19 ` Michal Suchanek 2020-03-19 12:19 ` [PATCH v11 8/8] MAINTAINERS: perf: Add pattern that matches ppc perf to the perf entry Michal Suchanek 2020-03-19 12:19 ` Michal Suchanek 2020-03-19 13:37 ` Andy Shevchenko 2020-03-19 13:37 ` Andy Shevchenko 2020-03-19 14:00 ` Michal Suchánek 2020-03-19 14:00 ` Michal Suchánek 2020-03-19 14:26 ` Andy Shevchenko 2020-03-19 14:26 ` Andy Shevchenko 2020-03-19 17:03 ` Joe Perches 2020-03-19 17:03 ` Joe Perches 2020-03-19 12:36 ` [PATCH v11 0/8] Disable compat cruft on ppc64le v11 Christophe Leroy 2020-03-19 12:36 ` Christophe Leroy 2020-03-19 14:01 ` Michal Suchánek 2020-03-19 14:01 ` Michal Suchánek 2020-04-03 7:25 ` Nicholas Piggin 2020-04-03 7:25 ` Nicholas Piggin 2020-04-03 7:26 ` Christophe Leroy 2020-04-03 7:26 ` Christophe Leroy 2020-04-03 9:43 ` Nicholas Piggin 2020-04-03 9:43 ` Nicholas Piggin 2020-04-05 0:40 ` Michael Ellerman 2020-04-05 0:40 ` Michael Ellerman 2020-03-20 10:20 ` [PATCH v12 0/8] Disable compat cruft on ppc64le v12 Michal Suchanek 2020-03-20 10:20 ` Michal Suchanek 2020-03-20 10:20 ` [PATCH v12 1/8] powerpc: Add back __ARCH_WANT_SYS_LLSEEK macro Michal Suchanek 2020-03-20 10:20 ` Michal Suchanek 2020-04-06 13:05 ` Michael Ellerman 2020-03-20 10:20 ` [PATCH v12 2/8] powerpc: move common register copy functions from signal_32.c to signal.c Michal Suchanek 2020-03-20 10:20 ` Michal Suchanek 2020-03-20 10:20 ` [PATCH v12 3/8] powerpc/perf: consolidate read_user_stack_32 Michal Suchanek 2020-03-20 10:20 ` Michal Suchanek 2020-03-20 10:20 ` [PATCH v12 4/8] powerpc/perf: consolidate valid_user_sp -> invalid_user_sp Michal Suchanek 2020-03-20 10:20 ` Michal Suchanek 2020-03-20 10:20 ` [PATCH v12 5/8] powerpc/64: make buildable without CONFIG_COMPAT Michal Suchanek 2020-03-20 10:20 ` Michal Suchanek 2020-04-07 5:50 ` Christophe Leroy 2020-04-07 5:50 ` Christophe Leroy 2020-04-07 9:57 ` Michal Suchánek 2020-04-07 9:57 ` Michal Suchánek 2020-03-20 10:20 ` [PATCH v12 6/8] powerpc/64: Make COMPAT user-selectable disabled on littleendian by default Michal Suchanek 2020-03-20 10:20 ` Michal Suchanek 2020-03-20 10:20 ` [PATCH v12 7/8] powerpc/perf: split callchain.c by bitness Michal Suchanek 2020-03-20 10:20 ` Michal Suchanek 2020-03-20 10:20 ` [PATCH v12 8/8] MAINTAINERS: perf: Add pattern that matches ppc perf to the perf entry Michal Suchanek 2020-03-20 10:20 ` Michal Suchanek 2020-03-20 10:33 ` Andy Shevchenko 2020-03-20 10:33 ` Andy Shevchenko 2020-03-20 11:23 ` Michal Suchánek 2020-03-20 11:23 ` Michal Suchánek 2020-03-20 12:42 ` Andy Shevchenko 2020-03-20 12:42 ` Andy Shevchenko 2020-03-20 14:42 ` Joe Perches 2020-03-20 14:42 ` Joe Perches 2020-03-20 16:28 ` Michal Suchánek 2020-03-20 16:28 ` Michal Suchánek 2020-03-20 16:31 ` Andy Shevchenko 2020-03-20 16:31 ` Andy Shevchenko 2020-03-20 16:42 ` Michal Suchánek 2020-03-20 16:42 ` Michal Suchánek 2020-03-20 16:47 ` Andy Shevchenko 2020-03-20 16:47 ` Andy Shevchenko 2020-03-20 21:36 ` Joe Perches 2020-03-20 21:36 ` Joe Perches 2020-03-19 11:52 [PATCH v11 0/8] Disable compat cruft on ppc64le v11 Michal Suchanek 2020-03-19 11:52 ` [PATCH v11 3/8] powerpc/perf: consolidate read_user_stack_32 Michal Suchanek 2020-03-19 11:52 ` Michal Suchanek
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=1585913065.zoacp2kzsv.astroid@bobo.none \ --to=npiggin@gmail.com \ --cc=Jonathan.Cameron@huawei.com \ --cc=acme@kernel.org \ --cc=alexander.shishkin@linux.intel.com \ --cc=allison@lohutok.net \ --cc=andriy.shevchenko@linux.intel.com \ --cc=arnd@arndb.de \ --cc=bauerman@linux.ibm.com \ --cc=benh@kernel.crashing.org \ --cc=bigeasy@linutronix.de \ --cc=cclaudio@linux.ibm.com \ --cc=christophe.leroy@c-s.fr \ --cc=davem@davemloft.net \ --cc=ebiederm@xmission.com \ --cc=erichte@linux.ibm.com \ --cc=gregkh@linuxfoundation.org \ --cc=gustavold@linux.ibm.com \ --cc=hbathini@linux.ibm.com \ --cc=jniethe5@gmail.com \ --cc=jolsa@redhat.com \ --cc=linux-fsdevel@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linuxppc-dev@lists.ozlabs.org \ --cc=mark.rutland@arm.com \ --cc=masahiroy@kernel.org \ --cc=mchehab+huawei@kernel.org \ --cc=mikey@neuling.org \ --cc=mingo@redhat.com \ --cc=mpe@ellerman.id.au \ --cc=msuchanek@suse.de \ --cc=namhyung@kernel.org \ --cc=nayna@linux.ibm.com \ --cc=paulus@samba.org \ --cc=peterz@infradead.org \ --cc=robh@kernel.org \ --cc=tglx@linutronix.de \ --cc=valentin.schneider@arm.com \ --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.