[coreboot-gerrit] Change in coreboot[master]: cpu/x86: Put guard around align for smm_save_state_size

[Name of user not set (Code Review)]

[e-mail] has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38734 )

Change subject: cpu/x86: Put guard around align for smm_save_state_size
......................................................................

Patch Set 3:

(2 comments)

responeded to Nico Huber's comments.

https://review.coreboot.org/c/coreboot/+/38734/3/src/cpu/x86/mp_init.c 
File src/cpu/x86/mp_init.c:

PS3:
> So is this an Intel-only tech? If so, the changes to this file should be […]

Yes, this is intel-only.

I see your point, but the change will involved multiple files (one for each family).  I can do that  as a part of the solution for the issue below.

https://review.coreboot.org/c/coreboot/+/38734/3/src/cpu/x86/mp_init.c@1056 
PS3, Line 1056: 		 * value of 0x400 for this value.
> You have to be careful what code you read. smmhandler.S, smmhandler.c, […]

Thanks for the opinion and suggestion.

I've been looking this morning's issue, in particular that >4 processors breaks this.  So, your suggestions are helpful.