From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by mx.groups.io with SMTP id smtpd.web11.19874.1644224953469014787 for ; Mon, 07 Feb 2022 01:09:13 -0800 Authentication-Results: mx.groups.io; dkim=fail reason="unable to parse pub key" header.i=@intel.com header.s=intel header.b=A4InrUt6; spf=pass (domain: intel.com, ip: 192.55.52.120, mailfrom: chasel.chiu@intel.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1644224953; x=1675760953; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-transfer-encoding:mime-version; bh=XbMJ+UYEMPh4fKvblq1vfL6UiU6VsDDI8lpCvRVhVTg=; b=A4InrUt6OeURaRhwuF/qI+5RXZHpTETV6EDC216JrrtnB8tF/v6qG9UK B6vOG19HR9s3D+/0e42PtL6J0cLcuGYpjm7BMSYFDJUmP3YPP/2LL3Bvu jl9/CkyR+LB47RHkAo5pSCbCmbwEz7NIRom3qmBibL9Rl4vCqJKu9ac1N 3MyRfI2KEJbpmJtHzLb88tWEQ/YWRbUS0Ia24iVK398Npn4aFF0DYXBWL 67vmbA5CquIJL6U9e5y2HOhjnB/qWrkfDY3xJuYGA/Wbs+E+SkdjKouJc PFlwei2Hcvg8Qs0c5FxxcYQ42upIQgVRfE6auzwvUOjcLl1TvAj+S1Rta w==; X-IronPort-AV: E=McAfee;i="6200,9189,10250"; a="247511124" X-IronPort-AV: E=Sophos;i="5.88,349,1635231600"; d="scan'208";a="247511124" Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga104.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 Feb 2022 01:09:12 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.88,349,1635231600"; d="scan'208";a="621481089" Received: from orsmsx605.amr.corp.intel.com ([10.22.229.18]) by FMSMGA003.fm.intel.com with ESMTP; 07 Feb 2022 01:09:12 -0800 Received: from orsmsx609.amr.corp.intel.com (10.22.229.22) by ORSMSX605.amr.corp.intel.com (10.22.229.18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.20; Mon, 7 Feb 2022 01:09:12 -0800 Received: from orsmsx612.amr.corp.intel.com (10.22.229.25) 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.2308.20; Mon, 7 Feb 2022 01:09:12 -0800 Received: from ORSEDG602.ED.cps.intel.com (10.7.248.7) by orsmsx612.amr.corp.intel.com (10.22.229.25) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.20 via Frontend Transport; Mon, 7 Feb 2022 01:09:12 -0800 Received: from NAM04-MW2-obe.outbound.protection.outlook.com (104.47.73.174) 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.2308.20; Mon, 7 Feb 2022 01:09:11 -0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=lpvCHwUWiks4lX610fu3Pie7cWnEh86cxJnKIdqxtl58+GCXN1G1oGUxbGI4zpLg6P/WmfJ25BaIfA3qf1rffnc3oAbnKkjgY+ucds9Z1MAy9L4TeRGPDkU4veL5BSm0J+imMp6YmTdl5jY34eFKFIY4ON/hXyNJln1jHxrGHDkqjOmrZRauQOxHbzby5Pd8LjIeV0wji9z5G9a0+yuc3kfOg7QMh4UsJ+ORaInBEf9opFv1ISLcWiehY3B2EY569II6YefXkAyNoBCyfgH8io0lq+c65/f2k3U3kBS1Yz5ZJdgZv3x3lJNLcGFkpIGVwbUBYQQ14bplJk1TQrYBnQ== 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=e11ub5qLWZy0KgGs/Z8TIutfAtUKBqdw/Y3zFLzvRDY=; b=Qqz1rSAV6YumA0f+2XAoNSDX6IQcj2Lohn2VRlD5h9qFGNrYYY5vlpDS/oKmji6IUpwytlJWf+/wMHgxXYpZICLA1rTnq8D47Cl+62iPaK/5/AL4L9T+/3uxv6KY+FLo58F0EhTR4n4zkkhMpTgO4N2GF7HorSiyY2C7Xd3YMx6rv/Ke2/KYN1Jy+99WfhEMdXxEbOUAyoWutY3gQnGpHLc9gGmnF+5ig3Bnu0TsZUxf5EemlommrcPLvn+PdTpGTziUpU+FjJ6hJpSmmT71VIftbDr4Sf3ApIfQDx5NOBuy+D5SxY1or101xdWjx7kEjaAlcZAJQC+TkZV+oCs/sw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none Received: from BN9PR11MB5483.namprd11.prod.outlook.com (2603:10b6:408:104::10) by BN8PR11MB3697.namprd11.prod.outlook.com (2603:10b6:408:8e::26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4951.11; Mon, 7 Feb 2022 09:09:10 +0000 Received: from BN9PR11MB5483.namprd11.prod.outlook.com ([fe80::c8ff:c95c:e601:eee3]) by BN9PR11MB5483.namprd11.prod.outlook.com ([fe80::c8ff:c95c:e601:eee3%5]) with mapi id 15.20.4951.018; Mon, 7 Feb 2022 09:09:10 +0000 From: "Chiu, Chasel" To: "Zhang, Xiaoqiang" , "devel@edk2.groups.io" CC: "Desimone, Nathaniel L" , "Oram, Isaac W" Subject: Re: [PATCH v1] MinPlatformPkg: Install memory relating PPIs Thread-Topic: [PATCH v1] MinPlatformPkg: Install memory relating PPIs Thread-Index: AQHYG/pg24FyZkDYAUeUnHtc4Dwo8KyHzGhQ Date: Mon, 7 Feb 2022 09:09:10 +0000 Message-ID: References: <20220207081109.1094-1-xiaoqiang.zhang@intel.com> In-Reply-To: <20220207081109.1094-1-xiaoqiang.zhang@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.6.200.16 authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=intel.com; x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 64cd80f5-6407-4ceb-99d5-08d9ea1980e8 x-ms-traffictypediagnostic: BN8PR11MB3697:EE_ x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:2449; x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 3m8Yw83UY5zhHpWmHowmpoLmY+Gb9rFU8Gj0LlHWsEEydx8L43CyCKjYhY3iWcUzXABfrTN9TwzZBPqZ7O6guTfoFTyGwCds/3DxWduwj4MrU3e2tr4YBqI31g5EfkbirNd5liy/tSj3kAJrwpfonNvynfjFDq4sC3AtYXlsC23dA2WUETEGLXztsr5ZSPfvZ4xATthKNMOvDcsG3D+Md2+eWMDjbsmmMiz9uO5PHbflsPcGqtd5Xwc+bQGFh84PjH/z8ezir4GUredP7c7EYmvhuO+lqZfg/V3yyxOJB4DwdI+NsT3DpKNLeVTqFPGIJMvg3JBEf4ZmY1pcmSU6Fez9brwlwCqhtPYCFtAF7FbNW7m00dNrIeP6faXpZOfDBQ4JCidK7Nbe/aGZR1tzccfXX/yzHqDP2qcGdH6gSxoTevTXTkNS3dOy9GE0seEbfG10LnrH8xvM7ktpmiCQWIXAke9afTHwJR/8VzpcUbCTNIGOvyrcVG+uOMxWozy36Ly8Xw5L0D0isBRgouQk8iJ8elqyXq6zbitW/v8VdTVywOylzwGAYtxQQkD/s5EOHSqeTXtZHtjINbdjt4VIokJISIkuVpSF8D1ApIB74jyCWVIke0q0zq1vlEsPINHViwknvmvQiuK1nPG3AkZie/loKCkLTnPAvlHJIxetmPm8Lr4GBjWf9oOmxLZWiA8E367/6v0jl/ek1SNR9cZW8iYn0udSTnN8MVG6SJTyEiuAJez+IJH08e8Y56u5pCQagZ0zPkyzdEoYQ3IV3GaZ4oWRdutk6Wf8wfv2fRJF/w214K1lqEXEQjhVrorikuHl x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BN9PR11MB5483.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230001)(366004)(38070700005)(33656002)(186003)(26005)(2906002)(64756008)(4326008)(66446008)(8676002)(8936002)(55016003)(86362001)(66946007)(66476007)(66556008)(76116006)(38100700002)(9686003)(83380400001)(54906003)(6506007)(508600001)(966005)(82960400001)(52536014)(71200400001)(53546011)(7696005)(107886003)(110136005)(122000001)(316002)(5660300002)(19627235002)(11716005);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?t+LsNr89UAEbTQmOh+Dd2DWqoLyRAj6Qy2jjUJYlSY0/jiTfZzLWgQfRxqxM?= =?us-ascii?Q?QDMnZleeJEPNaVS0ZkYZX39n9qgakOxvH3wzLLrN8Fx3Y7BjVUH0/qIhW5Qg?= =?us-ascii?Q?liSHAFLrNuFpCUwvOxMssReuJBivxzya00WbRvqgEYvxdhwFBuZM3kxPwPB3?= =?us-ascii?Q?BTrceHpo1iCaHL+aRdb/JERSwK9lzCuQ5PeByGDrYoG5afGj2jxZiTAq2btb?= =?us-ascii?Q?oS+xuSQdeGlzb3euzgjPBQp8x4ooabK+wWsI/c2Q/1r4DeMOQ8B0DirL7JJ2?= =?us-ascii?Q?yI4qKoDU/8QuHsnlNRqY0reCHwV0rn7RexVu3htQFIP6tnFqEHU3ZF7woOG9?= =?us-ascii?Q?KPrQeXGiHgBCrqt0ImfMwXWCaA8nNBeviggFYYX/f0Mt8RcN6+aIwPIQdZw9?= =?us-ascii?Q?0ZL9IBVjvIT0k8dTEwXrTq+OZmfW77g3hbRKc70+5+YYMeVkuAvAUmyJSaX7?= =?us-ascii?Q?cRilMihpmnNNUwAr9U3PP7jaAqrQLJNje6R9YMf+2b4xyvZifg8KGdx+H0Qm?= =?us-ascii?Q?cLnqennnZUbrvyegXljDcsFOpcZ2TujFhH9nEQ41sNgkEvgdi6OZVrxxuK2g?= =?us-ascii?Q?TxcBWkNddvG5tVaw7WD3NZR3HDIxIc78cSeKdJzYiQgZEuHkR6IdNiyQ95hL?= =?us-ascii?Q?D/flwdNhjwPD04uvSkiRWOkUxQfbf4z05wbgOCzO4VB8kryQ+pRW3ZWXl0J0?= =?us-ascii?Q?GFlaSVq5vOBX5STlEsPYriYHP3Gj81knwKtBp0FOvYxNAgRUGx5VwIUKZvnY?= =?us-ascii?Q?1aqN/0UHjsYZhRCHG1/fO9Io/dPI5VHq7g1TAj7VLVRlhV+QLx0jLOT/Cjsk?= =?us-ascii?Q?ZS52DjjDDG01wzhKKddz1vvOtcbl4nAaO2x5+ZWr1Ccj3HgMzHEnsDEOoLPP?= =?us-ascii?Q?oqL1159sKlh+MtU57sb8WwIev98qACaZu6kDvion0YfwLg8StM34zJsbCWR2?= =?us-ascii?Q?Dmmby/HtxNczXdv6cP0Wa39w9e+YjsWXlhK3fu5mqAkxufXXHBFp7dXJBWWL?= =?us-ascii?Q?NGOpaptoz8U9jzyc4aOnzU1nK48sulxTUOMGXcbzFmbL0W+0efT8xfGj2FB5?= =?us-ascii?Q?4jiiGO/srItWA7WqNFPuArflA+KAEz1VglfK0cV6FBfxxTdsqFsQn4mvIUMe?= =?us-ascii?Q?T/TcKg6jj/jhoMmw92JGLonHCV9ddqlTWHkZ0k3LVlQi8ZMUtY5C55POfL4j?= =?us-ascii?Q?O8VfPYIrsDs2HzLljnPLeT85Wz1VgNySYpt9j/vwyYyA6w3A5pZTcNaxNSK6?= =?us-ascii?Q?amj6HbPnyh1NAFHW2MHeQ2vqSu/N6BTLDsZke5UytyGmCdnksO1mUQi3Uf0u?= =?us-ascii?Q?2iUA9Sl0StzrWgaaS1UNMAn14LDO0kz5rDpWf3GGWfJ+V6A9C08aAywxIqr9?= =?us-ascii?Q?QmDPrPeRgx3pRPNP1bywZXKgPT3uY8gnqO3AgKTT2SjFIk9bFM7fkux0rUgb?= =?us-ascii?Q?3ss04qhWRna0okytCEdgaTekBfK0AbT25jV/PvcYjcIPHurHkqxoizEWGpXx?= =?us-ascii?Q?ntW/NGClDEplUVwjLO2PNS3VXc+Ueqye7uZ1kLEC5mkl8XmNwD/bLSMJla+C?= =?us-ascii?Q?cEXBM3jCe/e/b5e9sp07xNaMpyZKySqYVFfxKErwwE5OD3tuOwk5hDsKQgs2?= =?us-ascii?Q?5w=3D=3D?= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: BN9PR11MB5483.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 64cd80f5-6407-4ceb-99d5-08d9ea1980e8 X-MS-Exchange-CrossTenant-originalarrivaltime: 07 Feb 2022 09:09:10.6195 (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: DTNAtv5NErBLg72Dpm8vbhTsemmGqLg7bGimIB+EsZvnG+rMj5qvd49mJrdsW5aCMr85tjM3SIE44pjU9ZeZUA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN8PR11MB3697 Return-Path: chasel.chiu@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Thanks Xiaoqiang!=20 Reviewed-by: Chasel Chiu > -----Original Message----- > From: Zhang, Xiaoqiang > Sent: Monday, February 7, 2022 4:11 PM > To: devel@edk2.groups.io > Cc: Zhang, Xiaoqiang ; Chiu, Chasel > ; Desimone, Nathaniel L > ; Oram, Isaac W > Subject: [PATCH v1] MinPlatformPkg: Install memory relating PPIs >=20 > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D3824 >=20 > Install memory relating PPIs for EDKII native build and FSP dispatch mode >=20 > Signed-off-by: Xiaoqiang Zhang > Cc: Chasel Chiu > Cc: Nate DeSimone > Cc: Isaac Oram > --- >=20 > Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformInitPr= eMe > m.c | 5 ++++- >=20 > Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformInitPr= eMe > m.inf | 2 ++ > 2 files changed, 6 insertions(+), 1 deletion(-) >=20 > diff --git > a/Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformInit= Pre > Mem.c > b/Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformInit= Pre > Mem.c > index 6519fdd042..d8c96b52f4 100644 > --- > a/Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformInit= Pre > Mem.c > +++ > b/Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformInit= Pre > Mem.c > @@ -481,7 +481,10 @@ PlatformInitPreMem ( >=20 > BuildMemoryTypeInformation (); >=20 > - if (!PcdGetBool(PcdFspWrapperBootMode)) { > + if ((!PcdGetBool (PcdFspWrapperBootMode)) || (PcdGet8 > (PcdFspModeSelection) =3D=3D 0)) { > + // > + // Install memory relating PPIs for EDKII native build and FSP dispa= tch mode > + // > Status =3D PeiServicesInstallPpi (mMemPpiList); > ASSERT_EFI_ERROR (Status); > } > diff --git > a/Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformInit= Pre > Mem.inf > b/Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformInit= Pre > Mem.inf > index fb997838ef..86d8246f02 100644 > --- > a/Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformInit= Pre > Mem.inf > +++ > b/Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformInit= Pre > Mem.inf > @@ -34,11 +34,13 @@ > MdeModulePkg/MdeModulePkg.dec > MdePkg/MdePkg.dec > IntelSiliconPkg/IntelSiliconPkg.dec > + IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec >=20 > [Pcd] > gMinPlatformPkgTokenSpaceGuid.PcdFspWrapperBootMode ## > CONSUMES > gMinPlatformPkgTokenSpaceGuid.PcdStopAfterDebugInit ## CONSUM= ES > gMinPlatformPkgTokenSpaceGuid.PcdStopAfterMemInit ## CONSUM= ES > + gIntelFsp2WrapperTokenSpaceGuid.PcdFspModeSelection ## > CONSUMES >=20 > [FixedPcd] > gMinPlatformPkgTokenSpaceGuid.PcdPlatformEfiAcpiReclaimMemorySize ## > CONSUMES > -- > 2.32.0.windows.1