Re: [PATCH v3 1/5] x86/paravirt: move some functions and defines to alternative

From: Borislav Petkov
Date: Wed Oct 25 2023 - 06:34:45 EST


On Thu, Oct 19, 2023 at 11:15:16AM +0200, Juergen Gross wrote:
> +/* Low-level backend functions usable from alternative code replacements. */
> +DEFINE_ASM_FUNC(x86_nop, "", .entry.text);
> +EXPORT_SYMBOL_GPL(x86_nop);

This is all x86 code so you don't really need the "x86_" prefix - "nop"
is perfectly fine.

> +noinstr void x86_BUG(void)
> +{
> + BUG();
> +}
> +EXPORT_SYMBOL_GPL(x86_BUG);

That export is needed for?

Paravirt stuff in modules?

It builds here without it - I guess I need to do an allmodconfig.

--
Regards/Gruss,
Boris.

https://people.kernel.org/tglx/notes-about-netiquette