linux-kernel - Re: [PATCH v1] fs/resctrl: Add missing kconfig entry for CONFIG_RESCTRL_ASSIGN

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <de29108c-f0f8-45c5-8f33-52b3da79d75a@intel.com>
Date: Thu, 15 Jan 2026 07:37:58 -0800
From: Reinette Chatre <reinette.chatre@...el.com>
To: Ben Horgan <ben.horgan@....com>, <linux-kernel@...r.kernel.org>
CC: <tony.luck@...el.com>, <Dave.Martin@....com>, <james.morse@....com>,
	<babu.moger@....com>, <bp@...en8.de>
Subject: Re: [PATCH v1] fs/resctrl: Add missing kconfig entry for
 CONFIG_RESCTRL_ASSIGN_FIXED

Hi Ben,

On 1/15/26 2:32 AM, Ben Horgan wrote:
> On 1/14/26 19:09, Reinette Chatre wrote:
>> On 1/13/26 6:49 AM, Ben Horgan wrote:
>>> The commit 3b497c3f4f04 ("fs/resctrl: Introduce the interface to display
>>> monitoring modes") introduced CONFIG_RESCTRL_ASSIGN_FIXED but did not add
>>> the kconfig entry. Add this.
>>>
>>> Also, take the opportunity ensure that any user attempt to change the
>>> assign mode fails from the resctrl code rather than delegating to the arch
>>> specific code and let the user know by adding a message in last_cmd_status.
>>
>> Why *any* attempt? Avoiding delegating to arch seems ok as a goal but this change
>> additionally changes interface with user space. Current behavior when user writes
>> existing mode to the file is to just return success. For example, if current mode
>> is "default" and user writes "default". This patch changes this behavior to fail
>> instead. Is this intended?
> 
> Intended but as you point out, it would be best to accept the current
> value without error for maximum compatibility between architectures.
> I'll update the config option help text to reflect this. Was the help
> text also discussed previously?

It is not clear to me which help text you are referring to. Could you
please point me to it?

Reinette