Message ID | 9666.1339113149@neuling.org (mailing list archive) |
---|---|
State | Rejected, archived |
Headers | show |
On Fri, 2012-06-08 at 09:52 +1000, Michael Neuling wrote: > Fix sldi to take literal not register name. > > Was introduced by: > commit 19ccb76a1938ab364a412253daec64613acbf3df > Author: Paul Mackerras <paulus@samba.org> > Date: Sat Jul 23 17:42:46 2011 +1000 > > Signed-off-by: Michael Neuling <mikey@neuling.org> > --- > I got sick off the bike-shedding in a vacuum and started doing the > conversion. Found this as a result. > > I'll post the full conversion at some point, but this fix is useful > now. Euh... except that: > diff --git a/arch/powerpc/kvm/book3s_hv_rmhandlers.S b/arch/powerpc/kvm/book3s_hv_rmhandlers.S > index a84aafc..969caeb 100644 > --- a/arch/powerpc/kvm/book3s_hv_rmhandlers.S > +++ b/arch/powerpc/kvm/book3s_hv_rmhandlers.S > @@ -810,7 +810,7 @@ END_FTR_SECTION_IFSET(CPU_FTR_ARCH_201) > lwz r3,VCORE_NAPPING_THREADS(r5) > lwz r4,VCPU_PTID(r9) > li r0,1 > - sldi r0,r0,r4 > + sldi r0,r0,4 Isn't the right fix s/sldi/sld and keep r4 ? IE. I think Paul meant to shift 1 by the TID not shift 1 by 4 Cheers, Ben. > andc. r3,r3,r0 /* no sense IPI'ing ourselves */ > beq 43f > mulli r4,r4,PACA_SIZE /* get paca for thread 0 */
diff --git a/arch/powerpc/kvm/book3s_hv_rmhandlers.S b/arch/powerpc/kvm/book3s_hv_rmhandlers.S index a84aafc..969caeb 100644 --- a/arch/powerpc/kvm/book3s_hv_rmhandlers.S +++ b/arch/powerpc/kvm/book3s_hv_rmhandlers.S @@ -810,7 +810,7 @@ END_FTR_SECTION_IFSET(CPU_FTR_ARCH_201) lwz r3,VCORE_NAPPING_THREADS(r5) lwz r4,VCPU_PTID(r9) li r0,1 - sldi r0,r0,r4 + sldi r0,r0,4 andc. r3,r3,r0 /* no sense IPI'ing ourselves */ beq 43f mulli r4,r4,PACA_SIZE /* get paca for thread 0 */
Fix sldi to take literal not register name. Was introduced by: commit 19ccb76a1938ab364a412253daec64613acbf3df Author: Paul Mackerras <paulus@samba.org> Date: Sat Jul 23 17:42:46 2011 +1000 Signed-off-by: Michael Neuling <mikey@neuling.org> --- I got sick off the bike-shedding in a vacuum and started doing the conversion. Found this as a result. I'll post the full conversion at some point, but this fix is useful now.