Yes. I think BootServiceData is ok. If OVMF is built without SMRAM support, it's ovmf platform's responsibility to get it and save it into ACPI NVS.
Jeff -----Original Message----- From: Paolo Bonzini [mailto:[email protected]] On Behalf Of Paolo Bonzini Sent: Tuesday, July 28, 2015 3:34 PM To: Fan, Jeff; Laszlo Ersek; [email protected] Cc: Chen Fan; Justen, Jordan L Subject: Re: [edk2] [PATCH 38/58] UefiCpuPkg: CpuDxe: optionally save MTRR settings to AcpiNVS memory block On 28/07/2015 09:09, Fan, Jeff wrote: > I did not receive the patch 42. I have only gotten 38,39,40,41. > > OK, If this mtrr setting stored in ACPI NVS is for CpuS3DataDxe to > store into SMRAM, that's fine. > > Then, another question, what's requirement to save MTRR setting into > ACPI NVS on this case? And need one PCD to switch on/off it? Do you mean it could be BootServicesData? I'll let Laszlo answer this. The PCD is there to skip this code if OVMF is built without SMRAM support. Paolo _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
