From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by mx.groups.io with SMTP id smtpd.web10.8589.1688721945635426679 for ; Fri, 07 Jul 2023 02:25:46 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="unable to parse pub key" header.i=@intel.com header.s=intel header.b=VGqB4JGy; spf=pass (domain: intel.com, ip: 134.134.136.126, 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=1688721945; x=1720257945; h=from:to:subject:date:message-id:references:in-reply-to: content-transfer-encoding:mime-version; bh=cOdDOSPmPPsm7BOLx+1C4hCuZlfF5H1No0jHYfWDH/0=; b=VGqB4JGyWOfV791DNI++w0mBMgHGJDL8XxmdYaHIIphWinTZd3osL4c5 NpvohpKKIS8TllSip40yvXD1FgOlyMvzIb6U1EFJyxxCFx11eWSDPkh0q 7BPwbUWTugNNHmFDwQ1GN99DeKS9EMT+CV+PLNaCztYGCuTaYekDOhN7M 6m176p8ifyThrrWHDvgymIkkGOGVBsC2GZrHnvkbC+pXnVjp9SvjVqCMc alsZ2GyxInE5A0GJZfMgPunO/3IidOSXfSS7VJ+VhXruJlyEufdjAr8wp Xf1BNhBdnNUgr9WcFsLX3knfi46Bj1ifY19zKSy1tbsboX1NkegzCHeZK g==; X-IronPort-AV: E=McAfee;i="6600,9927,10763"; a="348643963" X-IronPort-AV: E=Sophos;i="6.01,187,1684825200"; d="scan'208";a="348643963" Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 Jul 2023 02:25:45 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10763"; a="893925976" X-IronPort-AV: E=Sophos;i="6.01,187,1684825200"; d="scan'208";a="893925976" Received: from fmsmsx601.amr.corp.intel.com ([10.18.126.81]) by orsmga005.jf.intel.com with ESMTP; 07 Jul 2023 02:25:44 -0700 Received: from fmsmsx611.amr.corp.intel.com (10.18.126.91) by fmsmsx601.amr.corp.intel.com (10.18.126.81) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.27; Fri, 7 Jul 2023 02:25:44 -0700 Received: from fmsmsx610.amr.corp.intel.com (10.18.126.90) by fmsmsx611.amr.corp.intel.com (10.18.126.91) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.27; Fri, 7 Jul 2023 02:25:43 -0700 Received: from fmsedg601.ED.cps.intel.com (10.1.192.135) 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.2507.27 via Frontend Transport; Fri, 7 Jul 2023 02:25:43 -0700 Received: from NAM10-BN7-obe.outbound.protection.outlook.com (104.47.70.102) by edgegateway.intel.com (192.55.55.70) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.27; Fri, 7 Jul 2023 02:25:41 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=b9OgcX8pKGgmFpu7UJh04TdOZHh4YGOdIBN2wW1xSXrEoTgCYrymmv5Re609sm3OnDTq56qgAM7KgnGuUdwBLFZVE4gudO3pAcyRfC0a2uxlKgMhdt9UTwvNaRI0P7fAP459CyOB40xNahMEP6el1gYM5URliDf5HDdvFv/HL2hgAnkQUC6OG8LHsS7q+rywNXuSzvoQW3Xx3MPGqltz0SsW2sxZ/BKY3FSWGuhc36zKfouuDbpfgJICBYBttDypP/2+ov6nByYpTQGiDknPgHeUs7GfzT+lC05qcszTRLMiFDbda3P2EYGMwXfWZIw+NVxiSufKVIOXmyRSLE2U4Q== 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=pExX8HLDQJw03rBzJGl8yQauAHB/4ahhNJtSmo1gQjQ=; b=nmBAm0a7fiyJF/8baIc7z5oAfCUiN8g+T/KXuBeiGO3aUW7gW+U9CL/qckIJ28nbKYJby4oN8HuGnXaEdzrxbzpsD+5VPkK/E4gWX8gBbWBW2EraN67T3c/s/RH4E+7cx9XC0yHz3GpqGM5u0p6K88vaaxMxbNC9eR25/17NhE9JEGtgXJJjroQXsNkjc1Tqt+i0EK4M9k8+nYFcOxjIqc5fdNm7re1IC/gHS0/xgCWoZr601LCeVwKfj8phNbaK+xGKzZPAKcy5e6+zKBbif8CWCheTgnopIhFsEzsj8XMEDw5yA86VhVUfDPMDhssY3VOF+++45Nq3xhCs+CFZhw== 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 PH7PR11MB7026.namprd11.prod.outlook.com (2603:10b6:510:209::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6544.24; Fri, 7 Jul 2023 09:25:40 +0000 Received: from MN6PR11MB8244.namprd11.prod.outlook.com ([fe80::5542:6ddb:ea31:33df]) by MN6PR11MB8244.namprd11.prod.outlook.com ([fe80::5542:6ddb:ea31:33df%4]) with mapi id 15.20.6565.025; Fri, 7 Jul 2023 09:25:39 +0000 From: "Ni, Ray" To: "devel@edk2.groups.io" , "kraxel@redhat.com" Subject: Re: [edk2-devel] [PATCH 0/4] Sync BSP's APIC mode to APs in MP init flow Thread-Topic: [edk2-devel] [PATCH 0/4] Sync BSP's APIC mode to APs in MP init flow Thread-Index: AQHZsLDc7kQDB4fbRk297i2KWyiyCq+uB8EA Date: Fri, 7 Jul 2023 09:25:39 +0000 Message-ID: References: <20230707052901.869-1-ray.ni@intel.com> <4nerjr3hcwfepwzllcsrm3vurwzdyvr7qz6b4lqfualkngt7he@vjtw4v2ixksc> In-Reply-To: <4nerjr3hcwfepwzllcsrm3vurwzdyvr7qz6b4lqfualkngt7he@vjtw4v2ixksc> 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_|PH7PR11MB7026:EE_ x-ms-office365-filtering-correlation-id: b0f57192-aa6c-4196-237c-08db7ecc20f0 x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: z9NFDievLoCkIHemg5zThA+/OHB+aYREEdyW2nhvFEIZqgtMNSjnS4JybXr9/Z8hdlE0V/dpqTGyevpCEC3/YhImGFK0bDmNavYzk5RlZk1i5szQF0k+SxsA4lnPomCUQf/JEPuFIougqUhdLKZYKsS1rGFmViqnjB6Q33StVQ2SJ8QKkAQ98/PHJRZPnqCRs7VWVK0PwnSQctk8Yr49mSr7OqZcsy4gBbLPS7j8OYIaBT/Zldn0w22MKzgifEpmRFZ3Plt5zPSGZtdPxP8qOjhi/MRRCFdslAbEt7Eb49UzWVPN/Skn2xiCDFm2xB/qcwLCTVebcAR+lgj7Oi+A3BcO+ar84Ykxp6MOZYKEhQW/UO/vhAa/iuo4vUKre6O6+jW96Je4axB+la/b+uuGAEGPE+MDP66CaYmNTIzeYDaHJ+VQX67i2Zflmgk1J0ZGc8wJvEj+wke2tlJf6w+JupATICJA71L/gnPPGM25/wCvPUK3CjEI55YzXfXa/DeaSNrTYaHNiXbCdDS02K1z2tllxH6roN20+FSOgGR2A9iF2aRD6Q7uTI9zzz72zADasBog6nRqOc6sYhJgjeyehte3rTdldxPuv90jaMTtCNH+6PjY54wqqvP9aksq07k+WKjd4xO1E5zz6M67TlzvaQ== 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)(39860400002)(346002)(376002)(136003)(396003)(366004)(451199021)(41300700001)(86362001)(38100700002)(71200400001)(83380400001)(122000001)(82960400001)(6506007)(53546011)(966005)(55016003)(26005)(38070700005)(110136005)(7696005)(9686003)(478600001)(316002)(66476007)(66446008)(66556008)(76116006)(66946007)(33656002)(64756008)(8676002)(8936002)(186003)(52536014)(2906002)(5660300002)(66899021);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?WX7116yZdeXSG/TuDYknYPMhXzFbHZXBgv20eF5zLjBNBCn9XqZ4+0Lyt8q+?= =?us-ascii?Q?P3UMhB//jtw2KvNZQOCkp7dD2Zv3mHMl7EHEL0/Myib9vFegnwApZ+zMhAzF?= =?us-ascii?Q?Eay5iZYePrRnZ+N7qfNScAi4uvLkpAKRBJP71MDIlKT6YE5X+IjzLh1XskUo?= =?us-ascii?Q?wA8pwqG5cFYGDPYlOjo+7QmyEY16C2eqJMiFUY0gPmyc7Nut4DIqrWa0+oX1?= =?us-ascii?Q?K1LCH+V1x5xKlq06lbu+7ICZFl1wIrr4PDDivqQ3Dibwye8sx/417K/Gx8uN?= =?us-ascii?Q?GmqRswniLRo8m0COUL1+owUMH/tGWf4Av/qKUrom7Y+j7BY1GQAE5F2pCDg2?= =?us-ascii?Q?OwXjZCtwZc62yEAdrVJ+lehaBHT8EJNCFOeNoO8lYqOn1ATb9OAn02yAi9Dk?= =?us-ascii?Q?A3zYpaXnwaA8qhAfqS+GJgDYZO1cXJblzvvkftXz4qhrgyuz6YO9ABtZ6BS4?= =?us-ascii?Q?CvJGapcVF29Ur/vcrkyuSkU0Qbisw+CjpqR5CVfS1XX/toK4ifl0m/K9ruid?= =?us-ascii?Q?Fnq1K3D3Mem1btTJRGbY4wmXOqFvs28THHGbN3P+TI7WDjWtlvAqLPpeU/Ee?= =?us-ascii?Q?L+Vt3xqegEyY9oUvimjdPUP3a1nkcbkD6VfVfcGmK+Cl0bSh95VBYqPBTuZJ?= =?us-ascii?Q?IwMSxm8YPgnb27b7N5BwKAadaORl8w9R77PnUZsl0w8zwm/yV7kXdLAg4pCx?= =?us-ascii?Q?SRjsYubppT/PHtHUQ8lwDfF4A82NBFAXcpaWJnRXZ7Xtn1UKKBKJ4E8agFFB?= =?us-ascii?Q?FyCGHFeIXkahRPTSbPyI0UmJVpNpqtfKdFh97FsOyxmvW/ZhFvwZCk2GtikS?= =?us-ascii?Q?TkxnDJEaGI7OYh0dHg1bDtoZwB8UhHgV9i9lfkrOvhf8tUGeqhg1B+LbroHY?= =?us-ascii?Q?uMo1RdPp0PCiBI02LMuXI5+/xPFIq0GPlmZVTuda7ChMBN8S6JDOJpdiDUxn?= =?us-ascii?Q?2P4brkY6cIGaqRQkthWm9g9h02pjtDS3lZBk/APIwLcQEm0qhOEyqbttzobo?= =?us-ascii?Q?0z7O2QodxxpdNPY84Z/1FsK6IrORVldkcR6E8PpJEeEVc3JeXh2Tl/8HaSsu?= =?us-ascii?Q?5JLqngSo9SzTTuXgsAGL0aO+KTCMYRvEET45kQuUTLh4HkfwJ/vfyNApJ/FG?= =?us-ascii?Q?gDzZnmcblp48GYrRPY0mDGuztCBZTppTdtJGzlI0kKzApSF/b4pbQr3mPkEP?= =?us-ascii?Q?Y3+bxTKIRLXE+SL8/1LyHB2hx66BgcNynhv+K8y2ZIj/LCXREbzhMLYQnxWk?= =?us-ascii?Q?dR5y9SMBH2Hvz2zdpPD95Jjn6v2aayBJczoqhJccXQ59ZgA8n19xwD5W120N?= =?us-ascii?Q?ERD6qvDh12ZShdA543n6pDouTon3OGqgHEEXlehyzgM3iNapFdUwcDCWmakI?= =?us-ascii?Q?E9uuTlhUM/uYAv9ufyWjrsDdh8CnyVpM2lMQlJmvgD/aMP1pT/F6nhUfhGMD?= =?us-ascii?Q?B7PQkUEvZAS7BLOQsH6b+MIaAJ8Udny3hPh2nO4ptGthjjGnxvcFkei6Splf?= =?us-ascii?Q?fb1omC7Kk6emJqdSyq8iBVbafFiRcTJdQeXLGIzjYpftI6DpyAjQhfblWwe+?= =?us-ascii?Q?yuLuNjPxyCexv3SdcCg=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: b0f57192-aa6c-4196-237c-08db7ecc20f0 X-MS-Exchange-CrossTenant-originalarrivaltime: 07 Jul 2023 09:25:39.3664 (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: JU56VnRAoZjOuiXJYCAXY/9oJnYBYMva73eOTv4gtDIZZAHr0AD7KsoORZ+0T2OHIGOhM3GvZpXffPByM9T74g== X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR11MB7026 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 Gerd, No. I don't plan to add PCD. I thought that initially but in the end I figured out: * PCD is a way to let platform configure the common logic behavior. * Why not treat "X2 APIC status in BSP" as a "hardware" PCD? With above thoughts, I prefer (a). Platform can choose whether to define a "platform-level" PCD to control platform behavior regarding whether to enable X2 APIC in BSP before CPU MP. Comments? Thanks, Ray > -----Original Message----- > From: devel@edk2.groups.io On Behalf Of Gerd > Hoffmann > Sent: Friday, July 7, 2023 4:56 PM > To: devel@edk2.groups.io; Ni, Ray > Subject: Re: [edk2-devel] [PATCH 0/4] Sync BSP's APIC mode to APs in MP i= nit > flow >=20 > On Fri, Jul 07, 2023 at 01:28:57PM +0800, Ni, Ray wrote: >=20 > [ empty cover letter ] >=20 > Summary of the patch series changes would be nice. >=20 > If I read things correctly this series will: >=20 > (a) allow platforms use x2apic mode by simply switching the BSP into > x2apic mode early enough (for example in PlatformPei). > (b) avoids waking up all APs a second time to set apic mode. >=20 > Is that correct? >=20 > Do you still plan to add a PCD to enable x2apic mode, or is that plan > obsoleted by (a) ? >=20 > take care, > Gerd >=20 >=20 >=20 >=20 >=20