From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=104.47.1.78; helo=eur01-ve1-obe.outbound.protection.outlook.com; envelope-from=alexei.fedorov@arm.com; receiver=edk2-devel@lists.01.org Received: from EUR01-VE1-obe.outbound.protection.outlook.com (mail-ve1eur01on0078.outbound.protection.outlook.com [104.47.1.78]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 0FF54202E6113 for ; Tue, 17 Oct 2017 01:37:30 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=armh.onmicrosoft.com; s=selector1-arm-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=IwpXzueUMD/TtEL60T2nTwJ3aouQJzYyCffaExCyQIM=; b=ArQ7g+qpsxClStIV0+46tUHjv2gzJsLKuCN4p7V9PZ8wFxu+aoyFHsn470IGZ7TINy9Fq5jsf6WAnCBL6VcjilqKyoOdnM3cfLw27hC2xiRQgVM1LN9t/4gZzV4CJWQqnMf5nRZbP5XXsBIesSs8pOyBRg85ERtmgMVPh03r1+0= Received: from DB5PR08MB1014.eurprd08.prod.outlook.com (10.166.14.11) by VI1PR0801MB1776.eurprd08.prod.outlook.com (10.168.67.141) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.77.7; Tue, 17 Oct 2017 08:41:03 +0000 Received: from DB5PR08MB1014.eurprd08.prod.outlook.com ([fe80::4c98:c840:d411:4259]) by DB5PR08MB1014.eurprd08.prod.outlook.com ([fe80::4c98:c840:d411:4259%14]) with mapi id 15.20.0077.022; Tue, 17 Oct 2017 08:41:03 +0000 From: Alexei Fedorov To: "Gao, Liming" , Ard Biesheuvel , "edk2-devel@lists.01.org" , "leif.lindholm@linaro.org" , Mitch Ishihara , "Evan Lloyd" , Matteo Carlini , "Sami Mujawar" , Girish Pathak CC: Sudeep Holla Thread-Topic: [edk2] [PATCH edk2-platforms] Platform/ARM: remove ReportStatusCodeLib resolutions Thread-Index: AQHTRqklopDcYs/b5U+UI1LTsi/OP6LnN3MAgACBAbo= Date: Tue, 17 Oct 2017 08:41:03 +0000 Message-ID: References: <20171016180347.22930-1-ard.biesheuvel@linaro.org>, <4A89E2EF3DFEDB4C8BFDE51014F606A14E16C97C@SHSMSX104.ccr.corp.intel.com> In-Reply-To: <4A89E2EF3DFEDB4C8BFDE51014F606A14E16C97C@SHSMSX104.ccr.corp.intel.com> Accept-Language: en-GB, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [217.140.96.140] x-ms-publictraffictype: Email x-microsoft-exchange-diagnostics: 1; VI1PR0801MB1776; 6:EHOvZ9AFyUfWulVKxvPQNJbokidyJJvB0H81kpBO8J3MYH5YJzdyHuVgSoD0I0VZyTqHf09lRK/NGIGQ48IHck4AC3LtEeGlnBDRGWp7wfInzYNh/0qbcX+UVI6eErUxrvdSowtk4dqbk2nGXs1/lxbprZy6sXbhjuV4xzWgB5tIMnGp2CKrjY0Owgn95rZYocunZpllq/6LODcMcuf+ysYQS/u3CYdqLoP3htPj3b6PFUU9A4ix1p9MJp3PoX1Kst7eZBtSVH+S+/MJ6oMl3w0OEMyjk3iLZezyIn7bKJ6LpMHKp8NkFsAREzHpBS2HAv5uyaz/lnkxBZm0xFbV7Q==; 5:VsJLvhd/4oblKj4Dg+cNSnhtZeuMrbRTpKuqatRPkqyJU10CxyJc7IbxavebawoEMtiI6JgjiWSiK41JgEYQomtZiGNNrt7V8LSTytR+cR9nSqXFOaURO9uxmE9Dxl9bsgBqY/sYWJB8TwhflG/eYA==; 24:/6d2Q7P3/55jqPK38fJ4IvwPArpEGvBgyG1ByNWe1Efm7JxVpXN6Lv6V27TQKAIi6n3mCJCteTiiCxO+Fyf+isY4mSODfVnfJOBil92y1D4=; 7:frspWTzUd5HgE1xuU0ww2Nqjbn0MzoEqCRkRUgMy5Bm+7QLHzr32XEkQzX0ReVKBQ6yOm6LCTjUbQgNy82JHTjHWk6CkP8sEt/vFRL5L64N6XnLqYjYXM5sjXIB68z9XJcPPFQZIk1WpFvDM1EFHsICabzdwr6wZZVn4cPhHdSVxGjFGW7KvG9TsmpaGV6N9MvLOOG/qHMKck+Fehd4HeJ3oLYZaKywpPqhO/sjsa50= x-ms-exchange-antispam-srfa-diagnostics: SSOS;SSOR; x-ms-office365-filtering-correlation-id: d03f800e-f70e-46ba-dba0-08d5153accd2 x-ms-office365-filtering-ht: Tenant x-microsoft-antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(2017030254152)(48565401081)(2017052603199)(201703131423075)(201703031133081)(201702281549075); SRVR:VI1PR0801MB1776; x-ms-traffictypediagnostic: VI1PR0801MB1776: x-exchange-antispam-report-test: UriScan:(158342451672863)(180628864354917)(162533806227266)(228905959029699); x-microsoft-antispam-prvs: x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(5005006)(8121501046)(3002001)(10201501046)(93006095)(93001095)(100000703101)(100105400095)(6055026)(6041248)(20161123562025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123564025)(20161123558100)(20161123560025)(20161123555025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:VI1PR0801MB1776; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:VI1PR0801MB1776; x-forefront-prvs: 04631F8F77 x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(346002)(39860400002)(376002)(377454003)(199003)(40434004)(13464003)(189002)(106356001)(68736007)(105586002)(74316002)(101416001)(6636002)(8676002)(5890100001)(7736002)(2501003)(5250100002)(110136005)(2201001)(86362001)(3660700001)(25786009)(4326008)(6246003)(53546010)(33656002)(316002)(2906002)(189998001)(606006)(5660300001)(3280700002)(55016002)(229853002)(14454004)(54896002)(9686003)(6506006)(2900100001)(6436002)(53936002)(6306002)(7696004)(99286003)(3846002)(54356999)(102836003)(76176999)(6116002)(236005)(50986999)(2950100002)(478600001)(81156014)(81166006)(72206003)(966005)(97736004)(8936002)(66066001)(921003)(1121003); DIR:OUT; SFP:1101; SCL:1; SRVR:VI1PR0801MB1776; H:DB5PR08MB1014.eurprd08.prod.outlook.com; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX:1; LANG:en; received-spf: None (protection.outlook.com: arm.com does not designate permitted sender hosts) authentication-results: spf=none (sender IP is ) smtp.mailfrom=Alexei.Fedorov@arm.com; spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM MIME-Version: 1.0 X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-originalarrivaltime: 17 Oct 2017 08:41:03.0044 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: f34e5979-57d9-4aaa-ad4d-b122a662184d X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0801MB1776 X-Content-Filtered-By: Mailman/MimeDel 2.1.22 Subject: Re: [PATCH edk2-platforms] Platform/ARM: remove ReportStatusCodeLib resolutions X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 17 Oct 2017 08:37:31 -0000 Content-Language: en-GB Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Ard, BaseReportStatusCodeLibNull.inf was replaced with DxeReportStatusCodeLib.in= f to support storing boot performance data required for FPDT ACPI table. As Liming already mentioned ResetSystemRuntimeDxe.inf should be linked to R= untimeDxeReportStatusCodeLib library: MdeModulePkg/Universal/ResetSystemRuntimeDxe/ResetSystemRuntimeDxe.inf { ReportStatusCodeLib|MdeModulePkg/Library/RuntimeDxeReportStatusCodeLi= b/RuntimeDxeReportStatusCodeLib.inf } Alexei. ________________________________ From: edk2-devel on behalf of Gao, Liming= Sent: 17 October 2017 01:53:46 To: Ard Biesheuvel; edk2-devel@lists.01.org; leif.lindholm@linaro.org Cc: Sudeep Holla Subject: Re: [edk2] [PATCH edk2-platforms] Platform/ARM: remove ReportStatu= sCodeLib resolutions Ard: MdeModulePkg\Library\RuntimeDxeReportStatusCodeLib\RuntimeDxeReportStatus= CodeLib.inf is designed for Runtime driver. If you require StatusCode at ru= ntime, you can use this library instance. Thanks Liming > -----Original Message----- > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Ar= d Biesheuvel > Sent: Tuesday, October 17, 2017 2:04 AM > To: edk2-devel@lists.01.org; leif.lindholm@linaro.org > Cc: Ard Biesheuvel ; sudeep.holla@arm.com > Subject: [edk2] [PATCH edk2-platforms] Platform/ARM: remove ReportStatusC= odeLib resolutions > > The generic ResetSystemRuntimeDxe may invoke ReportStatusCodeLib, and > this may happen at runtime. If the chosen resolution is not suitable > for runtime, this will result in a crash. > > Given that we don't actually use status codes, let's just switch to > the NULL instance for all modules and be done with it. > > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Ard Biesheuvel > --- > Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc | 6 +----- > 1 file changed, 1 insertion(+), 5 deletions(-) > > diff --git a/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc b/Platform/ARM/= VExpressPkg/ArmVExpress.dsc.inc > index 8bcb84869c84..b758c58c9872 100644 > --- a/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc > +++ b/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc > @@ -153,7 +153,7 @@ > CapsuleLib|MdeModulePkg/Library/DxeCapsuleLibNull/DxeCapsuleLibNull.in= f > CustomizedDisplayLib|MdeModulePkg/Library/CustomizedDisplayLib/Customi= zedDisplayLib.inf > > - ReportStatusCodeLib|MdeModulePkg/Library/DxeReportStatusCodeLib/DxeRep= ortStatusCodeLib.inf > + ReportStatusCodeLib|MdePkg/Library/BaseReportStatusCodeLibNull/BaseRep= ortStatusCodeLibNull.inf > > [LibraryClasses.common.SEC] > ArmPlatformSecExtraActionLib|ArmPlatformPkg/Library/DebugSecExtraActio= nLib/DebugSecExtraActionLib.inf > @@ -182,7 +182,6 @@ > MemoryAllocationLib|MdePkg/Library/PeiMemoryAllocationLib/PeiMemoryAll= ocationLib.inf > PeiCoreEntryPoint|MdePkg/Library/PeiCoreEntryPoint/PeiCoreEntryPoint.i= nf > PerformanceLib|MdeModulePkg/Library/PeiPerformanceLib/PeiPerformanceLi= b.inf > - OemHookStatusCodeLib|MdeModulePkg/Library/OemHookStatusCodeLibNull/Oem= HookStatusCodeLibNull.inf > PeCoffGetEntryPointLib|MdePkg/Library/BasePeCoffGetEntryPointLib/BaseP= eCoffGetEntryPointLib.inf > ExtractGuidedSectionLib|MdePkg/Library/PeiExtractGuidedSectionLib/PeiE= xtractGuidedSectionLib.inf > > @@ -195,7 +194,6 @@ > MemoryAllocationLib|MdePkg/Library/PeiMemoryAllocationLib/PeiMemoryAll= ocationLib.inf > PeimEntryPoint|MdePkg/Library/PeimEntryPoint/PeimEntryPoint.inf > PerformanceLib|MdeModulePkg/Library/PeiPerformanceLib/PeiPerformanceLi= b.inf > - OemHookStatusCodeLib|MdeModulePkg/Library/OemHookStatusCodeLibNull/Oem= HookStatusCodeLibNull.inf > PeCoffGetEntryPointLib|MdePkg/Library/BasePeCoffGetEntryPointLib/BaseP= eCoffGetEntryPointLib.inf > PeiResourcePublicationLib|MdePkg/Library/PeiResourcePublicationLib/Pei= ResourcePublicationLib.inf > ExtractGuidedSectionLib|MdePkg/Library/PeiExtractGuidedSectionLib/PeiE= xtractGuidedSectionLib.inf > @@ -358,8 +356,6 @@ > # DEBUG_ERROR 0x80000000 // Error > gEfiMdePkgTokenSpaceGuid.PcdDebugPrintErrorLevel|0x8000000F > > - gEfiMdePkgTokenSpaceGuid.PcdReportStatusCodePropertyMask|0x07 > - > gEmbeddedTokenSpaceGuid.PcdEmbeddedAutomaticBootCommand|"" > gEmbeddedTokenSpaceGuid.PcdEmbeddedDefaultTextColor|0x07 > gEmbeddedTokenSpaceGuid.PcdEmbeddedMemVariableStoreSize|0x10000 > -- > 2.11.0 > > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel IMPORTANT NOTICE: The contents of this email and any attachments are confid= ential and may also be privileged. If you are not the intended recipient, p= lease notify the sender immediately and do not disclose the contents to any= other person, use it for any purpose, or store or copy the information in = any medium. Thank you.