Re: [PATCH 2/7] x86/percpu: Clean up percpu_to_op()

2020-05-26 Thread Nick Desaulniers
On Thu, May 21, 2020 at 6:06 AM Brian Gerst wrote: > > On Wed, May 20, 2020 at 1:26 PM Nick Desaulniers > wrote: > > > > On Mon, May 18, 2020 at 8:38 PM Brian Gerst wrote: > > > > > > On Mon, May 18, 2020 at 5:15 PM Nick Desaulniers > > > wrote: > > > > > > > > On Sun, May 17, 2020 at 8:29 AM B

Re: [PATCH 2/7] x86/percpu: Clean up percpu_to_op()

2020-05-21 Thread Brian Gerst
On Wed, May 20, 2020 at 1:26 PM Nick Desaulniers wrote: > > On Mon, May 18, 2020 at 8:38 PM Brian Gerst wrote: > > > > On Mon, May 18, 2020 at 5:15 PM Nick Desaulniers > > wrote: > > > > > > On Sun, May 17, 2020 at 8:29 AM Brian Gerst wrote: > > > > > > > > The core percpu macros already have a

Re: [PATCH 2/7] x86/percpu: Clean up percpu_to_op()

2020-05-20 Thread Nick Desaulniers
On Mon, May 18, 2020 at 8:38 PM Brian Gerst wrote: > > On Mon, May 18, 2020 at 5:15 PM Nick Desaulniers > wrote: > > > > On Sun, May 17, 2020 at 8:29 AM Brian Gerst wrote: > > > > > > The core percpu macros already have a switch on the data size, so the > > > switch > > > in the x86 code is red

Re: [PATCH 2/7] x86/percpu: Clean up percpu_to_op()

2020-05-18 Thread Brian Gerst
On Mon, May 18, 2020 at 5:15 PM Nick Desaulniers wrote: > > On Sun, May 17, 2020 at 8:29 AM Brian Gerst wrote: > > > > The core percpu macros already have a switch on the data size, so the switch > > in the x86 code is redundant and produces more dead code. > > > > Also use appropriate types for

Re: [PATCH 2/7] x86/percpu: Clean up percpu_to_op()

2020-05-18 Thread Nick Desaulniers
On Sun, May 17, 2020 at 8:29 AM Brian Gerst wrote: > > The core percpu macros already have a switch on the data size, so the switch > in the x86 code is redundant and produces more dead code. > > Also use appropriate types for the width of the instructions. This avoids > errors when compiling wit

[PATCH 2/7] x86/percpu: Clean up percpu_to_op()

2020-05-17 Thread Brian Gerst
The core percpu macros already have a switch on the data size, so the switch in the x86 code is redundant and produces more dead code. Also use appropriate types for the width of the instructions. This avoids errors when compiling with Clang. Signed-off-by: Brian Gerst --- arch/x86/include/asm