Re: [PATCH] MIPS: include: mmu_context.h: Replace VIRTUALIZATION with KVM

[Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>]

hello.

On 10-06-2013 15:42, Markos Chandras wrote:

> The kvm_* symbols are only available if KVM is selected.

> Fixes the following linking problem on a randconfig:

> arch/mips/built-in.o: In function `local_flush_tlb_mm':
> (.text+0x18a94): undefined reference to `kvm_local_flush_tlb_all'
> arch/mips/built-in.o: In function `local_flush_tlb_range':
> (.text+0x18d0c): undefined reference to `kvm_local_flush_tlb_all'
> kernel/built-in.o: In function `__schedule':
> core.c:(.sched.text+0x2a00): undefined reference to `kvm_local_flush_tlb_all'
> mm/built-in.o: In function `use_mm':
> (.text+0x30214): undefined reference to `kvm_local_flush_tlb_all'
> fs/built-in.o: In function `flush_old_exec':
> (.text+0xf0a0): undefined reference to `kvm_local_flush_tlb_all'
> make: *** [vmlinux] Error 1

> Signed-off-by: Markos Chandras <markos.chandras@imgtec.com>
> Signed-off-by: Steven J. Hill <Steven.Hill@imgtec.com>
> ---
>   arch/mips/include/asm/mmu_context.h | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)

> diff --git a/arch/mips/include/asm/mmu_context.h b/arch/mips/include/asm/mmu_context.h
> index 8201160..ee5a93b 100644
> --- a/arch/mips/include/asm/mmu_context.h
> +++ b/arch/mips/include/asm/mmu_context.h
> @@ -111,13 +111,15 @@ static inline void enter_lazy_tlb(struct mm_struct *mm, struct task_struct *tsk)
>   static inline void
>   get_new_mmu_context(struct mm_struct *mm, unsigned long cpu)
>   {
> +#ifdef CONFIG_KVM
>   	extern void kvm_local_flush_tlb_all(void);
> +#endif

    #ifdef should not be needed around declaration.

WBR, Sergei