block: remove the update_bdev parameter to set_capacity_revalidate_and_notify
authorChristoph Hellwig <hch@lst.de>
Mon, 16 Nov 2020 14:56:56 +0000 (15:56 +0100)
committerJens Axboe <axboe@kernel.dk>
Mon, 16 Nov 2020 15:34:14 +0000 (08:34 -0700)
The update_bdev argument is always set to true, so remove it.  Also
rename the function to the slighly less verbose set_capacity_and_notify,
as propagating the disk size to the block device isn't really
revalidation.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Hannes Reinecke <hare@suse.de>
Reviewed-by: Petr Vorel <pvorel@suse.cz>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
block/genhd.c
drivers/block/loop.c
drivers/block/virtio_blk.c
drivers/block/xen-blkfront.c
drivers/nvme/host/core.c
drivers/scsi/sd.c
include/linux/genhd.h

index ec2a24799cd9e796e8f8a44ba79f522034caa959..4e039524f92b8f5f6113a73ad5ae9df48acdc719 100644 (file)
@@ -47,17 +47,15 @@ static void disk_del_events(struct gendisk *disk);
 static void disk_release_events(struct gendisk *disk);
 
 /*
- * Set disk capacity and notify if the size is not currently
- * zero and will not be set to zero
+ * Set disk capacity and notify if the size is not currently zero and will not
+ * be set to zero.  Returns true if a uevent was sent, otherwise false.
  */
-bool set_capacity_revalidate_and_notify(struct gendisk *disk, sector_t size,
-                                       bool update_bdev)
+bool set_capacity_and_notify(struct gendisk *disk, sector_t size)
 {
        sector_t capacity = get_capacity(disk);
 
        set_capacity(disk, size);
-       if (update_bdev)
-               revalidate_disk_size(disk, true);
+       revalidate_disk_size(disk, true);
 
        if (capacity != size && capacity != 0 && size != 0) {
                char *envp[] = { "RESIZE=1", NULL };
@@ -68,8 +66,7 @@ bool set_capacity_revalidate_and_notify(struct gendisk *disk, sector_t size,
 
        return false;
 }
-
-EXPORT_SYMBOL_GPL(set_capacity_revalidate_and_notify);
+EXPORT_SYMBOL_GPL(set_capacity_and_notify);
 
 /*
  * Format the device name of the indicated disk into the supplied buffer and
index fcc5e32f099304f75b546c9b4cf438a22d623093..9a27d4f1c08aacf059892d3f81f4f6ac67961017 100644 (file)
@@ -251,7 +251,7 @@ loop_validate_block_size(unsigned short bsize)
  */
 static void loop_set_size(struct loop_device *lo, loff_t size)
 {
-       if (!set_capacity_revalidate_and_notify(lo->lo_disk, size, true))
+       if (!set_capacity_and_notify(lo->lo_disk, size))
                kobject_uevent(&disk_to_dev(lo->lo_disk)->kobj, KOBJ_CHANGE);
 }
 
index a314b9382442b6411d5bd78b52103573287ee0d3..3e812b4c32e669e6afbccac2cfc4649e7a52fc9c 100644 (file)
@@ -470,7 +470,7 @@ static void virtblk_update_capacity(struct virtio_blk *vblk, bool resize)
                   cap_str_10,
                   cap_str_2);
 
-       set_capacity_revalidate_and_notify(vblk->disk, capacity, true);
+       set_capacity_and_notify(vblk->disk, capacity);
 }
 
 static void virtblk_config_changed_work(struct work_struct *work)
index 48629d3433b4c397fe57d86e14018ecfaf82e54b..79521e33d30ed5637b1ca983143ca7388cf28811 100644 (file)
@@ -2370,7 +2370,7 @@ static void blkfront_connect(struct blkfront_info *info)
                        return;
                printk(KERN_INFO "Setting capacity to %Lu\n",
                       sectors);
-               set_capacity_revalidate_and_notify(info->gd, sectors, true);
+               set_capacity_and_notify(info->gd, sectors);
 
                return;
        case BLKIF_STATE_SUSPENDED:
index f6c6479da0e9ec3932b4d7cacf9ce9c2dac57323..6c144e748f8caebe7d0f6b1b3305518a6901eced 100644 (file)
@@ -2053,7 +2053,7 @@ static void nvme_update_disk_info(struct gendisk *disk,
                        capacity = 0;
        }
 
-       set_capacity_revalidate_and_notify(disk, capacity, true);
+       set_capacity_and_notify(disk, capacity);
 
        nvme_config_discard(disk, ns);
        nvme_config_write_zeroes(disk, ns);
index e9b898a1a48061048d85eb27d8c3f89eae2782f6..679c2c02504763ba31a21c425d58bdfca02baba8 100644 (file)
@@ -3261,8 +3261,7 @@ static int sd_revalidate_disk(struct gendisk *disk)
 
        sdkp->first_scan = 0;
 
-       set_capacity_revalidate_and_notify(disk,
-               logical_to_sectors(sdp, sdkp->capacity), true);
+       set_capacity_and_notify(disk, logical_to_sectors(sdp, sdkp->capacity));
        sd_config_write_same(sdkp);
        kfree(buffer);
 
@@ -3272,7 +3271,7 @@ static int sd_revalidate_disk(struct gendisk *disk)
         * capacity to 0.
         */
        if (sd_zbc_revalidate_zones(sdkp))
-               set_capacity_revalidate_and_notify(disk, 0, true);
+               set_capacity_and_notify(disk, 0);
 
  out:
        return 0;
index 3cbc2781ef347657e00abb486a293457ccab1e69..46553d6d602563d41d355830291e18ca721ca1a5 100644 (file)
@@ -314,8 +314,7 @@ static inline int get_disk_ro(struct gendisk *disk)
 extern void disk_block_events(struct gendisk *disk);
 extern void disk_unblock_events(struct gendisk *disk);
 extern void disk_flush_events(struct gendisk *disk, unsigned int mask);
-bool set_capacity_revalidate_and_notify(struct gendisk *disk, sector_t size,
-               bool update_bdev);
+bool set_capacity_and_notify(struct gendisk *disk, sector_t size);
 
 /* drivers/char/random.c */
 extern void add_disk_randomness(struct gendisk *disk) __latent_entropy;