Attention is currently required from: Felix Held.
Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/77171?usp=email )
Change subject: [RFC] drivers/amd/agesa/state_machine: disable NULL breakpoint for binaryPI ......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1: Maybe we introduce a new version of the breakpoint message that puts out an info level message saying that there's a null dereference in a blob instead of disabling it completely? It seems somewhat questionable to me that any of the blobs would need to access the NULL page.
Even if we do go this route, maybe we want to just disable the deref breakpoint, not the instruction breakpoint? That would require updating code in null_breakpoint.c.