From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by mx.groups.io with SMTP id smtpd.web10.4414.1622702255334042678 for ; Wed, 02 Jun 2021 23:37:36 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=L+uaYICP; spf=pass (domain: intel.com, ip: 134.134.136.100, mailfrom: hao.a.wu@intel.com) IronPort-SDR: Tklw/Cb/CFJf0b7+W0CqimZa8F373b8e2TCTf1lWtaJoziikEosWWCHn7Ps2wUsmUEtx9Uf4mf g3JV8NRsJ3pw== X-IronPort-AV: E=McAfee;i="6200,9189,10003"; a="267844198" X-IronPort-AV: E=Sophos;i="5.83,244,1616482800"; d="scan'208";a="267844198" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Jun 2021 23:37:34 -0700 IronPort-SDR: hL2dOKPOaOvnlahX1uDpL91SAqK+NSC9AXtL7+20vh7r5x5PtFHD26H0SOmmlBgdpZKpYHpsUT c1rawEVi2ljQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.83,244,1616482800"; d="scan'208";a="480051307" Received: from fmsmsx606.amr.corp.intel.com ([10.18.126.86]) by orsmga001.jf.intel.com with ESMTP; 02 Jun 2021 23:37:34 -0700 Received: from fmsmsx610.amr.corp.intel.com (10.18.126.90) by fmsmsx606.amr.corp.intel.com (10.18.126.86) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.4; Wed, 2 Jun 2021 23:37:33 -0700 Received: from fmsedg602.ED.cps.intel.com (10.1.192.136) by fmsmsx610.amr.corp.intel.com (10.18.126.90) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.4 via Frontend Transport; Wed, 2 Jun 2021 23:37:33 -0700 Received: from NAM12-BN8-obe.outbound.protection.outlook.com (104.47.55.168) 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.4; Wed, 2 Jun 2021 23:37:32 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=cz42WcYHVtARzb4bnmAwPhsrliRuWpZ/IgzNVyRvuhfav6fq+AFEL8IPKmb+q8IelZknztCv7hlqRYCE4+QHn8pwreTWJtNb5s1IYabsrHOpEUSiIXK0AczdtKfq2/zV5ONBcv0Hi75WRtqaIVN2D0XgmdGPKSGtDHHrwcUCMc7OAdU9uV7qy57ebDBUpHFijZAR3IaGBdZkjN69lEcAZ1tFIrY54BK+mtzNLhUFuBxqtaLZzaa90G3I/C+xOIftnKSDWF9fc0pSuSegqHWAr8FqVxNdpSoOn8kGvcUljC9WYb0eIjDT/4++e9MklEYhJHULiS6qFv+87DXmhBDIPA== 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=3s2cunV/9skRfk454v4ZTrJayOhbaw3chH9Decl+8rM=; b=ai52zZt0iGT9Qm8BqzbSDZAIBuocqx/fuz12RVMQXjgvLPnzqnMT5s9Yb/y0Kz1ML/nPt1/BqKq2DjwkTujtyVLFyUN32hGkYw6etA/AtUCp3xSLPF0W3uAasOPymv6TeGO81eXDNC2CUGdFhCPyRUy2Edd9NVo9e+KMCccOgQGdnWzByZxYJiln6fLf6egaGTx/StY9AEw+ckzx4uIoAoZx3hEeQQ7jxXhwtW+MOAHZbZA44SdNs5nUz4Gtii0UD62AyuLfVa/H485maJhopuLEjYy4q60jorDRpiUKEe6GeK5eyZS21aMjqEB6uywjnYNAPzRoZz/Nx3Ql9XK9lA== 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=3s2cunV/9skRfk454v4ZTrJayOhbaw3chH9Decl+8rM=; b=L+uaYICPEPHM2k4X4RoQM9ifxDbcJKyW6MTF8nET/r/rYQaH9BH6th/UofChVLfAfgTfIoHyVOV2kehOzA4NFwcrE3L+GLwhg9Bn8WSxvXuzzv7oCb3HwcLekyv83kw47kkB6ctKepyVaoKDshbFaGZE/J7+4E+PvgOQ8va7tlk= Received: from DM6PR11MB3673.namprd11.prod.outlook.com (2603:10b6:5:145::26) by DM6PR11MB4706.namprd11.prod.outlook.com (2603:10b6:5:2a5::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4173.22; Thu, 3 Jun 2021 06:37:25 +0000 Received: from DM6PR11MB3673.namprd11.prod.outlook.com ([fe80::751f:5d15:f9fd:2d53]) by DM6PR11MB3673.namprd11.prod.outlook.com ([fe80::751f:5d15:f9fd:2d53%7]) with mapi id 15.20.4195.022; Thu, 3 Jun 2021 06:37:25 +0000 From: "Wu, Hao A" 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 Thread-Topic: [edk2-devel] [PATCH v2 1/3] MdeModulePkg/UniversalPayload: Add definition for extra info in payload Thread-Index: AQHXWEEM+sub46P/K025ZeNX5HT7WKsB0uXg Date: Thu, 3 Jun 2021 06:37:24 +0000 Message-ID: References: <20210603062259.1390-1-ray.ni@intel.com> <20210603062259.1390-2-ray.ni@intel.com> In-Reply-To: <20210603062259.1390-2-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: edk2.groups.io; dkim=none (message not signed) header.d=none;edk2.groups.io; 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: 43b8a200-082b-46d1-db39-08d9265a0caf x-ms-traffictypediagnostic: DM6PR11MB4706: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:8882; x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 9sGDjBsGVqRrsm098LNkQgQMzrI3j2O+wdpZem0MSh6M5rPsh4GrtJbH7198uMih124Vb0owEampiOHLvGSuKb1WEn1aHngaAYe4SuP6nI541jyCu+IM8jhrt+ILeVxZoHxreVemRQMFigkkTAz0+pHJlAWAdUjL1MHsRxmCy/eOyG6YewDpwKknYnL2M01Euy+kcic2RzsuzY80ecZRS1H9vL6euB5dEhaPu7rJDT4QoKB+DN3HnHNSeO+81FlVP002tpGYGn27kHpa6uDO36h6tgrTCNXagyTr2uw8+QP2aJ3aLrwJAduP0Tz35SHk6XbH/KIKM9KSkXLPZz9H0Dp1ztJM7R9ntMsNzXamAt5EVmG7IMfogsXjq1H2URCFeFSURgB57he+c95QRrsX3MsUMXNAmXWF96oB2v/7Tj4MWa4ePr3Z0F/4XwH3CFpHXBCqroWi91i5TjVekHnH/eR1Xp3aZPH7OyLhWYSozu1xk77Yy1PJ1j3ibBPJyFq41Ysud/6P6ie2LezZuqV+myGhVObH6suljW7PkjbJV8tn5r+KKrJqg3WJ7nZXtUw/HxBOBnzhJHPu0bdoXTZwdhQKO58qtWw9rOG+HlnpG94PeyQToiOStSYha/c6BNSck4Osrfnbf8wBWfF4KDCvdAZTGo7HcgG5bYd9pMn4jkQ5d/sWM3wZb2PXX5zCIerEOQ2W3u8+6mN6sZ7hmBsTVW9sUn1aOy4mJepCh7qCFERwWi71TmBZHT7fl0+4hUDSTBH55/cprsshs/I73ugPBg== x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DM6PR11MB3673.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(136003)(396003)(346002)(376002)(39860400002)(366004)(316002)(5660300002)(54906003)(110136005)(2906002)(9686003)(52536014)(33656002)(66446008)(64756008)(66556008)(66476007)(478600001)(7696005)(4326008)(8936002)(122000001)(38100700002)(6636002)(6506007)(71200400001)(83380400001)(966005)(8676002)(55016002)(53546011)(186003)(66946007)(86362001)(107886003)(26005)(76116006);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata: =?us-ascii?Q?j9eoTuGk2wSSx3ykeFlP3vROnIQpq77p4lV+RmxJg3aUZs5HeujYZMPrkE/O?= =?us-ascii?Q?sDqzoJ5zIlO6O/SvKr0mSHM5SuiFUx3s9unUQ93rOfOlRnEIwZPu25jsJ3QA?= =?us-ascii?Q?eD5G+E53D9B0xVaMHxh4HxSjQ+a0oRDhFQoAl/0EliRL2JjgnE+CQhLztSB9?= =?us-ascii?Q?o+alB11Pm5BDkkwqsVnikKzarOPvmde8pt3MAhQzUt/gMw54BFksgvdifkCc?= =?us-ascii?Q?u9ECmlog06NqT6oeEpsF5TAJ/3/ShtjRjnqxXNB3wVMEm54KRJ5IdPaoR7bh?= =?us-ascii?Q?SlJithBvYFtS7NLzbE197H7pZIwBsRcP+/r6Ms5BjfMqLCqVX93sbXEYcNnH?= =?us-ascii?Q?SEuNwPYP2koXfV1lUGH2DqZpLOPnbc+TrN7yhaXrb/PuW3l41TiShdXQqE5y?= =?us-ascii?Q?VRj0YP+xzNOixL5WhSwEn98lwCV0WWDnow9nFKUiABSAHEoY5wTlzEsg1neu?= =?us-ascii?Q?+Nv8Q46/QG7Rg7jBqB23HViIn/b1TZpqtEKF0HgOflWNy10S5pDnELoRBOjb?= =?us-ascii?Q?IoD2Gv0y/GHg5wPFzpi+Ut1imV4JFzas6hp3gnyeEcaeDGexTlpgluyvRsuJ?= =?us-ascii?Q?qHWG1oLlmGkGMK1GO37EJcxZ9REIkcZD+pPO3ndNZRmIcCICcIZRYY85HZ4x?= =?us-ascii?Q?Nj0zTOax/Es8ijeCq2aDgQ/sEEMNA7iMIzZZpbkrrNgU9YKPN5wbcJ3Mu4gv?= =?us-ascii?Q?H0noBzNrhTgwVKHKL94cOHivWLTZOq1ORCpoi0ZeQ3HTSlsdkR9BVtCUhL5t?= =?us-ascii?Q?2rUpwtW6c2YFYceen0Xv42fXWKbGWmp2jBAFLY/KCnUt4udSd9Zh9vqoD88A?= =?us-ascii?Q?KPldeeGDarbgPCl/bLlD6B4qE+GCPWEA7mL9x68D4EfIPFeBjoUk8wJs+YfG?= =?us-ascii?Q?SqSLXPuM6NMEypLq3HnzQi/iD4K7t4DMvVeiEHzsGshB1TUsNie2EAkYFSM0?= =?us-ascii?Q?1kQUUzZMJjIf/8WzXXfeyXsiJPAS7hHik0XEHDvS+HlYVFzO4cEKdzV+GtiJ?= =?us-ascii?Q?btY5k30QRy4Q/COnu4ViF2kQRErkyFDxVSEtOmcnhrvrf0d+s6rtdqNyEKBU?= =?us-ascii?Q?2z4KMOObVXrf/pXo3V5f9YPbBq9sQ6RDiOeZ64dAm/Q7suVEonWNbLomIHtB?= =?us-ascii?Q?vGqz3k358K2SPFj+LT1UMongXKfcE+KRsfz8XyUo++SEcesWtp5F7VWzmKcS?= =?us-ascii?Q?O5hMHYmpbUDinrgQxSwbFTHaZpfVM+Ga8CoHCnso6kiZJeAAkX8tbTzb6qmr?= =?us-ascii?Q?K6eIie97YunXxClhsNkKgx3Jo6ltNrRF+t7bvXPlL4ql7tuUPM6c1W68fmvz?= =?us-ascii?Q?Hay75+rRHSb0u0aHHTfq3ZBt?= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DM6PR11MB3673.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 43b8a200-082b-46d1-db39-08d9265a0caf X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Jun 2021 06:37:24.9850 (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: bM+XLdKSP+OmgG4QNPta5pVLpEFGwSm6bs/l2uXdbRAKWdkF39vBGbqCYEfCa6T5F0JOKQmXIJMI3gJopYMieg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR11MB4706 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: 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 >=20 > 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". >=20 > 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 indust= ry standard at this moment, some definitions will be placed under MdeModulePkg= . I have a request to update the MdeModulePkg part of the Maintainers.txt fil= e to list the reviewers for these UniversalPayload header files as well as th= ose UniversalPayload related implementation source files. For the timing of this Maintainers.txt file change, I am fine with either: a) Made in one of the UniversalPayload related patch series b) Made after all the implementations are completed Best Regards, Hao Wu >=20 > 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 >=20 > 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 gPeiAtaControllerPpiGui= d > =3D { 0xa45e60d1, 0xc719, 0x44aa, { 0xb0, 0x7a, 0xaa, 0x77, 0x7f, 0x85, 0= x90, > 0x6d }}-- > 2.31.1.windows.1 >=20 >=20 >=20 > -=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 >=20