From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by mx.groups.io with SMTP id smtpd.web09.983.1626210032837457008 for ; Tue, 13 Jul 2021 14:00:33 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=l0WFAeKD; spf=pass (domain: intel.com, ip: 192.55.52.136, mailfrom: guo.dong@intel.com) X-IronPort-AV: E=McAfee;i="6200,9189,10044"; a="189926244" X-IronPort-AV: E=Sophos;i="5.84,237,1620716400"; d="scan'208";a="189926244" Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 Jul 2021 14:00:29 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.84,237,1620716400"; d="scan'208";a="459721019" Received: from fmsmsx601.amr.corp.intel.com ([10.18.126.81]) by orsmga008.jf.intel.com with ESMTP; 13 Jul 2021 14:00:28 -0700 Received: from fmsmsx607.amr.corp.intel.com (10.18.126.87) by fmsmsx601.amr.corp.intel.com (10.18.126.81) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.10; Tue, 13 Jul 2021 14:00:28 -0700 Received: from fmsedg602.ED.cps.intel.com (10.1.192.136) by fmsmsx607.amr.corp.intel.com (10.18.126.87) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.10 via Frontend Transport; Tue, 13 Jul 2021 14:00:28 -0700 Received: from NAM11-BN8-obe.outbound.protection.outlook.com (104.47.58.171) by edgegateway.intel.com (192.55.55.71) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2242.10; Tue, 13 Jul 2021 14:00:22 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=NIVlMvj18LCLeDgHX6kV+VTsrY3w8qZlq6sQ/Q1upI5bZ6Yr847igpxmYUyXFBykK9F12lXGVaTD64glyRRKotGPAiJRIN8YVvTsK0VxF7o26qXHDEOVXJniSsoUmwukssrJ29pzyFe+VnDNFPyjDPPlOQJxMJ87Odkw/Q2uPuL6ZVc/mjePiOc3PExMHbO6Cxxc4zytdoEgQ1Hj2U/7FiWRClvnHglqqnlNSySsUWj6JBCHOFmK3KrlZYpaTSWKgmzLknRx3PR78B6GBi/dVLY4A/rRQPCncoApbBTX9OmKwndXFe5tGvZz2saRoUCS72/t8erA3DdvVd9V0tvoAA== 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=qEogoNthmzDjX/pV8g9rQUVR5Kb2ym7yhYCob/zviJE=; b=GicvnP7ky6C2r6Jw4jbv6FQPUW4eBlrY2m4px2wflZzXOQNkRuVNPWxbzxkxlVTQm8U2SwzSjoPCqt4EreDf1eQtWvFsjFl4I8mFGAGevpx2UW20P4qgN7JEf+z+D0jTIS5dwgUca/CTgCxeKvJ3pTmBuXylA2NB7hgXhn0vV/gSd/rXf5fXHKFin/Pm4EfQ0dl3w48bQvLFB74lGO+y+FVv3keWYFgpIuDb92Ll6SA2vRyVLoygtACQRTRdZi8SjIKaP0kCEv8OWIDKSp4Q4dtjkGVB+LMnowXkxVjxN4neIt/pket4WH8KOfbRrWr39hrL1gYU44qPdHlGxy4wQQ== 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=qEogoNthmzDjX/pV8g9rQUVR5Kb2ym7yhYCob/zviJE=; b=l0WFAeKDL+0XEOpray25qLqsvUB0/rrDVcUEOPfBnD0u9LaoPwv9gUyTKMqAWVwwpbdKULG8psFKwxaLwVbg8+JVRl4Zetfx19J+MCMTLj3tBlPLskXUe6EWUyvtOfuJrR5xPpEMhM5blhVf65hnOdCm+nY6i1pFlhgABGOPMlA= Received: from BYAPR11MB3622.namprd11.prod.outlook.com (2603:10b6:a03:fe::30) by BY5PR11MB4484.namprd11.prod.outlook.com (2603:10b6:a03:1c3::27) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4308.21; Tue, 13 Jul 2021 21:00:17 +0000 Received: from BYAPR11MB3622.namprd11.prod.outlook.com ([fe80::7520:e237:98fd:6442]) by BYAPR11MB3622.namprd11.prod.outlook.com ([fe80::7520:e237:98fd:6442%4]) with mapi id 15.20.4308.027; Tue, 13 Jul 2021 21:00:17 +0000 From: "Guo Dong" To: "Tan, Dun" , "devel@edk2.groups.io" CC: "Ni, Ray" , "Ma, Maurice" , "You, Benjamin" Subject: Re: [Patch V3 2/2] UefiPayloadPkg: Assign the length of UniversalPayload ExtraData Thread-Topic: [Patch V3 2/2] UefiPayloadPkg: Assign the length of UniversalPayload ExtraData Thread-Index: AQHXd7eUafeobN+OCE2bLcA4u+Sk6atBZJVA Date: Tue, 13 Jul 2021 21:00:17 +0000 Message-ID: References: <20210713071913.1060-1-dun.tan@intel.com> <20210713071913.1060-3-dun.tan@intel.com> In-Reply-To: <20210713071913.1060-3-dun.tan@intel.com> Accept-Language: en-US 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-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: f57885c6-4219-43b7-8bd0-08d946413816 x-ms-traffictypediagnostic: BY5PR11MB4484: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:595; x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: wSXJxItynS8P6VlEY+mQ7QK33c12DFjcec0UbH9nvfUi/0LpfAHQ8g6WTIIqPtD7Y/jsj/CA+Y8rWpxvGJNpzoDRSOW9fGARDtwGIi+8WBcKJJtAOQ9sC2ImTdnsKUx39GTSCD2bYtpz+cmV4DkA8w/VGTBCwM6CH3L6lQ/yp4qqt7GZ1sY9P1WoZQQOSjRaNRGmI/bsXYFrMpmQlICIgs8ZYUqIw2oIyf4mgZH6hGLvLeoQStefKH9MGDSZI7WiVY9zajS6uuUb8x42MT10g6Yajy2LsXts/wf6KL2ZZwskb/9N/r7bfz4KW10/vFozF4jE7bFDQlB6pJcllbJh/D35T1uz6QtnpaLOBvfD9S/Oz7vxpXtFLVuTKh3Ch5iCgntsS+RLc870XDAZ83NdtCPXjL26biNd99lOSp1MHeLbouRQrTCYqxrQbBRGNtr8ehPkVcfWbX+5WHQJMZ7taWe2JKyfRFaHbSO2gLXA+fACYQs5IUhQyei/+4YKE2iZyfj8LH5g/db9QOpNTBmHH1nv418uL/ZxV1qZH+XmiCWD54bHgfZmrHiAi3VfAKasA4o+d1uYbt2W8JPEFojFqSyK8Ez4Sr2H4FmmiS9+0CYyMeEJuYP8Wik4eossSJ8ALgz9364pOw1wWTAJ/l9vPA== x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BYAPR11MB3622.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(396003)(376002)(39860400002)(366004)(136003)(346002)(4326008)(8676002)(66946007)(66476007)(33656002)(110136005)(8936002)(83380400001)(52536014)(66556008)(64756008)(7696005)(5660300002)(55016002)(76116006)(53546011)(86362001)(478600001)(316002)(6506007)(107886003)(186003)(71200400001)(66446008)(38100700002)(54906003)(9686003)(2906002)(122000001);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?/ZUJxeavC+cp1kSJQ5HIgFApTP78niGmFWBekZTkjHu08fnsO1DylsECLvxl?= =?us-ascii?Q?oIJgA+xUbfeTarOqdvJTYXiMWLZjOre2Ehy9ig5vCwt4vr9FKxMLbEpOb5vm?= =?us-ascii?Q?BiolMAELWnx6JLjh2RlaG40/DZWXW3N0zO+yE8RbG+YRKQL7rLKJmoDTK70j?= =?us-ascii?Q?+L5yfaztufmVB85yNAwnV6UKYIbMaczBOwj2zmB243utdUyjPXe/SMU4vtTu?= =?us-ascii?Q?yCrZ3Kyrf5NmW/tFEqR73tG+0Fk2JC0UZkXRf46+ZxjtcD02DZUjRPo6HQr4?= =?us-ascii?Q?tftP2f1khr7TsPYZGV3jIfw6Bhmn1k11vQw4Pv3PLdNu4s+ypZz6aX9n2OoT?= =?us-ascii?Q?bMNCgIFzzilxXKOWukGSwM9fIuiujaTXZ2uwtCEF8cv1hAhFOXm/CqEEUfkF?= =?us-ascii?Q?oGYySTZ5uz0LefMpVm4oyFPUWeE/lv477j6uWbvnHt0HNp/lF5V/MTe7cJvX?= =?us-ascii?Q?E5LXsFGSiGnBOviVm47cXF1iBmURd4OaaV/JKWwOwaLhWGJczBzQunzpeO77?= =?us-ascii?Q?Jb/MbcLcNBwmqt0bxBf+qUlP4lvXc86qI7ACybQSlKM0E1iSQzUQKagw2ddN?= =?us-ascii?Q?muEHMIYcuA8OiA9vbqxBusdmtro4xpKVU2PTmRqGtNJRVDLor9ET3xbQBSNP?= =?us-ascii?Q?4fTsKef+QNWfD9HgV8VoRFv//RbzUamgwv6KXZndp3kb9srKZQWPZqgD7EWm?= =?us-ascii?Q?4hIzfEgsYfSIfFqvp76jUTEYNMQRW09j/j6OGrU1XaUv01XUYTaTiXJ3hxVT?= =?us-ascii?Q?TtEaIlLUY+kCLT+UIwhPUaSSK5tDAp9R0V3B0YjjVYv6T224jDEXp0dqmIo9?= =?us-ascii?Q?ffbRKdNHaz8x3xTBzzXGrwjhUgHIMK9rjxRrPg79iNZxTtllrXGbi96myhVT?= =?us-ascii?Q?TQCoeaJYRFZ6akDItJH7mV3c8knBRM7Cb33Rm3VnatPr7Jptzb6yZ+JBZOz2?= =?us-ascii?Q?evV+mqHkiwVCLFKvx1Gjx+RAgBEyuMcWCW339T7V1sbuHO41LdG1wc5EFhGF?= =?us-ascii?Q?j4CGNAROT+oMcDW9k4iioPGlJ14Tk7S9JPHD8eCx33HCbRFC2tYoMCnnPJKR?= =?us-ascii?Q?BhisCTiMpYTc/vWeOs8TawZ/FKnt/yPaIXMqQWWG/H7vwZckrF7RJ85sNaq+?= =?us-ascii?Q?1v8IPK5IBNGuGi/JOaqaWY0qrN0dDXFTOzIUPSX1aTih/pCOmRD680acAFX+?= =?us-ascii?Q?B/8q7d9EIlrNESGhmdsWf0kmPQLrsB5AG8S4mXwqnGEupjnCFo6+2Ewx2V1E?= =?us-ascii?Q?LF1b5huoDYU/bGviDRDI4yB80GxK43/S7nojS5JQY/0BrvKPL859xGAbV4Lp?= =?us-ascii?Q?PLJqRcWcVaTR/c7KyzGPGhndKVnb2EHh6iarLaYLwcnydC+yjTyolrgtVnCQ?= =?us-ascii?Q?JKeuWIjqcW+SiUg9A+teJP96ZXmc?= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: BYAPR11MB3622.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: f57885c6-4219-43b7-8bd0-08d946413816 X-MS-Exchange-CrossTenant-originalarrivaltime: 13 Jul 2021 21:00:17.6456 (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: 7aV+yeVd3XM8C7PFA0EZsZoj/Fi+VXwZ+IGUssQwFBRKd79wG26CPMoGkRKqIMO4ha7UfxMhEDupDkVms3ItMQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR11MB4484 Return-Path: guo.dong@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Signed-off-by: Guo Dong > -----Original Message----- > From: Tan, Dun > Sent: Tuesday, July 13, 2021 12:19 AM > To: devel@edk2.groups.io > Cc: Dong, Guo ; Ni, Ray ; Ma, > Maurice ; You, Benjamin > ; Tan, Dun > Subject: [Patch V3 2/2] UefiPayloadPkg: Assign the length of > UniversalPayload ExtraData >=20 > V1: Assign the length and revision of UniversalPayload ExtraData > V2: Force int to UINT16 > V3: Create a local variable to hold the size of ExtraData >=20 > Cc: Guo Dong > Cc: Ray Ni > Cc: Maurice Ma > Cc: Benjamin You >=20 > Signed-off-by: DunTan > --- > UefiPayloadPkg/PayloadLoaderPeim/PayloadLoaderPeim.c | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) >=20 > diff --git a/UefiPayloadPkg/PayloadLoaderPeim/PayloadLoaderPeim.c > b/UefiPayloadPkg/PayloadLoaderPeim/PayloadLoaderPeim.c > index c619470dbb..141ce86b46 100644 > --- a/UefiPayloadPkg/PayloadLoaderPeim/PayloadLoaderPeim.c > +++ b/UefiPayloadPkg/PayloadLoaderPeim/PayloadLoaderPeim.c > @@ -56,6 +56,7 @@ PeiLoadFileLoadPayload ( > UINTN Size; > UINT32 ExtraDataCount; > UINTN Instance; > + UINTN Length; >=20 > // > // ELF is added to file as RAW section for EDKII bootloader. > @@ -105,11 +106,14 @@ PeiLoadFileLoadPayload ( > // > // Report the additional PLD sections through HOB. > // > + Length =3D sizeof (UNIVERSAL_PAYLOAD_EXTRA_DATA) + ExtraDataCount * > sizeof (UNIVERSAL_PAYLOAD_EXTRA_DATA_ENTRY); > ExtraData =3D BuildGuidHob ( > &gUniversalPayloadExtraDataGuid, > - sizeof (UNIVERSAL_PAYLOAD_EXTRA_DATA) + ExtraDataCount * > sizeof (UNIVERSAL_PAYLOAD_EXTRA_DATA_ENTRY) > + Length > ); > ExtraData->Count =3D ExtraDataCount; > + ExtraData->Header.Revision =3D > UNIVERSAL_PAYLOAD_EXTRA_DATA_REVISION; > + ExtraData->Header.Length =3D (UINT16) Length; > if (ExtraDataCount !=3D 0) { > for (ExtraDataIndex =3D 0, Index =3D 0; Index < Context.ShNum; Index= ++) { > Status =3D GetElfSectionName (&Context, Index, &SectionName); > -- > 2.31.1.windows.1