On 23.04.2024 03:17, Marek Marczykowski-Górecki wrote: > On Wed, Jan 17, 2024 at 10:32:53AM +0100, Jan Beulich wrote: >> --- a/xen/arch/x86/cpu/mtrr/main.c >> +++ b/xen/arch/x86/cpu/mtrr/main.c >> @@ -328,7 +316,7 @@ int mtrr_add_page(unsigned long base, un >> } >> >> /* If the type is WC, check that this processor supports it */ >> - if ((type == X86_MT_WC) && !have_wrcomb()) { >> + if ((type == X86_MT_WC) && mtrr_have_wrcomb()) { > > Is reversing the condition intentional here?
Clearly not. Patch sent. Jan