* [RFC PATCH] arch: arm64: have memblocks out of kernel text use section map @ 2021-11-12 9:20 Huangzhaoyang 2021-11-12 9:31 ` Ard Biesheuvel 0 siblings, 1 reply; 3+ messages in thread From: Huangzhaoyang @ 2021-11-12 9:20 UTC (permalink / raw) To: Ard Biesheuvel, Catalin Marinas, Will Deacon, Anshuman Khandual, Andrew Morton, Nicholas Piggin, Mike Rapoport, Pavel Tatashin, Christophe Leroy, Jonathan Marek, Zhaoyang Huang, linux-mm, linux-kernel From: Zhaoyang Huang <zhaoyang.huang@unisoc.com> By comparing the swapper_pg_dir with k54 and previous versions,we find that the linear mappings within which the addr is out of kernel text section will use the smallest pte. It should arise for the sake of rodata_full, which set all memblock use NO_CONT_MAPPINGS. Signed-off-by: Zhaoyang Huang <zhaoyang.huang@unisoc.com> --- arch/arm64/mm/mmu.c | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/arch/arm64/mm/mmu.c b/arch/arm64/mm/mmu.c index cfd9deb..14e1bea 100644 --- a/arch/arm64/mm/mmu.c +++ b/arch/arm64/mm/mmu.c @@ -252,6 +252,8 @@ static void init_pmd(pud_t *pudp, unsigned long addr, unsigned long end, pmd_clear_fixmap(); } +static bool crash_mem_map __initdata; + static void alloc_init_cont_pmd(pud_t *pudp, unsigned long addr, unsigned long end, phys_addr_t phys, pgprot_t prot, @@ -259,7 +261,15 @@ static void alloc_init_cont_pmd(pud_t *pudp, unsigned long addr, { unsigned long next; pud_t pud = READ_ONCE(*pudp); + unsigned long len = end - addr; + phys_addr_t kernel_start = __pa_symbol(_stext); + phys_addr_t kernel_end = __pa_symbol(__init_begin); + if (debug_pagealloc_enabled() || crash_mem_map || IS_ENABLED(CONFIG_KFENCE)) + ; + else if (phys > kernel_end || phys + len < kernel_start) { + flags &= ~(NO_BLOCK_MAPPINGS | NO_CONT_MAPPINGS); + } /* * Check for initial section mappings in the pgd/pud. */ @@ -483,7 +493,6 @@ void __init mark_linear_text_alias_ro(void) PAGE_KERNEL_RO); } -static bool crash_mem_map __initdata; static int __init enable_crash_mem_map(char *arg) { -- 1.9.1 ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [RFC PATCH] arch: arm64: have memblocks out of kernel text use section map 2021-11-12 9:20 [RFC PATCH] arch: arm64: have memblocks out of kernel text use section map Huangzhaoyang @ 2021-11-12 9:31 ` Ard Biesheuvel 2021-11-12 9:45 ` Zhaoyang Huang 0 siblings, 1 reply; 3+ messages in thread From: Ard Biesheuvel @ 2021-11-12 9:31 UTC (permalink / raw) To: Huangzhaoyang Cc: Catalin Marinas, Will Deacon, Anshuman Khandual, Andrew Morton, Nicholas Piggin, Mike Rapoport, Pavel Tatashin, Christophe Leroy, Jonathan Marek, Zhaoyang Huang, Linux Memory Management List, Linux Kernel Mailing List On Fri, 12 Nov 2021 at 10:21, Huangzhaoyang <huangzhaoyang@gmail.com> wrote: > > From: Zhaoyang Huang <zhaoyang.huang@unisoc.com> > > By comparing the swapper_pg_dir with k54 I take it this means Linux v5.4 ? > and previous versions,we find > that the linear mappings within which the addr is out of kernel text section > will use the smallest pte. It should arise for the sake of rodata_full, which > set all memblock use NO_CONT_MAPPINGS. > OK so your intent seems to be to use block mappings for the linear alias of the kernel text and rodata, right? What does that achieve? It should make no difference in TLB pressure, as the linear alias is rarely referenced (we only kept it around for hibernate). So I guess we save a handful of pages for page tables. > Signed-off-by: Zhaoyang Huang <zhaoyang.huang@unisoc.com> > --- > arch/arm64/mm/mmu.c | 11 ++++++++++- > 1 file changed, 10 insertions(+), 1 deletion(-) > > diff --git a/arch/arm64/mm/mmu.c b/arch/arm64/mm/mmu.c > index cfd9deb..14e1bea 100644 > --- a/arch/arm64/mm/mmu.c > +++ b/arch/arm64/mm/mmu.c > @@ -252,6 +252,8 @@ static void init_pmd(pud_t *pudp, unsigned long addr, unsigned long end, > pmd_clear_fixmap(); > } > > +static bool crash_mem_map __initdata; > + > static void alloc_init_cont_pmd(pud_t *pudp, unsigned long addr, > unsigned long end, phys_addr_t phys, > pgprot_t prot, > @@ -259,7 +261,15 @@ static void alloc_init_cont_pmd(pud_t *pudp, unsigned long addr, > { > unsigned long next; > pud_t pud = READ_ONCE(*pudp); > + unsigned long len = end - addr; > + phys_addr_t kernel_start = __pa_symbol(_stext); > + phys_addr_t kernel_end = __pa_symbol(__init_begin); > > + if (debug_pagealloc_enabled() || crash_mem_map || IS_ENABLED(CONFIG_KFENCE)) > + ; > + else if (phys > kernel_end || phys + len < kernel_start) { > + flags &= ~(NO_BLOCK_MAPPINGS | NO_CONT_MAPPINGS); > + } Please don't use empty statements like this, and I'm not sure we even need this check: I wouldn't expect debug_pagealloc or KFENCE to ever require page granular mappings of this region either. Also, please add a comment to explain what the condition is meant to check (i..e, that the PMD entry covers a part of the linear alias of the kernel image, and so there is no need to map it down to pages or to avoid contiguous mappings) > /* > * Check for initial section mappings in the pgd/pud. > */ > @@ -483,7 +493,6 @@ void __init mark_linear_text_alias_ro(void) > PAGE_KERNEL_RO); > } > > -static bool crash_mem_map __initdata; > > static int __init enable_crash_mem_map(char *arg) > { > -- > 1.9.1 > ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [RFC PATCH] arch: arm64: have memblocks out of kernel text use section map 2021-11-12 9:31 ` Ard Biesheuvel @ 2021-11-12 9:45 ` Zhaoyang Huang 0 siblings, 0 replies; 3+ messages in thread From: Zhaoyang Huang @ 2021-11-12 9:45 UTC (permalink / raw) To: Ard Biesheuvel Cc: Catalin Marinas, Will Deacon, Anshuman Khandual, Andrew Morton, Nicholas Piggin, Mike Rapoport, Pavel Tatashin, Christophe Leroy, Jonathan Marek, Zhaoyang Huang, Linux Memory Management List, Linux Kernel Mailing List, ben.dai, lvqiang.huang, Ke Wang On Fri, Nov 12, 2021 at 5:31 PM Ard Biesheuvel <ardb@kernel.org> wrote: > > On Fri, 12 Nov 2021 at 10:21, Huangzhaoyang <huangzhaoyang@gmail.com> wrote: > > > > From: Zhaoyang Huang <zhaoyang.huang@unisoc.com> > > > > By comparing the swapper_pg_dir with k54 > > I take it this means Linux v5.4 ? > > > and previous versions,we find > > that the linear mappings within which the addr is out of kernel text section > > will use the smallest pte. It should arise for the sake of rodata_full, which > > set all memblock use NO_CONT_MAPPINGS. > > > > OK so your intent seems to be to use block mappings for the linear > alias of the kernel text and rodata, right? > > What does that achieve? It should make no difference in TLB pressure, > as the linear alias is rarely referenced (we only kept it around for > hibernate). So I guess we save a handful of pages for page tables. Thanks for the quick response and sorry for causing confusion with my poor comments. What I want to do is on the contrary, that is using block mapping on the addr OUT OF the kernel text, which could be affected by setting rodata_full(can_set_direct_map) so far. > > > Signed-off-by: Zhaoyang Huang <zhaoyang.huang@unisoc.com> > > --- > > arch/arm64/mm/mmu.c | 11 ++++++++++- > > 1 file changed, 10 insertions(+), 1 deletion(-) > > > > diff --git a/arch/arm64/mm/mmu.c b/arch/arm64/mm/mmu.c > > index cfd9deb..14e1bea 100644 > > --- a/arch/arm64/mm/mmu.c > > +++ b/arch/arm64/mm/mmu.c > > @@ -252,6 +252,8 @@ static void init_pmd(pud_t *pudp, unsigned long addr, unsigned long end, > > pmd_clear_fixmap(); > > } > > > > +static bool crash_mem_map __initdata; > > + > > static void alloc_init_cont_pmd(pud_t *pudp, unsigned long addr, > > unsigned long end, phys_addr_t phys, > > pgprot_t prot, > > @@ -259,7 +261,15 @@ static void alloc_init_cont_pmd(pud_t *pudp, unsigned long addr, > > { > > unsigned long next; > > pud_t pud = READ_ONCE(*pudp); > > + unsigned long len = end - addr; > > + phys_addr_t kernel_start = __pa_symbol(_stext); > > + phys_addr_t kernel_end = __pa_symbol(__init_begin); > > > > + if (debug_pagealloc_enabled() || crash_mem_map || IS_ENABLED(CONFIG_KFENCE)) > > + ; > > + else if (phys > kernel_end || phys + len < kernel_start) { > > + flags &= ~(NO_BLOCK_MAPPINGS | NO_CONT_MAPPINGS); > > + } > > Please don't use empty statements like this, and I'm not sure we even > need this check: I wouldn't expect debug_pagealloc or KFENCE to ever > require page granular mappings of this region either. > > Also, please add a comment to explain what the condition is meant to > check (i..e, that the PMD entry covers a part of the linear alias of > the kernel image, and so there is no need to map it down to pages or > to avoid contiguous mappings) > > > /* > > * Check for initial section mappings in the pgd/pud. > > */ > > @@ -483,7 +493,6 @@ void __init mark_linear_text_alias_ro(void) > > PAGE_KERNEL_RO); > > } > > > > -static bool crash_mem_map __initdata; > > > > static int __init enable_crash_mem_map(char *arg) > > { > > -- > > 1.9.1 > > ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-11-12 9:45 UTC | newest] Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-11-12 9:20 [RFC PATCH] arch: arm64: have memblocks out of kernel text use section map Huangzhaoyang 2021-11-12 9:31 ` Ard Biesheuvel 2021-11-12 9:45 ` Zhaoyang Huang
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox