Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <a75fefb5-530d-c802-4670-3c8262e9abb8@intel.com>
Date:   Wed, 24 Aug 2022 10:39:40 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Firefox/91.0 Thunderbird/91.12.0
Subject: Re: [PATCH v3 03/10] x86/resctrl: Add a new resource type
 RDT_RESOURCE_SMBA
Content-Language: en-US
To:     Babu Moger <babu.moger@amd.com>, <fenghua.yu@intel.com>,
        <tglx@linutronix.de>, <mingo@redhat.com>, <bp@alien8.de>
CC:     <eranian@google.com>, <dave.hansen@linux.intel.com>,
        <x86@kernel.org>, <hpa@zytor.com>, <corbet@lwn.net>,
        <linux-kernel@vger.kernel.org>, <linux-doc@vger.kernel.org>,
        <bagasdotme@gmail.com>
References: <166117559756.6695.16047463526634290701.stgit@bmoger-ubuntu>
 <166117578296.6695.6947071073613460712.stgit@bmoger-ubuntu>
From:   Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <166117578296.6695.6947071073613460712.stgit@bmoger-ubuntu>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?cjhEZVdhbnBaMEVaeXQwMzhMVm5LM01lekd6S2w2U3R1ai9OM0ZwVXBVNld1?=
 =?utf-8?B?anZIWFdwWWdRM1kyNFQwQnZQeEkwQldPRTVMYVNHOTFNOWUzd3phK1hjeXV4?=
 =?utf-8?B?bmhFL0FESWRMRXV1UGFPNVJtNHg3UFRtSk9iUHVSRktVL1V3VTErVk9aMjV3?=
 =?utf-8?B?TGZZSHVHc0NLRlVlZFViL3cxTDdXVmZEbGRQQlFiYXdGOEw4UUZYTDVKZm9n?=
 =?utf-8?B?RlE1VXZlVmhaSWxuUE9KNlh3NVgzL25iaDBHNVVKaGVpdkJKWEVzT1JCV3Ny?=
 =?utf-8?B?Z2VJQVZYUTlSWHNaQUFOOXBRTVpkMjdLMlM0ZWpOQThrdThBaUg5K3crUnBN?=
 =?utf-8?B?TU5GN2tXbkJmMVlQWU9FTTk2eUdvRlpNb05ZRnpnejNOc0VkTXZRcWdxTXEy?=
 =?utf-8?B?cmdqUXNOZTF1QmN4bHJoSlhVL3N6WHFBbCtMSWZFSlRlSytaZjN0SUZ0Vmdy?=
 =?utf-8?B?VmorZG5mTTNzbGMxUHMrNTJxcEwrMThDYXhmMFNiSTNMUkxWRkNGRmFqcGsr?=
 =?utf-8?B?QmZTZkhzKzU5dnh4SUpGUWkrTVgva2RxSmNEUS9VRVMwaVkwaGxua2ZQVnRY?=
 =?utf-8?B?UUM2L0doNDAwNzJ0YVJtZzJZTWpKVk9vY0JRck9VNDErbC92dE9yVW9nQlJI?=
 =?utf-8?B?RHh1QTY3dHVOcEszaDB5ZEdxOE0xdUIxVFhvcGh5cW95SUFQQjlTMU9PQXJS?=
 =?utf-8?B?WXBQQ1FkWE9JWDY0MytkNHVaaVZhY0xBNmwxd2RMWlJGdlpKNytYbit5ZFFF?=
 =?utf-8?B?MVBaTXd6Q0VMSjNBdWpoRm0yKy9CNDM0bkZTTG55ZzRqYVZ2V0J3dE91VU45?=
 =?utf-8?B?K0lWN0ZiQ0tJUDRrbmpUVDk0MzVZeFBNMDc5Q2ZQbFVQaSsvWlFKL1hGYWRs?=
 =?utf-8?B?c2hWeTFOS2psb2lET1VDak5xMUhxOWMvZ2owSlJKNDIyNlVhbVZRcmd6ZmRv?=
 =?utf-8?B?QlIzTkV0T2gyK2NhbDJlZXBvQXZmRld6bWQyMjBwZm1pNmNsanVuZXRGbkNn?=
 =?utf-8?B?VkFUV3dqMlBXcmFoWTI3cU8wa1VtSjRLcVRzYW9ZbCtzQm0zRHV6RXRzQWx5?=
 =?utf-8?B?QVFtQU56cVhBM2lxK2lKekt4N3BZcmhtVitmWnRiMWt4NkoyV1dGQ29ySUE1?=
 =?utf-8?B?ZmRxV2RWdFM2RjlxaC94elJIVjJWOXdCZHVhTXlRcTBMQ1Mzb1huNmFPNnk0?=
 =?utf-8?B?Qk5vay95WEtlOXRvMUZmUzhObnFnc1JvZXVRZXVGVFpMc2lGdzRQRjZTRnRC?=
 =?utf-8?B?NUhHK1kwU3dQMGx1YXZQRWdsNWhiUi9BMG9BMW1XSWV6aXhZWkhhWnNhbXFM?=
 =?utf-8?B?RjlWK3pEdm9kRWxkeFR5RlVFQ25zT2hEbkJvem9JaXJ2TXlIcVpRSFVtajk0?=
 =?utf-8?B?T0k0Z296OGNobzJsZ1dtQUpLNGZXVHdzUURSSS8rU2ZmQkIxcjFKMU1vdUwr?=
 =?utf-8?B?eGZnTWZYeGd0WFNLbXYyalIxY0c3WUdoRXZIc21tL2dpTTMwbXVpU3RDejI0?=
 =?utf-8?B?dnd6YmFaU2dVOWV6Y3FIbVlmUXBHRHRrb1VHQWQxYkFJemM2YXNTN0gxdFVq?=
 =?utf-8?B?bkM3Z0F1RjRuNDBuR2NtYVU5WjN6MUVLdEZqRDVuVGFHQ3JkaTM2NlRlZHh4?=
 =?utf-8?B?ZFVkVnZWKzE3SVVmSjdEMFVXVzZkUE5zbHZ3RE9NOVp2ZTBoeGJxS2pyL2p3?=
 =?utf-8?B?NERZZ1Bub2xOdlh1S1RrQnNqNXJNbXp1VHZ6VWJDemVheXFPRjdoNTg1MzVi?=
 =?utf-8?B?ZWcwNVdxWEh4Szc0UjRGV1NKL0x0WkYzcll6WVNsdUdBZ3lrc3g4L2RkeUFR?=
 =?utf-8?B?NUZoaHVza01hSFpKOFpRSTRkZkxmZ0hpaXgvVnFqOFlKZE5ISm5GWjZTZ2tV?=
 =?utf-8?B?V1EvdHd1WFY3ZzVTbkY1N3ZPQ1psd0VtUnNodG9hWnZzb2w0M2s5SWp6aTh1?=
 =?utf-8?B?eitXczZkL3dQRFRLUld3VjhKK2k5WHV2aWxGTDM0WHpPd3c4SzF5eER3dTd5?=
 =?utf-8?B?Y3pFeUx1bWRNbDFLRGkzQ1ZvYTFYMkJ4eldmc2lueEMwMExGOHNqanFMSVdX?=
 =?utf-8?B?KzZZZWVLZktsSVRNdGordEUxUjZMcEtDVFVYMThuaE12NUlzbi9xQ1hhUkFo?=
 =?utf-8?B?aWlLaGxNK2JNYXZONlZ5T0Y4VHdIdjRCWEtWMENQWnRaM1dxbDRzR0lDRGNL?=
 =?utf-8?B?dGc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: e5e9b834-4989-4f5d-3174-08da85f7a12a
X-MS-Exchange-CrossTenant-AuthSource: CY4PR11MB1862.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Aug 2022 17:39:43.6159
 (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: csyjIX37YuTo/aqat43OQm2YN5j6jndXMPK4DfjBA/4ymdh94760obAuEBO5Ijf07vhDJfQN2QBTOtW9tPt6pKTIbsx+TXCLXDkwXZwY4ko=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR11MB4903
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_MED,RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,
        SPF_NONE,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Babu,

On 8/22/2022 6:43 AM, Babu Moger wrote:
> Adds a new resource type RDT_RESOURCE_SMBA to handle the QoS
> enforcement policies on the external slow memory.
> 
> Signed-off-by: Babu Moger <babu.moger@amd.com>
> Reviewed-by: Ingo Molnar <mingo@kernel.org>
> ---
>  arch/x86/kernel/cpu/resctrl/core.c     |   12 ++++++++++++
>  arch/x86/kernel/cpu/resctrl/internal.h |    1 +
>  2 files changed, 13 insertions(+)
> 
> diff --git a/arch/x86/kernel/cpu/resctrl/core.c b/arch/x86/kernel/cpu/resctrl/core.c
> index a5c51a14fbce..6c38427b71a2 100644
> --- a/arch/x86/kernel/cpu/resctrl/core.c
> +++ b/arch/x86/kernel/cpu/resctrl/core.c
> @@ -100,6 +100,18 @@ struct rdt_hw_resource rdt_resources_all[] = {
>  			.fflags			= RFTYPE_RES_MB,
>  		},
>  	},
> +	[RDT_RESOURCE_SMBA] =
> +	{
> +		.r_resctrl = {
> +			.rid			= RDT_RESOURCE_SMBA,
> +			.name			= "SB",

This name seems very cryptic to me. I do not know how you picked this
name but thought it worth mentioning that names need not be limited to
two characters. There is already support to ensure fields are lined up
(see "max_name_width") in support of the longer "L2CODE", "L2DATA",
"L3CODE", and "L3DATA" resource names.

Reinette