Re: [RFC PATCH v2 2/4] blk-cgroup: use a request_queue rhashtable for blkg lookup

On Fri, Aug 14, 2026 at 03:50:04PM +0800, Yu Kuai wrote:

quoted

Are these fast path operations?  Otherwise we can walk all rhashtable
entries without an extra list, but it might be slower.

All users are from sysfs/cgroupfs API, I think they can be considered slow path,
however currently spinlock is held in these procedures, I think it's better to
convert them to blkg_lookup based iterate after spinlock is converted to the
blkcg_mutex.

Sounds good.  Maybe put that into the commit log?

quoted

Please add must_hold and/or lockdep annotations for this instead of just
a comment.

Also maybe mention that this does not acquire a reference and the caller
must already hold one?

Perhaps it's more accurate that the blkg is pinned by IO or caller already
hold one?

Sounds good.