Re: [PATCH v2] x86/paravirt: Cleanup native_patch()

From: Juergen Gross
Date: Tue Sep 11 2018 - 05:44:33 EST


On 11/09/18 11:15, Borislav Petkov wrote:
> When CONFIG_PARAVIRT_SPINLOCKS=n, it fires
>
> arch/x86/kernel/paravirt_patch_64.c: In function ânative_patchâ:
> arch/x86/kernel/paravirt_patch_64.c:89:1: warning: label âpatch_siteâ defined but not used [-Wunused-label]
> patch_site:
>
> but those labels can simply be removed by directly calling the
> respective functions there.
>
> Get rid of local variables too, while at it. Also, simplify function
> flow for better readability.
>
> Signed-off-by: Borislav Petkov <bp@xxxxxxx>
> Cc: Juergen Gross <jgross@xxxxxxxx>
> Cc: x86@xxxxxxxxxx
> Cc: virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx

Reviewed-by: Juergen Gross <jgross@xxxxxxxx>


Juergen