Re: [PATCH] powerpc: inline ppc64_runlatch_off

2010-08-08 Thread Olof Johansson
On Fri, Aug 06, 2010 at 11:28:19PM +1000, Anton Blanchard wrote: > > Hi Olof, > > > No semicolon here. > > Nice catch! > > Anton > > I'm sick of seeing ppc64_runlatch_off in our profiles, so inline it > into the callers. To avoid a mess of circular includes I didn't add > it as an inline func

Re: [PATCH] powerpc: inline ppc64_runlatch_off

2010-08-06 Thread Anton Blanchard
Hi Olof, > No semicolon here. Nice catch! Anton I'm sick of seeing ppc64_runlatch_off in our profiles, so inline it into the callers. To avoid a mess of circular includes I didn't add it as an inline function. Signed-off-by: Anton Blanchard --- Index: powerpc.git/arch/powerpc/include/asm/r

Re: [PATCH] powerpc: inline ppc64_runlatch_off

2010-08-06 Thread Olof Johansson
On Fri, Aug 06, 2010 at 02:53:15PM +1000, Anton Blanchard wrote: > > I'm sick of seeing ppc64_runlatch_off in our profiles, so inline the > heavily used part of it into the callers. To avoid a mess of circular includes > I didn't add it as an inline function. > > Signed-off-by: Anton Blanchard >

Re: [PATCH] powerpc: inline ppc64_runlatch_off

2010-08-05 Thread Anton Blanchard
Hi, > remind me why we need to do that runlatch thing on these CPUs at all ? The PMU uses it so events can be constructed that count only non idle cycles. I think the power management hardware on POWER6 and POWER7 also use the runlatch state to determine how busy a CPU is. Anton ___

Re: [PATCH] powerpc: inline ppc64_runlatch_off

2010-08-05 Thread Benjamin Herrenschmidt
On Fri, 2010-08-06 at 15:56 +1000, Anton Blanchard wrote: > Unfortunately we still need to prevent continual writes to it with a per > thread > flag because on some CPUs a write to the SPR in low priority mode will stall > another SMT thread. So we could get rid of the cpu feature comparison but >

Re: [PATCH] powerpc: inline ppc64_runlatch_off

2010-08-05 Thread Anton Blanchard
Hi Ben, > > I'm sick of seeing ppc64_runlatch_off in our profiles, so inline the > > heavily used part of it into the callers. To avoid a mess of circular > > includes I didn't add it as an inline function. > > Considering that it's just an asm instruction or two, should we make it > inline asm

Re: [PATCH] powerpc: inline ppc64_runlatch_off

2010-08-05 Thread Benjamin Herrenschmidt
On Fri, 2010-08-06 at 14:53 +1000, Anton Blanchard wrote: > I'm sick of seeing ppc64_runlatch_off in our profiles, so inline the > heavily used part of it into the callers. To avoid a mess of circular includes > I didn't add it as an inline function. Considering that it's just an asm instruction o

[PATCH] powerpc: inline ppc64_runlatch_off

2010-08-05 Thread Anton Blanchard
I'm sick of seeing ppc64_runlatch_off in our profiles, so inline the heavily used part of it into the callers. To avoid a mess of circular includes I didn't add it as an inline function. Signed-off-by: Anton Blanchard --- Index: powerpc.git/arch/powerpc/include/asm/reg.h ===