| Message ID | 20240607-caleb-upstreaming-v2-3-bae407f51f22@linaro.org |
|---|---|
| State | Superseded |
| Headers | show |
| Series | arm64: add a software pagetable walker | expand |
On Fri, Jun 07, 2024 at 10:42:51PM +0200, Caleb Connolly wrote: > Add some brief documentation on using dump_pagetables() to print out > U-Boot's pagetables during boot. > > Signed-off-by: Caleb Connolly <caleb.connolly@linaro.org> > --- > doc/arch/arm64.rst | 37 +++++++++++++++++++++++++++++++++++++ > 1 file changed, 37 insertions(+) > > diff --git a/doc/arch/arm64.rst b/doc/arch/arm64.rst > index 7c0713504c47..0c851304aad4 100644 > --- a/doc/arch/arm64.rst > +++ b/doc/arch/arm64.rst > @@ -47,8 +47,45 @@ Notes > > 6. CONFIG_ARM64 instead of CONFIG_ARMV8 is used to distinguish aarch64 and > aarch32 specific codes. > > +MMU > +--- > + > +U-Boot uses a simple page table for MMU setup. It uses the smallest number of bits > +possible for the virtual address based on the maximum memory address (see the logic > +in ``get_tcr()``). > + > +As with all platforms, U-Boot on ARM64 uses a 1:1 mapping of virtual to physical addresses. > +In general, the memory map is expected to remain static once the MMU is enabled. > + > +It is possible to debug the pagetable generated by U-Boot with the built in > +``dump_pagetable()`` and ``walk_pagetable()`` functions (the former being a simple > +wrapper for the latter). For example the following can be added to ``setup_all_pgtables()`` > +after the first call to ``setup_pgtables()``: > + > +.. code-block:: c > + > + dump_pagetable(gd->arch.tlb_addr, get_tcr(NULL, NULL)); > + > +This will result in a print like the following: > + > +.. code-block:: text > + > + Walking pagetable at 000000017df90000, va_bits: 36. Using 3 levels > + [0x17df91000] | Table | | > + [0x17df92000] | Table | | > + [0x000001000 - 0x000200000] | Pages | Device-nGnRnE | Non-shareable > + [0x000200000 - 0x040000000] | Block | Device-nGnRnE | Non-shareable > + [0x040000000 - 0x080000000] | Block | Device-nGnRnE | Non-shareable > + [0x080000000 - 0x140000000] | Block | Normal | Inner-shareable > + [0x17df93000] | Table | | > + [0x140000000 - 0x17de00000] | Block | Normal | Inner-shareable > + [0x17df94000] | Table | | > + [0x17de00000 - 0x17dfa0000] | Pages | Normal | Inner-shareable > + > +For more information, please refer to the function documentation in > +``arch/arm/include/asm/armv8/mmu.h``. Since the function comments are kernel-doc style, can you please rework this to leverage "kernel-doc::" and make sure it renders nicely? And this may show we should be putting some function comments in 2/3 of this series as well.
diff --git a/doc/arch/arm64.rst b/doc/arch/arm64.rst index 7c0713504c47..0c851304aad4 100644 --- a/doc/arch/arm64.rst +++ b/doc/arch/arm64.rst @@ -47,8 +47,45 @@ Notes 6. CONFIG_ARM64 instead of CONFIG_ARMV8 is used to distinguish aarch64 and aarch32 specific codes. +MMU +--- + +U-Boot uses a simple page table for MMU setup. It uses the smallest number of bits +possible for the virtual address based on the maximum memory address (see the logic +in ``get_tcr()``). + +As with all platforms, U-Boot on ARM64 uses a 1:1 mapping of virtual to physical addresses. +In general, the memory map is expected to remain static once the MMU is enabled. + +It is possible to debug the pagetable generated by U-Boot with the built in +``dump_pagetable()`` and ``walk_pagetable()`` functions (the former being a simple +wrapper for the latter). For example the following can be added to ``setup_all_pgtables()`` +after the first call to ``setup_pgtables()``: + +.. code-block:: c + + dump_pagetable(gd->arch.tlb_addr, get_tcr(NULL, NULL)); + +This will result in a print like the following: + +.. code-block:: text + + Walking pagetable at 000000017df90000, va_bits: 36. Using 3 levels + [0x17df91000] | Table | | + [0x17df92000] | Table | | + [0x000001000 - 0x000200000] | Pages | Device-nGnRnE | Non-shareable + [0x000200000 - 0x040000000] | Block | Device-nGnRnE | Non-shareable + [0x040000000 - 0x080000000] | Block | Device-nGnRnE | Non-shareable + [0x080000000 - 0x140000000] | Block | Normal | Inner-shareable + [0x17df93000] | Table | | + [0x140000000 - 0x17de00000] | Block | Normal | Inner-shareable + [0x17df94000] | Table | | + [0x17de00000 - 0x17dfa0000] | Pages | Normal | Inner-shareable + +For more information, please refer to the function documentation in +``arch/arm/include/asm/armv8/mmu.h``. Contributors ------------ * Tom Rini <trini@ti.com>
Add some brief documentation on using dump_pagetables() to print out U-Boot's pagetables during boot. Signed-off-by: Caleb Connolly <caleb.connolly@linaro.org> --- doc/arch/arm64.rst | 37 +++++++++++++++++++++++++++++++++++++ 1 file changed, 37 insertions(+)