File ovmf-bsc976253-postpone-shell.patch of Package ovmf

Overview Repositories Revisions Requests Users Attributes Meta

File ovmf-bsc976253-postpone-shell.patch of Package ovmf (Revision 55fd87b6d2f9db33eb7d65eaf09a0157)

Currently displaying revision 55fd87b6d2f9db33eb7d65eaf09a0157 , Show latest

From f84c6d67ccc846f20b03fbe46e5802a82c66d5be Mon Sep 17 00:00:00 2001
From: Gary Lin <glin@suse.com>
Date: Tue, 10 May 2016 11:18:34 +0800
Subject: [PATCH] OvmfPkg/PlatformBootManagerLib: Postpone the shell
 registration

We currently register the shell before creating the boot options for
the block devices and the network devices, so the boot manager boots
into the internal shell if the user doesn't specify the boot order.
However, Xen doesn't support fw_cfg, so there is no way to change the
boot order with the external command, and the firmware will always
boot into the internal shell if the user doesn't interfere the boot
process.

This patch postpones the shell registration after MdeModulePkg/BDS
creates all the boot options for the block and network devices, so
that firmware will try to boot the block/network devices first.

Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Jordan Justen <jordan.l.justen@intel.com>
Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Gary Lin <glin@suse.com>
---
 OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c b/OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c
index cf774a1..a16453d 100644
--- a/OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c
+++ b/OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c
@@ -184,12 +184,6 @@ PlatformRegisterOptionsAndKeys (
              NULL, (UINT16) BootOption.OptionNumber, 0, &Esc, NULL
              );
   ASSERT (Status == EFI_SUCCESS || Status == EFI_ALREADY_STARTED);
-  //
-  // Register UEFI Shell
-  //
-  PlatformRegisterFvBootOption (
-    PcdGetPtr (PcdShellFile), L"EFI Internal Shell", LOAD_OPTION_ACTIVE
-    );
 }
 
 EFI_STATUS
@@ -1304,6 +1298,13 @@ Routine Description:
 
   EfiBootManagerRefreshAllBootOption ();
 
+  //
+  // Register UEFI Shell
+  //
+  PlatformRegisterFvBootOption (
+    PcdGetPtr (PcdShellFile), L"EFI Internal Shell", LOAD_OPTION_ACTIVE
+    );
+
   SetBootOrderFromQemu (NULL);
 }
 
-- 
2.8.2

Places

File ovmf-bsc976253-postpone-shell.patch of Package ovmf (Revision 55fd87b6d2f9db33eb7d65eaf09a0157)

Places