From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by mx.groups.io with SMTP id smtpd.web08.22710.1644383051636253167 for ; Tue, 08 Feb 2022 21:04:11 -0800 Authentication-Results: mx.groups.io; dkim=fail reason="unable to parse pub key" header.i=@intel.com header.s=intel header.b=heokUQqW; spf=pass (domain: intel.com, ip: 192.55.52.136, mailfrom: xiaoqiang.zhang@intel.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1644383051; x=1675919051; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-transfer-encoding:mime-version; bh=RDuVLN725mmsT6BEjcIGocGcYgQJLRE/UY7gC8CwxdU=; b=heokUQqWPRBjYBwtGgPuMmxmByt4UP4byuv7TKg4YsanDpVMxyeSFqE3 zRxYM3HNWv0cBcfI8ceTOuS9BcBpbE8n1Io1K6VxbKoylkSmob8nx7aq/ DWMnNyLsONNieyxoBmJd+b3mQoVcL7LGpQiR05LtcWLuUCFn+gHsdYxgQ d4RsMEFt39sSEA0ihGGW7x1SwC3Gw7VhGTsVWlQ/HTc+mk4mQcbDZ7ZNn rrf4D3t6H0i+683XZTjvhsA5mvu2WFPczADkVJoxV6BJoQtutt4aLsXcY brDMtVSxa1YQPf/rMobPYvKqHnDPt+LYDCA7ZrvzL7WR/AQESKl3qveu1 w==; X-IronPort-AV: E=McAfee;i="6200,9189,10252"; a="229091254" X-IronPort-AV: E=Sophos;i="5.88,354,1635231600"; d="scan'208";a="229091254" Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 08 Feb 2022 21:04:10 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.88,354,1635231600"; d="scan'208";a="601475608" Received: from orsmsx605.amr.corp.intel.com ([10.22.229.18]) by fmsmga004.fm.intel.com with ESMTP; 08 Feb 2022 21:04:10 -0800 Received: from orsmsx610.amr.corp.intel.com (10.22.229.23) 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; Tue, 8 Feb 2022 21:04:10 -0800 Received: from ORSEDG601.ED.cps.intel.com (10.7.248.6) by orsmsx610.amr.corp.intel.com (10.22.229.23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.20 via Frontend Transport; Tue, 8 Feb 2022 21:04:10 -0800 Received: from NAM10-BN7-obe.outbound.protection.outlook.com (104.47.70.105) 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.2308.20; Tue, 8 Feb 2022 21:04:10 -0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=lBDQ7wZcgq1WSAhq40aB9022YtdyxArbYwMDxMtOVY9SjEARYscE8pU21Gk1gome0GUMJVADfCzLh6Fk6MYkJFzBPSmJmVf6ZIZk0RiBWlZGZvtDXMO+EyCEkJiTVEWnnAeaGtUkR8akBrYqfVOxswpzmPOMl33w3sF2vVthBkC3tRKzSFLJcqQBqhrm1kZVNwXNTnNL7hn6oF5OCziAnEp/6xx+AaCkzPL3PMt350sFkUKdKa1LZwwIfDwd8OKKq2+6AEy7f/oWpkCclBnaE7vsGRkVMTfWjoEMeFXFblLuGC1ytpi8rgnrSbOBYyG2d/05SkyYRsKXRnxIB0DHoA== 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=JlMcuwBE6PbczKcIVef/cpmtFzqXzVCS+fP70bP/cYU=; b=nnvc36XLfQJs+llc0vLUkIJFlB82ydAnk+Ftj3dm+X1/EOlY9Vp2mup60AIMOlNyzzLxhujp5rHdjjOJjyf3FEA5JVyCrrMtA8QPsnRiznDYXHoeJDZq6xWATT0mOhrfRe09BpcrY2C7xbi3URGdAZTHgy0IXgtkO0a6jIrVTdDCwwgKrmYf/RaUfQHuH+PGwTWbw5+cHus3wtPeohCDUWQvrKUkDRT5+uJJbrj9cq1aIaYWVpGQPilDLNAIz/StgT+iROz+PJReQKNYShvcXlvAdBQ9BAXpFDQaMi8aGe4DiRhhEO5MkuzBqUqM9e6DPXcVGhC5qlFmLhVLfGbl1Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none Received: from CH0PR11MB5379.namprd11.prod.outlook.com (2603:10b6:610:ba::19) by DM5PR11MB1786.namprd11.prod.outlook.com (2603:10b6:3:113::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4951.12; Wed, 9 Feb 2022 05:04:08 +0000 Received: from CH0PR11MB5379.namprd11.prod.outlook.com ([fe80::307a:5fc6:3e32:65c4]) by CH0PR11MB5379.namprd11.prod.outlook.com ([fe80::307a:5fc6:3e32:65c4%4]) with mapi id 15.20.4951.019; Wed, 9 Feb 2022 05:04:08 +0000 From: "Zhang, Xiaoqiang" To: "Desimone, Nathaniel L" , "devel@edk2.groups.io" CC: "Chiu, Chasel" , "Oram, Isaac W" Subject: Re: [PATCH v1] MinPlatformPkg: Install memory relating PPIs Thread-Topic: [PATCH v1] MinPlatformPkg: Install memory relating PPIs Thread-Index: AQHYG/pgV/pKwDhdQE+FgE/0+E6EUayKkdIAgAAHIqA= Date: Wed, 9 Feb 2022 05:04:08 +0000 Message-ID: References: <20220207081109.1094-1-xiaoqiang.zhang@intel.com> In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.6.200.16 dlp-reaction: no-action 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: 72c9c493-beae-4860-8dbb-08d9eb899a8a x-ms-traffictypediagnostic: DM5PR11MB1786:EE_ x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:119; x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: NoBGKVs0m9xWsNT5PJg1M5eDxbafFWG+htMG9DZrVyfxRqWi8T/dzkhegPDiZfBEozjEDVHNImgwGJ186HEMvhfb+C3dKvEiGR5Dg/XsflcSoIodav8kOErJVJcBYDVgMjMSto1N7ify1DVM5S0jyslTYs9VyTn0QIOavyf4iO664kVLMcMZF8fIiq5O273l4tDb5S+AhnTwB9X6/6JvilFHijhfG0NgA8em8HJaMNlHKxrVPANkftJg8MEEmBMOF16OY8f9Cx9UIitfGNlwTIK14as87g32VCcP1zi4xLM+0FWK2wfbOewsFXypNUnJdZm6D4MLfeLrC3UOojtptJELHoOI2YNOwT8+vmmTnYp4G7g9D0LadpUf8nJsnnonHk/jW2Hizm2/22vNoUTu3ayMRDcVX/hZra2SNQK3pFv9c5ETN0WlhLhSRLVHp83KB7wtXioO/LAIxRt898AmaAMTQgzYlqchSxmqnoxgP36Qlvea0T5VHuiN2QGQmzwNmFO/lHGP2P3fU6lD9fWHn+ZlhwCGuWUu3FO1BUrNE3DZWQ4xz6d0d5+CT6tS0njRsZy9jPN4s8XL/TqT6nfmyW4kp9s5dRN/8pLX0IP5dcMTr9yMV9Inj9bzMKKFaqZ5sRw7StNaXFBBK+oTiRMV5Xn2xPdgZ/mHuCkDtRGStLv7p/USzfFLAa9upJZc2os2YWSPOZP2CJ+2AAfBOm9+8A1n8D7C478/KCev0khvDruR6E/skJ8tm8nqwPUuQszSUTAuGZJj+Nccj1SZ+rfa+fYiMMoBqaLgylir98l/9h7g8/0KlSRENOq3lb4TavjOmUx6zD1rTXuoHikAum6cXA== x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:CH0PR11MB5379.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230001)(366004)(38100700002)(8936002)(4326008)(107886003)(8676002)(71200400001)(2906002)(508600001)(52536014)(66476007)(38070700005)(66446008)(64756008)(33656002)(26005)(186003)(82960400001)(66556008)(66946007)(53546011)(86362001)(76116006)(966005)(55016003)(54906003)(5660300002)(6506007)(9686003)(110136005)(19627235002)(7696005)(83380400001)(122000001)(316002);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?2Vi0G/WU8JpSWZBacRllSXQUgWs/UbamGQURLaT5Rx6AiKcrYid2z2AX1Zg9?= =?us-ascii?Q?MNEd+8wW6RMLiyneVsCgm5XsKiH+R458etbmzpsL3AZ4HZ8WhASFfwrwZJ91?= =?us-ascii?Q?tksG2MDn862t0QQXpCvcKwSPsyzuRetEy4J6JjC/t4DmQLULvevUsBkAbTlS?= =?us-ascii?Q?FXvbO0tZVDM8u+1CUFCKMkSNyZDNO694mZsFDmnIn7xjkEfB42c6tC6Ao9FU?= =?us-ascii?Q?ve0R3KxYE+CWGdZqu4t2zB9Pqto90+jKXplnsnCTgNlnAC9tLxHN45OVDCdm?= =?us-ascii?Q?/GvmSnuuiFGtHKUVTggG1CvRwxT8pZOTeRp9+6PsajBL7MXSWGnVIDrHDEg3?= =?us-ascii?Q?rWPTELTIMlThVdwTSxN06vyy6lwzhHr7vSrXwoJXtE0qiv8c1LfqNp5U+xdj?= =?us-ascii?Q?mMozEzoTYkrJwglA6cCJkX537MgIprOYVjIa27UeQNKWPyF/IZwfikD9zM6X?= =?us-ascii?Q?5et/cFR2VHF5D9gCqt2agTbyAxtH8Tf+W7udraaEFLzN6SYsSZipQrRLvLuS?= =?us-ascii?Q?YSiQ79QkJaSxYqKzL2lU7EvqwzsLKjIprJlPiFqs4hMAaOsWo3zVg8aBScgL?= =?us-ascii?Q?MAqh06d9a3nfZykUJ1dMBNid0CdzfcMdcX6zniYeV3LV9gOeEYTRIsQsjiy8?= =?us-ascii?Q?ytDWLGPVjYkKoiVbOblxc9z5nSneGCSwlRtfqF5pqmJyP69BNGrvNV2uXO0d?= =?us-ascii?Q?bI8HlSxPX+12iRHYih3fQcigdMbNB6zIguglcI+5BGDYaJ6132/8yI1I87lC?= =?us-ascii?Q?Zu7Sjt5Kla6fl5Phb7C4EfKAGfXZhsMl3gmWXXWAaQbwck0GPtzClEuAU5Rc?= =?us-ascii?Q?JgUnXlDk0JEZTskRvt7WyaA8o4sVG/meSfOh5vKTkGmPFNrdD2FFFd1+jPgq?= =?us-ascii?Q?d60iB0V1uivIwHV5y1gNq8OKcg2+DO9BLsn4jSz7+a+hF97xoEN8S3M2qGs0?= =?us-ascii?Q?qiyJLA/61XjWWjKuFFVIZc+fCruLcXJq2fODE9aV4w3N+P8g9hZeELL2x5Xd?= =?us-ascii?Q?b++5Pj/WjkXztNkhoZCSVhC3XF+UzhkxmOBfd60EQkGaVEIgwZ9Bo9H0CdT9?= =?us-ascii?Q?KkTOjzR8jAW2TACXDCGjEjn4hX1w4dKyHp1WpQBIylXJbpDlm6z3IgUtT9U1?= =?us-ascii?Q?0RSY5PzkXkBORp4/BykUhpmpO6spZ41k9MpSIS7bOgDON5e1IepI40PWE48X?= =?us-ascii?Q?RlPpCjtSTsSqnfZHExj+oA1hSaigN3SbRdBkm4eYlhWmBuDXO7Ll2Zig7ki4?= =?us-ascii?Q?NGy006VpnmmKqLIz4wZTgmzIA2Q28rsue9WeA36TIS/jgB2STZD3lD9XvaQ+?= =?us-ascii?Q?LAMhlqNIDG1bDRwCxC7tIp1J+xq59ML2chmLLsM4uCens4fvmeHWJD878L/K?= =?us-ascii?Q?HBkR715Ciofg6sBv9+GqMY38ChrKESYbjZucFWfr7kCFVt8ck8uiHwX9CvTX?= =?us-ascii?Q?8wKr9RKO3cg4Vkm6U+gv5AXaCmUtEiAq0EdWhLlNyGEaJHBCHv6Y1XuLfjBB?= =?us-ascii?Q?SNUgJE1oLkkkAuEpJX0dDnvzSgbMureJktQ5oUznBysaCP/MMTuj5sPzXLm8?= =?us-ascii?Q?uG/9LSj79rfntt2mOWDYCEbwB/wH/qWxZ/i65tdyn3ROUF6MHyqSXXziBeQZ?= =?us-ascii?Q?4I5i/mhyG0JqgDrbqM8WePMnKTpYbc04ZZiWzmJyiw0lLoJMZxb2kpTtCKII?= =?us-ascii?Q?WKv9MA=3D=3D?= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: CH0PR11MB5379.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 72c9c493-beae-4860-8dbb-08d9eb899a8a X-MS-Exchange-CrossTenant-originalarrivaltime: 09 Feb 2022 05:04:08.4516 (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: vPTM3ilJYSjl04Gc9gioxVHf7sI/AavXuE0XiYs90D+3so6pZcItBbv5PpX7Wm+5DomqoyovVqaM9XxRU0RUWguDgfU6UQag1MoLqWkFG6U= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR11MB1786 Return-Path: xiaoqiang.zhang@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Thanks Nate! Best Regards, Xiaoqiang Zhang -----Original Message----- From: Desimone, Nathaniel L =20 Sent: Wednesday, February 9, 2022 11:27 AM To: Zhang, Xiaoqiang ; devel@edk2.groups.io Cc: Chiu, Chasel ; Oram, Isaac W Subject: RE: [PATCH v1] MinPlatformPkg: Install memory relating PPIs Pushed: https://github.com/tianocore/edk2-platforms/commit/b6f9674 > -----Original Message----- > From: Zhang, Xiaoqiang > Sent: Monday, February 7, 2022 12:11 AM > To: devel@edk2.groups.io > Cc: Zhang, Xiaoqiang ; Chiu, Chasel=20 > ; Desimone, Nathaniel L=20 > ; Oram, Isaac W=20 > > 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=20 > mode >=20 > Signed-off-by: Xiaoqiang Zhang > Cc: Chasel Chiu > Cc: Nate DeSimone > Cc: Isaac Oram > --- >=20 > Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformInitPr= e > Mem.c | 5 ++++- >=20 > Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformIni > tPre > Mem.inf | 2 ++ > 2 files changed, 6 insertions(+), 1 deletion(-) >=20 > diff --git > a/Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformI > nitP > reMem.c > b/Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformI > nitP > reMem.c > index 6519fdd042..d8c96b52f4 100644 > --- > a/Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformI > nitP > reMem.c > +++ > b/Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformI > nitP > reMem.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=20 > + dispatch > mode > + // > Status =3D PeiServicesInstallPpi (mMemPpiList); > ASSERT_EFI_ERROR (Status); > } > diff --git > a/Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformI > nitP > reMem.inf > b/Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformI > nitP > reMem.inf > index fb997838ef..86d8246f02 100644 > --- > a/Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformI > nitP > reMem.inf > +++ > b/Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformI > nitP > reMem.inf > @@ -34,11 +34,13 @@ > MdeModulePkg/MdeModulePkg.dec > MdePkg/MdePkg.dec > IntelSiliconPkg/IntelSiliconPkg.dec > + IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec >=20 > [Pcd] > gMinPlatformPkgTokenSpaceGuid.PcdFspWrapperBootMode ## > CONSUMES > gMinPlatformPkgTokenSpaceGuid.PcdStopAfterDebugInit ## > CONSUMES > gMinPlatformPkgTokenSpaceGuid.PcdStopAfterMemInit ## > CONSUMES > + gIntelFsp2WrapperTokenSpaceGuid.PcdFspModeSelection ## > CONSUMES >=20 > [FixedPcd] > gMinPlatformPkgTokenSpaceGuid.PcdPlatformEfiAcpiReclaimMemorySize > ## CONSUMES > -- > 2.32.0.windows.1