hc
2024-01-03 2f7c68cb55ecb7331f2381deb497c27155f32faf
kernel/arch/x86/kernel/cpu/microcode/core.c
....@@ -55,7 +55,7 @@
5555 * All non cpu-hotplug-callback call sites use:
5656 *
5757 * - microcode_mutex to synchronize with each other;
58
- * - get/put_online_cpus() to synchronize with
58
+ * - cpus_read_lock/unlock() to synchronize with
5959 * the cpu-hotplug-callback call sites.
6060 *
6161 * We guarantee that only a single cpu is being
....@@ -315,7 +315,7 @@
315315 #endif
316316 }
317317
318
-void reload_early_microcode(void)
318
+void reload_early_microcode(unsigned int cpu)
319319 {
320320 int vendor, family;
321321
....@@ -329,7 +329,7 @@
329329 break;
330330 case X86_VENDOR_AMD:
331331 if (family >= 0x10)
332
- reload_ucode_amd();
332
+ reload_ucode_amd(cpu);
333333 break;
334334 default:
335335 break;
....@@ -390,101 +390,10 @@
390390 return ret;
391391 }
392392
393
-#ifdef CONFIG_MICROCODE_OLD_INTERFACE
394
-static int do_microcode_update(const void __user *buf, size_t size)
395
-{
396
- int error = 0;
397
- int cpu;
398
-
399
- for_each_online_cpu(cpu) {
400
- struct ucode_cpu_info *uci = ucode_cpu_info + cpu;
401
- enum ucode_state ustate;
402
-
403
- if (!uci->valid)
404
- continue;
405
-
406
- ustate = microcode_ops->request_microcode_user(cpu, buf, size);
407
- if (ustate == UCODE_ERROR) {
408
- error = -1;
409
- break;
410
- } else if (ustate == UCODE_NEW) {
411
- apply_microcode_on_target(cpu);
412
- }
413
- }
414
-
415
- return error;
416
-}
417
-
418
-static int microcode_open(struct inode *inode, struct file *file)
419
-{
420
- return capable(CAP_SYS_RAWIO) ? stream_open(inode, file) : -EPERM;
421
-}
422
-
423
-static ssize_t microcode_write(struct file *file, const char __user *buf,
424
- size_t len, loff_t *ppos)
425
-{
426
- ssize_t ret = -EINVAL;
427
- unsigned long nr_pages = totalram_pages();
428
-
429
- if ((len >> PAGE_SHIFT) > nr_pages) {
430
- pr_err("too much data (max %ld pages)\n", nr_pages);
431
- return ret;
432
- }
433
-
434
- get_online_cpus();
435
- mutex_lock(&microcode_mutex);
436
-
437
- if (do_microcode_update(buf, len) == 0)
438
- ret = (ssize_t)len;
439
-
440
- if (ret > 0)
441
- perf_check_microcode();
442
-
443
- mutex_unlock(&microcode_mutex);
444
- put_online_cpus();
445
-
446
- return ret;
447
-}
448
-
449
-static const struct file_operations microcode_fops = {
450
- .owner = THIS_MODULE,
451
- .write = microcode_write,
452
- .open = microcode_open,
453
- .llseek = no_llseek,
454
-};
455
-
456
-static struct miscdevice microcode_dev = {
457
- .minor = MICROCODE_MINOR,
458
- .name = "microcode",
459
- .nodename = "cpu/microcode",
460
- .fops = &microcode_fops,
461
-};
462
-
463
-static int __init microcode_dev_init(void)
464
-{
465
- int error;
466
-
467
- error = misc_register(&microcode_dev);
468
- if (error) {
469
- pr_err("can't misc_register on minor=%d\n", MICROCODE_MINOR);
470
- return error;
471
- }
472
-
473
- return 0;
474
-}
475
-
476
-static void __exit microcode_dev_exit(void)
477
-{
478
- misc_deregister(&microcode_dev);
479
-}
480
-#else
481
-#define microcode_dev_init() 0
482
-#define microcode_dev_exit() do { } while (0)
483
-#endif
484
-
485393 /* fake device for request_firmware */
486394 static struct platform_device *microcode_pdev;
487395
396
+#ifdef CONFIG_MICROCODE_LATE_LOADING
488397 /*
489398 * Late loading dance. Why the heavy-handed stomp_machine effort?
490399 *
....@@ -599,16 +508,27 @@
599508 */
600509 static int microcode_reload_late(void)
601510 {
602
- int ret;
511
+ int old = boot_cpu_data.microcode, ret;
512
+ struct cpuinfo_x86 prev_info;
603513
604514 atomic_set(&late_cpus_in, 0);
605515 atomic_set(&late_cpus_out, 0);
606516
607
- ret = stop_machine_cpuslocked(__reload_late, NULL, cpu_online_mask);
608
- if (ret == 0)
609
- microcode_check();
517
+ /*
518
+ * Take a snapshot before the microcode update in order to compare and
519
+ * check whether any bits changed after an update.
520
+ */
521
+ store_cpu_caps(&prev_info);
610522
611
- pr_info("Reload completed, microcode revision: 0x%x\n", boot_cpu_data.microcode);
523
+ ret = stop_machine_cpuslocked(__reload_late, NULL, cpu_online_mask);
524
+ if (!ret) {
525
+ pr_info("Reload succeeded, microcode revision: 0x%x -> 0x%x\n",
526
+ old, boot_cpu_data.microcode);
527
+ microcode_check(&prev_info);
528
+ } else {
529
+ pr_info("Reload failed, current microcode revision: 0x%x\n",
530
+ boot_cpu_data.microcode);
531
+ }
612532
613533 return ret;
614534 }
....@@ -629,7 +549,7 @@
629549 if (val != 1)
630550 return size;
631551
632
- get_online_cpus();
552
+ cpus_read_lock();
633553
634554 ret = check_online_cpus();
635555 if (ret)
....@@ -644,13 +564,16 @@
644564 mutex_unlock(&microcode_mutex);
645565
646566 put:
647
- put_online_cpus();
567
+ cpus_read_unlock();
648568
649569 if (ret == 0)
650570 ret = size;
651571
652572 return ret;
653573 }
574
+
575
+static DEVICE_ATTR_WO(reload);
576
+#endif
654577
655578 static ssize_t version_show(struct device *dev,
656579 struct device_attribute *attr, char *buf)
....@@ -668,7 +591,6 @@
668591 return sprintf(buf, "0x%x\n", uci->cpu_sig.pf);
669592 }
670593
671
-static DEVICE_ATTR_WO(reload);
672594 static DEVICE_ATTR(version, 0444, version_show, NULL);
673595 static DEVICE_ATTR(processor_flags, 0444, pf_show, NULL);
674596
....@@ -785,7 +707,7 @@
785707 if (uci->valid && uci->mc)
786708 microcode_ops->apply_microcode(cpu);
787709 else if (!uci->mc)
788
- reload_early_microcode();
710
+ reload_early_microcode(cpu);
789711 }
790712
791713 static struct syscore_ops mc_syscore_ops = {
....@@ -821,7 +743,9 @@
821743 }
822744
823745 static struct attribute *cpu_root_microcode_attrs[] = {
746
+#ifdef CONFIG_MICROCODE_LATE_LOADING
824747 &dev_attr_reload.attr,
748
+#endif
825749 NULL
826750 };
827751
....@@ -853,14 +777,14 @@
853777 if (IS_ERR(microcode_pdev))
854778 return PTR_ERR(microcode_pdev);
855779
856
- get_online_cpus();
780
+ cpus_read_lock();
857781 mutex_lock(&microcode_mutex);
858782
859783 error = subsys_interface_register(&mc_cpu_interface);
860784 if (!error)
861785 perf_check_microcode();
862786 mutex_unlock(&microcode_mutex);
863
- put_online_cpus();
787
+ cpus_read_unlock();
864788
865789 if (error)
866790 goto out_pdev;
....@@ -873,10 +797,6 @@
873797 goto out_driver;
874798 }
875799
876
- error = microcode_dev_init();
877
- if (error)
878
- goto out_ucode_group;
879
-
880800 register_syscore_ops(&mc_syscore_ops);
881801 cpuhp_setup_state_nocalls(CPUHP_AP_MICROCODE_LOADER, "x86/microcode:starting",
882802 mc_cpu_starting, NULL);
....@@ -887,18 +807,14 @@
887807
888808 return 0;
889809
890
- out_ucode_group:
891
- sysfs_remove_group(&cpu_subsys.dev_root->kobj,
892
- &cpu_root_microcode_group);
893
-
894810 out_driver:
895
- get_online_cpus();
811
+ cpus_read_lock();
896812 mutex_lock(&microcode_mutex);
897813
898814 subsys_interface_unregister(&mc_cpu_interface);
899815
900816 mutex_unlock(&microcode_mutex);
901
- put_online_cpus();
817
+ cpus_read_unlock();
902818
903819 out_pdev:
904820 platform_device_unregister(microcode_pdev);