Re: [PATCH] arch/x86/entry_fred: don't set up KVM IRQs if KVM is disabled

From: Max Kellermann
Date: Fri Feb 16 2024 - 18:00:42 EST


On Fri, Feb 16, 2024 at 10:45 PM Thomas Gleixner <tglx@xxxxxxxxxxxxx> wrote:
> #ifdeffing out the vector numbers is silly to begin with because these
> vector numbers stay assigned to KVM whether KVM is enabled or not.

There could be one non-silly use of this: if the macros are not
defined in absence of the feature, any use of it will lead to a
compiler error, which is good, because it may reveal certain kinds of
bugs.
(Though I agree that this isn't worth the code ugliness. I prefer to
avoid the preprocessor whenever possible. I hate how much the kernel
uses macros instead of inline functions.)