block: move make_it_fail to struct block_device
authorChristoph Hellwig <hch@lst.de>
Mon, 23 Nov 2020 15:28:47 +0000 (16:28 +0100)
committerJens Axboe <axboe@kernel.dk>
Tue, 1 Dec 2020 21:53:40 +0000 (14:53 -0700)
Move the make_it_fail flag to struct block_device an turn it into a bool
in preparation of killing struct hd_struct.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Jan Kara <jack@suse.cz>
Reviewed-by: Hannes Reinecke <hare@suse.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
block/blk-core.c
block/genhd.c
include/linux/blk_types.h
include/linux/genhd.h

index 9a3793d..9121390 100644 (file)
@@ -668,7 +668,8 @@ __setup("fail_make_request=", setup_fail_make_request);
 
 static bool should_fail_request(struct hd_struct *part, unsigned int bytes)
 {
-       return part->make_it_fail && should_fail(&fail_make_request, bytes);
+       return part->bdev->bd_make_it_fail &&
+               should_fail(&fail_make_request, bytes);
 }
 
 static int __init fail_make_request_debugfs(void)
index 0bd7026..f9c9577 100644 (file)
@@ -1292,7 +1292,7 @@ ssize_t part_fail_show(struct device *dev,
 {
        struct hd_struct *p = dev_to_part(dev);
 
-       return sprintf(buf, "%d\n", p->make_it_fail);
+       return sprintf(buf, "%d\n", p->bdev->bd_make_it_fail);
 }
 
 ssize_t part_fail_store(struct device *dev,
@@ -1303,7 +1303,7 @@ ssize_t part_fail_store(struct device *dev,
        int i;
 
        if (count > 0 && sscanf(buf, "%d", &i) > 0)
-               p->make_it_fail = (i == 0) ? 0 : 1;
+               p->bdev->bd_make_it_fail = i;
 
        return count;
 }
index c0591e5..b237f1e 100644 (file)
@@ -52,6 +52,9 @@ struct block_device {
        struct super_block      *bd_fsfreeze_sb;
 
        struct partition_meta_info *bd_meta_info;
+#ifdef CONFIG_FAIL_MAKE_REQUEST
+       bool                    bd_make_it_fail;
+#endif
 } __randomize_layout;
 
 #define bdev_whole(_bdev) \
index b4a5c05..349cf64 100644 (file)
@@ -56,9 +56,6 @@ struct hd_struct {
        struct block_device *bdev;
        struct device __dev;
        int policy, partno;
-#ifdef CONFIG_FAIL_MAKE_REQUEST
-       int make_it_fail;
-#endif
        struct rcu_work rcu_work;
 };