Re: [PATCH v7 3/6] powerpc/perf: consolidate read_user_stack_32

2019-09-02 Thread Michael Ellerman
Michal Suchánek writes: > On Mon, 02 Sep 2019 14:01:17 +1000 > Michael Ellerman wrote: >> Michael Ellerman writes: >> > Michal Suchanek writes: >> ... >> >> @@ -295,6 +279,12 @@ static inline int current_is_64bit(void) >> >> } >> >> >> >> #else /* CONFIG_PPC64 */ >> >> +static int

Re: [PATCH v7 3/6] powerpc/perf: consolidate read_user_stack_32

2019-09-02 Thread Michal Suchánek
On Mon, 02 Sep 2019 14:01:17 +1000 Michael Ellerman wrote: > Michael Ellerman writes: > > Michal Suchanek writes: > ... > >> @@ -295,6 +279,12 @@ static inline int current_is_64bit(void) > >> } > >> > >> #else /* CONFIG_PPC64 */ > >> +static int read_user_stack_slow(void __user *ptr,

Re: [PATCH v7 3/6] powerpc/perf: consolidate read_user_stack_32

2019-09-01 Thread Michael Ellerman
Michael Ellerman writes: > Michal Suchanek writes: ... >> @@ -295,6 +279,12 @@ static inline int current_is_64bit(void) >> } >> >> #else /* CONFIG_PPC64 */ >> +static int read_user_stack_slow(void __user *ptr, void *buf, int nb) >> +{ >> +return 0; >> +} >> +#endif /* CONFIG_PPC64 */ >

Re: [PATCH v7 3/6] powerpc/perf: consolidate read_user_stack_32

2019-09-01 Thread Michael Ellerman
Michal Suchanek writes: > There are two almost identical copies for 32bit and 64bit. > > The function is used only in 32bit code which will be split out in next > patch so consolidate to one function. > > Signed-off-by: Michal Suchanek > Reviewed-by: Christophe Leroy > --- > new patch in v6 >

[PATCH v7 3/6] powerpc/perf: consolidate read_user_stack_32

2019-08-30 Thread Michal Suchanek
There are two almost identical copies for 32bit and 64bit. The function is used only in 32bit code which will be split out in next patch so consolidate to one function. Signed-off-by: Michal Suchanek Reviewed-by: Christophe Leroy --- new patch in v6 --- arch/powerpc/perf/callchain.c | 25