OVMF BDS: Don't call BdsLibSaveMemoryTypeInformation

This call can cause a reset, and is most critical for ACPI S3/S4
resume situations.  OVMF does not support S3/S4.

OVMF does not have true non-volatile variable support, so
this call could cause a continuous reset situation in certain
scenarios.  (The BdsLibSaveMemoryTypeInformation may set an
non-volatile variable, and then reset with the assumption that
the variable will still exist during the next boot.)

Additionally, some version of QEMU appear to hang when the
port 64 reset is initiated.

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@10927 6f19259b-4bc3-4df7-8a09-765794883524

diff --git a/OvmfPkg/Library/PlatformBdsLib/BdsPlatform.c b/OvmfPkg/Library/PlatformBdsLib/BdsPlatform.c
index cce0dde4ec64e3ec860e079beb429184d42a9bcf..b0d045df9b5d2cb0cf59002278bd0343ba123bf5 100644 (file)
--- a/OvmfPkg/Library/PlatformBdsLib/BdsPlatform.c
+++ b/OvmfPkg/Library/PlatformBdsLib/BdsPlatform.c
@@ -110,7 +110,6 @@ Returns:
 --*/\r
 {\r
   DEBUG ((EFI_D_INFO, "PlatformBdsInit\n"));\r
-  BdsLibSaveMemoryTypeInformation ();\r
   InstallDevicePathCallback ();\r
   LoadVideoRom ();\r
 }\r
@@ -884,7 +883,7 @@ PlatformBdsRestoreNvVarsFromHardDisk (
     VisitingFileSystemInstance,\r
     NULL\r
     );\r
-  \r
+\r
 }\r
 \r
 \r