Re: [PATCH] powerpc: force inlining of csum_partial() to avoid multiple csum_partial() with GCC10

2020-12-21 Thread Michael Ellerman
On Thu, 15 Oct 2020 10:52:20 + (UTC), Christophe Leroy wrote: > ppc-linux-objdump -d vmlinux | grep -e "" -e > "<__csum_partial>" > > With gcc9 I get: > > c0017ef8 <__csum_partial>: > c00182fc: 4b ff fb fd bl c0017ef8 <__csum_partial> > c0018478:

Re: [PATCH] powerpc: force inlining of csum_partial() to avoid multiple csum_partial() with GCC10

2020-10-15 Thread Christophe Leroy
Le 15/10/2020 à 15:25, Segher Boessenkool a écrit : Hi! On Thu, Oct 15, 2020 at 10:52:20AM +, Christophe Leroy wrote: With gcc9 I get: With gcc10 I get: gcc10 defines multiple versions of csum_partial() which are just an unconditionnal branch to __csum_partial(). It doesn't

Re: [PATCH] powerpc: force inlining of csum_partial() to avoid multiple csum_partial() with GCC10

2020-10-15 Thread Christophe Leroy
Le 15/10/2020 à 15:25, Segher Boessenkool a écrit : Hi! On Thu, Oct 15, 2020 at 10:52:20AM +, Christophe Leroy wrote: With gcc9 I get: With gcc10 I get: gcc10 defines multiple versions of csum_partial() which are just an unconditionnal branch to __csum_partial(). It doesn't

Re: [PATCH] powerpc: force inlining of csum_partial() to avoid multiple csum_partial() with GCC10

2020-10-15 Thread Segher Boessenkool
Hi! On Thu, Oct 15, 2020 at 10:52:20AM +, Christophe Leroy wrote: > With gcc9 I get: > With gcc10 I get: > gcc10 defines multiple versions of csum_partial() which are just > an unconditionnal branch to __csum_partial(). It doesn't inline it, yes. Could you open a GCC PR for this

[PATCH] powerpc: force inlining of csum_partial() to avoid multiple csum_partial() with GCC10

2020-10-15 Thread Christophe Leroy
ppc-linux-objdump -d vmlinux | grep -e "" -e "<__csum_partial>" With gcc9 I get: c0017ef8 <__csum_partial>: c00182fc: 4b ff fb fd bl c0017ef8 <__csum_partial> c0018478: 4b ff fa 80 b c0017ef8 <__csum_partial> c03e8458: