Re: [PATCH] [v2] x86: apic: avoid -Wshadow warning in header
From: Paolo Bonzini
Date: Thu Oct 29 2020 - 03:22:05 EST
On 28/10/20 22:20, Arnd Bergmann wrote:
> Avoid this by renaming the global 'apic' variable to the more descriptive
> 'x86_system_apic'. It was originally called 'genapic', but both that
> and the current 'apic' seem to be a little overly generic for a global
> variable.
The 'apic' affects only the current CPU, so one of 'x86_local_apic',
'x86_lapic' or 'x86_apic' is probably preferrable.
I don't have huge objections to renaming 'apic' variables and arguments
in KVM to 'lapic'. I do agree with Sean however that it's going to
break again very soon.
Paolo