On 28/05/2024 4:29 pm, Fouad Hilly wrote: > Introduce --force option to xen-ucode to force skipping microcode version > check, which > allows the user to update x86 microcode even if both versions are the same or > downgrade. > xc_microcode_update() refactored to accept flags and utilize > xenpf_microcode_update2. > > Signed-off-by: Fouad Hilly <fouad.hi...@cloud.com>
I think it would be better to stop the subject at "... to xen-ucode". The commit message itself covers what has changed. > diff --git a/tools/misc/xen-ucode.c b/tools/misc/xen-ucode.c > index 6f9dd2a7e431..b878edf2399a 100644 > --- a/tools/misc/xen-ucode.c > +++ b/tools/misc/xen-ucode.c > @@ -13,6 +13,8 @@ > #include <xenctrl.h> > #include <getopt.h> > > +#include <xen/platform.h> > + > static xc_interface *xch; > > static const char intel_id[] = "GenuineIntel"; > @@ -24,7 +26,8 @@ static void usage(FILE *stream, const char *name) > "Usage: %s [microcode file] [options]\n" > "options:\n" > " -h, --help display this help and exit\n" > - " -s, --show-cpu-info show CPU information and exit\n", > + " -s, --show-cpu-info show CPU information and exit\n" > + " -f, --force force to skip microcode version check\n", I'd phrase this as "skip certain checks; do not use unless you know exactly what you are doing" which makes it very clear that people get to keep all pieces if they try this. Otherwise (and subject to the style cleanup in the previous patch), Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com>
