Attention is currently required from: Raul Rangel, Julius Werner, Rob Barnes, Patrick Rudolph, Karthik Ramasubramanian. Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59321 )
Change subject: treewide: Replace spinlock calls with mutex ......................................................................
Patch Set 5:
(6 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/59321/comment/f5399dd0_099e85ab PS5, Line 22: I think printk() is exactly where you would want to yield in romstage for DMA performance, and that was the only case where I had expected to see use of a mutex.
File src/cpu/intel/microcode/microcode.c:
https://review.coreboot.org/c/coreboot/+/59321/comment/91885047_71caf618 PS5, Line 14: struct mutex microcode_lock; static
File src/cpu/x86/lapic/lapic_cpu_init.c:
https://review.coreboot.org/c/coreboot/+/59321/comment/b9674204_8752875a PS5, Line 218: struct mutex start_cpu_lock; static
File src/cpu/x86/mp_init.c:
https://review.coreboot.org/c/coreboot/+/59321/comment/770dbf3e_a5bd440f PS5, Line 662: struct mutex smm_relocation_lock; static
File src/device/device.c:
https://review.coreboot.org/c/coreboot/+/59321/comment/20a4bfbc_f9a0feb7 PS5, Line 74: struct mutex dev_lock; static
File src/drivers/pc80/rtc/post.c:
https://review.coreboot.org/c/coreboot/+/59321/comment/7dca46ad_39f9855b PS5, Line 38: struct mutex cmos_post_lock; static