linux-kernel - Re: [PATCH v7 47/49] fs/resctrl: Remove unnecessary includes

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] [day] [month] [year] [list]

Message-ID: <af1a273a-0db5-491b-9fbf-48100e29169b@arm.com>
Date: Fri, 14 Mar 2025 17:42:16 +0000
From: James Morse <james.morse@....com>
To: Fenghua Yu <fenghuay@...dia.com>, x86@...nel.org,
 linux-kernel@...r.kernel.org
Cc: Reinette Chatre <reinette.chatre@...el.com>,
 Thomas Gleixner <tglx@...utronix.de>, Ingo Molnar <mingo@...hat.com>,
 Borislav Petkov <bp@...en8.de>, H Peter Anvin <hpa@...or.com>,
 Babu Moger <Babu.Moger@....com>, shameerali.kolothum.thodi@...wei.com,
 D Scott Phillips OS <scott@...amperecomputing.com>,
 carl@...amperecomputing.com, lcherian@...vell.com,
 bobo.shaobowang@...wei.com, tan.shaopeng@...itsu.com,
 baolin.wang@...ux.alibaba.com, Jamie Iles <quic_jiles@...cinc.com>,
 Xin Hao <xhao@...ux.alibaba.com>, peternewman@...gle.com,
 dfustini@...libre.com, amitsinght@...vell.com,
 David Hildenbrand <david@...hat.com>, Rex Nie <rex.nie@...uarmicro.com>,
 Dave Martin <dave.martin@....com>, Koba Ko <kobak@...dia.com>,
 Shanker Donthineni <sdonthineni@...dia.com>
Subject: Re: [PATCH v7 47/49] fs/resctrl: Remove unnecessary includes

Hi Fenghua,

On 07/03/2025 02:37, Fenghua Yu wrote:
> On 2/28/25 11:59, James Morse wrote:
>> When splitting and moving the resctrl code to live in fs/resctrl and
>> arch/x86, some code was duplicated. This was done to keep the parser
>> in the script that does the moving simple. These extra includes are

> As for "parser in the script", my concern is people may forget or don't know the context
> of the python script in the future and get confused when reading this patch.

> Is it possible to merge this patch into patch 45 which introduces the macros? It simplify
> the patch set and avoid future confuse?

The plan would be for these patches to get merged together by the person that applies them.


Thanks,

James