x86/ucode: Drop the sanity check for interrupts being disabled
authorAndrew Cooper <andrew.cooper3@citrix.com>
Fri, 27 Mar 2020 12:02:09 +0000 (12:02 +0000)
committerAndrew Cooper <andrew.cooper3@citrix.com>
Fri, 27 Mar 2020 13:13:46 +0000 (13:13 +0000)
commitac71d99c6aebfbeb321b32ba84125ac6f2699a41
tree8ada29237cac814fef14afcfdca2fe82d8595acd
parent1f97b6b9f1b5978659c5735954c37c130e7bb151
x86/ucode: Drop the sanity check for interrupts being disabled

Of the substantial number of things which can go wrong during microcode load,
this is not one.  Loading occurs entirely within the boundary of a single
WRMSR instruction.  Its certainly not a BUG()-worthy condition.

Xen has legitimate reasons to not want interrupts enabled at this point, but
that is to do with organising the system rendezvous.  As these are private low
level helpers invoked only from the microcode core logic, forgo the check
entirely.

While dropping system.h, clean up the processor.h include which was an
oversight in the previous header cleanup.

Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Reviewed-by: Jan Beulich <jbeulich@suse.com>
xen/arch/x86/cpu/microcode/amd.c
xen/arch/x86/cpu/microcode/intel.c