[L] Change in coreboot[main]: mb/qemu-q35/smihandler.c: add support for SMIs on QEMU

[Krystian Hebel (Code Review)]

Attention is currently required from: Arthur Heymans, Michał Kopeć, Michał Żygowski, Patrick Rudolph, Paul Menzel, Sergii Dmytruk.

Krystian Hebel has posted comments on this change by Krystian Hebel. ( https://review.coreboot.org/c/coreboot/+/82558?usp=email )

Change subject: mb/qemu-q35/smihandler.c: add support for SMIs on QEMU
......................................................................


Patch Set 7:

(1 comment)

Patchset:

PS7:
> https://review.coreboot. […]
> Unfortunately no, not for QEMU. It would still be an exception due to differences mentioned in first comment block in smihandler.c added here. It could be worked around by using different revisions, but then again it would mean that some board specific workarounds have to be placed in the common code.

I tried to bring those commits back, but new SMM handlers were added after those commits that would need to be aligned (AMD PSP communication). Since this wasn't a trivial change and it didn't fix QEMU issues I dropped that idea.

That single commit could be brought back after adding separate save state ops for QEMU, basically this change. On the other hand, we already have cpu/qemu-x86 which would keep the common code relatively clean. It would still need to differentiate between q35 and i440fx, though, as only the former has SMM. If you think this is acceptable, I can bring back that one change, but I won't be able to find the time to update the rest of relation chain, nor have a way of testing it on different platforms.



-- 
To view, visit https://review.coreboot.org/c/coreboot/+/82558?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I2fc7b74bdc13be8d76bc536283ab5a14fffec45f
Gerrit-Change-Number: 82558
Gerrit-PatchSet: 7
Gerrit-Owner: Krystian Hebel <[e-mail]>
Gerrit-Reviewer: Michał Kopeć <[e-mail]>
Gerrit-Reviewer: Michał Żygowski <[e-mail]>
Gerrit-Reviewer: Sergii Dmytruk <[e-mail]>
Gerrit-Reviewer: build bot (Jenkins) <[e-mail]>
Gerrit-CC: Arthur Heymans <[e-mail]>
Gerrit-CC: Patrick Rudolph <[e-mail]>
Gerrit-CC: Paul Menzel <[e-mail]>
Gerrit-Attention: Michał Żygowski <[e-mail]>
Gerrit-Attention: Patrick Rudolph <[e-mail]>
Gerrit-Attention: Paul Menzel <[e-mail]>
Gerrit-Attention: Michał Kopeć <[e-mail]>
Gerrit-Attention: Arthur Heymans <[e-mail]>
Gerrit-Attention: Sergii Dmytruk <[e-mail]>
Gerrit-Comment-Date: Thu, 13 Jun 2024 10:41:16 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Arthur Heymans <[e-mail]>