Received-SPF: pass (google.com: domain of linux-kernel+bounces-127301-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
Message-ID: <387a313b-9a1f-4997-acc7-932291f69ea7@intel.com>
Date: Mon, 1 Apr 2024 19:26:38 -0700
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH] Documentation/x86: Document resctrl bandwidth control
 units are MiB
To: "Luck, Tony" <tony.luck@intel.com>
CC: "linux-doc@vger.kernel.org" <linux-doc@vger.kernel.org>, "Yu, Fenghua"
	<fenghua.yu@intel.com>, Peter Newman <peternewman@google.com>, James Morse
	<james.morse@arm.com>, Babu Moger <babu.moger@amd.com>, Drew Fustini
	<dfustini@baylibre.com>, "x86@kernel.org" <x86@kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"patches@lists.linux.dev" <patches@lists.linux.dev>
References: <20240322182016.196544-1-tony.luck@intel.com>
 <56a93ec2-dc01-49be-b917-5134f5794062@intel.com>
 <Zgbe2FFwyHMmmsyM@agluck-desk3>
 <cf59f587-9ca2-4f0d-b412-69b559acbabb@intel.com>
 <d6a649fd-0bd3-4777-acb3-2b9362131796@intel.com>
 <SJ1PR11MB6083D3C5616E60D3D03CCEB5FC3F2@SJ1PR11MB6083.namprd11.prod.outlook.com>
Content-Language: en-US
From: Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <SJ1PR11MB6083D3C5616E60D3D03CCEB5FC3F2@SJ1PR11MB6083.namprd11.prod.outlook.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?NVdKYXZaL2NZMm5CUlNrWDZ5NzZDdzNNZGtWMzhGcWtrQkxvamtXWE1ZbjNz?=
 =?utf-8?B?dlI5eU1CSGpnc3VZSHpjbVVoOVBIc01OQTNxQWxyaHMxNUZhZ2pKUDhEVXg1?=
 =?utf-8?B?eXYrUmhNQ1JoQVdSWTVDK1Evc1Q3d2UvM2xVU1dqVWo2SHYzUzVwMjdaa05Y?=
 =?utf-8?B?STlRd1o3cWV5TCs5SmZSNWtlMFhCL0sxd0NTNWZTTVk0UUJtTkZmUEJiUm03?=
 =?utf-8?B?UkpuV0hxSndTbkI2bGpxeW00a0VRUE9QQUZ6S0FqdEpxenc4MklzdllXQkpE?=
 =?utf-8?B?UldtZE5RZ1VoMnhmREJoMzM5MFo5bU0rTm5kUHFHRUo4ZEllMkE4WFUyUUla?=
 =?utf-8?B?TGpWRXMySlNXcE9rSFlscHZrOURRL0p3b1ErTmowdE5GNzQxcmhnWEcvL3U2?=
 =?utf-8?B?cGNsZVY4ZVk0WkZSZkZMdjRmVXdQSzR6U2Y3aTVxUkVkd0NGNnYrVWlaOVZQ?=
 =?utf-8?B?Y2o5UG1VRjRwZkxQazkwNTNicHIzd2gxQlF3RUFJVkFDR2ZMUEM4S0cyQUFS?=
 =?utf-8?B?VElQYmw0RXhpdlRmREVkTVphWXVONHpUN1hJcmRCVWdOMEtPVjRpY3dwcGQx?=
 =?utf-8?B?NzYxaWRwOTY3OHZ5dUkxMlN1QVRhSmNORGtCelRQMFY1aXhyTndETWV3WlJ5?=
 =?utf-8?B?VnhlVWRLakl0VlovVXR3RGhTR2lld0pSNFVZQnJDVTRmMTg5ODRsemNIYmM3?=
 =?utf-8?B?S0IwWXhHVk90Ym5paEM3SXVHSCtuNEh2bm9HN2VNdm0yTDU3eWFPYlhZVkkx?=
 =?utf-8?B?NlM1a0RKVTZSeUNnc0M1akg5YXBDdHB5VDFINStweCt4cERONng5MkZ6WWdL?=
 =?utf-8?B?WFVVZysrMGx6K0FDc2FDN0tFbXpET1RuYVRjZFBLSmVsOUxzajMra1lxbWV1?=
 =?utf-8?B?eXdGcTZKei80eCt5VjlSek5IMmtIZnVEK0laMkJnUzFKWGhZcjFhZjlXNzhz?=
 =?utf-8?B?RzdqTk9iQ1Q1dkV4eEIvYWFtdjE4aXBOMDlsL3lUcm43Q25qZUh3VFlacXZP?=
 =?utf-8?B?bzcxRW9KNE5GNjRtUGl1YnI3cXBXdStrOEROY0lKcUNxOWpUVXF2Ky9uMjZJ?=
 =?utf-8?B?Y2RRK3hHaGszT3BRdyt5YzN1TGtyU1U4NGlPdTVJYUJaVEtPZDNoN0hBQjgy?=
 =?utf-8?B?ZUFMYVNZbGNpQlpCay91UTZ3OTA0a1dFVGNONFV6SFFSSVltMGlhaVQzQ0Fn?=
 =?utf-8?B?eDM1cmZKZ29aWHNMLzNaMTBmUWtGWUg1S0FPeTNwVTdkU0ZZS252NldCN3hY?=
 =?utf-8?B?eFovZThCb1Q3MUJqdXlmSjRjWmVzSCtxZFNTYS8vMDhGUlN3N2dOYyt0bS85?=
 =?utf-8?B?NFpHcHRtVUNOeXgwMnB6S2Q5dDFHRTd5VUpPd1l5Q3djY3ZFbDBobS91Y215?=
 =?utf-8?B?Zi9kQi92R2tsTkhuRXlSbzB0SUxIOWE1NjhrRk4wMWdKeHNzRWl6OWc5K01u?=
 =?utf-8?B?ejVOUnhUYURiVkFmM3BWTVF5Zjh0Z3RRQklzYTliWHFJOVNORTdLQ2FESGpN?=
 =?utf-8?B?NkRCczdLQ3g4c2pYTlRaWW40TTdOUVVKZHA4L2ptalFsYnBhOWFZL0tTYjRM?=
 =?utf-8?B?ZFFFTDlJdzg2RHY5NHZ4empXdUs5MFRSeDJNMVNYd3pFR2dzb1JaRStxQ2RV?=
 =?utf-8?B?VEZ2VXJBRDZMTXFqOUFBZW5XOEQyUTluekpUQTBPeG5PVjdCeG12T3RFYzRl?=
 =?utf-8?B?SlVLSWpzL2cyTVMwc2VTdGNmVDhEZ2Z1TGZRSGhwVGVvZlhPT0w0d1hMUVZF?=
 =?utf-8?B?Q0Y1QXRpcXpTZkFHZnRIdHlkWDRSWTRMa0g3Y0ttK3dEVWs1T2ZVMFlCZUxE?=
 =?utf-8?B?RTJSdWZtQVJCSWt4TGtWQ3prNmw3R3k4N2pPTXcwdDFrUmJCVHNkSCtEcUQw?=
 =?utf-8?B?dG1aYnJyT0lQMHNZZ1hKY2dxZU4vdVBLaU9PS1N0Mkc3Vm9iVmFVNENFak40?=
 =?utf-8?B?WWxrdncrSUJBdUIwZFV1WkNZNGhvWHIrRG04Q2pML2FEeThvYmtDVUhUK0hB?=
 =?utf-8?B?SUdMTU81eWxvaVF3VmN5SlNnaDM3UXZ6WnJJUHd4K1FOdW9hQjZNK1E4cjhk?=
 =?utf-8?B?Z05DamZKUklTeHdVbWt1T2kweGxWdG5TazJiM0lkdWhGSFQ2K0JxazRybXh6?=
 =?utf-8?B?SDl2b3djRkNrR0xETlFSbU5JMFI4T1J6SWtIdzNwelFkek1EaWhlcWc3K051?=
 =?utf-8?B?YXc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: c5ee19ab-29a7-4f40-3df9-08dc52bc54c9
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Apr 2024 02:26:41.1230
 (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: IxxRF4UY7b3e24a39eH7xrYXBhoFIV6VOXnwEIl7fs1AbIjNrNAKcU8+hlBlRDU9ZjD15Zqkf6xuh5dMGloR/3d7fCQQg6G4oKRHoI4yDvU=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR11MB6500
X-OriginatorOrg: intel.com

Hi Tony,

On 4/1/2024 4:03 PM, Luck, Tony wrote:
>> It appears that there is no custom here and it may just be somebody's preference?
> 
> Reinette,
> 
> Thanks for digging around. I had thought there was general consensus that
> memory was measured in 2^20, storage in 10^6 and networking in either
> 10^6 or 10^9 (but bits rather than bytes.
> 
> But, as you've found, there doesn't seem to be to be even that much of
> a custom.
> 
> Maybe a case for https://xkcd.com/927/ (since it is April 1st, I propose
> everyone standardize on Teranibbles per fortnight[1] :-) )
> 
> But back to the patch. As there is no standard, changing the documentation
> to accurately represent the code looks like a good option.

It is not obvious to me what the right thing is to do. The documentation has
stated since the inception of the software controller that that it accepts
bandwidth in MBps. You demonstrated this is the case but it is not obvious to
me that this is a documentation problem or an issue that needs to
be fixed in the code. I am not familiar with a precedent in this regard.

Changing the documentation does seem like the least controversial approach.
The consequence is that resctrl documentation itself now switches back and
forth between the units ... it uses MiBps for the software controller and
GBps when referring to AMD and talking about memory bandwidth in general
(see section Memory bandwidth Allocation and monitoring). I hope that it
is clear enough that MiBps is just related to the software
controller.

Reinette