Re: [PATCH for 5.10-stable] x86/paravirt: Fix build due to __text_gen_insn() backport

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Mar 05, 2024 at 12:27:11PM +0100, Borislav Petkov wrote:
> On Mon, Mar 04, 2024 at 10:49:33AM -0800, Omar Sandoval wrote:
> > v5.10.211 is failing to build with the attached .config with the
> > following error:
> 
> Ok, let's try this:
> 
> ---
> From: "Borislav Petkov (AMD)" <bp@xxxxxxxxx>
> 
> The Link tag has all the details but basically due to missing upstream
> commits, the header which contains __text_gen_insn() is not in the
> includes in paravirt.c, leading to:
> 
>   arch/x86/kernel/paravirt.c: In function 'paravirt_patch_call':
>   arch/x86/kernel/paravirt.c:65:9: error: implicit declaration of function '__text_gen_insn' \
>   [-Werror=implicit-function-declaration]
>    65 |         __text_gen_insn(insn_buff, CALL_INSN_OPCODE,
>       |         ^~~~~~~~~~~~~~~
> 
> Add the missing include.
> 
> Reported-by: Omar Sandoval <osandov@xxxxxxxxxxx>
> Signed-off-by: Borislav Petkov (AMD) <bp@xxxxxxxxx>
> Link: https://lore.kernel.org/r/ZeYXvd1-rVkPGvvW@telecaster
> ---
>  arch/x86/kernel/paravirt.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/x86/kernel/paravirt.c b/arch/x86/kernel/paravirt.c
> index 5bea8d93883a..f0e4ad8595ca 100644
> --- a/arch/x86/kernel/paravirt.c
> +++ b/arch/x86/kernel/paravirt.c
> @@ -31,6 +31,7 @@
>  #include <asm/special_insns.h>
>  #include <asm/tlb.h>
>  #include <asm/io_bitmap.h>
> +#include <asm/text-patching.h>
>  
>  /*
>   * nop stub, which must not clobber anything *including the stack* to
> -- 
> 2.43.0
> 
> -- 
> Regards/Gruss,
>     Boris.
> 
> https://people.kernel.org/tglx/notes-about-netiquette

Tested-by: Omar Sandoval <osandov@xxxxxxxxxxx>

Thanks!




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux