Subrata Banik has posted comments on this change by Subrata Banik. ( https://review.coreboot.org/c/coreboot/+/83048?usp=email )
Change subject: libpayload: Refactor exception_state for 64-bit compatibility ......................................................................
Patch Set 1:
(1 comment)
File payloads/libpayload/arch/x86/exception.c:
https://review.coreboot.org/c/coreboot/+/83048/comment/c59d5878_823026c2?usp... : PS1, Line 173: #if CONFIG(LP_ARCH_X86_32)
FWIW, I'm fine excluding the registers from the print out if they're meaningless. We just need to make sure the data structure remains equal to what GDB expects.
sure, i will revive this Cl (just the printing part alone on top of the main CB:83036 Cl) later.
I have address this open along with https://review.coreboot.org/c/coreboot/+/83036/4..5/payloads/libpayload/arch... changes. As we don't need to display those segment registers, we may not need to fill those in as well hence, marked skipped inside the stack frame prep logic.