linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] powerpc/mm/radix: Invalidate ERAT on tlbiel for POWER9 DD1
@ 2016-11-16  4:18 Michael Neuling
  2016-11-16  4:35 ` Benjamin Herrenschmidt
  0 siblings, 1 reply; 3+ messages in thread
From: Michael Neuling @ 2016-11-16  4:18 UTC (permalink / raw)
  To: mpe; +Cc: benh, aneesh.kumar, linuxppc-dev, mikey

On POWER9 DD1, when we do a local TLB invalidate we also need to explicitly
invalidate the ERAT.

Signed-off-by: Michael Neuling <mikey@neuling.org>
---
 arch/powerpc/mm/tlb-radix.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/arch/powerpc/mm/tlb-radix.c b/arch/powerpc/mm/tlb-radix.c
index bda8c43be7..4b087fc3c7 100644
--- a/arch/powerpc/mm/tlb-radix.c
+++ b/arch/powerpc/mm/tlb-radix.c
@@ -50,6 +50,10 @@ static inline void _tlbiel_pid(unsigned long pid, unsigned long ric)
 	for (set = 0; set < POWER9_TLB_SETS_RADIX ; set++) {
 		__tlbiel_pid(pid, set, ric);
 	}
+	if (cpu_has_feature(CPU_FTR_POWER9_DD1))
+		asm volatile("isync \n"
+			     PPC_SLBIA(0x7)
+			     : : :"memory");
 	return;
 }
 
@@ -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");
 }
 
 static inline void _tlbie_va(unsigned long va, unsigned long pid,
-- 
2.9.3

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] powerpc/mm/radix: Invalidate ERAT on tlbiel for POWER9 DD1
  2016-11-16  4:18 [PATCH] powerpc/mm/radix: Invalidate ERAT on tlbiel for POWER9 DD1 Michael Neuling
@ 2016-11-16  4:35 ` Benjamin Herrenschmidt
  2016-11-16  5:14   ` Michael Neuling
  0 siblings, 1 reply; 3+ messages in thread
From: Benjamin Herrenschmidt @ 2016-11-16  4:35 UTC (permalink / raw)
  To: Michael Neuling, mpe; +Cc: aneesh.kumar, linuxppc-dev

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

> @@ -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");
>  }

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] powerpc/mm/radix: Invalidate ERAT on tlbiel for POWER9 DD1
  2016-11-16  4:35 ` Benjamin Herrenschmidt
@ 2016-11-16  5:14   ` Michael Neuling
  0 siblings, 0 replies; 3+ messages in thread
From: Michael Neuling @ 2016-11-16  5:14 UTC (permalink / raw)
  To: Benjamin Herrenschmidt, mpe; +Cc: aneesh.kumar, linuxppc-dev

T24gV2VkLCAyMDE2LTExLTE2IGF0IDE1OjM1ICsxMTAwLCBCZW5qYW1pbiBIZXJyZW5zY2htaWR0
IHdyb3RlOgo+IE9uIFdlZCwgMjAxNi0xMS0xNiBhdCAxNToxOCArMTEwMCwgTWljaGFlbCBOZXVs
aW5nIHdyb3RlOgo+ID4gK8KgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoGFzbSB2b2xhdGls
ZSgiaXN5bmMgXG4iCj4gPiArwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDC
oMKgwqDCoMKgwqDCoMKgIFBQQ19TTEJJQSgweDcpCj4gPiArwqDCoMKgwqDCoMKgwqDCoMKgwqDC
oMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgIDogOiA6Im1lbW9yeSIpOwo+ID4gwqDC
oMKgwqDCoMKgwqDCoHJldHVybjsKPiA+IMKgfQo+ID4gCj4gCj4gV2UgcHJvYmFibHkgZG9uJ3Qg
bmVlZCB0aGUgaXN5bmMncwoKWWVwLCB0ZXN0aW5nIGNvbmZpcm1zIHRoYXQgdG9vLiBJJ2xsIHJl
c2VuZC4KClRoYW5rcywKTWlrZXkKCj4gCj4gPiBAQCAtODMsNiArODcsMTAgQEAgc3RhdGljIGlu
bGluZSB2b2lkIF90bGJpZWxfdmEodW5zaWduZWQgbG9uZyB2YSwKPiA+IHVuc2lnbmVkIGxvbmcg
cGlkLAo+ID4gwqDCoMKgwqDCoMKgwqDCoGFzbSB2b2xhdGlsZShQUENfVExCSUVMKCUwLCAlNCwg
JTMsICUyLCAlMSkKPiA+IMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqAg
OiA6ICJyIihyYiksICJpIihyKSwgImkiKHBycyksICJpIihyaWMpLCAiciIocnMpCj4gPiA6ICJt
ZW1vcnkiKTsKPiA+IMKgwqDCoMKgwqDCoMKgwqBhc20gdm9sYXRpbGUoInB0ZXN5bmMiOiA6IDoi
bWVtb3J5Iik7Cj4gPiArwqDCoMKgwqDCoMKgwqBpZiAoY3B1X2hhc19mZWF0dXJlKENQVV9GVFJf
UE9XRVI5X0REMSkpCj4gPiArwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgYXNtIHZvbGF0
aWxlKCJpc3luYyBcbiIKPiA+ICvCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDC
oMKgwqDCoMKgwqDCoMKgwqAgUFBDX1NMQklBKDB4NykKPiA+ICvCoMKgwqDCoMKgwqDCoMKgwqDC
oMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqAgOiA6IDoibWVtb3J5Iik7Cj4gPiDC
oH0KPiAKPiA=

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-11-16  5:14 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-11-16  4:18 [PATCH] powerpc/mm/radix: Invalidate ERAT on tlbiel for POWER9 DD1 Michael Neuling
2016-11-16  4:35 ` Benjamin Herrenschmidt
2016-11-16  5:14   ` Michael Neuling

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).