Re: [RFC PATCH v1 00/17] blk-cgroup: protect blkgs with blkcg_mutex
"yu kuai" <[email protected]> Thu, 9 Jul 2026 18:08:36 +0800
| Newsgroups | org.kernel.vger.linux-bcache,dev.linux.lists.dm-devel,org.infradead.lists.linux-nvme,org.kernel.vger.cgroups,org.kernel.vger.linux-block |
|---|---|
| Message-ID | <[email protected]> |
Hi, =E5=9C=A8 2026/7/9 14:09, Christoph Hellwig =E5=86=99=E9=81=93: > On Sun, Jul 05, 2026 at 03:51:07AM +0800, Yu Kuai wrote: >> From: Yu Kuai <[email protected]> >> >> This RFC moves queue-local blkg topology synchronization from >> q->queue_lock to q->blkcg_mutex. >> >> q->queue_lock is a hot block-layer spinlock used by request queue runtim= e >> paths, > I don't think it is hot any more. If it is in one of your workloads > we have a deep problem somewhere. That being said, futher removing > uses of his old catch-all lock is always good, hopefully we can > eventually remove it entirely. > > So this series looks great to me conceptually, but I'm unfortunately > not a very qualified reviewer for the blk-cgroup code. Thanks a lot for taking a look at this RFC set, it's very helpful. It's true that blk-cgroup review is not active for a long time. I'd like to help but I really don't have time to check every block layer patches. > --=20 Thanks, Kuai