On Thu, Dec 11, 2025 at 10:24:35AM +0100, Oliver Steffen wrote:
On Thu, Dec 11, 2025 at 9:46 AM Stefano Garzarella <[email protected]> wrote:
On Thu, Dec 11, 2025 at 09:15:17AM +0100, Oliver Steffen wrote:
[...]
>diff --git a/target/i386/sev.c b/target/i386/sev.c
>index fd2dada013..ffeb9f52a2 100644
>--- a/target/i386/sev.c
>+++ b/target/i386/sev.c
>@@ -1892,7 +1892,7 @@ static int sev_common_kvm_init(ConfidentialGuestSupport
*cgs, Error **errp)
> */
> if (x86machine->igvm) {
> if (IGVM_CFG_GET_CLASS(x86machine->igvm)
>- ->process(x86machine->igvm, machine->cgs, true, errp) ==
>+ ->process(x86machine->igvm, machine->cgs, true, NULL,
errp) ==
Why here we don't need to pass it?
Here we only read the IGVM to figure out the initial vcpu configuration
(the `onlyVpContext` parameter is true) to initialize kvm,
The actual IGVM processing is done later.
okay, I see, thanks!
Should I mention in the comment above why madt is NULL here ?
Yes, please :-)
Stefano
