Neat strategy to use the early stack setup to check the later stuff. The stack is within the safe area, right?
2 comments:
File src/cpu/x86/smm/sinkhole.c:
Patch Set #1, Line 31: printk(BIOS_EMERG, "Wrong LAPIC base detected! dying...\n");
hmm do we really want to link console code into the stub?
We don't need to be too friendly about this as there's really no reason for anybody to do this legitimately: Do weird stuff, get hurt. I'm fine with that.
Because of that I'd go with the crash code we already have. Maybe add a symbol there (added a comment at that place) and use that to jump there from here?
File src/cpu/x86/smm/smm_stub.S:
I'd just jump here from the later test in mitigate sinkhole.
To view, visit change 37289. To unsubscribe, or for help writing mail filters, visit settings.