Re: [PATCH] arm: asm/tlb.h: Fix build error implicit func declaration

From: Ingo Molnar
Date: Wed Jul 25 2018 - 06:10:00 EST



* Anders Roxell <anders.roxell@xxxxxxxxxx> wrote:

> Building on arm 32 with LPAE enabled we don't include asm-generic/tlb.h,
> where we have tlb_flush_remove_tables_local and tlb_flush_remove_tables
> defined.
> mm/memory.c: In function âtlb_remove_table_smp_syncâ:
> mm/memory.c:339:2: error: implicit declaration of function âtlb_flush_remove_tables_localâ; did you mean âtlb_remove_tableâ? [-Werror=implicit-function-declaration]
> tlb_flush_remove_tables_local(mm);
> ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> tlb_remove_table
> mm/memory.c: In function âtlb_table_flushâ:
> mm/memory.c:372:2: error: implicit declaration of function âtlb_flush_remove_tablesâ; did you mean âtlb_remove_tableâ? [-Werror=implicit-function-declaration]
> tlb_flush_remove_tables(tlb->mm);
> ^~~~~~~~~~~~~~~~~~~~~~~
> tlb_remove_table
>
> To fix this issue we define them in arm 32's specific asm/tlb.h file.
>
> Fixes: 2ff6ddf19c0e ("x86/mm/tlb: Leave lazy TLB mode at page table free time")
> Signed-off-by: Anders Roxell <anders.roxell@xxxxxxxxxx>
> ---
>
> Ingo, would it make sense for you to pick it up into the x86 tree since
> that contains the bug ?

Done and sorry about that! Will push your fix out to -next as well ASAP.

Thanks,

Ingo