[S] Change in coreboot[main]: cpu/x86/smm: Pass full SMRAM region info to SMM runtime

[Benjamin Doron (Code Review)]

Attention is currently required from: Angel Pons, Arthur Heymans, Christian Walter, Felix Held, Fred Reitberger, Jason Glenesk, Jeff Daly, Johnny Lin, Jérémy Compostella, Lean Sheng Tan, Matt DeVillier, Shuo Liu, Tim Chu, Vanessa Eusebio.

Benjamin Doron has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80703?usp=email )

Change subject: cpu/x86/smm: Pass full SMRAM region info to SMM runtime
......................................................................

Patch Set 3:

(2 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/80703/comment/0e10e1f9_e571f28e :
PS2, Line 7: cpu/x86/smm/smm_module_loader.c: Pass full SMRAM region info to SMM runtime
> I would suggest something like: […]

Okay, sounds good.

https://review.coreboot.org/c/coreboot/+/80703/comment/91a3e0b2_0dcf0981 :
PS2, Line 17: integrity of the S3 resume path.
> Sorry, I mean the current smm_region_overlaps_handler() only provides SMRAM, but not staging cache a […]

Ah, I see what you're talking about. The stage cache and chipset area are all subregions of SMRAM, which also contains one more area: the handler. You can see how this calculated in tseg_region.c of this folder. The objective of this patch is to cover every subregion of SMRAM, not just the handler region.