[RFC] powerpc/mm: Enable TLB flush during native_register_proc_table()

Subsystems: linux for powerpc (32-bit and 64-bit), the rest

2 messages, 2 authors, 2018-01-15 · open the first message on its own page

[RFC] powerpc/mm: Enable TLB flush during native_register_proc_table()

From: Anshuman Khandual <hidden>
Date: 2018-01-10 04:53:44

From: Anshuman Khandual <redacted>

Any changes to the partition table must be followed by appropriate TLB
flush which is not happening at present in native_register_proc_table()
path. Fix this by calling mmu_partition_table_set_entry() which does
take care of TLB flushing after the partition table update.

Signed-off-by: Anshuman Khandual <redacted>
---
- From code inspection
- Compile, boot tested without CONFIG_PPC_RADIX_MMU
- Though it seems to be not getting called from any where

 arch/powerpc/mm/hash_native_64.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/arch/powerpc/mm/hash_native_64.c b/arch/powerpc/mm/hash_native_64.c
index 640cf56..89be5a6 100644
--- a/arch/powerpc/mm/hash_native_64.c
+++ b/arch/powerpc/mm/hash_native_64.c
@@ -758,12 +758,14 @@ static void native_flush_hash_range(unsigned long number, int local)
 static int native_register_proc_table(unsigned long base, unsigned long page_size,
 				      unsigned long table_size)
 {
-	unsigned long patb1 = base << 25; /* VSID */
+	unsigned long patb0, patb1;
 
+	patb1 = base << 25;	    /* VSID */
 	patb1 |= (page_size << 5);  /* sllp */
 	patb1 |= table_size;
 
-	partition_tb->patb1 = cpu_to_be64(patb1);
+	patb0 = be64_to_cpu(partition_tb[0].patb0);
+	mmu_partition_table_set_entry(0, patb0, patb1);
 	return 0;
 }
 
-- 
1.8.3.1

Re: [RFC] powerpc/mm: Enable TLB flush during native_register_proc_table()

From: Aneesh Kumar K.V <hidden>
Date: 2018-01-15 06:14:23

Anshuman Khandual [off-list ref] writes:
From: Anshuman Khandual <redacted>

Any changes to the partition table must be followed by appropriate TLB
flush which is not happening at present in native_register_proc_table()
path. Fix this by calling mmu_partition_table_set_entry() which does
take care of TLB flushing after the partition table update.

Signed-off-by: Anshuman Khandual <redacted>
---
- From code inspection
- Compile, boot tested without CONFIG_PPC_RADIX_MMU
- Though it seems to be not getting called from any where

For hash the patb1 is zero and hence it is not called. This is left over
from the segment table implementation. I guess we can remove
native_register_proc_table.


quoted hunk
 arch/powerpc/mm/hash_native_64.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/arch/powerpc/mm/hash_native_64.c b/arch/powerpc/mm/hash_native_64.c
index 640cf56..89be5a6 100644
--- a/arch/powerpc/mm/hash_native_64.c
+++ b/arch/powerpc/mm/hash_native_64.c
@@ -758,12 +758,14 @@ static void native_flush_hash_range(unsigned long number, int local)
 static int native_register_proc_table(unsigned long base, unsigned long page_size,
 				      unsigned long table_size)
 {
-	unsigned long patb1 = base << 25; /* VSID */
+	unsigned long patb0, patb1;
 
+	patb1 = base << 25;	    /* VSID */
 	patb1 |= (page_size << 5);  /* sllp */
 	patb1 |= table_size;
 
-	partition_tb->patb1 = cpu_to_be64(patb1);
+	patb0 = be64_to_cpu(partition_tb[0].patb0);
+	mmu_partition_table_set_entry(0, patb0, patb1);
 	return 0;
 }
 
-- 
1.8.3.1
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help