Re: [PATCH 4/7] x86/percpu: Clean up percpu_add_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

Re: [PATCH 4/7] x86/percpu: Clean up percpu_add_op()

2020-05-17 Thread kbuild test robot
Hi Brian, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on dennis-percpu/for-next] [also build test WARNING on tip/auto-latest linus/master linux/master v5.7-rc5 next-20200515] [if your patch is applied to the wrong git tree, please drop us a note to help

Re: [PATCH 4/7] x86/percpu: Clean up percpu_add_op()

2020-05-17 Thread kbuild test robot
Hi Brian, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on dennis-percpu/for-next] [also build test WARNING on tip/auto-latest linus/master linux/master v5.7-rc5 next-20200515] [if your patch is applied to the wrong git tree, please drop us a note to help

[PATCH 4/7] x86/percpu: Clean up percpu_add_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 ---