Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:7 as permitted sender) client-ip=2620:137:e000::3:7;
Message-ID: <b768e6bf-04c8-ad90-bfb1-39d78073be2a@intel.com>
Date:   Mon, 11 Sep 2023 14:53:33 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Firefox/102.0 Thunderbird/102.15.0
Subject: Re: [PATCH v9 8/9] x86/resctrl: Display CLOSID for resource group
Content-Language: en-US
To:     Fenghua Yu <fenghua.yu@intel.com>, Babu Moger <babu.moger@amd.com>,
        <corbet@lwn.net>, <tglx@linutronix.de>, <mingo@redhat.com>,
        <bp@alien8.de>
CC:     <dave.hansen@linux.intel.com>, <x86@kernel.org>, <hpa@zytor.com>,
        <paulmck@kernel.org>, <akpm@linux-foundation.org>,
        <quic_neeraju@quicinc.com>, <rdunlap@infradead.org>,
        <damien.lemoal@opensource.wdc.com>, <songmuchun@bytedance.com>,
        <peterz@infradead.org>, <jpoimboe@kernel.org>,
        <pbonzini@redhat.com>, <chang.seok.bae@intel.com>,
        <pawan.kumar.gupta@linux.intel.com>, <jmattson@google.com>,
        <daniel.sneddon@linux.intel.com>, <sandipan.das@amd.com>,
        <tony.luck@intel.com>, <james.morse@arm.com>,
        <linux-doc@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        <bagasdotme@gmail.com>, <eranian@google.com>,
        <christophe.leroy@csgroup.eu>, <jarkko@kernel.org>,
        <adrian.hunter@intel.com>, <quic_jiles@quicinc.com>,
        <peternewman@google.com>
References: <20230907235128.19120-1-babu.moger@amd.com>
 <20230907235128.19120-9-babu.moger@amd.com>
 <d0bd1c6f-f1df-5355-5fc8-4fbd3be7d1e2@intel.com>
From:   Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <d0bd1c6f-f1df-5355-5fc8-4fbd3be7d1e2@intel.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?SGVFSzhIUkNpbzI3ZWliQ2pJaElzbXhlemFNcjVlanVCQUUzQ1h4RHpmeUYv?=
 =?utf-8?B?Z2RMTld4VTVMMlA1MEZIMW1XOHNKUHEwNDB5aGd3TStoR2xVdi9kaVphd3lU?=
 =?utf-8?B?M3EwMGMreHMvUGs2N0VlZVozNlVrT1F5SU5qM0paRmFmSmhTTWFqTzltek41?=
 =?utf-8?B?bnU1UGtkelhyNEU5STdNYlJwUVZiNkJnZTlpa2cxbmZQZDFOcmQ0MWZndk0x?=
 =?utf-8?B?M1A2dGlOK1ZlNHlTSjZCNUdVdDNyWFFHbDFLbFRqRHFwZGYwS1diRDNpNFJp?=
 =?utf-8?B?MDZ6TFNDU0ozd2R0NDlXY3d4SURIakRZS3JFbVJFQVgyK0doOHIwSHQrUzln?=
 =?utf-8?B?MWJVVzkrZDA4YytIWlR4anZzcmhXM1NhWGZDbUJ5cFYxTFlxZUc2eTFrZ0Jm?=
 =?utf-8?B?QXBiaWJDUXJnS1dvV3JlWjF5YnZZcDRUWjZZR0dhYXlYL0dqTXFxdmpvZ1Ay?=
 =?utf-8?B?Q2RVZUVtREhnWW1STUkramZMVW5VN2YzSGhpUUNBUDRwWGJ5UVpSRTZjbGdO?=
 =?utf-8?B?bVFGa1M0RWNjd1VMVVNZblRucm9DYnU5WmFySzdaM0JTWkswVWVONlRCcEFI?=
 =?utf-8?B?YzB5SEZ2WEo0TGFQTXZJOW5YVzJHWEFIZENrTGJ2Z2phaWtxdWs2TVFGM1cr?=
 =?utf-8?B?L005UVRwcUtJSGFKbkNyNnduYmdXclRuQVRvVGlzbXY4RHQ3Q3dhbFRLVkJk?=
 =?utf-8?B?VGIza1RxR0VFWExvTC9WTkdCbHdQVm9CTDRocTFHa3V4OFc5VWN0M3dxWEpp?=
 =?utf-8?B?T2NITHNtak5HUm1ZcVRMZ2p6NGVkengycVVXV3ltS3laaC9ueDZNNFFwTkZU?=
 =?utf-8?B?UUppNVhRblM3c29qbDgxNGtPRHZsZTRyWlVmMkw5YmJjbE1mK2huZmQ4TThy?=
 =?utf-8?B?cGJnTXgyZjNzcVptM25wdU9DQWZKVzRMeGt6SjdHdnBzY3ZsMWV4d0h0bVVX?=
 =?utf-8?B?Si84dlRTejhLMVhROHhZa3k0ckJ0aE1hSWMvNUg5bERZSzBGamYzWWd5N2Ur?=
 =?utf-8?B?c0QzNVJQQUx5MGFlWEJodCsrcjU2cXA5Q1BaSC9XOHNKUHBnWDVkVzBUcjN6?=
 =?utf-8?B?M1RDckVDUVdUMW9DR0Z5VkNtdCtqN1Vxa0wvTXVMMDJJbXZFTnFmT2t3ZStR?=
 =?utf-8?B?QlRuQXVQYzFrbFdUS0xRQ0hMUllZWWt5Ym5ZQ3NtU25ITFZ0aUdYZnVXb1pz?=
 =?utf-8?B?Q2k3ZnFFeXhIMWt5b0ttTGNib0d3SHI4RzZ0Y0U4TWhqdHgrSFBhdVVVTGND?=
 =?utf-8?B?UVhmajMvZ0l0ZDF1YnA3amxsTE5UZk9XVVdhOUtIeE0xRjlXL3NXZllDd2tp?=
 =?utf-8?B?VUttdTBtUWF2QnR2RFpOSm5BNmJrV1UvL2F3MkNMbE92VGQ4SHhma3M2VXJq?=
 =?utf-8?B?UDlJUW1KZE5QTE5nWXZiSXY3ZzhQUXVydlgzci9SSUd1cnF5MktwaHg1RWlm?=
 =?utf-8?B?eUdkbzgxSC9iMHFGWS91aGt3bUJUblBLSW1tdTgvclVrdTFVc05YRFE5T0Rh?=
 =?utf-8?B?WmtaR2MyR1V1MEZBTDZBdnpxKzhQRWtFR2JDd0xPN0pHU3VlUDQxK1pITjFq?=
 =?utf-8?B?VVorV0UxZFAyV045YituVjVYVDJMb3c5Y0IzZkd3OVVTUkx1c1IzaHFDeXgv?=
 =?utf-8?B?dE5sZTY5OVFuOEU1bHd5VGhwZ0NNbGZKWkNaVS8yNnBjMGZwU3lacTFVU0dK?=
 =?utf-8?B?QWtPd1dxSFBzeEowaEFLTW41RnkyYysrUEJLdzE3MldDNFZoTmNjSGFsS2lp?=
 =?utf-8?B?a2g2Z3hlSk9Hdlo1UXdUSk1waEh0VFJoUHEzd2prMUZtZkxGdW03T0NKZlNy?=
 =?utf-8?B?eXkzSUtqQlFkRjBkTlN6WVdNODVLcHlMaDN0Nk5CNWZ3cjh1SHM0Z2NyZU9x?=
 =?utf-8?B?NTV5Ni9YOUY3eFVvR3ZvdzRYcXozNXBZZzJ4NnNqdTlUN1ZCZmwrL0RKUEky?=
 =?utf-8?B?WXhoWVBVazVKd1hzZ3IyQkdybnNXMHk4MHI2NDRjSzVlcTN6TFlOM3ZGYVRB?=
 =?utf-8?B?UzB3OXY4V1ZsYmcvV1JFeHVSSW1YN2xDZGdqRVlQRlZneUZCbm1vM3h0RlVp?=
 =?utf-8?B?c24vRFdZZ1BCTFFFUGpFVjRQMnVYNCtQbTZWZ1cwN0dqWW40VnFvMG05ZmlS?=
 =?utf-8?B?VmZ1cHM0N0lIOVVuSjl3eklVZW53L21QZlEybHAzNm1kV2JWMUdETm1YYndl?=
 =?utf-8?B?T3c9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: a75e82f5-e2d9-47d2-1030-08dbb3118d82
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Sep 2023 21:53:37.4481
 (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: gDn/ndJ9pNKVwBtwvCXRgkG2+EWuyZoQavpAXHehvSIMJod+lDNBgiNx0YWThtvJE1btVJEPXMPFFeUoeQjEuidhHD+dWvwwn9QEX3sjQBo=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR11MB7350
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-3.6 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_BLOCKED,SPF_HELO_NONE,SPF_NONE 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
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (snail.vger.email [0.0.0.0]); Mon, 11 Sep 2023 21:19:22 -0700 (PDT)

Hi Fenghua,

On 9/11/2023 11:08 AM, Fenghua Yu wrote:
> On 9/7/23 16:51, Babu Moger wrote:

...
>> @@ -1863,6 +1879,13 @@ static struct rftype res_common_files[] = {
>>           .seq_show    = rdtgroup_size_show,
>>           .fflags        = RFTYPE_CTRL_BASE,
>>       },
>> +    {
>> +        .name        = "ctrl_hw_id",
>> +        .mode        = 0444,
>> +        .kf_ops        = &rdtgroup_kf_single_ops,
>> +        .seq_show    = rdtgroup_closid_show,
> 
> Is it better to rename "rdtgroup_closid_show" as arch neutral name
> "rdtgroup_ctrl_hw_id_show"? So the name is arch neutral and reflects
> the ctrl_hw_id. So this can eventually go to generic fs code without
> renaming it. The getting closid implementation in the function will
> be arch specific.

This is not so obvious to me. We have to draw the line somewhere. This
series draws the line between the kernel and user interface. That is,
from user perspective it is "ctrl_hw_id" and from kernel perspective
it is "closid".

I think renaming rdtgroup_closid_show() to rdtgroup_ctrl_hw_id_show() may
be mixing up the two because the function called rdtgroup_ctrl_hw_id_show() 
would operate on rdtgroup->closid - so in a sense renaming the function
to rdtgroup_ctrl_hw_id_show() would move the name further from what the
function does. Would that mean that we need to refactor the whole kernel
the naming to match between kernel space and user space by changing
rdtgroup->closid to rdtgroup->ctrl_hw_id? That escalates quite far and
wide and I wonder if it is not just simpler to keep the naming
boundary between kernel space and user space.

Reinette