Attention is currently required from: Angel Pons, Benjamin Doron, Christian Walter, Lean Sheng Tan, Leon Groß, Patrick Rudolph.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/77905?usp=email )
Change subject: mb/emulation: Add SIMICS QSP support
......................................................................
Patch Set 13:
(1 comment)
Patchset:
PS4:
I'm not sure if I follow. You expect current users of Simics to adopt this?
Currently, with QEMU it's as simple as running `make qemu`. Are there plans
to make Simics as easy to use? if not, wouldn't it be better to teach people
about QEMU?
i mean it is no harm to add more support to the platform for coreboot i guess?
Every addition also adds at least some maintenance overhead. I've often
seen redundant additions to be abandoned and bit rot.
It is a nice addition as well. As it does have specific use cases (although quite limited): as Intel or silicon vendors usually use Simics for the RTL simulation before the actual silicon is available, actively used by the internal engineers for that.
Do they use QSP?
EDK2 and Slimbootloader already have nice support for Simics:
https://slimbootloader.github.io/supported-hardware/qsp.html
https://github.com/tianocore/edk2-platforms/tree/master/Platform/Intel/Simic...
And others have support for other open-source emulators, e.g. Bochs.
Still we don't add those, why?
--
To view, visit
https://review.coreboot.org/c/coreboot/+/77905?usp=email
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I175b20bb4746608e4d868aa96492fc06c149bd36
Gerrit-Change-Number: 77905
Gerrit-PatchSet: 13
Gerrit-Owner: Leon Groß
leon.gross@9elements.com
Gerrit-Reviewer: Angel Pons
th3fanbus@gmail.com
Gerrit-Reviewer: Arthur Heymans
arthur@aheymans.xyz
Gerrit-Reviewer: Benjamin Doron
benjamin.doron00@gmail.com
Gerrit-Reviewer: Christian Walter
christian.walter@9elements.com
Gerrit-Reviewer: Lean Sheng Tan
sheng.tan@9elements.com
Gerrit-Reviewer: Patrick Rudolph
patrick.rudolph@9elements.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Nico Huber
nico.h@gmx.de
Gerrit-CC: Paul Menzel
paulepanter@mailbox.org
Gerrit-Attention: Patrick Rudolph
patrick.rudolph@9elements.com
Gerrit-Attention: Benjamin Doron
benjamin.doron00@gmail.com
Gerrit-Attention: Christian Walter
christian.walter@9elements.com
Gerrit-Attention: Leon Groß
leon.gross@9elements.com
Gerrit-Attention: Angel Pons
th3fanbus@gmail.com
Gerrit-Attention: Lean Sheng Tan
sheng.tan@9elements.com
Gerrit-Comment-Date: Tue, 31 Oct 2023 16:36:37 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Nico Huber
nico.h@gmx.de
Comment-In-Reply-To: Patrick Rudolph
patrick.rudolph@9elements.com
Comment-In-Reply-To: Lean Sheng Tan
sheng.tan@9elements.com
Gerrit-MessageType: comment