powerpc/mm: Fix printing of process table address
PATCH 1/1 powerpc/mm: Fix printing of process table address
New changes to %p cause the information printed at boot about radix-mmu partition and process table opaque, if we really want to protect those values, we should make dmesg more secure using CONFIG_SECURITY_DMESG_RESTRICT
Before the patches:
radix-mmu: Partition table (ptrval)
radix-mmu: Process table (ptrval) and radix root for kernel: (ptrval)
After the patches
radix-mmu: Partition table c0000000ffff0000
radix-mmu: Process table c0000000fe000000 and radix root for kernel: c0000000015a0000
--- arch/powerpc/mm/pgtable-radix.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)
arch/powerpc/mm/pgtable-radix.c +3 -2
@@ -329,7 +329,8 @@ static void __init radix_init_pgtable(void)* physical address here. */ register_process_table (__pa (process_tb),0 ,PRTB_SIZE_SHIFT -12 ); -pr_info ("Process table %p and radix root for kernel: %p\n" , process_tb, init_mm.pgd); +pr_info ("Process table %px and radix root for kernel: %px\n" , + process_tb, init_mm.pgd); asmvolatile ("ptesync" : : :"memory" ); asm volatile(PPC_TLBIE_5(%0,%1,2,1,1) : : "r" (TLBIEL_INVAL_SET_LPID), "r" (0));
@@ -363,7 +364,7 @@ static void __init radix_init_partition_table(void)mmu_partition_table_set_entry (0 , dw0,0 );pr_info ("Initializing Radix MMU\n" ); -pr_info ("Partition table %p\n" , partition_tb); +pr_info ("Partition table %px\n" , partition_tb); }void __init radix_init_native (void )
Signature · 1 line
2.13.6
Le 28/03/2018 à 05:09, Balbir Singh a écrit :
Quoted 10 lines
New changes to %p cause the information printed at boot about radix-mmu partition and process table opaque, if we really want to protect those values, we should make dmesg more secure using
CONFIG_SECURITY_DMESG_RESTRICTBefore the patches:
radix-mmu: Partition table (ptrval)
radix-mmu: Process table (ptrval) and radix root for kernel: (ptrval)
Quoted 6 lines
After the patches
radix-mmu: Partition table
c0000000ffff0000
radix-mmu: Process tablec0000000fe000000and radix root for kernel:c0000000015a0000Signed-off-by Balbir Singh <[email protected]>
This patch was superseded by ec5b705c4836 ("powerpc/mm/radix: remove useless kernel messages")
Quoted 27 lines
--- arch/powerpc/mm/pgtable-radix.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)arch/powerpc/mm/pgtable-radix.c +3
-2@@ -329,7 +329,8 @@ static void __init radix_init_pgtable(void)* physical address here. */ register_process_table (__pa (process_tb),0 ,PRTB_SIZE_SHIFT -12 ); -pr_info ("Process table %p and radix root for kernel: %p\n" , process_tb, init_mm.pgd); +pr_info ("Process table %px and radix root for kernel: %px\n" , + process_tb, init_mm.pgd); asmvolatile ("ptesync" : : :"memory" ); asm volatile(PPC_TLBIE_5(%0,%1,2,1,1) : : "r" (TLBIEL_INVAL_SET_LPID), "r" (0));@@ -363,7 +364,7 @@ static void __init radix_init_partition_table(void)mmu_partition_table_set_entry (0 , dw0,0 );pr_info ("Initializing Radix MMU\n" ); -pr_info ("Partition table %p\n" , partition_tb); +pr_info ("Partition table %px\n" , partition_tb); }void __init radix_init_native (void )