linux-kernel - Re: [PATCH v2 1/3] fs/resctrl: Add helpers to check io

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <hwisd6ld3wpajp24ruubd7nyleyfiq2s35xwztamatx4g2bryr@yhrmurfcyxts>
Date: Thu, 18 Dec 2025 18:22:49 -0500
From: Aaron Tomlin <atomlin@...mlin.com>
To: Reinette Chatre <reinette.chatre@...el.com>
Cc: tony.luck@...el.com, Dave.Martin@....com, james.morse@....com, 
	babu.moger@....com, tglx@...utronix.de, mingo@...hat.com, bp@...en8.de, 
	dave.hansen@...ux.intel.com, sean@...e.io, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 1/3] fs/resctrl: Add helpers to check io_alloc support
 and enabled state

On Tue, Dec 16, 2025 at 09:01:32PM -0800, Reinette Chatre wrote:
> How does this patch benefit the goal of this submission, which is to set
> identical CBM on all domains?
> 
> If this change benefited this submission I should find these new helpers
> used in later patches but they are not. This seems to be an unrelated and
> unnecessary change mixed with this submission. It does not fix a bug nor
> does it support this submission and thus just adds noise when considering
> the new feature for inclusion. This patch can be dropped. 

Hi Reinette,

Thank you for your feedback regarding the relevance of this patch to the
overall objective.

To be clear, this change was included as part of the broader series. Whilst
I grant that this patch is not a strong requirement, I deem it a useful
prerequisite clean-up. In my view, streamlining the existing infrastructure
before layering on new features is a sound practice that prevents the
accumulation of technical debt.

Given that this refactoring provides a cleaner foundation, I would propose
that it remain in the series rather than being excluded. I believe the
long-term benefit to the maintainability of the resctrl code outweighs the
concern of it being "noise" in the context of this specific feature.

Kind regards,
-- 
Aaron Tomlin

Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)