From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by mx.groups.io with SMTP id smtpd.web10.48960.1680246689167449098 for ; Fri, 31 Mar 2023 00:11:29 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="unable to parse pub key" header.i=@intel.com header.s=intel header.b=DRvb0FFi; spf=pass (domain: intel.com, ip: 134.134.136.31, mailfrom: ray.ni@intel.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1680246689; x=1711782689; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-transfer-encoding:mime-version; bh=WUOajnQTUV4QhNZUa05todp/tS7vTSwYUHVAeYbyNmI=; b=DRvb0FFivRRR52nODjTC/kS6QvVxICSNGGUtigTJv4ktzVSZlXahvOFA hf5yVNAVdKaYBO6cH5d4wIXo8DlON5THtw7NgKagI1EX/AnWEfdVty6nl 2CIKGPPg/ogEk97lO7cc66Bk5D5KS/fbOp3DGbss5HjK7PbsAoTQ900+N Plio/aB8/fiQwUlouGaYbiV4swoR0F8CUF++LrXSUJW8FfLWi1LSI9xww vwm5nYRDVSXGiW3k/tG+FFnLMvbcBxJafZG68PNVUi+4mnodNV9Fz0EvX 0OY72y9J6DcD70BtZeSzaG0pb+LhQ+pd7DeCiGxBBEv90Y1dhSTkFLiPm A==; X-IronPort-AV: E=McAfee;i="6600,9927,10665"; a="404093640" X-IronPort-AV: E=Sophos;i="5.98,307,1673942400"; d="scan'208";a="404093640" Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 31 Mar 2023 00:10:14 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10665"; a="685004466" X-IronPort-AV: E=Sophos;i="5.98,307,1673942400"; d="scan'208";a="685004466" Received: from orsmsx602.amr.corp.intel.com ([10.22.229.15]) by orsmga002.jf.intel.com with ESMTP; 31 Mar 2023 00:08:50 -0700 Received: from orsmsx602.amr.corp.intel.com (10.22.229.15) by ORSMSX602.amr.corp.intel.com (10.22.229.15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.21; Fri, 31 Mar 2023 00:08:50 -0700 Received: from ORSEDG602.ED.cps.intel.com (10.7.248.7) by orsmsx602.amr.corp.intel.com (10.22.229.15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.21 via Frontend Transport; Fri, 31 Mar 2023 00:08:50 -0700 Received: from NAM12-BN8-obe.outbound.protection.outlook.com (104.47.55.172) by edgegateway.intel.com (134.134.137.103) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.21; Fri, 31 Mar 2023 00:08:49 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=hTiMS2nafgmmsBfu/yQR/Xno9r5cG2sZStQ81p8MJV6z5p7935LctWbwKTR6rq1x7rGXlPTlrHU3dKLGdmMIifY6Snv/kiJu7IDVlkJafJEGH9+OlmONZQOMXcy9weNx25ly2ioMzLc29VmLOj6mYMVRRcQlQkkYz8Hr4dGNK3mHIzJMS0X/pFTpbBE3UEv4hVHYoLEP4LvrDg6APL2EdYygBsA+8F+LsvfJJETCL0N9N50EUcNxGuBAgDmEj2MQEFnP5dK8rcYa2Ri74GxZpBp+/2/B15AFeUiv+aIxEBshBZIFaV2BNytpgP/wU+jlXxUzAHNMxRvIr+FYgLTisA== 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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=fEHHLSuwMq493GqjKrGPUhDK1cYQ5OGjQC630SI2FQk=; b=YPMTehX3vogqlI0c8AmzWITWAuVzTSpyWB1t8M9a4rGejOgtH2UNX2juoxU9Uz8fNhkNEr2IMHz/32w8WLuKU6o96jc/39Rvm9O/oolXpOjiMoNBDgFLw2EKjTGZH9+pHL4FB332qb+Ceqh9k27MMdOg7R+dcI8MOBNnyWOY0LzKGcWbSdsaMgi6Mm3O73uSHSjMPisNuOVWFMCP38Nji9nw4m1upOfpGP3LHj8LtPIgTC9xqXqKqLLMZxvCNCNkANSSJVPXJN4SumZPS7SrsLwgDzkHIGqfnLleGArSPkwHQYD4IlFhfk1NH2E/2NiyJfQrhy0mDeuXiCcipuHk8g== 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 Received: from MN6PR11MB8244.namprd11.prod.outlook.com (2603:10b6:208:470::14) by SJ0PR11MB5769.namprd11.prod.outlook.com (2603:10b6:a03:420::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6178.38; Fri, 31 Mar 2023 07:08:47 +0000 Received: from MN6PR11MB8244.namprd11.prod.outlook.com ([fe80::ae07:e96a:4a24:8a69]) by MN6PR11MB8244.namprd11.prod.outlook.com ([fe80::ae07:e96a:4a24:8a69%8]) with mapi id 15.20.6222.035; Fri, 31 Mar 2023 07:08:47 +0000 From: "Ni, Ray" To: "Tan, Dun" , "devel@edk2.groups.io" , "Kinney, Michael D" CC: "Gao, Liming" , "Wang, Jian J" Subject: Re: [PATCH 6/9] MdeModulePkg: Add UefiCpuPkg.dec to pass DependencyCheck Thread-Topic: [PATCH 6/9] MdeModulePkg: Add UefiCpuPkg.dec to pass DependencyCheck Thread-Index: AQHZYR8z2XAsvgF6AUyW/DmfsKXe268UfB7A Date: Fri, 31 Mar 2023 07:08:46 +0000 Message-ID: References: <20230328024302.2085-1-dun.tan@intel.com> <20230328024302.2085-7-dun.tan@intel.com> In-Reply-To: <20230328024302.2085-7-dun.tan@intel.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=intel.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: MN6PR11MB8244:EE_|SJ0PR11MB5769:EE_ x-ms-office365-filtering-correlation-id: 627beed9-f380-4e94-0551-08db31b6c576 x-ld-processed: 46c98d88-e344-4ed4-8496-4ed7712e255d,ExtAddr x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: RYHYGVyLYEh/5Nyk7Z2ISfckmmjiuUVfyWD2nXXv6Sop0sg3NCi6pblTbcTlvtHnV9gI3sdp1URkWYep9+FnuzJLQlA/vWk/iFf0WErlX4zFrIWsmI0g73ivn1XFdCZCjxIr4gg9nUseF2o/n95QEB75a8ufQKLH1ASdLWJLOcJGTnLGLjP5F+NNYQqshiYl/3uAre8tTJklu6px5OdoH/pI+1ukiON29CGDvkK7LqKfFEe9sR5l/Qb7b8p4qb/PZOy6LvVdZVSBtlsHRYI2qLo/m0jsCVxNR6LTu2OdDI/q1yoJfBUJsUpkv8MklAwrw6bOfBUu6Fdmo2MiuhlkFef8BsoBHxtRLq1z2sFe7u9GXWoqL3w1U3TzedAX9Zkgc+dZGBonVdGoBMEPKG/zPk0Pw3eFb7qJbrdPhgOaSww8be1dG7iAmtE21E6BMbzU6BauvYWiRH+7kSdxV0ZaBtmX7dCNgKDvI6lOu+hiUP2jKR8NQz5SqHkBWtDqgEf5h/nCdh74mGaUl0nJDcdyb7ITIvgXuj3w79fTB4gM8BdO4rMzN3JRtu8Xr2v5nfO5NppvpRKSd7cpKBZdp6i2j/b/2JQ9uLQFIsg2AvCoqT/02wOGNM1IgRPaHzgePJmm x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:MN6PR11MB8244.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230028)(136003)(346002)(39860400002)(396003)(376002)(366004)(451199021)(82960400001)(4326008)(122000001)(52536014)(8676002)(8936002)(186003)(76116006)(66476007)(478600001)(53546011)(66946007)(66556008)(66446008)(55016003)(110136005)(7696005)(45080400002)(54906003)(38070700005)(6636002)(64756008)(71200400001)(9686003)(2906002)(83380400001)(316002)(33656002)(41300700001)(38100700002)(5660300002)(107886003)(6506007)(86362001)(26005);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?UMEEaHCSS5p9o3dcN8MRjlAhLr7eo1L+pqXxt/s9IHmWH7vpO2RFq20JyS2y?= =?us-ascii?Q?aTbgoBIFQQef2MXI/lWpT78cV/2xHiH/XKx7Lhed620dtEBB1eU3wh2O7a5J?= =?us-ascii?Q?tTmHAAhagrRJ5Pc317fFUEyhrgqksda1Mq3qHFU5ZrgSdNCFI1VcXgDTantP?= =?us-ascii?Q?Na0IS8Iqz5SxiAdr7IjtNjRT8KLJpMf0diCwnC+7WsMDnjxhKsl9Dhn0Mrhb?= =?us-ascii?Q?naGHLoPqP+hl/5xAK6LROCX8iCkuqDHurdqI51YAeyg9LdI1IpBu55eKZHZF?= =?us-ascii?Q?kObBm0Xp+VH9YQrmJ9zlZaidP1wp6xdYPECyMOszf7J6lMyiS429t3mUcIZA?= =?us-ascii?Q?HDTffrxk1luLkKy87AG9T1kC6CykMutha4SNCd5pvY2pu+v0JsCiEt9pX940?= =?us-ascii?Q?XAtPTL9nkH/Oe9tnpKsP1qaWjB8anGY4rSGNjDuoabalzVFEiQUc3qfguv2l?= =?us-ascii?Q?TymS/RtK2N/8ws8zQ5rVF9Ds3KwkcEuJlW9AQF/sMn/JcgTClhRPoojrnOXu?= =?us-ascii?Q?oYpwJe2b7R+ZrY38/Bu9/EhUldPR5AgsSGBxq1y6qBDHSBftFu/q93zqMJGD?= =?us-ascii?Q?MDasWv7dLNo5moCfwIphfyk8XVL2CRIfZAAXY3R6+BoQwRcI0Nn0Vf9R9Ugj?= =?us-ascii?Q?fVk74kiV7iZZpxfZsK6RlKfZZ9Wdfa2DoM8l7t490Ji4FeehGjfjumh7jjrC?= =?us-ascii?Q?F03Y8sFaDCbmDKXw+EezPK5GO5Qa4ZL6Eu0j3Y/Lqks6FFEjflcx4i5KIwyS?= =?us-ascii?Q?46ABZZoyDHwTj9qTYJEH1rBWIk+nGSQylCGxwk412sYkZ700NIAbVWAwuAnG?= =?us-ascii?Q?DePUU0hC3ibg5RNBK/2k5C4ydqeL7pufXLlh7FjzaO/4Mk02pSXqlMfCOogY?= =?us-ascii?Q?Of1ZZGy/bKoPwvbeJTedb8w+UzLDZRGZP9h1zzPkIU6IrZ/3FzPvTvJsY+3r?= =?us-ascii?Q?FuT3lJtJJJ+o/54uOakAMAUnTSSIwaBVs0qcISpikLPKJWKWHjpMlXe2+AnK?= =?us-ascii?Q?4jyKKQpXfX/wQUUA5H67SerF6r1c3GJjDCuZD/srD9zs2Gj4z5di3mHQGiRN?= =?us-ascii?Q?szbWRG5PaBJiqp5ziQADZnG4y3St0E6GFeKNpzGRxvUiOTvmH96uTqAAWTZV?= =?us-ascii?Q?AXsHB/I8xj+NAtj/OtscpuA84K46F8coDY+KhL3cyEySxKYqHPcr1SvvUeSc?= =?us-ascii?Q?gUzDxRLEvSc9qaWL/k3i7Ikswuwm6UAFugdYRzXYjI8mW0Ckk3x33UMkqotw?= =?us-ascii?Q?DysnC9lrQ6yedRBT2TRRPpPoffSKFtx+fV7A6FGFpCK+s+a3K3b7rn56B7qL?= =?us-ascii?Q?tGeooPDcJKoJG7HxIhucbDauJEFaaKlOGqfRuQl8VOnCDXPqzrNkr33sLDPR?= =?us-ascii?Q?bcUa4NJz4ntMYnnLCUJUdj5vvBOrJYXO89d7zBdN9GejgbaMOywUrgQxC1hV?= =?us-ascii?Q?XLJuivXn9OMU9TxstHFrvIMhvCjAIIw2KeJ5FekY/DS4IGRMuAMzNxOpMcJB?= =?us-ascii?Q?tufIcw8RRiUKGv4+3yZWj7gg5oOwPV+KYV1J6SlLAHZB9qIg+d7zACkH9sbf?= =?us-ascii?Q?Ufw37LLe9HmRKjJFbpk=3D?= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: MN6PR11MB8244.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 627beed9-f380-4e94-0551-08db31b6c576 X-MS-Exchange-CrossTenant-originalarrivaltime: 31 Mar 2023 07:08:46.9447 (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: zNl7ccl6bu1GG0q1quCTKUyq4rLUKR2siN6pMpr15FVhg57gPZQZOWgGOPzb7z44N4vofWOyuVo2Ro9rdOO4Vg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR11MB5769 Return-Path: ray.ni@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Mike, What's your comment for this patch? PageTableLib is in UefiCpuPkg. DxeIpl requires PageTableLib for Ia32 and X6= 4 versions. I prefer that we can move the pkg dependency rule discussion as a separate = topic and only review the code logic in this patch series. Do you think that we can loosen the dependency for the DxeIpl because it's = really arch specific? Thanks, Ray > -----Original Message----- > From: Tan, Dun > Sent: Tuesday, March 28, 2023 10:43 AM > To: devel@edk2.groups.io > Cc: Gao, Liming ; Ni, Ray ; W= ang, Jian J > Subject: [PATCH 6/9] MdeModulePkg: Add UefiCpuPkg.dec to pass DependencyC= heck >=20 > Add UefiCpuPkg/UefiCpuPkg.dec in MdeModulePkg.ci.yaml to pass > DependencyCheck since DxeIpl in MdeModulePkg needs to consume > CpuPageTableLib in UefiCpuPkg. >=20 > Signed-off-by: Dun Tan > Cc: Liming Gao > Cc: Ray Ni > Cc: Jian J Wang > --- > MdeModulePkg/MdeModulePkg.ci.yaml | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) >=20 > diff --git a/MdeModulePkg/MdeModulePkg.ci.yaml b/MdeModulePkg/MdeModulePk= g.ci.yaml > index f69989087b..d2616f4cdc 100644 > --- a/MdeModulePkg/MdeModulePkg.ci.yaml > +++ b/MdeModulePkg/MdeModulePkg.ci.yaml > @@ -2,7 +2,7 @@ > # CI configuration for MdeModulePkg > # > # Copyright (c) Microsoft Corporation > -# Copyright (c) 2020, Intel Corporation. All rights reserved.
> +# Copyright (c) 2020 - 2023, Intel Corporation. All rights reserved.
> # (C) Copyright 2021 Hewlett Packard Enterprise Development LP
> # SPDX-License-Identifier: BSD-2-Clause-Patent > ## > @@ -51,7 +51,8 @@ > "MdePkg/MdePkg.dec", > "MdeModulePkg/MdeModulePkg.dec", > "StandaloneMmPkg/StandaloneMmPkg.dec", > - "ArmPkg/ArmPkg.dec" # this should be fixed by promoting an = abstraction > + "ArmPkg/ArmPkg.dec", # this should be fixed by promoting an= abstraction > + "UefiCpuPkg/UefiCpuPkg.dec" > ], > # For host based unit tests > "AcceptableDependencies-HOST_APPLICATION":[ > -- > 2.31.1.windows.1