From nobody Tue May 7 02:21:11 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of groups.io designates 66.175.222.12 as permitted sender) client-ip=66.175.222.12; envelope-from=bounce+27952+40101+1787277+3901457@groups.io; helo=web01.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zoho.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+40101+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=intel.com ARC-Seal: i=1; a=rsa-sha256; t=1557223735; cv=none; d=zoho.com; s=zohoarc; b=TIL4uvWnbkuoSdWFZMR+CPOD2PkesQA+wFXtl+uIZtxR+ZIDQWDHENGjZ15MICt3g7iNahhWm1OrSqsIjbgTUZqTZxETv4HMSR0YfmiiGgL3fbQmFmwEAO2ZTxIpqH2rDJi8g/gYG56+1oD9UEkW5B3RqM+zxOyY+LyeUOFuWQs= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1557223735; h=Cc:Date:From:In-Reply-To:List-Id:List-Unsubscribe:Message-ID:Reply-To:References:Sender:Subject:To:ARC-Authentication-Results; bh=1Rr24k4MXzeZ8UzFqQZwBbT+uaTfRnKZbkrY0Dftfns=; b=UUiYJZ9yKI7yPwYl5jgzKyFCX5FQofaL9HKUi6n5xjW3ZA4qyHGqdFAr4UB2x0rrtuumto3FsIoQ45g8No/sv+HA4H0IOcIj0OCxFcEjNzPQIhdSPHQzEViojs5/FWRfYUDgmtAYKaEqAzfEFqBgtNe4Idy6QJqvpShxBwj32wE= ARC-Authentication-Results: i=1; mx.zoho.com; dkim=pass; spf=pass (zoho.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+40101+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) header.from= Received: from web01.groups.io (web01.groups.io [66.175.222.12]) by mx.zohomail.com with SMTPS id 1557223735740236.716613795395; Tue, 7 May 2019 03:08:55 -0700 (PDT) Return-Path: X-Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by groups.io with SMTP; Tue, 07 May 2019 03:08:54 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 07 May 2019 03:08:54 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.60,441,1549958400"; d="scan'208";a="169040158" X-Received: from shwdeopenpsi174.ccr.corp.intel.com ([10.239.157.33]) by fmsmga002.fm.intel.com with ESMTP; 07 May 2019 03:08:53 -0700 From: "Xu, Wei6" To: devel@edk2.groups.io Cc: Ni Ray , Wu Hao A , Xu Wei6 Subject: [edk2-devel][Patch 1/2] FatPkg/FatPei: Remove BootInRecoveryMode PPI DEPEX. Date: Tue, 7 May 2019 18:08:13 +0800 Message-Id: <20190507100814.18236-2-wei6.xu@intel.com> In-Reply-To: <20190507100814.18236-1-wei6.xu@intel.com> References: <20190507100814.18236-1-wei6.xu@intel.com> Precedence: Bulk List-Unsubscribe: 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,wei6.xu@intel.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1557223735; bh=MiAKTClKZ2Az7l0ahi1f8woS5v7+myOEtIcD2FNHKjQ=; h=Cc:Date:From:Reply-To:Subject:To; b=wyOGytMmS1YQzpaOVxmPLxOqHNSt8nhjke87t5Zo1ht2lP+QfdEikjDq4QMMMZaRsU/ JuAiCZturClhtY26t8DI+a1GpPqGVWXxi6QtPyCD4VMNZ0KZP6u09upB7TXVUwVPRW5Ax LXX9vw4a1zaGY5e2ATUqGSmHuoqeMEK/ayY= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D1777 BootInRecoveryMode Ppi dependency should be removed from PEI storage stack = drivers. Besides recovery, more features are using the storage stack. Platform can apply Ppi dependency in Recovery FV which contains full storag= e driver stack. Platform storage stack modules, such as host controller init PEI module, sh= ould also remove this dependency. CC: Ni Ray CC: Wu Hao A Signed-off-by: Xu Wei6 Reviewed-by =EF=BC=9A Chao Zhang Reviewed-by: Hao A Wu --- FatPkg/FatPei/FatPei.inf | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/FatPkg/FatPei/FatPei.inf b/FatPkg/FatPei/FatPei.inf index b554aae040..4560dbaab4 100644 --- a/FatPkg/FatPei/FatPei.inf +++ b/FatPkg/FatPei/FatPei.inf @@ -71,9 +71,9 @@ =20 [Pcd] gEfiMdeModulePkgTokenSpaceGuid.PcdRecoveryFileName ## CONSUMES =20 [Depex] - gEfiPeiMemoryDiscoveredPpiGuid AND gEfiPeiBootInRecoveryModePpiGuid + gEfiPeiMemoryDiscoveredPpiGuid =20 [UserExtensions.TianoCore."ExtraFiles"] FatPeiExtra.uni --=20 2.16.2.windows.1 -=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 (#40101): https://edk2.groups.io/g/devel/message/40101 Mute This Topic: https://groups.io/mt/31530232/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- From nobody Tue May 7 02:21:11 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of groups.io designates 66.175.222.12 as permitted sender) client-ip=66.175.222.12; envelope-from=bounce+27952+40102+1787277+3901457@groups.io; helo=web01.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zoho.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+40102+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=intel.com ARC-Seal: i=1; a=rsa-sha256; t=1557223738; cv=none; d=zoho.com; s=zohoarc; b=J/Ic41h5tPpwbuCc8GOiwSc4hzqnY5jM4rPLF5pGyL+fdnR7NO1H1F1MCmK7x5AUvFzOytl38DxBnuoNWUOmGeZegSfrxPiDok5DKlHhW4JP+USu3D3FUXaML9c+7Ly3j/yk1+adC6CfhMluIDs2Xn50xTUXxOTMQA0G4gD+ayw= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1557223738; h=Cc:Date:From:In-Reply-To:List-Id:List-Unsubscribe:Message-ID:Reply-To:References:Sender:Subject:To:ARC-Authentication-Results; bh=IrzDy8QxLPXwTHGOy/DHMhxLtXRlyRNonL9ZawXa4zQ=; b=co0ZEViBzPxqy74kcp9QYVgylDuo1NqXyQhTG3ewjNBowTQU6HNZXp9AQW7caVmaUq1/m8da8x3ejp4swLehbLuto5mrSil9vzbLg76LbWUZvJyiqZu2xAuK1A3J/Rct5hPJ7ixursKyNviLGhBEFSM3YWYcPryCsNMoj5e3yeg= ARC-Authentication-Results: i=1; mx.zoho.com; dkim=pass; spf=pass (zoho.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+40102+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) header.from= Received: from web01.groups.io (web01.groups.io [66.175.222.12]) by mx.zohomail.com with SMTPS id 1557223738211532.3076325746667; Tue, 7 May 2019 03:08:58 -0700 (PDT) Return-Path: X-Received: from mga01.intel.com (mga01.intel.com []) by groups.io with SMTP; Tue, 07 May 2019 03:08:57 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 07 May 2019 03:08:57 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.60,441,1549958400"; d="scan'208";a="169040186" X-Received: from shwdeopenpsi174.ccr.corp.intel.com ([10.239.157.33]) by fmsmga002.fm.intel.com with ESMTP; 07 May 2019 03:08:56 -0700 From: "Xu, Wei6" To: devel@edk2.groups.io Cc: Ni Ray , Wu Hao A , Xu Wei6 Subject: [edk2-devel][Patch 2/2] MdeModulePkg: Remove BootInRecoveryMode PPI DEPEX from PEI storage stack. Date: Tue, 7 May 2019 18:08:14 +0800 Message-Id: <20190507100814.18236-3-wei6.xu@intel.com> In-Reply-To: <20190507100814.18236-1-wei6.xu@intel.com> References: <20190507100814.18236-1-wei6.xu@intel.com> Precedence: Bulk List-Unsubscribe: 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,wei6.xu@intel.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1557223737; bh=VTJwZ2nDUNbDWe3skhFYFg2FKl2+PdmPgT75WHZjyVY=; h=Cc:Date:From:Reply-To:Subject:To; b=Cpk1+awDDFUCT0mnImIjt1Gq/VC0AoAvhLMXUlvVdOi1JIm8G2YLfCPmtx34dpC/BDn LuTMGLQkl5uvtOZlMWmouOp9geowoOZiuFoPPw2yn0QMnAWsy6F8NzsQFzoFah8uHIoXd IFBOgxfQK76hijGsLRQPW8kb2Quqwcl6fc4= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D1777 BootInRecoveryMode Ppi dependency should be removed from PEI storage stack = drivers. Besides recovery, more features are using the storage stack. Platform can apply Ppi dependency in Recovery FV which contains full storag= e driver stack. Platform storage stack modules, such as host controller init PEI module, sh= ould also remove this dependency. CC: Ni Ray CC: Wu Hao A Signed-off-by: Xu Wei6 Reviewed-by =EF=BC=9A Chao Zhang Reviewed-by: Hao A Wu --- MdeModulePkg/Bus/Pci/EhciPei/EhciPei.inf | 2 +- MdeModulePkg/Bus/Pci/IdeBusPei/IdeBusPei.inf | 2 +- MdeModulePkg/Bus/Pci/UhciPei/UhciPei.inf | 2 +- MdeModulePkg/Bus/Pci/XhciPei/XhciPei.inf | 2 +- MdeModulePkg/Bus/Usb/UsbBotPei/UsbBotPei.inf | 2 +- MdeModulePkg/Bus/Usb/UsbBusPei/UsbBusPei.inf | 2 +- MdeModulePkg/Universal/Disk/CdExpressPei/CdExpressPei.inf | 2 +- 7 files changed, 7 insertions(+), 7 deletions(-) diff --git a/MdeModulePkg/Bus/Pci/EhciPei/EhciPei.inf b/MdeModulePkg/Bus/Pc= i/EhciPei/EhciPei.inf index a810c3c6bc..0fc09ffca4 100644 --- a/MdeModulePkg/Bus/Pci/EhciPei/EhciPei.inf +++ b/MdeModulePkg/Bus/Pci/EhciPei/EhciPei.inf @@ -57,9 +57,9 @@ gPeiUsbControllerPpiGuid ## CONSUMES gEdkiiIoMmuPpiGuid ## CONSUMES gEfiEndOfPeiSignalPpiGuid ## CONSUMES =20 [Depex] - gEfiPeiMemoryDiscoveredPpiGuid AND gPeiUsbControllerPpiGuid AND gEfiPeiB= ootInRecoveryModePpiGuid + gEfiPeiMemoryDiscoveredPpiGuid AND gPeiUsbControllerPpiGuid =20 [UserExtensions.TianoCore."ExtraFiles"] EhciPeiExtra.uni diff --git a/MdeModulePkg/Bus/Pci/IdeBusPei/IdeBusPei.inf b/MdeModulePkg/Bu= s/Pci/IdeBusPei/IdeBusPei.inf index 074f12586c..ad50d4c0c9 100644 --- a/MdeModulePkg/Bus/Pci/IdeBusPei/IdeBusPei.inf +++ b/MdeModulePkg/Bus/Pci/IdeBusPei/IdeBusPei.inf @@ -54,9 +54,9 @@ =20 [Pcd] gEfiMdeModulePkgTokenSpaceGuid.PcdSataSpinUpDelayInSecForRecoveryPath = ## CONSUMES =20 [Depex] - gEfiPeiMemoryDiscoveredPpiGuid AND gEfiPeiBootInRecoveryModePpiGuid AND = gPeiAtaControllerPpiGuid + gEfiPeiMemoryDiscoveredPpiGuid AND gPeiAtaControllerPpiGuid =20 [UserExtensions.TianoCore."ExtraFiles"] IdeBusPeiExtra.uni diff --git a/MdeModulePkg/Bus/Pci/UhciPei/UhciPei.inf b/MdeModulePkg/Bus/Pc= i/UhciPei/UhciPei.inf index accb0e2460..06515b7ca7 100644 --- a/MdeModulePkg/Bus/Pci/UhciPei/UhciPei.inf +++ b/MdeModulePkg/Bus/Pci/UhciPei/UhciPei.inf @@ -52,9 +52,9 @@ gEdkiiIoMmuPpiGuid ## CONSUMES gEfiEndOfPeiSignalPpiGuid ## CONSUMES =20 =20 [Depex] - gEfiPeiMemoryDiscoveredPpiGuid AND gPeiUsbControllerPpiGuid AND gEfiPeiB= ootInRecoveryModePpiGuid + gEfiPeiMemoryDiscoveredPpiGuid AND gPeiUsbControllerPpiGuid =20 [UserExtensions.TianoCore."ExtraFiles"] UhciPeiExtra.uni diff --git a/MdeModulePkg/Bus/Pci/XhciPei/XhciPei.inf b/MdeModulePkg/Bus/Pc= i/XhciPei/XhciPei.inf index 89bb28ee53..f216538f4d 100644 --- a/MdeModulePkg/Bus/Pci/XhciPei/XhciPei.inf +++ b/MdeModulePkg/Bus/Pci/XhciPei/XhciPei.inf @@ -52,9 +52,9 @@ gPeiUsbControllerPpiGuid ## CONSUMES gEdkiiIoMmuPpiGuid ## CONSUMES gEfiEndOfPeiSignalPpiGuid ## CONSUMES =20 [Depex] - gEfiPeiMemoryDiscoveredPpiGuid AND gPeiUsbControllerPpiGuid AND gEfiPeiB= ootInRecoveryModePpiGuid + gEfiPeiMemoryDiscoveredPpiGuid AND gPeiUsbControllerPpiGuid =20 [UserExtensions.TianoCore."ExtraFiles"] XhciPeiExtra.uni diff --git a/MdeModulePkg/Bus/Usb/UsbBotPei/UsbBotPei.inf b/MdeModulePkg/Bu= s/Usb/UsbBotPei/UsbBotPei.inf index f9f44a883e..de2df5fe3c 100644 --- a/MdeModulePkg/Bus/Usb/UsbBotPei/UsbBotPei.inf +++ b/MdeModulePkg/Bus/Usb/UsbBotPei/UsbBotPei.inf @@ -54,9 +54,9 @@ ## CONSUMES ## NOTIFY gPeiUsbIoPpiGuid =20 [Depex] - gEfiPeiMemoryDiscoveredPpiGuid AND gPeiUsbIoPpiGuid AND gEfiPeiBootInRec= overyModePpiGuid + gEfiPeiMemoryDiscoveredPpiGuid AND gPeiUsbIoPpiGuid =20 [UserExtensions.TianoCore."ExtraFiles"] UsbBotPeiExtra.uni diff --git a/MdeModulePkg/Bus/Usb/UsbBusPei/UsbBusPei.inf b/MdeModulePkg/Bu= s/Usb/UsbBusPei/UsbBusPei.inf index 617aa3bb10..31d7c2e2fe 100644 --- a/MdeModulePkg/Bus/Usb/UsbBusPei/UsbBusPei.inf +++ b/MdeModulePkg/Bus/Usb/UsbBusPei/UsbBusPei.inf @@ -52,9 +52,9 @@ gPeiUsbIoPpiGuid ## PRODUCES gPeiUsbHostControllerPpiGuid ## SOMETIMES_CONSUMES gPeiUsb2HostControllerPpiGuid ## SOMETIMES_CONSUMES =20 [Depex] - gEfiPeiMemoryDiscoveredPpiGuid AND gEfiPeiBootInRecoveryModePpiGuid AND = gPeiUsb2HostControllerPpiGuid OR gPeiUsbHostControllerPpiGuid + gEfiPeiMemoryDiscoveredPpiGuid AND gPeiUsb2HostControllerPpiGuid OR gPei= UsbHostControllerPpiGuid =20 [UserExtensions.TianoCore."ExtraFiles"] UsbBusPeiExtra.uni diff --git a/MdeModulePkg/Universal/Disk/CdExpressPei/CdExpressPei.inf b/Md= eModulePkg/Universal/Disk/CdExpressPei/CdExpressPei.inf index 11ab49b33b..6a8f3c2da8 100644 --- a/MdeModulePkg/Universal/Disk/CdExpressPei/CdExpressPei.inf +++ b/MdeModulePkg/Universal/Disk/CdExpressPei/CdExpressPei.inf @@ -63,9 +63,9 @@ =20 [Pcd] gEfiMdeModulePkgTokenSpaceGuid.PcdRecoveryFileName ## CONSUMES =20 [Depex] - gEfiPeiMemoryDiscoveredPpiGuid AND gEfiPeiBootInRecoveryModePpiGuid + gEfiPeiMemoryDiscoveredPpiGuid =20 [UserExtensions.TianoCore."ExtraFiles"] CdExpressPeiExtra.uni --=20 2.16.2.windows.1 -=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 (#40102): https://edk2.groups.io/g/devel/message/40102 Mute This Topic: https://groups.io/mt/31530233/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-