block: don't autoload drivers on blk-cgroup configuration
authorChristoph Hellwig <hch@lst.de>
Wed, 23 Apr 2025 05:37:42 +0000 (07:37 +0200)
committerJens Axboe <axboe@kernel.dk>
Thu, 24 Apr 2025 13:35:23 +0000 (07:35 -0600)
Loading a driver just to configure blk-cgroup doesn't make sense, as that
assumes and already existing device.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Christian Brauner <brauner@kernel.org>
Acked-by: Tejun Heo <tj@kernel.org>
Link: https://lore.kernel.org/r/20250423053810.1683309-5-hch@lst.de
Signed-off-by: Jens Axboe <axboe@kernel.dk>
block/blk-cgroup.c

index e172aeda41837e3f4200b4b4712fd723f80a6cfb..ce93706555c5b907f15e755011cdab80e02f3b5d 100644 (file)
@@ -797,7 +797,7 @@ int blkg_conf_open_bdev(struct blkg_conf_ctx *ctx)
                return -EINVAL;
        input = skip_spaces(input);
 
-       bdev = blkdev_get_no_open(MKDEV(major, minor), true);
+       bdev = blkdev_get_no_open(MKDEV(major, minor), false);
        if (!bdev)
                return -ENODEV;
        if (bdev_is_partition(bdev)) {