Received-SPF: pass (google.com: domain of linux-kernel+bounces-149519-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Message-ID: <3391a909-d8a4-4ee6-9836-b16c0ad8944e@intel.com>
Date: Wed, 17 Apr 2024 22:16:45 -0700
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v1 12/31] x86/resctrl: Move max_{name,data}_width into
 resctrl code
To: Dave Martin <Dave.Martin@arm.com>
CC: James Morse <james.morse@arm.com>, <x86@kernel.org>,
	<linux-kernel@vger.kernel.org>, Fenghua Yu <fenghua.yu@intel.com>, "Thomas
 Gleixner" <tglx@linutronix.de>, Ingo Molnar <mingo@redhat.com>, "Borislav
 Petkov" <bp@alien8.de>, H Peter Anvin <hpa@zytor.com>, Babu Moger
	<Babu.Moger@amd.com>, <shameerali.kolothum.thodi@huawei.com>, "D Scott
 Phillips OS" <scott@os.amperecomputing.com>, <carl@os.amperecomputing.com>,
	<lcherian@marvell.com>, <bobo.shaobowang@huawei.com>,
	<tan.shaopeng@fujitsu.com>, <baolin.wang@linux.alibaba.com>, Jamie Iles
	<quic_jiles@quicinc.com>, Xin Hao <xhao@linux.alibaba.com>,
	<peternewman@google.com>, <dfustini@baylibre.com>, <amitsinght@marvell.com>,
	David Hildenbrand <david@redhat.com>, Rex Nie <rex.nie@jaguarmicro.com>
References: <20240321165106.31602-1-james.morse@arm.com>
 <20240321165106.31602-13-james.morse@arm.com>
 <fc4ee516-54b7-47cb-b881-00ce10d311a9@intel.com>
 <ZhfwjBJPeTvO04BL@e133380.arm.com>
 <d269b5d2-bd6d-44b2-8d99-0e0a2790bf50@intel.com>
 <Zh/flJb3obutHTbD@e133380.arm.com>
Content-Language: en-US
From: Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <Zh/flJb3obutHTbD@e133380.arm.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?YUpDMUNvVFlOc3o0YkMyTXR3cEJFVjFYbjlQTGZmZVlyQ0YyaFhCNm13OUJK?=
 =?utf-8?B?eVY1MDFJUnBUTVR6REt2QjFQZ2xCMTNza1BWbFh5MWRoUzEyWlJrOG9LVnJI?=
 =?utf-8?B?WFc2OG80TWI3QzZuS1NMWkYyLzFZejYwNE0yOE5Sc2IxZTVjVlBPcGM0VGtT?=
 =?utf-8?B?U204RWV0SDdpK2YvZmVaQ1p0RmQ2bHQ2S1RPajlLR0Q0Myt1am1uUm5KamJl?=
 =?utf-8?B?S2k4UU9aa3dIQmVwalEwZmVLSis2YlVBNitYYlRDUGE1K0JocEZtQkFDSnV3?=
 =?utf-8?B?c1V6bE5vaWFjYXphQmQxQTVGMkkyTmt3RHR6YWJtUzUzb0cvM2R5VnFyOG1l?=
 =?utf-8?B?QVJ1UE85d2lLVnYzUGVxMDRzVWd1Z1lyQmx6NUdEY3RELzNCL1RyZnl5azE2?=
 =?utf-8?B?VitpdGh0TVM2dGhSa3VVek8yalhiWUQrY0UzWHlnU0l4OVR0NWRzcDVpakM0?=
 =?utf-8?B?K3cvR25WZHQ3NjRTMGxXM0NSUlB0RmF6b0daeWdCRlBHelpVZW93d09janlQ?=
 =?utf-8?B?Q01TUDV4cE12bm8yV3VnakcxS2wzNWp3QVVIaGdRaGRCQTZ3WS9lSUM4VHZt?=
 =?utf-8?B?NnVZZzQ3NEd3TDlKSmMxTmovNVk3QnpySDd0Rjc1TnFvZXdyOXZkN3hyWTgw?=
 =?utf-8?B?anMyNkdVd2ZoMDlBd3dXTkp0TURxQm9XVHF3K2Y5MEUxd0NmUXVOWklHUjdn?=
 =?utf-8?B?eHNleGJNRXhQTzlWa0JUSkVJZjErSlh2cmZDTVRLTTRwbE5zNDVNMHBWalFR?=
 =?utf-8?B?TUt2MG1kb00ySXJqWnVYUjFEQ2dHVW5ST3JzdWMzNWZ3S3JESUswbnVUTTQw?=
 =?utf-8?B?aU93NGxHOXBFMHpjTUNNQldUSXRlZVpnVHB3K3hHMHF0MFFHY01FZmxONTFF?=
 =?utf-8?B?YmN3dmhaczE0S1JBbzlXbm9KOWhTZlJyTVFqZmJ3S1VWcEk3TmF6NHlLcG9C?=
 =?utf-8?B?T1ZULzJvQkhYYjBZZzhFNVdJSFErNmFJano4L3ppWGsreHBKL0lzZCtjVkdD?=
 =?utf-8?B?T0IrYjd0bWd6NXBoSFRFbFBMTmVrcVlYR3I0TDkwU0dCVldUTkQ0OFF6QW5p?=
 =?utf-8?B?SXRJL3RkNmZvME1qdU1Ba0JoNHhmcDE3aExKdzFhc21vVVJVcmlhYTBPOW00?=
 =?utf-8?B?YWFQMUR3UEdhNnVFeXE0WlpjQTRqcHQwMmM1M25pWXIrTWhPNEJGQmxpbEhP?=
 =?utf-8?B?dzc3TDJWNE5yd1F3U1dCTVpNVStqZ0pQbTMyY21yN1VpYzdWSy9aUk05K3Zh?=
 =?utf-8?B?aWJGNjliMlExRkh2bW5ZcUl1ekVrSmZueVZIdGpBVFNSOFkwUDNVVW1JZUJu?=
 =?utf-8?B?Uk9kMXJ0bkkrZHRwb3B2TWFuOXh5UVNuU3Y5VC9yRG5HS0pGc0FmN1JFN0lU?=
 =?utf-8?B?ZFplR2l1clU2TFZGREJwTFFOTm5UUHd5ZjN1RE0xUDlnNWM0TVVxTng1Z3Iw?=
 =?utf-8?B?QXlEaDdvM1IvUWpFcmcrYVBrNTJ6c3NGNlM1OU9GL3JwMDA3K3c0QmNJQWJB?=
 =?utf-8?B?ZFFPekZxSmMzVTcrdEt2RVhMOTVUWFBtMFpZZFgwSmIyN0ZRT0NobmNuSk5S?=
 =?utf-8?B?ZXpMeW42bER2R0V5UzIwQ3VyNk14ekRTVDlJZ3NzQnNNS2pmVVZQaTNSR2ZO?=
 =?utf-8?B?K1lhNG95OURpdUNpL1JwSzlhVVh0VkFzUmdUeWdIN2xjcjRPak1RNXh3Y3hy?=
 =?utf-8?B?RHJ0ajhTSUtVckJreEp2b3M3R21lQzFXYklXUmRUbUlYMjY4RjlISXlFdWIv?=
 =?utf-8?B?NHJjS1VuMC9jenIyMW5DeUFkUEJ1QjA0VTdRTmFSWFV3ZUJIdkhGSEsvWmVx?=
 =?utf-8?B?Q1o0Y2t2S3Z0azdDUVpKanpsNnZUMVBOTmNhQ3RtOTFuKytOWDJXb3ZtMEp4?=
 =?utf-8?B?MHFUeEV1TW85MjM4ZGYxTlR6dmgzQ2RCdEpnWVV4YzZjR2E4SjVObWhsbmpL?=
 =?utf-8?B?SVFFTW1OdmZ4SHozZ0NIWDB4SmgyODg3MzBKWStGL01qM3BtR2tkZVc1a3pG?=
 =?utf-8?B?dnB1cWQ3MTFQTWVGUDdvZklVKzN0Q3lkV1VkY29WMEs5VGNLTkt6cUhGNmZq?=
 =?utf-8?B?eUEyNTJXa1l0Y21UR3FYL1hDbXhMMGhFTjdFMGpQbWJjRlowbDUxWUhQUHJn?=
 =?utf-8?B?L0Q1V1Y2VWowOGVmbUN5cHczejF4NHdFVlNJNEY1b094RVdnYW5BN1NSaDVi?=
 =?utf-8?B?MkE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 01202f35-a32b-4605-3d66-08dc5f66bef4
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Apr 2024 05:16:47.5591
 (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: my5GQx5hndNpjvVpW05EquDOe6HVoB3CiMd4FJlmnnKdarQFjyi8dTj7FqtbQNEhjnBOhT9CLdNLGVa7WVy54UkiPJhBpMwWEaxo5VBkmSU=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA3PR11MB8004
X-OriginatorOrg: intel.com

Hi Dave,

On 4/17/2024 7:41 AM, Dave Martin wrote:
> On Thu, Apr 11, 2024 at 10:38:20AM -0700, Reinette Chatre wrote:
>> On 4/11/2024 7:15 AM, Dave Martin wrote:
>>> On Mon, Apr 08, 2024 at 08:19:15PM -0700, Reinette Chatre wrote:
>>>> Hi James,
>>>>
>>>> On 3/21/2024 9:50 AM, James Morse wrote:
>>>>> @@ -2595,6 +2601,12 @@ static int schemata_list_add(struct rdt_resource *r, enum resctrl_conf_type type
>>>>>  	if (cl > max_name_width)
>>>>>  		max_name_width = cl;
>>>>>  
>>>>> +	/*
>>>>> +	 * Choose a width for the resource data based on the resource that has
>>>>> +	 * widest name and cbm.
>>>>
>>>> Please check series to ensure upper case is used for acronyms.
>>>
>>> [...]
>>>
>>>> Reinette
>>>
>>> This patch is just moving existing code around AFAICT.  See:
>>> commit de016df88f23 ("x86/intel_rdt: Update schemata read to show data in tabular format")
>>>
>>> Since no new usage of any term is being introduced here, can it be
>>> left as-is?
>>>
>>> There seem to be other uses of "cbm" with this sense in the resctrl
>>> code already.
>>
>> I am not asking to change all existing usages of these terms but in
>> any new changes, please use upper case for acronyms.
> 
> While there is a general argument to be made here, it sounds from this
> like you are not requesting a change to this patch; can you confirm?

Sorry for confusion. I do think in a small change like this it is a good
opportunity to make sure the style is clean. Since this changes the code
anyway, it might as well ensure the style is clean. So, yes, could
you please use CBM instead of cbm.

The final patch of this series is in a different category altogether and I
do not think style changes will be appropriate in it.

Reinette