From nobody Sat May 4 03:44:23 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+78261+1787277+3901457@groups.io; helo=mail02.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+78261+1787277+3901457@groups.io ARC-Seal: i=1; a=rsa-sha256; t=1627438706; cv=none; d=zohomail.com; s=zohoarc; b=igi8d4PshBNoqjS3mEGoI3X6Zyt3t5D66EEpD8u6SxM2U2Saq/wNDdbfo9VlTdCrtFhMebnSFhTjcLiRkYEF2OHn+aMmbsPWlMJwRm2lUmfUXG4dsy/aBue13d8aJu05S7u73lQn+wvv0V0/5D6t7PX9Rmena8d60mHLH1sz7wA= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1627438706; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:References:Sender:Subject:To; bh=ivfxOP4psJ0gCmPN41Mq6qvl/QWj5/dYnX2o4hbjXDY=; b=l4zcPjHiQXtQIM+JmTM9yVnvA8LHcHNY5TbfzeFBDidI2rH+pXcCrZCuUGeWeYT3R6MZUHyffxG9qFuxpYeUmLK50FK700ublzqqznaJa+DtuOlLb6HlLonQawgIjWVlsYS7aTdyZIYM1aYqQm6M7aB6h7PiX6i1xVXiW4HREOk= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+78261+1787277+3901457@groups.io Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1627438706242516.5107980026442; Tue, 27 Jul 2021 19:18:26 -0700 (PDT) Return-Path: X-Received: by 127.0.0.2 with SMTP id cYwTYY1788612xgZeE4lGKD2; Tue, 27 Jul 2021 19:18:25 -0700 X-Received: from mail105.syd.optusnet.com.au (mail105.syd.optusnet.com.au [211.29.132.249]) by mx.groups.io with SMTP id smtpd.web10.1353.1627438705194857390 for ; Tue, 27 Jul 2021 19:18:25 -0700 X-Received: from d8.hgw.net.au (pa49-197-59-136.pa.qld.optusnet.com.au [49.197.59.136]) (Authenticated sender: chris.willing@optusnet.com.au) by mail105.syd.optusnet.com.au (Postfix) with ESMTPA id CD9E010453F8; Wed, 28 Jul 2021 12:18:23 +1000 (AEST) From: "Christoph Willing" To: devel@edk2.groups.io Cc: Christoph Willing Subject: [edk2-devel] [PATCH 1/1] OvmfPkg PlatformBootManagerLib: Move TryRunningQemuKernel() Date: Wed, 28 Jul 2021 12:02:33 +1000 Message-Id: <20210728020232.127332-2-chris.willing@linux.com> In-Reply-To: <20210728020232.127332-1-chris.willing@linux.com> References: <20210728020232.127332-1-chris.willing@linux.com> MIME-Version: 1.0 X-Optus-CM-Score: 0 X-Optus-CM-Analysis: v=2.3 cv=Tu+Yewfh c=1 sm=1 tr=0 a=Td1b0GxWWDVT73Tj8akgaQ==:117 a=Td1b0GxWWDVT73Tj8akgaQ==:17 a=hqBzw_eTAAAA:8 a=NufY4J3AAAAA:8 a=fqpOqWsNvOZOpF2C5gcA:9 a=bkWp_v3HvcftT6DRAIDL:22 a=TPcZfFuj8SYsoCJAFAiX:22 Precedence: Bulk List-Unsubscribe: List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,chris.willing@linux.com X-Gm-Message-State: vroi2bKIHWuSLK5uWF8rinA2x1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1627438705; bh=SeJRUmMIH78MXmD3d0ClYeO8l/mMzqeiDaO9QzY5umA=; h=Cc:Date:From:Reply-To:Subject:To; b=ZcII6h9Zf3/vq/JzmCex1Ofc0miEOeE6J8r/ksErtQovAPUIyqUSFKdcbF8bzM7S+56 zbpOuEHYCo0q2F1tHB5SQHHLRs3W4skn5JFMvWliFU7wsFnRrarvH0Txl/tvqzeYagxvA 05wjzG/+iAk1gOcPUDAhh73AHlNH8l+Ps8s= X-ZohoMail-DKIM: pass (identity @groups.io) X-ZM-MESSAGEID: 1627438707700100002 Content-Type: text/plain; charset="utf-8" Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3D3504 This change ensures that PlatformBdsConnectSequence() is run before TryRunningQemuKernel(). It reverses commit a34a886962561f6d8550b2a1bb193798ca456431 which aimed to improve UEFI boot time. Unfortunately that commit now results in boot failure when using qemu's -kernel option, therefore rendering the -initrd & -append options useless. Such failures are seen in all releases from edk2-stable201808 to edk2-stable202105 as well as the master branch. The last releases which actually work correctly are vUDK2017 and vUDK2018 as they don't contain the a34a88 commit. This change has been tested with edk2-stable201808, edk2-stable201811, edk2-stable202105 as well as master (at 21/7/2021). It results in correct booting using the -kernel option in all the test cases. No measurable change in boot times was discernible compared with the unaffected vUDK2017 & vUDK2018 releases. Signed-off-by: Christoph Willing --- OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c b/OvmfPkg= /Library/PlatformBootManagerLib/BdsPlatform.c index b0e9742937..f08ace87b3 100644 --- a/OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c +++ b/OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c @@ -1517,14 +1517,14 @@ PlatformBootManagerAfterConsole ( Tcg2PhysicalPresenceLibProcessRequest (NULL); =20 // - // Process QEMU's -kernel command line option + // Perform some platform specific connect sequence // - TryRunningQemuKernel (); + PlatformBdsConnectSequence (); =20 // - // Perform some platform specific connect sequence + // Process QEMU's -kernel command line option // - PlatformBdsConnectSequence (); + TryRunningQemuKernel (); =20 EfiBootManagerRefreshAllBootOption (); =20 --=20 2.32.0 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#78261): https://edk2.groups.io/g/devel/message/78261 Mute This Topic: https://groups.io/mt/84497355/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-