From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) by mx.groups.io with SMTP id smtpd.web11.2503.1622768579002514171 for ; Thu, 03 Jun 2021 18:02:59 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=ylp5SP98; spf=pass (domain: intel.com, ip: 192.55.52.151, mailfrom: hao.a.wu@intel.com) IronPort-SDR: hfMLnVBKvT2Lxr72fTArKsB4eRvsFt7SCurZvlZ+VUKHbAb6GK66RM/HycOMDBcx/eqZEsrC8z DR1TS9AJv3PA== X-IronPort-AV: E=McAfee;i="6200,9189,10004"; a="184567533" X-IronPort-AV: E=Sophos;i="5.83,246,1616482800"; d="scan'208";a="184567533" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga107.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 03 Jun 2021 18:02:49 -0700 IronPort-SDR: J//J4cAEKD5eFooC2+ZUFVonjk1ulZ/E553nV4tbQqcXTOmdr580/fORFsh/I16mtbUy1pg/lL G6NsmJePB2lA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.83,246,1616482800"; d="scan'208";a="480445617" Received: from orsmsx606.amr.corp.intel.com ([10.22.229.19]) by orsmga001.jf.intel.com with ESMTP; 03 Jun 2021 18:02:49 -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; Thu, 3 Jun 2021 18:02:49 -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; Thu, 3 Jun 2021 18:02:49 -0700 Received: from NAM04-DM6-obe.outbound.protection.outlook.com (104.47.73.41) 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; Thu, 3 Jun 2021 18:02:48 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=EdVcyDOiHig36vj6TCgC5d4XMssw7yAE/XVyQ46JevwkhXCKYTqahAVAHlUy9wcjabtmplnrcHkEI9uxUjyJBNtE/MkNacfs4AEJkEsfbPFjbhU1BordoOVOlDaCgy8PpmE9BQXkgd687RlvI8P/XwfqtrYQ6jgK6GblE9dIGT2LDSHKbSyDBkijfbJ9txksydR+sXvUxv9pdQgOjHl6lRvSoTI7mxwBdnvEx0XJ3TXhRYdwgkllD1U+L3Ln9wG2VWbfZEIgn6P6QRygZdqCwrYOd3o9ELQgTKaNu0nBU0p6z3OPJqZ8BoVtB4Mrp348hCqKqMwUxoq/5Nb//BYJYg== 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=qR2EnJMNnkI9ra6w4Uuk+lqBk634nA2TjKZjHuxwTOw=; b=SOpvTEkdq+He5GxsbQLHeGYKGt1+Aav8afVDxSobUcu4DlZGMmPphGa0mr/20t1aFsQKL8OoghZNJ2+xa09sZkIDKWbc0QaQGeGWVmS0rPMnQgS5K3eqgj9YlcrG43s3yb5hglUYt4DMnwAqjWQo8rIrE4ymmgsw3nO6HPEIn2oBMENl2TNJvgxPybQmgM7tw62Y1xYZTiPEgn/tbaialTQ8pP7vLE2YrjFVgnGsgutz19mXBXNLV5yh9AupLWWxO5hqA48ePY862fA9X3kts+e8om592Mq86i6g/6+d+BjcZTbe6re5rbQLTzpnnKbpzMygrn3yeMzmrsEzpErAeQ== 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=qR2EnJMNnkI9ra6w4Uuk+lqBk634nA2TjKZjHuxwTOw=; b=ylp5SP98D8cADmATMOPR1QrffDidNKZUcfCnnXU+g0sy0DKV3uMw3j8ED20XuPLtEjR1YcCVPzOW/FYcMKn4C2PKCm3m2Mv4k7la0Pkig1gKRaa/64K20dLBEDUh57PnAfCllbgQlba4vb7Dhi7LSanUIRnKiCJMh7GnHhQn/a4= Received: from BN8PR11MB3666.namprd11.prod.outlook.com (2603:10b6:408:8c::19) by BN6PR11MB4052.namprd11.prod.outlook.com (2603:10b6:405:7a::37) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4173.22; Fri, 4 Jun 2021 01:02: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.4173.035; Fri, 4 Jun 2021 01:02:46 +0000 From: "Wu, Hao A" To: "Ni, Ray" , "devel@edk2.groups.io" , "Liu, Zhiguang" CC: "Kinney, Michael D" , Liming Gao , "Ma, Maurice" , "Dong, Guo" Subject: Re: [edk2-devel] [PATCH v2 1/3] MdeModulePkg/UniversalPayload: Add definition for extra info in payload Thread-Topic: [edk2-devel] [PATCH v2 1/3] MdeModulePkg/UniversalPayload: Add definition for extra info in payload Thread-Index: AQHXWEEM+sub46P/K025ZeNX5HT7WKsB0uXggAE21YCAAAA6wA== Date: Fri, 4 Jun 2021 01:02:46 +0000 Message-ID: References: <20210603062259.1390-1-ray.ni@intel.com> <20210603062259.1390-2-ray.ni@intel.com> In-Reply-To: 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.143.16] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: fd90e7e3-8880-459d-1664-08d926f47774 x-ms-traffictypediagnostic: BN6PR11MB4052: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:9508; x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 2TGg86os3ti/a+mUSae+B8tmfYUGu79a+7gnZlh36IwzDygMTU4Bu8xOTNITKqwJPpCcxY9l2f3uBUVYBr55AKlBMziVd+F2nXVLf2qAo8ZyrytiX7exyn3oSG88J7+jbjCMk56FJ4m8mdrYzl21rFKaBP98YytGlG5V8U6X9OpH7CkvrJsXnDoALN0M0TCreQhpHwQuLhRfeVOlVtemA8W1rMo0x/01FOACwPB+uBPcQ7jwLLW6N4LeYYd7/gAROcY8jMBZaUw0hKBWFPDt4SHQQNjcFVflnnQsXGS8qND3YNqKbGzok2hsfYCMu3TWQBojpF3SGXH4omZV2hokR5/AaysH7eK6rguCkUeXSdqHMMN4s7fMxPW0eiwjxyIT2bGSrKud/letXPCZKwy3rFrk+cza3UzkoANMdxGvQ5pesIhg2toAeowDSyXyDoe6RZFjPuLxd6mRGpMRAEpWwubNel0e+iYkf9VsjEwLt7bn1pnBl1v2MrFCQlDlaZPUoEdA3n95Py3/pXPwST58605g6RoH05pqKsBkSMtI8U9Gx3X5YJIm8dfQ3F+NnevJjhytcebnX1T299OpGA9UY9+7SDLeGnsHQm6JqXUoUJzTOMIQIymIzlbf8gYkylbr3ugLamKgIPBkrH32JQfLD1my5Q7rbOAwwJLv6G2vhhfmhkh12wDhWInIXaMYINOMzI07GQRCO2PIbfz1INAX+w2J2OObZdnb1QGGszo+0MbmHZSBNpXXcLdK378hE6IyKYCE7K0M/fUIrcOYxDQebA== 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:(346002)(366004)(376002)(396003)(39860400002)(136003)(33656002)(966005)(54906003)(110136005)(122000001)(4326008)(66946007)(316002)(478600001)(76116006)(9686003)(71200400001)(66556008)(6506007)(86362001)(2906002)(26005)(6636002)(66476007)(7696005)(53546011)(83380400001)(64756008)(66446008)(8936002)(5660300002)(38100700002)(55016002)(107886003)(52536014)(8676002)(186003);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata: =?us-ascii?Q?kO6lAbDtSyh0vpLg8d5rMrP+37AsgVF8sU1XR6nvxwlCbN3jtDeFHvbQMgh+?= =?us-ascii?Q?G6XPlJgXtd3xOOfxL5UMwliaoLKU0v4c5dAuO0szTEqfevpbMrY9cDPJAqT9?= =?us-ascii?Q?B+BoAcdgyc28g69t3+q9YUhSCNP0w7pcwqycK2x8vGxfvMnetHdKTq1IPs/R?= =?us-ascii?Q?Qv9pxlJkffot/QSBZ19MciCYdyM8D0haVJiAEGd2wlVQTjQwliASzfPBmU7p?= =?us-ascii?Q?ZBzVuDQgoNiuELrdDFVz66pNMSmdY5Fcz5QHhCghYnP8D2CUAESxVpC16f0w?= =?us-ascii?Q?imL2zy6abEZoTVaA8sTmruHGF2STnZOXBsIBeGJVmVN+EuKboFNrtb828TUF?= =?us-ascii?Q?KvozK3IIjg7mlENn5FRtH+QCkPivjLU/BaQRiHaDHxh+csYnPeTiYs3XNaxd?= =?us-ascii?Q?XsxCNQWs8FTE9ouvwOmzJgaKJdW5tg5ymg2PJ5frUfrMtzdelgNWwF5hs1p0?= =?us-ascii?Q?G2fGBAfTlUM/sCYV4+Px6xQLIHcKPE3onnLKr8LZd8j4796RG6Dv7yiz4I0Z?= =?us-ascii?Q?al1IP/o8VJMonmHd49e+E84xMJ0qntNIwLaInCILIssQrv5E02C1hCiYokAE?= =?us-ascii?Q?RsmDMzc3rPBaZR5khfPEicpnCcySDsFg5PLApoFUCdzlYXRQkgCtnQSIaQcj?= =?us-ascii?Q?3LHf9DXczRhhOgsLgt/Y1+1wPw8+Ei8XAKDfdJcWEdFIphKa4CHb2wd0NqGa?= =?us-ascii?Q?ZFrTzEcqeUNvg9OuAKGRegHEa//P/rFDDpptLyZSUGyrXSvXmYLfpRpq4sro?= =?us-ascii?Q?2c/o54iWwRNBoE2X5h0yvYcxpwf0bhGcUSOpgMY7EheUWh5t2Agx8WP2TRRB?= =?us-ascii?Q?eSs3UIuwQQiBkT9S95cRgiJ7KhbAifpCW4dVl5eHcTIP3Pubz6hsTrghzqPk?= =?us-ascii?Q?B5TNX10bgBeFuQm6ypuz4AltOW/W3IxN2BRjs0A8F5cs4FX40pzJCHSLM97X?= =?us-ascii?Q?DxJ0a4/eD5M5nlhCFAlcNEFMm3qU6PCrsAoYe46LbPaknUt8hWhnoMDnaVk4?= =?us-ascii?Q?364/6HLPeqXRwRHoEswHITIcy37a7LTMYg38lK4YOtgqRuXWjPrX2nkWGQpK?= =?us-ascii?Q?3t69sghrv+Z5Ky4O2c5Z3odbRQMLOUCOtLu8Tae2Qushtyy0injDzFQkpFbV?= =?us-ascii?Q?mDe3CXa+GPndMhrQkH5R8gq6/CzwvSYbEVFLGtvmhuuUutAXbTuKRNBo53E8?= =?us-ascii?Q?gjxiSiUbCrlnLKiS+kxL0/lsOiURrhOjARmV/Ik1R8wO6mB5tY3npcXV6TGT?= =?us-ascii?Q?zxUx1yJDUcCkoJ6yc4zILGoB7u/WBw964fY/AlN4wU9C+uoKEX8GJ6/522zF?= =?us-ascii?Q?8ikxHJLHsQrd0EwpU8ULYyE3?= 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: fd90e7e3-8880-459d-1664-08d926f47774 X-MS-Exchange-CrossTenant-originalarrivaltime: 04 Jun 2021 01:02:46.2400 (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: GnkaVqYsHmYVGcS3rruBfmUJGOOe93oOjzhfNr5td7Bdoy6oc+PVyJYopZDykewAFhg6E/6JEuyOQ6TjNaK8MA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN6PR11MB4052 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 Hello Ray, Yes, it looks to good me. Thanks a lot for the support. Best Regards, Hao Wu > -----Original Message----- > From: Ni, Ray > Sent: Friday, June 4, 2021 9:01 AM > To: Wu, Hao A ; devel@edk2.groups.io; Liu, Zhiguang > > Cc: Kinney, Michael D ; Liming Gao > ; Ma, Maurice ; Dong, > Guo > Subject: RE: [edk2-devel] [PATCH v2 1/3] MdeModulePkg/UniversalPayload: > Add definition for extra info in payload >=20 > Hao, > Sure. I will send patch to change Maintainers.txt putting Zhiguang and me= on > the reviewer list for Include/UniversalPayload after this is merged. >=20 > For the specific changes in AcpiTableDxe and SmbiosDxe, I will discuss wi= th > Zhiguang and send changes to append him on reviewer list of ACPI Modules > and SMBIOS Modules. >=20 > Ok to you? >=20 > Thanks, > Ray >=20 > > -----Original Message----- > > From: Wu, Hao A > > Sent: Thursday, June 3, 2021 2:37 PM > > To: devel@edk2.groups.io; Ni, Ray ; Liu, Zhiguang > > > > Cc: Kinney, Michael D ; Liming Gao > > ; Ma, Maurice ; > Dong, > > Guo > > Subject: RE: [edk2-devel] [PATCH v2 1/3] > > MdeModulePkg/UniversalPayload: Add definition for extra info in > > payload > > > > > -----Original Message----- > > > From: devel@edk2.groups.io On Behalf Of Ni, > > > Ray > > > Sent: Thursday, June 3, 2021 2:23 PM > > > To: devel@edk2.groups.io > > > Cc: Kinney, Michael D ; Liming Gao > > > ; Liu, Zhiguang ; > > > Ma, Maurice ; Dong, Guo > > > > Subject: [edk2-devel] [PATCH v2 1/3] MdeModulePkg/UniversalPayload: > > > Add definition for extra info in payload > > > > > > The payload is in ELF format per the universal payload spec. > > > PLD_INFO_HEADER is stored in the ELF payload as a separate section > > > named ".upld_info". > > > > > > Extra data needed by payload is stored in sections whose name starts > > > with ".upld.". > > > > > > Hello Ray and Zhiguang, > > > > There are a couple of on-going patch series on UniversalPayload. > > Since the specifications for UniversalPayload does not classified as > > industry standard at this moment, some definitions will be placed under > MdeModulePkg. > > > > I have a request to update the MdeModulePkg part of the > > Maintainers.txt file to list the reviewers for these UniversalPayload > > header files as well as those UniversalPayload related implementation > source files. > > > > For the timing of this Maintainers.txt file change, I am fine with eith= er: > > a) Made in one of the UniversalPayload related patch series > > b) Made after all the implementations are completed > > > > Best Regards, > > Hao Wu > > > > > > > > > > Signed-off-by: Ray Ni > > > Cc: Michael D Kinney > > > Cc: Liming Gao > > > Cc: Zhiguang Liu > > > Cc: Maurice Ma > > > Cc: Guo Dong > > > --- > > > .../Include/UniversalPayload/ExtraData.h | 28 ++++++++++++++ > > > .../UniversalPayload/UniversalPayload.h | 38 > +++++++++++++++++++ > > > MdeModulePkg/MdeModulePkg.dec | 3 ++ > > > 3 files changed, 69 insertions(+) > > > create mode 100644 > > > MdeModulePkg/Include/UniversalPayload/ExtraData.h > > > create mode 100644 > > > MdeModulePkg/Include/UniversalPayload/UniversalPayload.h > > > > > > diff --git a/MdeModulePkg/Include/UniversalPayload/ExtraData.h > > > b/MdeModulePkg/Include/UniversalPayload/ExtraData.h > > > new file mode 100644 > > > index 0000000000..07b7426683 > > > --- /dev/null > > > +++ b/MdeModulePkg/Include/UniversalPayload/ExtraData.h > > > @@ -0,0 +1,28 @@ > > > +/** @file++ Copyright (c) 2021, Intel Corporation. All rights > > > +reserved.
+ > > > SPDX-License-Identifier: BSD-2-Clause-Patent++**/+#ifndef > > > __EXTRA_DATA_H__+#define __EXTRA_DATA_H__++extern GUID > > > gPldExtraDataGuid;++#pragma pack(1)++typedef struct {+ CHAR8 > > > Identifier[16];+ EFI_PHYSICAL_ADDRESS Base;+ UINT64 = Size;+} > > > PLD_EXTRA_DATA_ENTRY;++typedef struct {+ PLD_GENERIC_HEADER > > > PldHeader;+ UINT32 Count;+ PLD_EXTRA_DATA_ENTRY > Entry[0];+} > > > PLD_EXTRA_DATA;++#pragma pack()++#endifdiff --git > > > a/MdeModulePkg/Include/UniversalPayload/UniversalPayload.h > > > b/MdeModulePkg/Include/UniversalPayload/UniversalPayload.h > > > new file mode 100644 > > > index 0000000000..b8e9fe11a1 > > > --- /dev/null > > > +++ b/MdeModulePkg/Include/UniversalPayload/UniversalPayload.h > > > @@ -0,0 +1,38 @@ > > > +/** @file++ Copyright (c) 2021, Intel Corporation. All rights > > > +reserved.
+ > > > SPDX-License-Identifier: BSD-2-Clause-Patent++**/+#ifndef > > > __UNIVERSAL_PAYLOAD_H__+#define > > > __UNIVERSAL_PAYLOAD_H__++typedef VOID (EFIAPI > > > *UNIVERSAL_PAYLOAD_ENTRY) (VOID *HobList);++#define > PLD_IDENTIFIER > > > SIGNATURE_32('U', 'P', 'L', 'D')+#define PLD_INFO_SEC_NAME > > > ".upld_info"+#define PLD_EXTRA_SEC_NAME_PREFIX > ".upld."+#define > > > PLD_EXTRA_SEC_NAME_PREFIX_LENGTH (sizeof > > > (PLD_EXTRA_SEC_NAME_PREFIX) - 1)++#pragma pack(1)++typedef > struct {+ > > > UINT32 Identifier;+ UINT32 = HeaderLength;+ > > > UINT16 SpecRevision;+ UINT8 = Reserved[2];+ > > > UINT32 Revision;+ UINT32 = Attribute;+ UINT32 > > > Capability;+ CHAR8 ProducerId[16];+ CHAR8 > > > ImageId[16];+} PLD_INFO_HEADER;++typedef struct {+ UINT8 Revision;+ > > > UINT8 Reserved[3];+} PLD_GENERIC_HEADER;++#pragma > pack()++#endifdiff > > > --git a/MdeModulePkg/MdeModulePkg.dec > > > b/MdeModulePkg/MdeModulePkg.dec index 8d38383915..d823ad0ba7 > 100644 > > > --- a/MdeModulePkg/MdeModulePkg.dec > > > +++ b/MdeModulePkg/MdeModulePkg.dec > > > @@ -404,6 +404,9 @@ [Guids] > > > ## Include/Guid/MigratedFvInfo.h gEdkiiMigratedFvInfoGuid =3D > > > { 0xc1ab12f7, 0x74aa, 0x408d, { 0xa2, 0xf4, 0xc6, 0xce, 0xfd, 0x17, > > > 0x98, > > > 0x71 } } + ## Include/UniversalPayload/ExtraData.h+ > > > gPldExtraDataGuid =3D {0x15a5baf6, 0x1c91, 0x467d, {0x9d, 0xfb, 0x31,= 0x9d, > 0x17, 0x8d, 0x4b, > > > 0xb4}}+ [Ppis] ## Include/Ppi/AtaController.h gPeiAtaControllerPp= iGuid > > > =3D { 0xa45e60d1, 0xc719, 0x44aa, { 0xb0, 0x7a, 0xaa, 0x77, 0x7f, > > > 0x85, 0x90, 0x6d }}-- > > > 2.31.1.windows.1 > > > > > > > > > > > > -=3D-=3D-=3D-=3D-=3D-=3D > > > Groups.io Links: You receive all messages sent to this group. > > > View/Reply Online (#76009): > > > https://edk2.groups.io/g/devel/message/76009 > > > Mute This Topic: https://groups.io/mt/83277975/1768737 > > > Group Owner: devel+owner@edk2.groups.io > > > Unsubscribe: https://edk2.groups.io/g/devel/unsub > > > [hao.a.wu@intel.com] - =3D-=3D-=3D-=3D-=3D-=3D > > >