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.web11.52065.1686547881184732268 for ; Sun, 11 Jun 2023 22:31:21 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="unable to parse pub key" header.i=@intel.com header.s=intel header.b=EyxOBuDq; 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=1686547881; x=1718083881; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-transfer-encoding:mime-version; bh=zINnf60fa20Ip21+loaPafMiXG39Z2PZEv8lOKUnPRQ=; b=EyxOBuDqTqlZf+eOMdA0akLDLEJgpHgnRnPNJQKgwfCt9QsIserZen+5 2NnlBIKWWbljcmfWuc4n0opH6G01WbCGUIiIAqWXO5c0N+0uOX42aQp6U pw+LcQaKvnX9H2AprIVLHJX/eKV21DqZS7xMTV8lYKXme03XlvUGJe/mq ZQWixSXWUwB5xkdVDI1wYmk9rY5keZZ+zVQakvI4fo0jcENdbd6oYGv59 0UHxwTghZH/uB0UCp4hEscD0KftMa5meoRaSBXGmfPc0vQwKQPCNiIXVR sNGmLRYhGmp+IsEpDuDhIV4GazOeORBowFyHJyqleIr/K6jpYCrmC1awT g==; X-IronPort-AV: E=McAfee;i="6600,9927,10738"; a="421542236" X-IronPort-AV: E=Sophos;i="6.00,236,1681196400"; d="scan'208";a="421542236" Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Jun 2023 22:31:16 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10738"; a="835340130" X-IronPort-AV: E=Sophos;i="6.00,236,1681196400"; d="scan'208";a="835340130" Received: from fmsmsx601.amr.corp.intel.com ([10.18.126.81]) by orsmga004.jf.intel.com with ESMTP; 11 Jun 2023 22:31:16 -0700 Received: from fmsmsx610.amr.corp.intel.com (10.18.126.90) 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.23; Sun, 11 Jun 2023 22:31:15 -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.23 via Frontend Transport; Sun, 11 Jun 2023 22:31:15 -0700 Received: from NAM12-MW2-obe.outbound.protection.outlook.com (104.47.66.41) 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.23; Sun, 11 Jun 2023 22:31:15 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=jausZBo+PV8xVUNoTii+hBSYp2mmMSjlM7aivBD23yMLVlAxT2IrnM3nnn7dkMO3YyN70Sg8N1td2JYEy8gxGHWKw85v0x+Wp0+nR7WZb49uZ+yy9O6JM4QqYqLVIHv7s1o++uo+SK/Z20pPpG3Lvx4qKH6xWLs2fijXlIGaI9qafm9u+1uWI9OMrnfiyq6YMvYCJstDhjneeTqcQ+DvNb9Ir+MqA49mdpHzlfloc5xlRwDQ1/Dhi8ywrebTjWcVp/8gI7JT0dSKDfnvOBYOjY2mM2s0mMNJMIw6mgPa0jebjKksjvZmaShQ8kZqfRrxszYq39vgVgGJl70NhhLf6w== 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=6QkIA+P7UOMNRJDK45sYwTAoOtvZHFqAUm98r1nT4Yo=; b=a3swFzxKf/ChcBgZywqDxTG+SpQX1m2NH3I7GS1G+fTmrchknCJO5n/AE5zPEhHBrSConEHy6psqqf3WpGcNdKB0whHWfMaT1u/3tVv+GJ7k6Bl5oZ6KnaxAaICZBqOuPgirqWLDrzTfSP833m8NUcPgSgDQFzNE3AUdWzTNFt6GMOHwTgTGV9kTglE4XhQTcjOnGf3zvWRyAaJFVKrw/BX70l437vFEA5Y92qFnhIOEFG8e0CuiPx1yV9Gv8Fq4PJz6B5FKhQKY0GtdGTiNQ/fqrunskflHhDOlF3MBaS/c8v2s3XtzN19JZeESZSO+1X4Eh6dgXz6/BYyb1IbD3w== 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 DM6PR11MB4595.namprd11.prod.outlook.com (2603:10b6:5:2ac::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6455.44; Mon, 12 Jun 2023 05:31:14 +0000 Received: from MN6PR11MB8244.namprd11.prod.outlook.com ([fe80::892b:b8e6:bab7:635d]) by MN6PR11MB8244.namprd11.prod.outlook.com ([fe80::892b:b8e6:bab7:635d%2]) with mapi id 15.20.6455.034; Mon, 12 Jun 2023 05:31:14 +0000 From: "Ni, Ray" To: "Wu, Jiaxin" , "devel@edk2.groups.io" CC: "Dong, Eric" , "Kumar, Rahul R" , Gerd Hoffmann Subject: Re: [edk2-devel] [PATCH V2 1/6] UefiCpuPkg/CpuSmm: Add perf-logging for time-consuming BSP procedures Thread-Topic: [edk2-devel] [PATCH V2 1/6] UefiCpuPkg/CpuSmm: Add perf-logging for time-consuming BSP procedures Thread-Index: AQHZmtg2C6N4xD+060aZjLWRbwjZB6+Go/d7gAAET2g= Date: Mon, 12 Jun 2023 05:31:14 +0000 Message-ID: References: <20230531113438.1797-1-ray.ni@intel.com> <20230531113438.1797-2-ray.ni@intel.com> In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: msip_labels: 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_|DM6PR11MB4595:EE_ x-ms-office365-filtering-correlation-id: 301bdb1a-2675-48fa-9e0a-08db6b063d18 x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: sZy4d3+gjUzeYXxkg4TH2Ey7Iy8YED4+3dNrXI9DwFI02sXFOuf43jlkGAOZT2JmUFwyXvUM4psXK4bMEHT3ntctzCvJbFpfnB3lcK56q1ASIvOdiHc5D/HcbYNC5VdemYA/lyVwa1DBA2r6etUCJif5oRDKt22CFR2hrdhb98NMTEDEb7I+CH+PCLC+95viZ5ylhcfYAiKZ5OOWL7XciEwIGg9X5CyIeEUWCfAvVmCQpRsO7mqtpQjOnPSALEBuIhGPeJsnEK29TkhpXCT26VIwuPKQ0E7Y4j9v6pQONH1tbfYJH2iIuTiF/MitCiqwDpWUCWGEmsq2ljBrdHkB209VGgSHPGzMtfc599xJu94C90Q0zjUkrr6WMJlBynMcj38zp9WrfjSrrbMpPD8Wdad6G3irasdJJfOuNymxbNAZaIEdrYD6NlkTpmKmFOxBVbgMnC+9Oz4hk82iHzsoAeWl/h3Y6m45mrxJzWT4aQctub5XPtPDCJH9pe5xGeX0rgNUTtZYS+0xVQ69SZFU73r4UQtmXImg+TwBQdckAoFB0ggYZpTdm1r1sAvbxAbp/V01I75PzgqazuOssjRUzkzuGoQwve3+KVEDu0OmZ7bU6S/NffjZ29QQcATmRS1z 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)(396003)(136003)(376002)(346002)(366004)(39860400002)(451199021)(4326008)(91956017)(66446008)(66556008)(66946007)(64756008)(316002)(66476007)(76116006)(41300700001)(186003)(2940100002)(2906002)(110136005)(54906003)(478600001)(8676002)(8936002)(5660300002)(52536014)(71200400001)(7696005)(53546011)(9686003)(33656002)(122000001)(6506007)(26005)(55016003)(83380400001)(86362001)(38070700005)(82960400001)(38100700002);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?JlRVFH4+t8uKIklywlD7mLYPBMC07IAGSEzI8V8imat+JrqEv+1Ke7lQPj?= =?iso-8859-1?Q?SE61L0wPIdSIYLLazTOcPXLykU5X4XPHp0A5IVHcl/tmCQQfJMCxl6gM55?= =?iso-8859-1?Q?9V7zylJt0Mh8AmTbsU7jKHjQhRNpSL8pUaJNF0UUxXEcLtHYrycYzAIXtK?= =?iso-8859-1?Q?oyY3HCQy7qoNZjTfokNg1Zp1C5Nw6+8RfDEZfQFRmLnVvrTN237UM66+Xk?= =?iso-8859-1?Q?LZXw6abwPlaF5HfRL9P/qxDABC+otzOxtx0f0afXnMiMWrjsP2Xq2VGG1z?= =?iso-8859-1?Q?XN22CHhUj7vSxsobb7U+OiLB6r3fAMp/gSU27PCYINN83g9U8m6OkC9fnw?= =?iso-8859-1?Q?YbcqvrZpfMmvZgqgXWau+WrSB9bEeQs21ZYWZif+sV9GrfmRuHoNpvKeUo?= =?iso-8859-1?Q?cJEM0csBJwontqnCr4qzoCXOxw8B3xjnAJV8hRxAvKfWe5IHOUPa+rfMYo?= =?iso-8859-1?Q?U0gh6z88fi/h/F0RtkDYBHdlDxvouP04SdVQ/AKyLo4lt3LOmTzvzEvlYf?= =?iso-8859-1?Q?jmh8rB6pnyvk+wbdH5b4fQWnsf5jwS3drOwgHqNtxIOjUomQ/YXLmdyx36?= =?iso-8859-1?Q?e/kgllk14i50WzUq8Nx6Mg/CPtdQcXfaaPmJw1o56zHTO3WZ0PG++uYinn?= =?iso-8859-1?Q?sT7e7Oer96H9L6MR3UG/1A4496Rm7rpREC7oErhNbvyjSh1jMh7syN0+KO?= =?iso-8859-1?Q?sKGKM0vgjdO7Ou6KoRB1rciOFIpvcNXvZM5awqeYAFI0y6qLlO1ruqRooA?= =?iso-8859-1?Q?8sIK52r9voO8fgL4u45mtWnVqxzPMsNYXoyvIqUaWHjReiNCrDqPEsUZAF?= =?iso-8859-1?Q?H3FWYaud/tUJt1tic8oGiBn3C0Maq03M7BGvvSl2p00WkE0M8/9kqIEVFl?= =?iso-8859-1?Q?iUs7DrOlBzytsjFkobMjlm/QZHzTDbYjnsaagY13FYaO1XltLMRfI8MxdT?= =?iso-8859-1?Q?fpyXE8xzbXyqsG6A6BAmsuXCrdHLKNANhd44NnnjJwzlS8pbnwFMy3ed5p?= =?iso-8859-1?Q?vPWqv8JnAD3jMIVeEIhttwsKqeB3A60P8M8GqExrG+3INw3kh3G2qzmlas?= =?iso-8859-1?Q?nN/dp9fuSsrMEp1BmIZZ67wdjUSWi77xRMI2ebbPWxF/9U6+FUSWSQcVHl?= =?iso-8859-1?Q?rftH3t3kNTYZLZNuQMzMqfuYnUbOC87fgbJoExCv8icvgXstzqRTgMeqVN?= =?iso-8859-1?Q?oEyewo0n93hxPid/9mV7bA4EJVjC6aF38mdmPTQr8Zop6IA40OEgdhjI0+?= =?iso-8859-1?Q?MV4vz4xq50UYrPpqmNapUp3GU0DO9nNPf4aBbq9gES6vGX1PNoCefeTi9x?= =?iso-8859-1?Q?MSPphfQ3ttTQ7D6y6JGQHxe+QNsLDWeyX+ca9Sh6DvToJi4B8/wBJ+uisI?= =?iso-8859-1?Q?U8sfDwLr5ktg+yxySn+fXbOg3ql8MIOLiP1IVo4nabudgzjZNZqBvvWUED?= =?iso-8859-1?Q?7JgPflQ+C3O7U8V0vkolZLGkE8aSvz3AcwxTZDTKuYNrXM+RXugCPhf9/J?= =?iso-8859-1?Q?QYkohNX+c2Nacsl7eD0b99DokIJve+G+nwveSHgsATjp/X8shDxYPeLZOz?= =?iso-8859-1?Q?gZd8fVNVGtYUVSIA3acToXQ64APYcI5X0l7YcT0esloDGkcoBj3n6Y+v77?= =?iso-8859-1?Q?BefbW97SPqv3A=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: 301bdb1a-2675-48fa-9e0a-08db6b063d18 X-MS-Exchange-CrossTenant-originalarrivaltime: 12 Jun 2023 05:31:14.1719 (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: 6A6kPAMGupWuxWRVerxA3rIFsxxRr1gnRRjDj++Q7GZCdt5NQHAJruo7jkS+EL7IQS547vTAS9+WhEkf3a+tMQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR11MB4595 Return-Path: ray.ni@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Jiaxin,=0A= you are right!=0A= =0A= ExecuteFirstSmiInit doesn't belong to SmmRelocateBases.=0A= I will update commit message.=0A= =0A= ________________________________________=0A= From: Ni, Ray =0A= Sent: Monday, June 12, 2023 13:16=0A= To: Wu, Jiaxin; devel@edk2.groups.io=0A= Cc: Dong, Eric; Kumar, Rahul R; Gerd Hoffmann=0A= Subject: Re: [edk2-devel] [PATCH V2 1/6] UefiCpuPkg/CpuSmm: Add perf-loggin= g for time-consuming BSP procedures=0A= =0A= =0A= =0A= ________________________________________=0A= From: Wu, Jiaxin =0A= Sent: Friday, June 9, 2023 21:42=0A= To: devel@edk2.groups.io; Ni, Ray=0A= Cc: Dong, Eric; Kumar, Rahul R; Gerd Hoffmann=0A= Subject: RE: [edk2-devel] [PATCH V2 1/6] UefiCpuPkg/CpuSmm: Add perf-loggin= g for time-consuming BSP procedures=0A= =0A= >=0A= > The patch adds perf-logging for the following potential=0A= > time-consuming BSP procedures:=0A= > * PiCpuSmmEntry=0A= > - SmmRelocateBases=0A= > * ExecuteFirstSmiInit=0A= >=0A= =0A= > ExecuteFirstSmiInit is not belong to the SmmRelocateBases. Others good to= me.=0A= ExecuteFirstSmiInit contributes to the overall running time when the CpuSmm= driver relocates the SMM Base.=0A= =0A= =0A= > * BSPHandler=0A= > - SmmWaitForApArrival=0A= > - PerformRemainingTasks=0A= > * InitPaging=0A= > * SetMemMapAttributes=0A= > * SetUefiMemMapAttributes=0A= > * SetPageTableAttributes=0A= > * ConfigSmmCodeAccessCheck=0A= > * SmmCpuFeaturesCompleteSmmReadyToLock=0A= >=0A=