On Tue, Apr 01, 2025 at 09:01:29AM -0400, Xiaoyao Li wrote: > Date: Tue, 1 Apr 2025 09:01:29 -0400 > From: Xiaoyao Li <xiaoyao...@intel.com> > Subject: [PATCH v8 19/55] i386/tdx: Don't initialize pc.rom for TDX VMs > X-Mailer: git-send-email 2.34.1 > > For TDX, the address below 1MB are entirely general RAM. No need to > initialize pc.rom memory region for TDs. > > Signed-off-by: Xiaoyao Li <xiaoyao...@intel.com> > --- > This is more as a workaround of the issue that for q35 machine type, the > real memslot update (which requires memslot deletion )for pc.rom happens > after tdx_init_memory_region. It leads to the private memory ADD'ed > before get lost. I haven't work out a good solution to resolve the > order issue. So just skip the pc.rom setup to avoid memslot deletion. > --- > hw/i386/pc.c | 29 ++++++++++++++++------------- > 1 file changed, 16 insertions(+), 13 deletions(-)
Fine for me, Reviewed-by: Zhao Liu <zhao1....@intel.com>
