From: "Liang, Kan" <kan.liang@linux.intel.com> To: Steven Price <steven.price@arm.com>, Peter Zijlstra <peterz@infradead.org> Cc: x86@kernel.org, "Arnd Bergmann" <arnd@arndb.de>, "Ard Biesheuvel" <ard.biesheuvel@linaro.org>, "Catalin Marinas" <catalin.marinas@arm.com>, kirill@shutemov.name, "Dave Hansen" <dave.hansen@linux.intel.com>, "Will Deacon" <will.deacon@arm.com>, linux-kernel@vger.kernel.org, linux-mm@kvack.org, "Jérôme Glisse" <jglisse@redhat.com>, "Ingo Molnar" <mingo@redhat.com>, "Borislav Petkov" <bp@alien8.de>, "Andy Lutomirski" <luto@kernel.org>, "H. Peter Anvin" <hpa@zytor.com>, "James Morse" <james.morse@arm.com>, "Thomas Gleixner" <tglx@linutronix.de>, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 03/13] mm: Add generic p?d_large() macros Date: Mon, 18 Feb 2019 22:44:10 -0500 [thread overview] Message-ID: <8a74c111-b099-8d18-5fb0-422909a1367a@linux.intel.com> (raw) In-Reply-To: <aad21496-a86b-ca91-70b7-0c23ea6fefd3@arm.com> On 2/18/2019 9:19 AM, Steven Price wrote: > On 18/02/2019 11:31, Peter Zijlstra wrote: >> On Fri, Feb 15, 2019 at 05:02:24PM +0000, Steven Price wrote: >>> From: James Morse <james.morse@arm.com> >>> >>> Exposing the pud/pgd levels of the page tables to walk_page_range() means >>> we may come across the exotic large mappings that come with large areas >>> of contiguous memory (such as the kernel's linear map). >>> >>> For architectures that don't provide p?d_large() macros, provided a >>> does nothing default. >> >> Kan was going to fix that for all archs I think.. > Yes, I'm still working on a generic function to retrieve page size. The generic p?d_large() issue has been fixed. However, I found that the pgd_page() is not generic either. I'm still working on it. I will update you on the other thread when all issues are fixed. > The latest series I can find from Kan is still x86 specific. I'm happy > to rebase onto something else if Kan has an implementation already > (please point me in the right direction). Otherwise Kan is obviously > free to base on these changes. > My implementation is similar as yours. I'm happy to re-base on your changes. Could you please also add a generic p4d_large()? Thanks, Kan > Steve > >> See: >> >> http://lkml.kernel.org/r/20190204105409.GA17550@hirez.programming.kicks-ass.net >> >>> Signed-off-by: James Morse <james.morse@arm.com> >>> Signed-off-by: Steven Price <steven.price@arm.com> >>> --- >>> include/asm-generic/pgtable.h | 10 ++++++++++ >>> 1 file changed, 10 insertions(+) >>> >>> diff --git a/include/asm-generic/pgtable.h b/include/asm-generic/pgtable.h >>> index 05e61e6c843f..7630d663cd51 100644 >>> --- a/include/asm-generic/pgtable.h >>> +++ b/include/asm-generic/pgtable.h >>> @@ -1186,4 +1186,14 @@ static inline bool arch_has_pfn_modify_check(void) >>> #define mm_pmd_folded(mm) __is_defined(__PAGETABLE_PMD_FOLDED) >>> #endif >>> >>> +#ifndef pgd_large >>> +#define pgd_large(x) 0 >>> +#endif >>> +#ifndef pud_large >>> +#define pud_large(x) 0 >>> +#endif >>> +#ifndef pmd_large >>> +#define pmd_large(x) 0 >>> +#endif >>> + >>> #endif /* _ASM_GENERIC_PGTABLE_H */ >>> -- >>> 2.20.1 >>> >> >> _______________________________________________ >> linux-arm-kernel mailing list >> linux-arm-kernel@lists.infradead.org >> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel >> >
WARNING: multiple messages have this Message-ID (diff)
From: "Liang, Kan" <kan.liang@linux.intel.com> To: Steven Price <steven.price@arm.com>, Peter Zijlstra <peterz@infradead.org> Cc: "Dave Hansen" <dave.hansen@linux.intel.com>, "James Morse" <james.morse@arm.com>, "Arnd Bergmann" <arnd@arndb.de>, "Ard Biesheuvel" <ard.biesheuvel@linaro.org>, "Catalin Marinas" <catalin.marinas@arm.com>, x86@kernel.org, "Will Deacon" <will.deacon@arm.com>, linux-kernel@vger.kernel.org, linux-mm@kvack.org, "Jérôme Glisse" <jglisse@redhat.com>, "Ingo Molnar" <mingo@redhat.com>, "Borislav Petkov" <bp@alien8.de>, "Andy Lutomirski" <luto@kernel.org>, "H. Peter Anvin" <hpa@zytor.com>, kirill@shutemov.name, "Thomas Gleixner" <tglx@linutronix.de>, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 03/13] mm: Add generic p?d_large() macros Date: Mon, 18 Feb 2019 22:44:10 -0500 [thread overview] Message-ID: <8a74c111-b099-8d18-5fb0-422909a1367a@linux.intel.com> (raw) In-Reply-To: <aad21496-a86b-ca91-70b7-0c23ea6fefd3@arm.com> On 2/18/2019 9:19 AM, Steven Price wrote: > On 18/02/2019 11:31, Peter Zijlstra wrote: >> On Fri, Feb 15, 2019 at 05:02:24PM +0000, Steven Price wrote: >>> From: James Morse <james.morse@arm.com> >>> >>> Exposing the pud/pgd levels of the page tables to walk_page_range() means >>> we may come across the exotic large mappings that come with large areas >>> of contiguous memory (such as the kernel's linear map). >>> >>> For architectures that don't provide p?d_large() macros, provided a >>> does nothing default. >> >> Kan was going to fix that for all archs I think.. > Yes, I'm still working on a generic function to retrieve page size. The generic p?d_large() issue has been fixed. However, I found that the pgd_page() is not generic either. I'm still working on it. I will update you on the other thread when all issues are fixed. > The latest series I can find from Kan is still x86 specific. I'm happy > to rebase onto something else if Kan has an implementation already > (please point me in the right direction). Otherwise Kan is obviously > free to base on these changes. > My implementation is similar as yours. I'm happy to re-base on your changes. Could you please also add a generic p4d_large()? Thanks, Kan > Steve > >> See: >> >> http://lkml.kernel.org/r/20190204105409.GA17550@hirez.programming.kicks-ass.net >> >>> Signed-off-by: James Morse <james.morse@arm.com> >>> Signed-off-by: Steven Price <steven.price@arm.com> >>> --- >>> include/asm-generic/pgtable.h | 10 ++++++++++ >>> 1 file changed, 10 insertions(+) >>> >>> diff --git a/include/asm-generic/pgtable.h b/include/asm-generic/pgtable.h >>> index 05e61e6c843f..7630d663cd51 100644 >>> --- a/include/asm-generic/pgtable.h >>> +++ b/include/asm-generic/pgtable.h >>> @@ -1186,4 +1186,14 @@ static inline bool arch_has_pfn_modify_check(void) >>> #define mm_pmd_folded(mm) __is_defined(__PAGETABLE_PMD_FOLDED) >>> #endif >>> >>> +#ifndef pgd_large >>> +#define pgd_large(x) 0 >>> +#endif >>> +#ifndef pud_large >>> +#define pud_large(x) 0 >>> +#endif >>> +#ifndef pmd_large >>> +#define pmd_large(x) 0 >>> +#endif >>> + >>> #endif /* _ASM_GENERIC_PGTABLE_H */ >>> -- >>> 2.20.1 >>> >> >> _______________________________________________ >> linux-arm-kernel mailing list >> linux-arm-kernel@lists.infradead.org >> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel >> > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2019-02-19 3:44 UTC|newest] Thread overview: 72+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-02-15 17:02 [PATCH 00/13] Convert x86 & arm64 to use generic page walk Steven Price 2019-02-15 17:02 ` Steven Price 2019-02-15 17:02 ` [PATCH 01/13] arm64: mm: Add p?d_large() definitions Steven Price 2019-02-15 17:02 ` Steven Price 2019-02-18 11:16 ` Mark Rutland 2019-02-18 11:16 ` Mark Rutland 2019-02-18 11:29 ` Peter Zijlstra 2019-02-18 11:29 ` Peter Zijlstra 2019-02-18 13:45 ` Mark Rutland 2019-02-18 13:45 ` Mark Rutland 2019-02-18 14:11 ` Steven Price 2019-02-18 14:11 ` Steven Price 2019-02-18 14:29 ` Mark Rutland 2019-02-18 14:29 ` Mark Rutland 2019-02-18 15:06 ` Peter Zijlstra 2019-02-18 15:06 ` Peter Zijlstra 2019-02-18 15:30 ` Steven Price 2019-02-18 15:30 ` Steven Price 2019-02-18 17:04 ` Mark Rutland 2019-02-18 17:04 ` Mark Rutland 2019-02-18 17:22 ` Steven Price 2019-02-18 17:22 ` Steven Price 2019-02-15 17:02 ` [PATCH 02/13] x86/mm: " Steven Price 2019-02-15 17:02 ` Steven Price 2019-02-15 17:02 ` [PATCH 03/13] mm: Add generic p?d_large() macros Steven Price 2019-02-15 17:02 ` Steven Price 2019-02-18 11:14 ` Mark Rutland 2019-02-18 11:14 ` Mark Rutland 2019-02-18 12:53 ` Peter Zijlstra 2019-02-18 12:53 ` Peter Zijlstra 2019-02-18 14:20 ` Steven Price 2019-02-18 14:20 ` Steven Price 2019-02-18 11:31 ` Peter Zijlstra 2019-02-18 11:31 ` Peter Zijlstra 2019-02-18 14:19 ` Steven Price 2019-02-18 14:19 ` Steven Price 2019-02-19 3:44 ` Liang, Kan [this message] 2019-02-19 3:44 ` Liang, Kan 2019-02-20 13:56 ` Steven Price 2019-02-20 13:56 ` Steven Price 2019-02-15 17:02 ` [PATCH 04/13] mm: pagewalk: Add p4d_entry() and pgd_entry() Steven Price 2019-02-15 17:02 ` Steven Price 2019-02-15 17:02 ` [PATCH 05/13] mm: pagewalk: Allow walking without vma Steven Price 2019-02-15 17:02 ` Steven Price 2019-02-15 17:02 ` [PATCH 06/13] mm: pagewalk: Add 'depth' parameter to pte_hole Steven Price 2019-02-15 17:02 ` Steven Price 2019-02-18 11:23 ` Mark Rutland 2019-02-18 11:23 ` Mark Rutland 2019-02-18 15:23 ` Steven Price 2019-02-18 15:23 ` Steven Price 2019-02-20 11:35 ` William Kucharski 2019-02-20 11:35 ` William Kucharski 2019-02-20 14:10 ` Steven Price 2019-02-20 14:10 ` Steven Price 2019-02-15 17:02 ` [PATCH 07/13] mm: pagewalk: Add test_p?d callbacks Steven Price 2019-02-15 17:02 ` Steven Price 2019-02-15 17:02 ` [PATCH 08/13] arm64: mm: Convert mm/dump.c to use walk_page_range() Steven Price 2019-02-15 17:02 ` Steven Price 2019-02-15 17:02 ` [PATCH 09/13] x86/mm: Point to struct seq_file from struct pg_state Steven Price 2019-02-15 17:02 ` Steven Price 2019-02-15 17:02 ` [PATCH 10/13] x86/mm+efi: Convert ptdump_walk_pgd_level() to take a mm_struct Steven Price 2019-02-15 17:02 ` Steven Price 2019-02-15 17:02 ` [PATCH 11/13] x86/mm: Convert ptdump_walk_pgd_level_debugfs() to take an mm_struct Steven Price 2019-02-15 17:02 ` Steven Price 2019-02-15 17:02 ` [PATCH 12/13] x86/mm: Convert ptdump_walk_pgd_level_core() " Steven Price 2019-02-15 17:02 ` Steven Price 2019-02-15 17:02 ` [PATCH 13/13] x86: mm: Convert dump_pagetables to use walk_page_range Steven Price 2019-02-15 17:02 ` Steven Price 2019-02-15 17:16 ` Dave Hansen 2019-02-15 17:16 ` Dave Hansen 2019-02-15 17:32 ` Steven Price 2019-02-15 17:32 ` Steven Price
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=8a74c111-b099-8d18-5fb0-422909a1367a@linux.intel.com \ --to=kan.liang@linux.intel.com \ --cc=ard.biesheuvel@linaro.org \ --cc=arnd@arndb.de \ --cc=bp@alien8.de \ --cc=catalin.marinas@arm.com \ --cc=dave.hansen@linux.intel.com \ --cc=hpa@zytor.com \ --cc=james.morse@arm.com \ --cc=jglisse@redhat.com \ --cc=kirill@shutemov.name \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mm@kvack.org \ --cc=luto@kernel.org \ --cc=mingo@redhat.com \ --cc=peterz@infradead.org \ --cc=steven.price@arm.com \ --cc=tglx@linutronix.de \ --cc=will.deacon@arm.com \ --cc=x86@kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.