| Message ID | 1527856758-27169-4-git-send-email-cpandya@codeaurora.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Fri, Jun 01, 2018 at 06:09:16PM +0530, Chintan Pandya wrote: > Add helper macros to give virtual references to page > tables. These will be used while freeing dangling > page tables. > > Signed-off-by: Chintan Pandya <cpandya@codeaurora.org> > --- > arch/arm64/include/asm/pgtable.h | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/arch/arm64/include/asm/pgtable.h b/arch/arm64/include/asm/pgtable.h > index 7c4c8f3..ef4047f 100644 > --- a/arch/arm64/include/asm/pgtable.h > +++ b/arch/arm64/include/asm/pgtable.h > @@ -580,6 +580,9 @@ static inline phys_addr_t pgd_page_paddr(pgd_t pgd) > > #endif /* CONFIG_PGTABLE_LEVELS > 3 */ > > +#define pmd_page_vaddr(pmd) __va(pmd_page_paddr(pmd)) > +#define pud_page_vaddr(pud) __va(pud_page_paddr(pud)) Are these actually needed, or do pte_offset_kernel and pmd_offset do the job already? Will
On 6/4/2018 5:43 PM, Will Deacon wrote: > On Fri, Jun 01, 2018 at 06:09:16PM +0530, Chintan Pandya wrote: >> Add helper macros to give virtual references to page >> tables. These will be used while freeing dangling >> page tables. >> >> Signed-off-by: Chintan Pandya <cpandya@codeaurora.org> >> --- >> arch/arm64/include/asm/pgtable.h | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/arch/arm64/include/asm/pgtable.h b/arch/arm64/include/asm/pgtable.h >> index 7c4c8f3..ef4047f 100644 >> --- a/arch/arm64/include/asm/pgtable.h >> +++ b/arch/arm64/include/asm/pgtable.h >> @@ -580,6 +580,9 @@ static inline phys_addr_t pgd_page_paddr(pgd_t pgd) >> >> #endif /* CONFIG_PGTABLE_LEVELS > 3 */ >> >> +#define pmd_page_vaddr(pmd) __va(pmd_page_paddr(pmd)) >> +#define pud_page_vaddr(pud) __va(pud_page_paddr(pud)) > > Are these actually needed, or do pte_offset_kernel and pmd_offset do the > job already? > I introduced these macros for consistency across different arch. Looking at pte_offset_kernel, it seems to use READ_ONCE() which looks little costly for its intended use (in next patch) where we already have dereferenced value. Do you still suggest to remove this ? > Will > Chintan
On Mon, Jun 04, 2018 at 07:13:48PM +0530, Chintan Pandya wrote: > > > On 6/4/2018 5:43 PM, Will Deacon wrote: > >On Fri, Jun 01, 2018 at 06:09:16PM +0530, Chintan Pandya wrote: > >>Add helper macros to give virtual references to page > >>tables. These will be used while freeing dangling > >>page tables. > >> > >>Signed-off-by: Chintan Pandya <cpandya@codeaurora.org> > >>--- > >> arch/arm64/include/asm/pgtable.h | 3 +++ > >> 1 file changed, 3 insertions(+) > >> > >>diff --git a/arch/arm64/include/asm/pgtable.h b/arch/arm64/include/asm/pgtable.h > >>index 7c4c8f3..ef4047f 100644 > >>--- a/arch/arm64/include/asm/pgtable.h > >>+++ b/arch/arm64/include/asm/pgtable.h > >>@@ -580,6 +580,9 @@ static inline phys_addr_t pgd_page_paddr(pgd_t pgd) > >> #endif /* CONFIG_PGTABLE_LEVELS > 3 */ > >>+#define pmd_page_vaddr(pmd) __va(pmd_page_paddr(pmd)) > >>+#define pud_page_vaddr(pud) __va(pud_page_paddr(pud)) > > > >Are these actually needed, or do pte_offset_kernel and pmd_offset do the > >job already? > > > > I introduced these macros for consistency across different arch. > > Looking at pte_offset_kernel, it seems to use READ_ONCE() which looks > little costly for its intended use (in next patch) where we already have > dereferenced value. Do you still suggest to remove this ? It's only an additional load instruction on the freeing path and it matches what we do in other page table code, so I'd rather use the existing API unless we have numbers to show otherwise. Will
diff --git a/arch/arm64/include/asm/pgtable.h b/arch/arm64/include/asm/pgtable.h index 7c4c8f3..ef4047f 100644 --- a/arch/arm64/include/asm/pgtable.h +++ b/arch/arm64/include/asm/pgtable.h @@ -580,6 +580,9 @@ static inline phys_addr_t pgd_page_paddr(pgd_t pgd) #endif /* CONFIG_PGTABLE_LEVELS > 3 */ +#define pmd_page_vaddr(pmd) __va(pmd_page_paddr(pmd)) +#define pud_page_vaddr(pud) __va(pud_page_paddr(pud)) + #define pgd_ERROR(pgd) __pgd_error(__FILE__, __LINE__, pgd_val(pgd)) /* to find an entry in a page-table-directory */
Add helper macros to give virtual references to page tables. These will be used while freeing dangling page tables. Signed-off-by: Chintan Pandya <cpandya@codeaurora.org> --- arch/arm64/include/asm/pgtable.h | 3 +++ 1 file changed, 3 insertions(+)