[coreboot-gerrit] Change in coreboot[master]: cpu/x86/smm: Use struct region to check overlapping sections

[Arthur Heymans (Code Review)]

Attention is currently required from: Paul Menzel, Arthur Heymans, Tim Wawrzynczak.
Hello build bot (Jenkins), Tim Wawrzynczak,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/63602

to look at the new patch set (#20).

Change subject: cpu/x86/smm: Use struct region to check overlapping sections
......................................................................

cpu/x86/smm: Use struct region to check overlapping sections

This allows for some runtime checks on all SMM elements and removes
the need for manual checks.

We can drop completely separate codepaths on SMM_TSEG & SMM_ASEG as the
only difference is where permanent handler gets placed.

TESTED on prodrive/hermes and qemu with SSM_ASEG with 4 cores & SMM_TSEG
with 128 cores. This code figured out quite some problems with
overlapping regions so I think this is the right approach.

Change-Id: Ib7e2e3ae16c223ecfd8d5bce6ff6c17c53496925
Signed-off-by: Arthur Heymans 
---
M src/cpu/x86/smm/smm_module_loader.c
1 file changed, 122 insertions(+), 152 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/02/63602/20