From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.14]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8D98A311C38 for ; Tue, 23 Jun 2026 07:48:58 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=198.175.65.14 ARC-Seal:i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782200940; cv=fail; b=JAONEqqT0syBNVpGSCzXdnqx37e0Q8146vkB5zlW+WESPZng166jCG2esMc+GmFVJZzr9qFVM5PIe3vG1gs9HPKN1EpnyI2xAPEJcz7uqqLUo2ZYXaPeTU/bXVa+066iHxRwGQJ0RvejCL8vpUSsf/gUmd/R4HX++QWcgH3lw10= ARC-Message-Signature:i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782200940; c=relaxed/simple; bh=r7xAzmfjXEhokoAE+lrEVBK67iWbzgBwarNO6vxLIO8=; h=Message-ID:Date:Subject:To:CC:References:From:In-Reply-To: Content-Type:MIME-Version; b=pQp7AQAJf0IZyYIZqPls1wGiz/nlKLi8ACwvH9gbo93sLSyXu8QP8M5+raR4w8aohf2+B/4js9h+q8O2DCAk0hcbGS8PDlMfMFu+/EOO8xBlPO38XtZNNW9aGiQ0NMkJDx33WYyhTMKyte2N+CxARgHGfSzlFCZHj+RlG217VPc= ARC-Authentication-Results:i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com; spf=pass smtp.mailfrom=intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=MW5rq1Ts; arc=fail smtp.client-ip=198.175.65.14 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="MW5rq1Ts" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1782200939; x=1813736939; h=message-id:date:subject:to:cc:references:from: in-reply-to:content-transfer-encoding:mime-version; bh=r7xAzmfjXEhokoAE+lrEVBK67iWbzgBwarNO6vxLIO8=; b=MW5rq1Ts4pfoNMSrE/UgT0ev+eAVDaSGFytjySBAMfn30OAia+8K+f3M eVey7CmviAmn/9s40HSwQAAIBECB6aNwXq2mcfoqaaz+qll0CX/x7xoH+ F2cAeSnrTSfZBK9zTmv+nKUO3YpMrTcxZ9MySlLAFWbjA/8PYsHRTbPr3 L9xjvIw96KRniLBjlw/k/xgdVm8jT/Bv/Q9EP95sAJT+Ia3YNEkQqJeeU R7uX8iTlQLoVwvH3hEO9gmANGgv3sbvTFboVe5fwKeVqKQtE0AiDhSCzR dzFz7LoO5SqQeImzSm6AVKhmZyIH3jugbcSQmVVHYSxxyExZg3rfU6Fxh g==; X-CSE-ConnectionGUID: 8hoB4Yh1Teqn46uImQWtkg== X-CSE-MsgGUID: sRA4dv2JSS+lUfAG0QRSCw== X-IronPort-AV: E=McAfee;i="6800,10657,11825"; a="86844334" X-IronPort-AV: E=Sophos;i="6.24,220,1774335600"; d="scan'208";a="86844334" Received: from fmviesa006.fm.intel.com ([10.60.135.146]) by orvoesa106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Jun 2026 00:48:57 -0700 X-CSE-ConnectionGUID: 3P8s6kt8QTSng0bf8Q7gog== X-CSE-MsgGUID: 3qcKRoq7TFCjH8SfsJzT3A== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.24,220,1774335600"; d="scan'208";a="245082195" Received: from orsmsx901.amr.corp.intel.com ([10.22.229.23]) by fmviesa006.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Jun 2026 00:48:57 -0700 Received: from ORSMSX903.amr.corp.intel.com (10.22.229.25) by ORSMSX901.amr.corp.intel.com (10.22.229.23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.37; Tue, 23 Jun 2026 00:48:56 -0700 Received: from ORSEDG902.ED.cps.intel.com (10.7.248.12) by ORSMSX903.amr.corp.intel.com (10.22.229.25) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.37 via Frontend Transport; Tue, 23 Jun 2026 00:48:56 -0700 Received: from DM5PR21CU001.outbound.protection.outlook.com (52.101.62.12) by edgegateway.intel.com (134.134.137.112) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.37; Tue, 23 Jun 2026 00:48:56 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=hSjv5e7c92M0WkIM+0NnE1yA7mC2l7xtXG7NzOHHQbOEmY0eKjWBRCiWq55IMcz0PdRBkQOLWe82Ry9MDF9vuCkt9yvqG54EFbg2XRvaa5BvAJPzCmz665JvfuRS+mYWOYVyotWVLMuq7zwAXtMq7oLwHEUju3PNuz3o8wJ8mZRTPZwdF6hdLsLCpOU4NE30fovGBhK22d24fC8lJMu2zTivErQon+pad8/RcpJgb6DMq45/thXvuoSHZMTdMArUW5YFPQ+QCQ78j3lYqp9A7Q83y1HZiBtNvJvDWRFU93hFaWtFCLcgk4Xf6awW4FkzEn/Pktazq1rTQTqD5yFRPA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; 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=IjBFhqv2VkRSl1SYS4JLL0fYqTar/Qx7invf+VXLwdo=; b=Oa4BBgM3LafiSWlKWlVn08FATyK6WZC2HmndVC/01lkOYAhnr5IAraubpgr0IkH17/hwbDAdjaBS/QCqqvm913A4Q3Hvc9iBC/QpimiuyhH33XDDVO5smEgzbHsf3hgQE6d0HW48YEch9gqcjPp9UcHHvxpxAlXHNCdRtF6a4LCPp/htn0hmpWRdNfBckmTq2accNvuC+pe4PcsNL7r9wZZ8fxahVBDTzz300JxY3aleOdSz3I2AbytjfJ8zzUqeQWS0sg7I4wl4E3ThKyxm1/zlPacHJ1MBJ43RhGYbq3DbGDn6i3vJ10vOenM4F/bm76MaOsrh/FSOAt2wGSpgvg== 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 Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=intel.com; Received: from DM4PR11MB6020.namprd11.prod.outlook.com (2603:10b6:8:61::19) by PH8PR11MB6949.namprd11.prod.outlook.com (2603:10b6:510:227::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.21.113.13; Tue, 23 Jun 2026 07:48:52 +0000 Received: from DM4PR11MB6020.namprd11.prod.outlook.com ([fe80::3058:1480:e4ac:5765]) by DM4PR11MB6020.namprd11.prod.outlook.com ([fe80::3058:1480:e4ac:5765%4]) with mapi id 15.21.0139.009; Tue, 23 Jun 2026 07:48:52 +0000 Message-ID: <4ee138d9-3d42-4452-a24d-b183690836ea@intel.com> Date: Tue, 23 Jun 2026 15:48:43 +0800 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v4 3/6] x86/resctrl: Rename prev_msr to prev_mon_val To: Reinette Chatre CC: , , , , , , , , , , , , , Chen Yu References: <2e9bd6a1a012b937e9205908ab6bbb09a21e0f71.1781332698.git.yu.c.chen@intel.com> <3ee018f0-689d-4175-bfd2-dda62a4aa45b@intel.com> Content-Language: en-US From: "Chen, Yu C" In-Reply-To: Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 8bit X-ClientProxiedBy: TP0P295CA0021.TWNP295.PROD.OUTLOOK.COM (2603:1096:910:5::12) To DM4PR11MB6020.namprd11.prod.outlook.com (2603:10b6:8:61::19) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DM4PR11MB6020:EE_|PH8PR11MB6949:EE_ X-MS-Office365-Filtering-Correlation-Id: 31c6c7df-3553-4752-e50d-08ded0fbdece 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;ARA:13230040|376014|1800799024|7416014|366016|23010399003|22082099003|18002099003|4143699003|56012099006|11063799006; X-Microsoft-Antispam-Message-Info: e3TbWTc9cyarvZ+g2b/S179QRvhMcIxc7PCUPiWuXhvlm493KRY1WgpQts3R7gQ6NOP5WgR39hy5vlvCvNopXrhxJdk42eWQUd4k52i+Usmh+01hAsNdHrCCdZLz29uBOS8Jh5B38Y5OXrxT/VQSfCSRZn6nOSwiOWm0cnITO9DBigyEKSb+GPn9uoeERnSu+hBjAVzYIZcGGzR5wCpVChvrRdGpWJcFDxMAkQWM03oI3JZ5ZiAJ866MM49ps91zEpprtOqZzmxKRHwi4VY1FY1eSH6EmV5V/oysB0Q6hqZaRuwazvrgXkTD/6OmTgOAGskmKZoxRQOTC4x7Y91oAWqr23k9vFD28dCXQ+4AUsRDNj3tAIL2jjoCuH/VsT3pjmveNNgK7j5bJxLDdEIpfuH+U+PZVr8tMWXLR2+gouYJ90Dd6J3TvpYDzTvM4K86jNfWk3Ruz0ebkSMF6rrIUXQX/vtqFYJ1gx4QkWDpfpXrv1c4xOD/RGkifEoV64ovW+rw4btSeTQAaHXMRIkljTdLQgZbQAO1eMREK+LFfAZ4W3hHz2RNrDXIZNsx5ePHQbpKGcAlGQJyqzaq7ZoSgy5V2uI6Azpq29ztSquZLEbdNmZ84681RrApp7KEWWJS6DWvdL4ceSMUr3fbQSUQdyKN6CLGbqZ0bW+/QIdfyYU= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DM4PR11MB6020.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(376014)(1800799024)(7416014)(366016)(23010399003)(22082099003)(18002099003)(4143699003)(56012099006)(11063799006);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?ak5yakF5VmI5VVdGN0w4NnNuWFhTbDQ0QUJGc0dKTWFFV1lPM25lSDhiVXl1?= =?utf-8?B?ZEVCdkFYMEx6TGd2WHRIdFltTTVmM1pIdXRrV055dVFCbWV5ZXR2K2RyNGV2?= =?utf-8?B?L0toUklMNDM5SVZ0aGlJODBmNUcvS1JOOWtRUEdkc3FpSmpOU0FxMUdnS0dL?= =?utf-8?B?azI3SGxNYkFIdVF6NTJjUDYxT0c5Uk5rVVpIRTF4TDROd0FiT1RiN3l2dDVq?= =?utf-8?B?WmRMbzZXRFcvTHpEMkhNQm95Z042dWtZYWQxQlNPQmZtQ3hJejU0bUYwcUJq?= =?utf-8?B?Y3JhWFM1ZlJjUFQ0SGNaWDlYdDJGbFg0eno2ZnZ5d2dsUjZUVzBwdnZybkFY?= =?utf-8?B?dU9vblZ3dzZRSXhHMGVzL0ZueDRidG85UXYrR2Z6UWJHMTA3SVNiOEtJTlg2?= =?utf-8?B?TWJyVlM4ZGxmVVB5alhDaUw2S3g1SjN2dDdSZFZYU0s5cWdXd0J3MlFvYkMx?= =?utf-8?B?aEs3M09EaEM5aWZiemt0cVR4RzJmVElsT1hEVk5SWkhMZkRKeWYralZmdWxq?= =?utf-8?B?OStrWUV3YWhDR2tNUDFUb0JEZlFNcHp2dUVUNEVFaFFWSTJaOHEvUWFBSHUr?= =?utf-8?B?aUF4STNhdmVWTHVpSDF3c0FZZXU4Q1RiM1RSTEdSS3I3eVExNTY0WG9ab0tS?= =?utf-8?B?NzE0OUhJMVl1bExIbDFKTUNzWTE2bGNNQnRXamVOakltTStiUzYzUUxhekh1?= =?utf-8?B?SWZjeXFHZzc4T09BWnRTbW5nUzFsdUNVbWJ2WTdwLzh5UmFlZGFRYlpKR3BJ?= =?utf-8?B?d3B3elJDZUxJSllmK3h6U0xZcWRza2lGV0MrOXhGWWc1WVFyV2xyS3RHZ1ln?= =?utf-8?B?QTY3RUo2aWt0cldVY1F5Q0dtdjJRK0hLQTVFY2xlRkJTRlVEY3FhWHlXalJp?= =?utf-8?B?SUF0eDFnUGpRMnFMZ2pvOUhRd2VlNHZUR08wMWNrbncxd1RpcU1HbXd1RnNv?= =?utf-8?B?RXJQdzlDTEdoWkhNaHBJbVQvVkxQb2YwVjAvYnd5T3JUdE9vS2ozQnNXblMx?= =?utf-8?B?ZHg4LzV0dnJaeHF2cm56SUFYV05UaFFJZDUxeUJobHZSNTJUTmxMN1NQRzg3?= =?utf-8?B?Nm1FMkROOXFzdXlnTHk5bFlML0l3Q3A3cUlGOEpFelNsaFp0TUF3MGs3ckQz?= =?utf-8?B?d0RkN3kzd0lPVWt4L1BvSWllN25DLzRkOWpaVCsyZ0dzenEyUnQ4TWw5MXhO?= =?utf-8?B?QmpheVdwNzhUd0sxU3BvcWFnL28xTVhqZGlFZEdsNjZkMitaczRNUXBMUktL?= =?utf-8?B?Ykg1a0ExM3E4WDdocWZZUzlaR253MTRIblRlMXU3NGJKVkxEaU1sTzRZZjVn?= =?utf-8?B?My8zdHFsbUcvbXc3aWpnME9ONmpuTW9JenFRR3owMm1iK1lmR1Q1Z1p3ZE9O?= =?utf-8?B?VlhIcVd1OUcrbzNibEFWZ25EMnVNbG5xQmJjUHJ0L1hScjJCVkxManJDbFFY?= =?utf-8?B?bXZQYWMwc1p0V2VQWWpvZytJSGhJR0ZWbm9hWUdnN3htL2Y2ZlpMeThRcTBS?= =?utf-8?B?M0lVT3drNWlTN2RmMG1TWEUrTjd2T29NYXpIdUJ2MlN2QS9wVWZ3Rm8xdG5B?= =?utf-8?B?V1BDdTJhWVIyWVA3OGlEanRTT05mUkczVStpaUloSmE3c0VJaDIzMERKZXFE?= =?utf-8?B?S2RHYXVhWFZmUnJiOHBMdUd5cm5DQnNma0kvdUFBRHowSnlsaW4xK3Q4bWFN?= =?utf-8?B?d2wzOFlXcGFRQjZ2Vy9ZRVZSRFNHUUk2LzRzZnVTazVoOEt1blJSNnVGVjha?= =?utf-8?B?cURsaFhtODR4THVzendKd3kvWGFmR2cvMER0UWhNbHpldlFpWktTK05FUWxW?= =?utf-8?B?YlZ4VmFVcXFuejhjMWZUYm1MbkxuYlJOM3lSS291VVVSMHRveENRTXdMZlBJ?= =?utf-8?B?alp6UFU3UmpZZk9hblF5dnN2YkxqVk43aW0rSEZ1T3d4ZUVQem9CUUE3MHpr?= =?utf-8?B?aWJSYzA1dkRmd0pXaHpOT1NlOFo3RGRsN1dVT3l3cXp4bDFNU29FaVhCdVFo?= =?utf-8?B?M09IRW1wQTl0SFk4a2ZmV2V4VUxwOUtoQy9wSzUyUjRscE5VVjZXazlNdXg5?= =?utf-8?B?UVpHZFoxVmUxNndBYjRGQlNQTUJZaTlpd2FLZmd2MTJ4M1phT1NFWE1KMG1D?= =?utf-8?B?UkRuRHZtZTVFd3YzZDFsT0ZveThzdmlUTXkyQmJaY3d3UmR4TGRTUmk1Vy9x?= =?utf-8?B?cndMRlhzdWYzaVYzQXVmVW03UHVseXZQOXg4RTdNd3hZK0xkbnJGZ0Yxa2gy?= =?utf-8?B?MkR3UmVuRWYwSDFaUlR5OWE2MFMvaER2SlNZS21JcDU3TUREQjBTZlMrMmVP?= =?utf-8?B?cWZPVGJJN29CWGFXTCt0T3Y4bEVaTGYrUGx6OThWaFl1RUI3Znc5Zz09?= X-Exchange-RoutingPolicyChecked: gLXVO8Eopz9gN83MkCWrtJ5HcawMQpCD/YMoU02nnb+/lRwb+31+6Qu4SVRyC7huSLQyP2qr9LmK6lvLUNxcnTJLMATKBeXIGO8Xfr/R0J4vEcWhVTNHR4aOMKN8gipBlEEryN5ytaXjsuTrCd/cXDZt89kTZf1zt56cP5V1cxTZSCe1rGGI/UlR7zDVPt8fWt3AxjfqzNqRmewNbpFWBb9D/+wKckW4ZYhNlQ5PiSNphhYDOcQlEHEKTrEVvpzU+GPLOvLvaZeh41cq8+Jj0bD0yEJETDDjtxnm/mgtmTyxXHjOkBoRKB1yr2Cli32vlCKusvGsqX9rbBnewRW3VA== X-MS-Exchange-CrossTenant-Network-Message-Id: 31c6c7df-3553-4752-e50d-08ded0fbdece X-MS-Exchange-CrossTenant-AuthSource: DM4PR11MB6020.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Jun 2026 07:48:52.7966 (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: 4LPh8/pPws3+9cuRAox6JdbZ9ovr/+bXgSYgPAvP3GLDFbfE2VeoY4Mkm+qy54NyaVevYXRsj3Op5y7zEPaDCw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH8PR11MB6949 X-OriginatorOrg: intel.com Hi Reinette, On 6/23/2026 5:29 AM, Reinette Chatre wrote: > Hi Chenyu, > >>> >>> On 6/13/26 12:57 AM, Chen Yu wrote: >>>> Rename the prev_msr field in struct arch_mbm_state to prev_mon_val. >>> >>> This change as described seems out of place when considering the following >>> from cover letter: >>>   This patch set focuses on the first part: enabling MMIO-based access for >>>   Cache Monitoring Technology (CMT), *while CAT/MBM/MBA are still using MSR*. >>> >>> It is thus confusing to find a change related to switching MBM to use MSR >>> in a series that claims that it does not do this. >>> >>> The patch looks fine but reading the changelog and the first few hunks makes >>> this patch look out of place. Could you please update the changelog to be >>> accurate about what this patch does? >>> >> >> OK, let me change it to: >> >> x86/resctrl: Rename prev_msr to prev_mon_val >> >> Rename the prev_msr field in struct arch_mbm_state to prev_mon_val >> to decouple the field name from the specific access method. > > This patch changes many more names to support this switch to MMIO yet the changelog > only highlights one of the changes which is the struct member specific to MBM while > the work explicitly states that MBM monitoring is not impacted by this series. > > Consider for comparison a more generic (just a draft to present an idea, please do not just copy&paste): > x86/resctrl: Replace "msr" in monitoring data identifiers > > Monitoring data is only consumed via MSR and many identifiers handling > the monitoring data contains "msr" as part of their names, for example > "msr_val". > > Replace "msr" in monitoring data identifiers as appropriate to support > their use for monitoring data accessed via MMIO. > >> >> Use a generic name for the stored previous monitoring counter value >> so that the field remains accurate regardless of whether the value >> is read through an MSR or another interface in the future. >> >> No functional change. >> Would the following be appropriate? x86/resctrl: Replace "msr" in monitoring data identifiers Monitoring counter values are currently obtained via MSR and many identifiers involved in processing these values contain "msr" as part of their names, for example "prev_msr" and "msr_val". The ERDT ACPI table describes MMIO registers for monitoring access. A subsequent patch adds MMIO-based L3 occupancy reads. Rename "msr" based identifiers to interface-agnostic names so they remain accurate as monitoring values may no longer originate exclusively from MSR. No functional change. thanks, Chenyu