]> xenbits.xensource.com Git - xen.git/commitdiff
xen/arm: mm: Remove special case for CPU0 in dump_hyp_walk()
authorStefano Stabellini <sstabellini@kernel.org>
Thu, 19 Nov 2020 19:07:46 +0000 (19:07 +0000)
committerJulien Grall <jgrall@amazon.com>
Sat, 23 Jan 2021 11:41:29 +0000 (11:41 +0000)
There is no need to have a special case for CPU0 when converting the
page-table virtual address into a physical address. The helper
virt_to_maddr() is able to translate any address as long as the root
page-tables is mapped in the virtual address. This is the case for all
the CPUs at the moment.

So use the same BUG_ON() regardless the CPU.

Signed-off-by: Stefano Stabellini <stefano.stabellini@xilinx.com>
[julien: Rework the commit message]
Signed-off-by: Julien Grall <jgrall@amazon.com>
Reviewed-by: Bertrand Marquis <bertrand.marquis@arm.com>
xen/arch/arm/mm.c

index 9c4b26bf079b93af7ed0f3cc7818b22cbf7a0f73..4dd886f7c80db4d2d7be4be4971b0c71ad183752 100644 (file)
@@ -284,10 +284,7 @@ void dump_hyp_walk(vaddr_t addr)
            "on CPU%d via TTBR 0x%016"PRIx64"\n",
            addr, smp_processor_id(), ttbr);
 
-    if ( smp_processor_id() == 0 )
-        BUG_ON( (lpae_t *)(unsigned long)(ttbr - phys_offset) != pgtable );
-    else
-        BUG_ON( virt_to_maddr(pgtable) != ttbr );
+    BUG_ON( virt_to_maddr(pgtable) != ttbr );
     dump_pt_walk(ttbr, addr, HYP_PT_ROOT_LEVEL, 1);
 }