Messages in this thread Patch in this message |  | | Subject | Re: [PATCH -next] module: fix build error when CONFIG_SMP is disabled | From | "Sunnanyong (Nanyong Sun, Intelligent Computing Solution Development Dep)" <> | Date | Sat, 29 May 2021 12:43:05 +0800 |
| |
On 2021/5/29 10:03, Bixuan Cui wrote:
> diff --git a/arch/riscv/include/asm/tlbflush.h b/arch/riscv/include/asm/tlbflush.h > index c84218ad7afc..9d5801f6e2c3 100644 > --- a/arch/riscv/include/asm/tlbflush.h > +++ b/arch/riscv/include/asm/tlbflush.h > @@ -44,6 +44,12 @@ static inline void flush_tlb_range(struct vm_area_struct *vma, > local_flush_tlb_all(); > } > > +static inline void flush_pmd_tlb_range(struct vm_area_struct *vma, > + unsigned long start, unsigned long end) > +{ > + local_flush_tlb_all(); > +} > + > #define flush_tlb_mm(mm) flush_tlb_all() > #endif /* !CONFIG_SMP || !CONFIG_MMU */
Move the prototype of flush_pmd_tlb_range from pgtable.h to tlbflush.h
can also fix this problem, and it seems that declare flush_pmd_tlb_range in tlbflush.h
is better.
We could
diff --git a/arch/riscv/include/asm/pgtable.h b/arch/riscv/include/asm/pgtable.h index 29e2c836848d..eda31002c4b1 100644 --- a/arch/riscv/include/asm/pgtable.h +++ b/arch/riscv/include/asm/pgtable.h @@ -622,11 +622,6 @@ static inline pmd_t pmdp_establish(struct vm_area_struct *vma, { return __pmd(atomic_long_xchg((atomic_long_t *)pmdp, pmd_val(pmd))); } - -#define __HAVE_ARCH_FLUSH_PMD_TLB_RANGE -void flush_pmd_tlb_range(struct vm_area_struct *vma, unsigned long start, - unsigned long end); - #endif /* CONFIG_TRANSPARENT_HUGEPAGE */
/* diff --git a/arch/riscv/include/asm/tlbflush.h b/arch/riscv/include/asm/tlbflush.h index c84218ad7afc..b9b9c64ac974 100644 --- a/arch/riscv/include/asm/tlbflush.h +++ b/arch/riscv/include/asm/tlbflush.h @@ -33,6 +33,11 @@ void flush_tlb_mm(struct mm_struct *mm); void flush_tlb_page(struct vm_area_struct *vma, unsigned long addr); void flush_tlb_range(struct vm_area_struct *vma, unsigned long start, unsigned long end); +#ifdef CONFIG_TRANSPARENT_HUGEPAGE +#define __HAVE_ARCH_FLUSH_PMD_TLB_RANGE +void flush_pmd_tlb_range(struct vm_area_struct *vma, unsigned long start, + unsigned long end); +#endif /* CONFIG_TRANSPARENT_HUGEPAGE */ #else /* CONFIG_SMP && CONFIG_MMU */
#define flush_tlb_all() local_flush_tlb_all() In this way , flush_p(m/u)d_tlb_range will be defined in include/linux/pgtable.h if
CONFIG_SMP is not set.
|  |