On Wed, 2016-11-16 at 15:35 +1100, Benjamin Herrenschmidt wrote: > On Wed, 2016-11-16 at 15:18 +1100, Michael Neuling wrote: > > + asm volatile("isync \n" > > + PPC_SLBIA(0x7) > > + : : :"memory"); > > return; > > } > > > > We probably don't need the isync's
Yep, testing confirms that too. I'll resend. Thanks, Mikey > > > @@ -83,6 +87,10 @@ static inline void _tlbiel_va(unsigned long va, > > unsigned long pid, > > asm volatile(PPC_TLBIEL(%0, %4, %3, %2, %1) > > : : "r"(rb), "i"(r), "i"(prs), "i"(ric), "r"(rs) > > : "memory"); > > asm volatile("ptesync": : :"memory"); > > + if (cpu_has_feature(CPU_FTR_POWER9_DD1)) > > + asm volatile("isync \n" > > + PPC_SLBIA(0x7) > > + : : :"memory"); > > } > >