Attention is currently required from: Arthur Heymans, Philipp Hug, Ron Minnich.
Maximilian Brune has posted comments on this change by Maximilian Brune. ( https://review.coreboot.org/c/coreboot/+/83056?usp=email )
Change subject: arch/riscv/pmp: Add print macro ......................................................................
Patch Set 2:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/83056/comment/9960e47c_f5eb6d76?usp... : PS2, Line 9: This adds a printk macro in the PMP code to avoid all harts writing onto : the serial which causes crappy/broken output. :
What is the problem exactly? Are spinlocks broken?
The problem is that currently all harts/cores are printing that information on the console. But we only need it to be printed by one of them (at least in my opinion since all harts/cores are configured the same). So instead of synchronizing it and let it be printed by all of them I chose to only let it be printed by the working hart (aka BSP).