On Tue, Jun 25, 2024 at 07:38:34PM GMT, Li Lingfeng <lilingfeng@xxxxxxxxxxxxxxx> wrote: > Thanks for your advice. It's indeed more appropriate to use "flush" instead > of "cancel" here, I will change it soon. I saw your v2. Didn't you forget to change also the function name? > I didn't quite get what you mean. Do you mean this patch will cause a change > in mechanics, and it is necessary to add an explanation? > > (After deleting the original cgroup, > Before: the limit of the throttled bios can't be changed and the bios will > complete under this limit; > Now: the limit will be canceled and the throttled bios will be flushed > immediately.) I mean -- can the new mechanics be exploited to bypass throttling by sending IO from a process, migrate it between cgroups and rmdir them? That should be covered in the commit log. Thanks, Michal
Attachment:
signature.asc
Description: PGP signature
