x86/microcode: Adjust late loading result reporting message

[ Upstream commit 6eab3abac7043226e5375e9ead0c7607ced6767b ]

During late microcode loading, the "Reload completed" message is issued
unconditionally, regardless of success or failure.

Adjust the message to report the result of the update.

  [ bp: Massage. ]

Fixes: 9bd681251b7c ("x86/microcode: Announce reload operation's completion")
Suggested-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Ashok Raj <ashok.raj@intel.com>
Signed-off-by: Borislav Petkov (AMD) <bp@alien8.de>
Reviewed-by: Tony Luck <tony.luck@intel.com>
Link: https://lore.kernel.org/lkml/874judpqqd.ffs@tglx/
Signed-off-by: Sasha Levin <sashal@kernel.org>

diff --git a/arch/x86/kernel/cpu/microcode/core.c b/arch/x86/kernel/cpu/microcode/core.c
index 755aab64872c84325f760e202a116fcefe6442d0..53d2d2f145ed79fe796d174dd47f0cc2186c8846 100644 (file)
--- a/arch/x86/kernel/cpu/microcode/core.c
+++ b/arch/x86/kernel/cpu/microcode/core.c
@@ -507,11 +507,14 @@ static int microcode_reload_late(void)
        store_cpu_caps(&prev_info);
 
        ret = stop_machine_cpuslocked(__reload_late, NULL, cpu_online_mask);
-       if (ret == 0)
+       if (!ret) {
+               pr_info("Reload succeeded, microcode revision: 0x%x -> 0x%x\n",
+                       old, boot_cpu_data.microcode);
                microcode_check(&prev_info);
-
-       pr_info("Reload completed, microcode revision: 0x%x -> 0x%x\n",
-               old, boot_cpu_data.microcode);
+       } else {
+               pr_info("Reload failed, current microcode revision: 0x%x\n",
+                       boot_cpu_data.microcode);
+       }
 
        return ret;
 }