From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by mx.groups.io with SMTP id smtpd.web08.5461.1623108546654278117 for ; Mon, 07 Jun 2021 16:29:06 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=CC+DapiX; spf=pass (domain: intel.com, ip: 134.134.136.126, mailfrom: hao.a.wu@intel.com) IronPort-SDR: qnnoWR8LVlAm0RIegtaZd/5Ke3zA3V9qSXJIfY6E1nI/k1VLTilPvKjBVyFWpR+AEeLR/C3nth l+i4jZcT25vw== X-IronPort-AV: E=McAfee;i="6200,9189,10008"; a="192062801" X-IronPort-AV: E=Sophos;i="5.83,256,1616482800"; d="scan'208";a="192062801" Received: from orsmga006.jf.intel.com ([10.7.209.51]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 Jun 2021 16:28:47 -0700 IronPort-SDR: 64F1pBPEDpuH2APu5GvfOXi0+7CsIAO/3sZDGQSJZ/Zzwio/7Be8Nmr3XTYwBh+qW2ND077Hwi 1ioftg/twkdA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.83,256,1616482800"; d="scan'208";a="401792501" Received: from orsmsx606.amr.corp.intel.com ([10.22.229.19]) by orsmga006.jf.intel.com with ESMTP; 07 Jun 2021 16:28:47 -0700 Received: from orsmsx609.amr.corp.intel.com (10.22.229.22) by ORSMSX606.amr.corp.intel.com (10.22.229.19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.4; Mon, 7 Jun 2021 16:28:47 -0700 Received: from ORSEDG601.ED.cps.intel.com (10.7.248.6) by orsmsx609.amr.corp.intel.com (10.22.229.22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.4 via Frontend Transport; Mon, 7 Jun 2021 16:28:47 -0700 Received: from NAM10-MW2-obe.outbound.protection.outlook.com (104.47.55.106) by edgegateway.intel.com (134.134.137.102) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2242.4; Mon, 7 Jun 2021 16:28:47 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=PD9s+yolrPHS6RhAbRbLLVROVtI0dH2lBzPEoVa4khlDR8Ns4XnHHeY04l8K5De4PKAK8fqOBS4gR/f4vMiNHJEZyARbkROXeJ6LCMPc3by7xyZNrk3Gln+ol5ghBjDHxbEJBK6gnTKLUVwHytoqchyzBBYY47NlKlobRC3VM4I+hnrFwCJ3j7ews7yPd99vJ6T2zrHzzxOS1Y/pBLCVsGcRf+4DmdY02OfUhmCfzG9pejoDQXb9rH0GUmAHNLZTfyZnXZ/7oVNd+ifyO2bps2BKj3GO8xQgWvGumdtkL1AIl9nGZZ11Mlki/8gWyph15sCtRxvzPkItZzitb0WEyA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=ezR9nEVIBSJMlDdOc8L/LmTB4ttYX6prqAQfq83QpL0=; b=XCh0ML9265dDSmGHqMAwlkC2eGmHyECE8oaEwSDIpGDJaUqYLBnBTWb0dkJ0dRpgi4LLLkFWXKWYlga1L1TSVcwfjQh46I3ApPGcpbYs62smuGJF2EyucbS1Qph/3m3eAbVsRMfinYkzC2SVAid+yxBelIQMecqquTeMxIoXWXL5Q1/oV5JZeCMH7H2dhKeTRSdi3R+wY3Tl4XPYTLtuCLhPPyJsSvf9iI4sxelz/kFkcIm6iI74O9YyZDAdkm05oWS6Th6hyywuNmsX+zPsfXP1gFzt8r8WcP735lYFffJisjwfn/S02XJl8fW9kpM9all+FqzakZblAlI7QIIbFA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel.onmicrosoft.com; s=selector2-intel-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=ezR9nEVIBSJMlDdOc8L/LmTB4ttYX6prqAQfq83QpL0=; b=CC+DapiXzgNx2fbZXHA+HtcLg8hnrqYiihQd2ix2V60S5JOowej4L0DKRu22PfOVArBxj4t1ZJU9x+/XtlzcfjfXLeskepq5OqXt1x3LrcTmYdXlLHLEQ+zomXqKtDGhFPi62BSidyeDCUHYqh19i/0J6sGEYN8Lw5iqjp2jJGE= Received: from BN8PR11MB3666.namprd11.prod.outlook.com (2603:10b6:408:8c::19) by BN6PR11MB1634.namprd11.prod.outlook.com (2603:10b6:405:c::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4195.23; Mon, 7 Jun 2021 23:28:46 +0000 Received: from BN8PR11MB3666.namprd11.prod.outlook.com ([fe80::b9d0:5694:1b70:c031]) by BN8PR11MB3666.namprd11.prod.outlook.com ([fe80::b9d0:5694:1b70:c031%4]) with mapi id 15.20.4195.030; Mon, 7 Jun 2021 23:28:46 +0000 From: "Wu, Hao A" To: "Ni, Ray" , "devel@edk2.groups.io" CC: "Wang, Jian J" , "Bi, Dandan" , Liming Gao Subject: Re: [PATCH v2 3/3] PeiCore: Remove assertion when failing to load PE image Thread-Topic: [PATCH v2 3/3] PeiCore: Remove assertion when failing to load PE image Thread-Index: AQHXWED6RMNEBfqEOU679eQBAWAFjKsJOSlQ Date: Mon, 7 Jun 2021 23:28:46 +0000 Message-ID: References: <20210603062259.1390-1-ray.ni@intel.com> <20210603062259.1390-4-ray.ni@intel.com> In-Reply-To: <20210603062259.1390-4-ray.ni@intel.com> Accept-Language: en-US, zh-CN X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-reaction: no-action dlp-version: 11.5.1.3 authentication-results: intel.com; dkim=none (message not signed) header.d=none;intel.com; dmarc=none action=none header.from=intel.com; x-originating-ip: [192.198.142.16] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 976266c6-14c3-4068-fb84-08d92a0bff52 x-ms-traffictypediagnostic: BN6PR11MB1634: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:6790; x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: kJBXoYrFPU17zD7p7xQpyZTdwVcQ3jl2lPdFPEP+sq+ksdlE620z/Q2DE40lkVUPRXYIrUqX5MEQWUSsAxNzAAGV505FqII+rmpc+DTZNFA+p/XZrPjPTAD4bG5q2PBI9SYWCZBW1Dn/z4j9iZTWbuy20TRudWVis1q6IZrKlBZXH3GgC9bVQ4ubiDI/2iXEIvzGaxMhpd/epkRE33DrJuf2ecni1wOx/VI+DFGeZ8FHenibc+KTdvHCg814aeQNWiCKhnKDl1YfRlJ9fGF8lnEKMGj53vV3y90gKwq4kFGOq2jruXLx1Jz3pY1CJ0yE7yci9goFl1XeWHnbJmb+VRvtXbnO1MZq7SNZDGrU9w+p9Jjp9ly457BH5zXqwJ+jSUjWs1oA1lIQ83Pdm+I6OMIkKuiufFxVUQ2U+jnpBdILp6pBkr28Y4+7152XDbOzn+O27uMQInOO8I7eqa+R+/PSn8LgsDgv/FQEPp1DggPM4eAPmSC+RDiYynhfIpMVZ5zehE4q/AjJetjkdDyu+Pw4ebeCbsoI7tewGNs2YJKluL5PlIoqxgpxWYv/a0QGv/v1sXpeEmP4iworeKNoWDrH1wybtQr8vVouvasTwiE= x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BN8PR11MB3666.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(396003)(366004)(136003)(346002)(39860400002)(376002)(52536014)(110136005)(54906003)(5660300002)(66946007)(186003)(122000001)(38100700002)(55016002)(4326008)(76116006)(7696005)(71200400001)(66446008)(64756008)(53546011)(83380400001)(66476007)(33656002)(8676002)(66556008)(6506007)(2906002)(316002)(26005)(8936002)(478600001)(9686003)(86362001);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata: =?us-ascii?Q?jLOzU6tlfc41sahSen7GQh6057mowzmiBeXPR7SSz3e7eIJr1uJ7fAr/X7oT?= =?us-ascii?Q?OEvIiKIl13nuuQfLjUQQw7sH6otnNLPMZAFWo0MdqUi3MfBkNvJbk3e+QD2G?= =?us-ascii?Q?aTTUe05eZDK71xX1I7Hl1voYwJvFGzQWZW/LJo3QewvvZhleqRCEMbITo3Up?= =?us-ascii?Q?wcvaK+CwDvxOsQOB4n6EAW7bv1V/YlICkRZNf1lPTrGGo4VThntLgM3klIl6?= =?us-ascii?Q?XMrJJ3rvRJLKuh9SYaWlRXF45psUDAa6J8OabZJmgTc9tGlvgCcwPtP2NL02?= =?us-ascii?Q?SsXNCBK63KeQRpyK8fQrt7H/Vgi5z0jDFGwbIJ3KVBL1JTz3b8f45PQbdGNV?= =?us-ascii?Q?g1QpR5VMe1960vpbg7p5aY5St71uEm/7Wo09Orgt69v7rsuyEJR3RPi8ntAx?= =?us-ascii?Q?RySG9MH9fcAc90fSkfGVkhdm38Wj0A1AKP22pNegiDpbOr5BWm9M5oSLhv/b?= =?us-ascii?Q?p23Pq9TuuScBOQDTVc1NVlnGMdc0FqPPh7AYS3Cc3NiYcJu4ZGlKeOjALwOg?= =?us-ascii?Q?eQU8WPMhMMuHv8u/sdJfOInC52J6oOgWGFtk/XXZGxijrBKMsRy8sMoOCiNi?= =?us-ascii?Q?B4MgZDFLT/+40PB6jrX74YFtVRZNbfAuQcxXEsAtNfjvbVd6Qq0irumJpYrS?= =?us-ascii?Q?WHCEj1U5cfJdyhOEgWupsn+/xWe9uaVy4g9EQkI7wPP1OaQBRrJuf6oCAoGI?= =?us-ascii?Q?Huysmc8VhBKtn/Ko62InUijsHlehOxKvb/LjqTLUbw7L0ohGm29oex2Gmn18?= =?us-ascii?Q?nPdj3/8nTBEFQR9+MFOhjnolzbpEOtuobmd4GECbPztH4jeEP/0ifZJJSk2F?= =?us-ascii?Q?no21gHRZXeYfKci6g1y8EPUJna5Cy97kxKnDFfNOtQrUoJ0U5UcbHhgV6xJ/?= =?us-ascii?Q?S7Ph6IRU/lwxrl98XxjLYbf0LIampnd7h6J4mHnVzU1oejiG/toEutYLx6ny?= =?us-ascii?Q?nsi1vCn2czkm+w9Pz9GlJQYvyfb9vxtqacSd41+j0ceE1pUac+QFuFkiGwIt?= =?us-ascii?Q?yuEsQS416/NIAAs0ZCypepfd1f9hXYMtGHx1R/TJ4qfXH+SYEMxMoQoBO2kk?= =?us-ascii?Q?4bsngw2VD9A9DNyK7tqUN4i6VRfJaBLRHNu8zV9isd0WzU58vRdKM83v1cNd?= =?us-ascii?Q?kDwGczms6pX2L962xRFFG9MhYGSBgI1zhNPzFpPnNgWqqYg57+Wabg6L+HvO?= =?us-ascii?Q?aqxOOV2fShxEo27yW4UMF3ixNs9FdDBQ/eCrYht4IHHtKa9gX3JDRPOlOTYw?= =?us-ascii?Q?PIbvtu9YAkZLRLQJa8E5IqJRGK4kqJm7dWEmSq+UzDr5+8Fdvo7hX1YL1ohT?= =?us-ascii?Q?HAVTfV1YNP8LWu/WudAz/+5T?= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: BN8PR11MB3666.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 976266c6-14c3-4068-fb84-08d92a0bff52 X-MS-Exchange-CrossTenant-originalarrivaltime: 07 Jun 2021 23:28:46.5471 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: MfTnPQ+RTDlRQ8CeS69Txd+FNqU/qJQwtcUhPM44HPBBRloYblScwiq+8AHLuT2fXlsg1Q2YruLrFrW+EuAjdw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN6PR11MB1634 Return-Path: hao.a.wu@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable > -----Original Message----- > From: Ni, Ray > Sent: Thursday, June 3, 2021 2:23 PM > To: devel@edk2.groups.io > Cc: Wang, Jian J ; Wu, Hao A ; > Bi, Dandan ; Liming Gao > Subject: [PATCH v2 3/3] PeiCore: Remove assertion when failing to load PE > image >=20 > EFI_PEI_LOAD_FILE_PPI is invoked by DxeIpl for loading DxeCore. > It's possible that the instance produced by PeiCore fails to load but > other instances of EFI_PEI_LOAD_FILE_PPI can load. Acked-by: Hao A Wu Best Regards, Hao Wu >=20 > Signed-off-by: Ray Ni > Cc: Jian J Wang > Cc: Hao A Wu > Cc: Dandan Bi > Cc: Liming Gao > --- > MdeModulePkg/Core/Pei/Image/Image.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) >=20 > diff --git a/MdeModulePkg/Core/Pei/Image/Image.c > b/MdeModulePkg/Core/Pei/Image/Image.c > index 1d15774527..5af3895191 100644 > --- a/MdeModulePkg/Core/Pei/Image/Image.c > +++ b/MdeModulePkg/Core/Pei/Image/Image.c > @@ -1,7 +1,7 @@ > /** @file >=20 > Pei Core Load Image Support >=20 >=20 >=20 > -Copyright (c) 2006 - 2019, Intel Corporation. All rights reserved.
>=20 > +Copyright (c) 2006 - 2021, Intel Corporation. All rights reserved.
>=20 > SPDX-License-Identifier: BSD-2-Clause-Patent >=20 >=20 >=20 > **/ >=20 > @@ -661,9 +661,6 @@ PeiLoadImageLoadImage ( > &ImageEntryPoint >=20 > ); >=20 >=20 >=20 > - ASSERT_EFI_ERROR (Status); >=20 > - >=20 > - >=20 > if (EFI_ERROR (Status)) { >=20 > return Status; >=20 > } >=20 > -- > 2.31.1.windows.1