Skip to content

Commit ba48aa3

Browse files
KAGA-KOKObp3tk0v
authored andcommitted
x86/microcode: Clean up mc_cpu_down_prep()
This function has nothing to do with suspend. It's a hotplug callback. Remove the bogus comment. Drop the pointless debug printk. The hotplug core provides tracepoints which track the invocation of those callbacks. Signed-off-by: Thomas Gleixner <[email protected]> Signed-off-by: Borislav Petkov (AMD) <[email protected]> Link: https://lore.kernel.org/r/[email protected]
1 parent 2e19973 commit ba48aa3

File tree

1 file changed

+1
-7
lines changed
  • arch/x86/kernel/cpu/microcode

1 file changed

+1
-7
lines changed

arch/x86/kernel/cpu/microcode/core.c

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -498,16 +498,10 @@ static int mc_cpu_online(unsigned int cpu)
498498

499499
static int mc_cpu_down_prep(unsigned int cpu)
500500
{
501-
struct device *dev;
502-
503-
dev = get_cpu_device(cpu);
501+
struct device *dev = get_cpu_device(cpu);
504502

505503
microcode_fini_cpu(cpu);
506-
507-
/* Suspend is in progress, only remove the interface */
508504
sysfs_remove_group(&dev->kobj, &mc_attr_group);
509-
pr_debug("%s: CPU%d\n", __func__, cpu);
510-
511505
return 0;
512506
}
513507

0 commit comments

Comments
 (0)