Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/36986 )
Change subject: mysteries_intel: Add a section about SMM_BWP ......................................................................
Patch Set 2: Code-Review+2
(1 comment)
Maybe mention that the "protection" is that a SMI gets triggered when trying to change the write protection status, and the handler just enables it again. I think it worked like that.
What you describe is actually an older mechanism that predates SMM_BWP. The latter became necessary because there was a race on SMP systems where one thread could write while the other was re-enabling the write protection in SMM.
https://review.coreboot.org/c/flashrom/+/36986/2/Documentation/mysteries_int... File Documentation/mysteries_intel.txt:
https://review.coreboot.org/c/flashrom/+/36986/2/Documentation/mysteries_int... PS2, Line 17: Flashrom
"flashrom" should be in lowercase, like on line 32
I'm not so sure...