Re: [PATCH v3 1/3] x86/resctrl: Factor rdtgroup lock for multi-file ops

From: Reinette Chatre
Date: Fri Feb 10 2023 - 18:21:55 EST


Hi Peter,

On 1/25/2023 2:13 AM, Peter Newman wrote:
> rdtgroup_kn_lock_live() can only release a kernfs lock for a single file

how about s/kernfs lock/kernfs reference/?

> before waiting on the rdtgroup_mutex, limiting its usefulness for
> operations on multiple files, such as rename.
>
> Factor the work needed to respectively break and unbreak active
> protection on an individual file into rdtgroup_kn_{get,put}().
>
> This should not result in any functional change.
>

"should" has been a trigger word on occasion. You can just
write "No functional change." (also in second patch).

Reinette