The Linux Kernel Mailing List
 help / color / mirror / Atom feed
From: Reinette Chatre <reinette.chatre@intel.com>
To: Borislav Petkov <bp@alien8.de>
Cc: <tony.luck@intel.com>, <james.morse@arm.com>,
	<Dave.Martin@arm.com>, <babu.moger@amd.com>, <tglx@linutronix.de>,
	<dave.hansen@linux.intel.com>, <x86@kernel.org>, <hpa@zytor.com>,
	<ben.horgan@arm.com>, <fustini@kernel.org>, <fenghuay@nvidia.com>,
	<peternewman@google.com>, <yu.c.chen@intel.com>,
	<linux-kernel@vger.kernel.org>, <patches@lists.linux.dev>
Subject: Re: [PATCH v5 00/11] x86,fs/resctrl: Fix long-standing issues
Date: Wed, 1 Jul 2026 13:56:06 -0700	[thread overview]
Message-ID: <dab1290e-24f8-443f-9ca3-9116792f320f@intel.com> (raw)
In-Reply-To: <20260701203725.GCakV6hQ9-x2BTBEnx@fat_crate.local>

Hi Boris,

On 7/1/26 1:37 PM, Borislav Petkov wrote:
> On Wed, Jul 01, 2026 at 12:40:27PM -0700, Borislav Petkov wrote:
>> On Wed, Jul 01, 2026 at 10:55:38AM -0700, Reinette Chatre wrote:
>>> I agree there are serious issues here. What prevents me from advocating for them
>>> all to go to stable is that the fixes to the intricate races required changes to
>>> resctrl's core locking flows. I absolutely tested and scrutinized these changes but
>>> having them go straight to all the stable kernels feels like a big jump to me.
>>
>> Makes sense to me.
>>
>>> The fixes that do not involve locking changes are below. They could be considered
>>> for stable.
>>> 1/11: x86,fs/resctrl: Prevent out-of-bounds access while offlining CPU when SNC enabled
>>> 4/11: fs/resctrl: Free mon_data structures on rdt_get_tree() failure
>>> 5/11: fs/resctrl: Fix use-after-free during unmount
>>> 8/11: fs/resctrl: Fix double-add of pseudo-locked region's RMID to free list
>>
>> Ok, lemme hoist those up into urgent first.
> 
> Yeah, not easy, see below.
> 
> This is because 3/11 which is moving the functions, is missing now.
> 
> We can do a forward declaration but I'd let you reshuffle things the way you
> want them and send me the urgent fixes by placing them at the beginning of the
> set so that I can stick them into urgent.

Will do. Since these patches are now targeting urgent and stable, a forward declaration
would make the backporting easier. I do prefer to keep 3/11 on top of that though. I will
prepare the next version of this series to do this. 

> 
> I'll queue 1/11 only now.

Thank you very much.

Reinette


      reply	other threads:[~2026-07-01 20:56 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-09 21:02 [PATCH v5 00/11] x86,fs/resctrl: Fix long-standing issues Reinette Chatre
2026-06-09 21:02 ` [PATCH v5 01/11] x86,fs/resctrl: Prevent out-of-bounds access while offlining CPU when SNC enabled Reinette Chatre
2026-06-09 21:02 ` [PATCH v5 02/11] x86,fs/resctrl: Document safe RCU list traversal Reinette Chatre
2026-07-01 11:45   ` Ben Horgan
2026-06-09 21:02 ` [PATCH v5 03/11] fs/resctrl: Move functions to avoid forward references in subsequent fixes Reinette Chatre
2026-06-09 21:02 ` [PATCH v5 04/11] fs/resctrl: Free mon_data structures on rdt_get_tree() failure Reinette Chatre
2026-06-09 21:02 ` [PATCH v5 05/11] fs/resctrl: Fix use-after-free during unmount Reinette Chatre
2026-06-09 21:02 ` [PATCH v5 06/11] fs/resctrl: Fix deadlock on errors during mount Reinette Chatre
2026-06-09 21:02 ` [PATCH v5 07/11] fs/resctrl: Prevent use-after-free in rdtgroup_kn_put() Reinette Chatre
2026-06-09 21:02 ` [PATCH v5 08/11] fs/resctrl: Fix double-add of pseudo-locked region's RMID to free list Reinette Chatre
2026-06-09 21:02 ` [PATCH v5 09/11] fs/resctrl: Prevent deadlock and use-after-free in info file handlers Reinette Chatre
2026-06-09 21:02 ` [PATCH v5 10/11] x86/resctrl: Ensure domain fully initialized before placed on RCU list Reinette Chatre
2026-06-09 21:02 ` [PATCH v5 11/11] fs/resctrl: Fix UAF from worker threads when domains are removed Reinette Chatre
2026-06-10 17:48 ` [PATCH v5 00/11] x86,fs/resctrl: Fix long-standing issues Reinette Chatre
2026-06-10 19:21   ` Borislav Petkov
2026-06-10 20:34     ` Reinette Chatre
2026-06-10 23:10       ` Borislav Petkov
2026-06-10 23:17         ` Reinette Chatre
2026-06-30 21:47     ` Reinette Chatre
2026-06-30 22:57       ` Borislav Petkov
2026-07-01 17:09         ` Borislav Petkov
2026-07-01 17:55           ` Reinette Chatre
2026-07-01 19:40             ` Borislav Petkov
2026-07-01 20:37               ` Borislav Petkov
2026-07-01 20:56                 ` Reinette Chatre [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=dab1290e-24f8-443f-9ca3-9116792f320f@intel.com \
    --to=reinette.chatre@intel.com \
    --cc=Dave.Martin@arm.com \
    --cc=babu.moger@amd.com \
    --cc=ben.horgan@arm.com \
    --cc=bp@alien8.de \
    --cc=dave.hansen@linux.intel.com \
    --cc=fenghuay@nvidia.com \
    --cc=fustini@kernel.org \
    --cc=hpa@zytor.com \
    --cc=james.morse@arm.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=patches@lists.linux.dev \
    --cc=peternewman@google.com \
    --cc=tglx@linutronix.de \
    --cc=tony.luck@intel.com \
    --cc=x86@kernel.org \
    --cc=yu.c.chen@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox