From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by mx.groups.io with SMTP id smtpd.web08.7041.1632969603276441946 for ; Wed, 29 Sep 2021 19:40:03 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=fJPLiuoM; spf=pass (domain: intel.com, ip: 192.55.52.93, mailfrom: jiewen.yao@intel.com) X-IronPort-AV: E=McAfee;i="6200,9189,10122"; a="221881957" X-IronPort-AV: E=Sophos;i="5.85,334,1624345200"; d="scan'208";a="221881957" Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 29 Sep 2021 19:40:01 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.85,334,1624345200"; d="scan'208";a="617844446" Received: from fmsmsx605.amr.corp.intel.com ([10.18.126.85]) by fmsmga001.fm.intel.com with ESMTP; 29 Sep 2021 19:40:01 -0700 Received: from fmsmsx610.amr.corp.intel.com (10.18.126.90) by fmsmsx605.amr.corp.intel.com (10.18.126.85) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12; Wed, 29 Sep 2021 19:40:01 -0700 Received: from fmsmsx603.amr.corp.intel.com (10.18.126.83) 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.12; Wed, 29 Sep 2021 19:40:00 -0700 Received: from fmsedg602.ED.cps.intel.com (10.1.192.136) by fmsmsx603.amr.corp.intel.com (10.18.126.83) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12 via Frontend Transport; Wed, 29 Sep 2021 19:40:00 -0700 Received: from NAM12-DM6-obe.outbound.protection.outlook.com (104.47.59.174) 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.12; Wed, 29 Sep 2021 19:40:00 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=axmk33Z/wTkoM65pj+3GPkd6oG4INilozw/WpwZIXKdljjCBFLJsVBSErDi0vEG/yhkrCO7TOAz81nXH9y3lM4ABmC+WCHXim7JADJ68nAsqovlpglHh/qEGQIhtM7AisSVsEYOQvC6KVePS3TK21LCdUwFr6UXyInoSz9ly0Bbs2RbuAPJ6a3qVU7b1NT/HKNGkzD+svYe8mXRQJUoxOjhTTnAKu2ZRx0wQ82x5+tYsVgfyunfzUZJbSl2Ecpp8KBWIZffL2006fD+zubooItlkzFED/f8+jWCedhdQFGPahu0GEWZLRMJvxNxXtZpTP50yLqVLfhyJ0AvrcTH9lg== 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; bh=YwDO4hyJSSHjBeBZkJSsXfKvL/oU6JrpEhsmR6mqIZc=; b=LtNRZTcC3LVNopymJoG7F2S2ew8shoXg7ZuU1UqEjU5pRcd9gbwZJQJ1UGfYQz5hKBjzXP9hDku3qavJzxBuW93Ev/LAikyOtmVbvC3AEcxWAaJKvimjWz7N72aPKaoQaYqJIJBJ9L9wKtq25e1/WTv12DieJyPbdtdychnbMye2JqXuDWWYzBAOupgD6uupOuNjiQ0L90dQgAMCFBmT2gTdbWgK9Auainh6tEPeawH7ypeY4gpze428Vmp54OqAi1/4Acq1wROlB9sz5ngaBFRHiCKzX95VeHmzMdPr+yBrTuv2zV8xkckuYYJ7RAPouqCJrm4P2i7AWIoQA29yjg== 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=YwDO4hyJSSHjBeBZkJSsXfKvL/oU6JrpEhsmR6mqIZc=; b=fJPLiuoM7Tf+kEfEkZ/TCD0enCbzjkfvQDv5ybXBe7FDmKYbZmiT7ftKzKzOmggZ7CeqS5fF7NRJ7NCUp6YldEJExRFz1nkNNxkDwfjStVhci47N2kslJGVSr92ghxvjk2al0MwsUd7is47E9vtfPXUHou8FX43us5/5pgwaL3M= Received: from PH0PR11MB4885.namprd11.prod.outlook.com (2603:10b6:510:35::14) by PH0PR11MB5014.namprd11.prod.outlook.com (2603:10b6:510:31::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4566.14; Thu, 30 Sep 2021 02:39:57 +0000 Received: from PH0PR11MB4885.namprd11.prod.outlook.com ([fe80::754e:42e9:16cd:1306]) by PH0PR11MB4885.namprd11.prod.outlook.com ([fe80::754e:42e9:16cd:1306%6]) with mapi id 15.20.4544.021; Thu, 30 Sep 2021 02:39:57 +0000 From: "Yao, Jiewen" To: "devel@edk2.groups.io" , "Chang, Abner" CC: Ard Biesheuvel , Leif Lindholm , Sami Mujawar , "Justen, Jordan L" , Gerd Hoffmann , "Schaefer, Daniel" , Sunil V L , Liming Gao , "Liu, Zhiguang" , "Kinney, Michael D" Subject: Re: [edk2-devel] [PATCH V3 00/12] Migrate ArmVirtPkg modules to OvmfPkg Thread-Topic: [edk2-devel] [PATCH V3 00/12] Migrate ArmVirtPkg modules to OvmfPkg Thread-Index: AQHXtZyniu4vyKNgf0iJbS9pUkMJ2qu73WCQ Date: Thu, 30 Sep 2021 02:39:56 +0000 Message-ID: References: <20210930004548.3033-1-abner.chang@hpe.com> In-Reply-To: <20210930004548.3033-1-abner.chang@hpe.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-version: 11.6.200.16 dlp-product: dlpe-windows dlp-reaction: no-action 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-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: ca382f6c-b8c9-4926-e5e2-08d983bb9754 x-ms-traffictypediagnostic: PH0PR11MB5014: x-ld-processed: 46c98d88-e344-4ed4-8496-4ed7712e255d,ExtAddr x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:7219; x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 8NcK8mEPVqIZxqGIM1xaaLvmkQwUWFGzoNjQ7tvsuao3GrNOeq3I4mC7dfw02yteeG4OGaGpD+jmxnP7IjsGFfhwHLBOHK5Ug7mEcG0t6HWCnvGcmA1HU+84haIQ9dv8yXDbE8zfsotiYeVYfOMl3DH6UmyPqWOk2JNPEVQ7gB0zIgtNcdB5yDbstK0iyye7CHRjRYJIwS1WOLIam0w5qRfWFuoOwoX7uVOQgaChlgcCtM2Fo/y7mIN+TiJuzPWNHKriTX9Rt17FyBoJiWaoKzmp6Mh4iYd3LEELpLdumsQ2yp/PeSN/xcvPvTZkat9IDlXnvXXlEnww7dJ2pCyicZB8FSnz4epERarCLhsb1QeWOcMgiiRn/4T8QMXfY5W+JfRCeEjoTs9IemWzUSL15X8fiynpEGJB82MMzT9RSrb69eZZ1DbTLnw+xWSEwF6zbOePbT99RK+/Wh1U6Gpq0ftrr7/o9+p8ip7hY0T+KhIaGx45Bb/0VyypakQVNx3NznKEEkpU+WAlN/L4DXvjYliHfAllYO1bycBxmu9s7ul1JtEaue/dGhk6swZubicNR/3gz+t6qYJVJDXdGm8hwJAnm9iDmjnr5sTOlgzeW/mhMTxuHn2OLgZSOKQXoVD8V320Rjv2p7UJDCCsjVake+ttJsK0O4mj6nOB1HlOohq07POvDDa8lQvQTityuGSBZMNxzxdtdY7d9cpVnf7KZT+evg3/fbZIwQgURBSePj09nYT5hAJjT60zyr+1ojmY9wg1HnVZojPucKOKMUuiNKKUei3mdQN2ampGffK/6q0PyjyYHY9xms14AFTWtw+1GKUjU9RULa8vqSk/rwLwWQZkByDBx5G6coQysiPaBHg= x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PH0PR11MB4885.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(366004)(122000001)(33656002)(2906002)(110136005)(54906003)(8676002)(5660300002)(66446008)(9686003)(66556008)(66476007)(7696005)(186003)(64756008)(4326008)(19627235002)(6506007)(66946007)(26005)(53546011)(8936002)(52536014)(71200400001)(76116006)(38100700002)(55016002)(38070700005)(107886003)(508600001)(966005)(86362001)(83380400001)(316002)(156123004);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?0BSAtPxSZJIiTJ3nB82oZ+K8UeEyP9YZK+w2ASp7z6/rhWlTOvv47uPtdl8k?= =?us-ascii?Q?O5hSlRZK94MFlFQWoONeuFGaCRDJ6c+uiWpoatLosOpDKT0vTR9qyjIhYtmn?= =?us-ascii?Q?xcxLui96NjMB1LdrCrfgOLYz/T4osrwd2AyjXwkzHHVKQPQmVzettTt4bSi4?= =?us-ascii?Q?w4Wl2yJ4pUpth04lio418yVy8QEp2dQOupBjETLDzOw8B7WlB3nvYhIWrTLt?= =?us-ascii?Q?bGEzEd58ShlEh0mc98Ihm4/BI/KrFLkFcRSOR8DCzzXm9ofgfa+a4Xsbgsi8?= =?us-ascii?Q?MA2VA2XZKnHE5ln0Q90GTQRDWbkKHg3kgY6NvpzM1JbBi+64MgEOMVjcwwho?= =?us-ascii?Q?aJhIbWkVPz5SR7NlHyWgZL9lyoU6MtRi/uPdc1mbyhZ5m3d3wN4OWWQy2kgl?= =?us-ascii?Q?DHO2jr/t/PGY9ZqbCcOTyKBW8kmOBslT3ZK1jK+3XmgxJQS9qRaVoATcVSlD?= =?us-ascii?Q?E6hahpUBRY4Lv5ycTFtM1M+njL3ScvVi68VLo+zKYT2cs0A8SDt2Km0UduoK?= =?us-ascii?Q?FPgwr3n3f0bG+y8KE/g71MXLtd2Jv12QY1AXaj1xce6y2GQ22jehE36PIZvS?= =?us-ascii?Q?mubcCbprEBcIwjKWzb3nvdqktkIYZabjXNTbRwrovdMoUMTwyKUpuCFocGV0?= =?us-ascii?Q?Ji8ozpXyQApdtFRcbHQmSljD090SioehjWyDfRktyjkZgZb6Ps2kjoswqkZX?= =?us-ascii?Q?/tnqiFOd6c+8ymFtL8hyOZJV47xJaqpB2wTjH4kwBQ1rP/KZrENsm4PbN2jr?= =?us-ascii?Q?c4OK3xpQngA2VIUev34zemqvov10YyZwsG4s18sW1mPip68oRkDZRYMCYCsB?= =?us-ascii?Q?zCzfT0sQaxX/Tg9I4VfSmWIMk1UWmiFzP12WDlbvoNFRwIRLEfA2YqLgAowj?= =?us-ascii?Q?ucjlYYm0EMTP3fYT4I+UOqAAw6Jri0yWXeAWOb9oXstzc8oDlyH7shfDEsN4?= =?us-ascii?Q?EiUxgEhh+aKBli5KKJXMdpzeU7jG25vOWS+EFz9lDSejGVw8Tl3n2yWY+D56?= =?us-ascii?Q?kubSbaVtznwLe6HeAb9LlTqAj1GRPSjucWSaeapcdkgrTp8J6wxltWDVVSDk?= =?us-ascii?Q?N49YSq+j8+4pcyyCOVFcJS2GSaxYckDv/nnYwoKVCZ5wDWe8nu5lUGTptFo2?= =?us-ascii?Q?z4I0BVn7RSFwepgSUd46A/obVmth5G4VUzgaHoHDhTlOzaSgAgSnpdKBuzGa?= =?us-ascii?Q?S/Mb8xrJNjlTQxWsCfOQAe9Namfcmc3nBJrFHMoTF1VfM0Yse4dhTrnvkC/q?= =?us-ascii?Q?1QajPm/eGXwRkXtGgrfob59jQqvcKqAFuQ4ao2Yx7Co1P2dRgaIqjTb6K5+X?= =?us-ascii?Q?pYy+29ySj5DB9dreH2wvjjqZ?= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: PH0PR11MB4885.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: ca382f6c-b8c9-4926-e5e2-08d983bb9754 X-MS-Exchange-CrossTenant-originalarrivaltime: 30 Sep 2021 02:39:56.9159 (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: oWZ87iqLvOBY6lx0PFDyV7eoFP7x8qJ+xUAJVUh4bM0oOMqCM0AGFhjooL8gYLgIHSI8/lJmaM8XNeO5pqDpRw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR11MB5014 Return-Path: jiewen.yao@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable For OvmfPkg change, Acked-by: Jiewen Yao Need ARM/RISC-V expert to give Reviewed-by. Thank you Yao Jiewen > -----Original Message----- > From: devel@edk2.groups.io On Behalf Of Abner > Chang > Sent: Thursday, September 30, 2021 8:46 AM > To: devel@edk2.groups.io > Cc: Ard Biesheuvel ; Leif Lindholm > ; Sami Mujawar ; Yao, Jiewen > ; Justen, Jordan L ; Ger= d > Hoffmann ; Schaefer, Daniel ; > Sunil V L ; Liming Gao > ; Liu, Zhiguang ; Kinne= y, > Michael D > Subject: [edk2-devel] [PATCH V3 00/12] Migrate ArmVirtPkg modules to > OvmfPkg >=20 > In V3: Address comments on V2. > In V2: Remove HPE license on the files that just moved around or > the changes in the file are just code removal. >=20 > edk2 BZ #: 3665 > edk2 platform corresponding changes will be submitted after > this pactch set is reviewed. >=20 > This pacthes set is to migrate some modules from ArmVirtPkg > to under OvmfPkg for the upcoming RiscVVirtPkg that can leverage > those modules without the dependency with Arm*Pkg. >=20 > The modules moved from ArmVirtPkg to OvmfPkg are, > - FdtClientDxe > - PciPcdProducerLib > - HighMemDxe > - QemuFwCfgLib > - FdtPciHostBridgeLib > - VirtioFdtDxe >=20 > Below PCDs are moved to under MdePkg and leverage by RiscVVirtPkg. > This change also remove the dependency on ArmPkg of OvmfPkg. > - PcdPciIoTranslation > - PcdPciIoTranslation > - PcdPciMmio32(64)Translation >=20 > Signed-off-by: Abner Chang > Cc: Ard Biesheuvel > Cc: Leif Lindholm > Cc: Sami Mujawar > Cc: Jiewen Yao > Cc: Jordan Justen > Cc: Gerd Hoffmann > Cc: Daniel Schaefer > Cc: Sunil V L > Cc: Liming Gao > Cc: Zhiguang Liu > Cc: Michael D Kinney >=20 > Abner Chang (12): > ArmVirtPkg/FdtClintDxe: Move FdtClientDxe to EmbeddedPkg > MdePkg: Add PcdPciIoTranslation PCD > ArmPkg: Use PcdPciIoTranslation PCD from MdePkg > ArmVirtPkg/FdtPciPcdProducerLib: Relocate PciPcdProducerLib to OvmfPkg > ArmVirtPkg/HighMemDxe: Relocate HighMemDxe to OvmfPkg > OvmfPkg/HighMemDxe: Add RISC-V in the supported arch. > ArmVirtPkg/QemuFwCfgLib: Relocate QemuFwCfgLib to OvmfPkg > OvmfPkg/QemuFwCfgLibMMIO: Add RISC-V arch support > MdePkg: Add PcdPciMmio32(64)Translation PCDs > ArmVirtPkg/FdtPciHostBridgeLib: Relocate FdtPciHostBridgeLib to > OvmfPkg/Fdt > OvmfPkg/FdtPciHostBridgeLib: Add RISC-V in the supported arch. > ArmVirtPkg/VirtioFdtDxe: Relocate VirtioFdtDxe to OvmfPkg/Fdt >=20 > ArmPkg/ArmPkg.dec | 15 ++++++-------- > ArmVirtPkg/ArmVirtPkg.dec | 3 --- > EmbeddedPkg/EmbeddedPkg.dec | 1 + > MdePkg/MdePkg.dec | 12 +++++++++++ > ArmVirtPkg/ArmVirtCloudHv.dsc | 18 ++++++++--------- > ArmVirtPkg/ArmVirtKvmTool.dsc | 18 ++++++++--------- > ArmVirtPkg/ArmVirtQemu.dsc | 20 +++++++++---------- > ArmVirtPkg/ArmVirtQemuKernel.dsc | 20 +++++++++---------- > ArmVirtPkg/ArmVirtXen.dsc | 2 +- > EmbeddedPkg/EmbeddedPkg.dsc | 1 + > ArmVirtPkg/ArmVirtCloudHv.fdf | 6 +++--- > ArmVirtPkg/ArmVirtKvmTool.fdf | 6 +++--- > ArmVirtPkg/ArmVirtXen.fdf | 2 +- > ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc | 6 +++--- > .../ArmPciCpuIo2Dxe/ArmPciCpuIo2Dxe.inf | 2 +- > .../ArmVirtGicArchLib/ArmVirtGicArchLib.inf | 1 + > .../ArmVirtPL031FdtClientLib.inf | 1 + > .../ArmVirtPsciResetSystemLib.inf | 1 + > .../ArmVirtTimerFdtClientLib.inf | 1 + > .../KvmtoolRtcFdtClientLib.inf | 1 + > .../NorFlashKvmtoolLib/NorFlashKvmtoolLib.inf | 1 + > .../NorFlashQemuLib/NorFlashQemuLib.inf | 1 + > .../XenAcpiPlatformDxe/XenAcpiPlatformDxe.inf | 1 + > ArmVirtPkg/XenioFdtDxe/XenioFdtDxe.inf | 1 + > .../Drivers}/FdtClientDxe/FdtClientDxe.inf | 1 - > .../FdtPciHostBridgeLib.inf | 11 +++++----- > .../FdtPciPcdProducerLib.inf | 5 ++--- > .../Fdt}/HighMemDxe/HighMemDxe.inf | 7 ++++--- > .../Fdt}/VirtioFdtDxe/VirtioFdtDxe.inf | 2 +- > .../Library/QemuFwCfgLib/QemuFwCfgLibMmio.inf | 6 +++--- > .../Include/Protocol/FdtClient.h | 0 > .../Drivers}/FdtClientDxe/FdtClientDxe.c | 0 > .../FdtPciHostBridgeLib/FdtPciHostBridgeLib.c | 0 > .../FdtPciPcdProducerLib.c | 0 > .../Fdt}/HighMemDxe/HighMemDxe.c | 3 ++- > .../Fdt}/VirtioFdtDxe/VirtioFdtDxe.c | 0 > .../Library/QemuFwCfgLib/QemuFwCfgLibMmio.c | 7 ++++--- > Maintainers.txt | 6 ++++++ > 38 files changed, 106 insertions(+), 83 deletions(-) > rename {ArmVirtPkg =3D> EmbeddedPkg/Drivers}/FdtClientDxe/FdtClientDxe.i= nf > (92%) > rename {ArmVirtPkg/Library =3D> > OvmfPkg/Fdt}/FdtPciHostBridgeLib/FdtPciHostBridgeLib.inf (77%) > rename {ArmVirtPkg/Library =3D> > OvmfPkg/Fdt}/FdtPciPcdProducerLib/FdtPciPcdProducerLib.inf (87%) > rename {ArmVirtPkg =3D> OvmfPkg/Fdt}/HighMemDxe/HighMemDxe.inf (83%) > rename {ArmVirtPkg =3D> OvmfPkg/Fdt}/VirtioFdtDxe/VirtioFdtDxe.inf (92%) > rename ArmVirtPkg/Library/QemuFwCfgLib/QemuFwCfgLib.inf =3D> > OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgLibMmio.inf (86%) > rename {ArmVirtPkg =3D> EmbeddedPkg}/Include/Protocol/FdtClient.h (100%) > rename {ArmVirtPkg =3D> EmbeddedPkg/Drivers}/FdtClientDxe/FdtClientDxe.c > (100%) > rename {ArmVirtPkg/Library =3D> > OvmfPkg/Fdt}/FdtPciHostBridgeLib/FdtPciHostBridgeLib.c (100%) > rename {ArmVirtPkg/Library =3D> > OvmfPkg/Fdt}/FdtPciPcdProducerLib/FdtPciPcdProducerLib.c (100%) > rename {ArmVirtPkg =3D> OvmfPkg/Fdt}/HighMemDxe/HighMemDxe.c (95%) > rename {ArmVirtPkg =3D> OvmfPkg/Fdt}/VirtioFdtDxe/VirtioFdtDxe.c (100%) > rename ArmVirtPkg/Library/QemuFwCfgLib/QemuFwCfgLib.c =3D> > OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgLibMmio.c (93%) >=20 > -- > 2.17.1 >=20 >=20 >=20 >=20 >=20