On Thu 26-11-20 14:03:59, Christoph Hellwig wrote: > This will allow for a more symmetric calling convention going forward. > > Signed-off-by: Christoph Hellwig <[email protected]>
Looks good to me. You can add: Reviewed-by: Jan Kara <[email protected]> Honza > --- > fs/block_dev.c | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/fs/block_dev.c b/fs/block_dev.c > index 86a61a2141f642..d0783c55a0ce65 100644 > --- a/fs/block_dev.c > +++ b/fs/block_dev.c > @@ -1458,6 +1458,7 @@ static int __blkdev_get(struct block_device *bdev, > struct gendisk *disk, > if (!(disk->flags & GENHD_FL_UP) || > !bdev->bd_part || !bdev->bd_part->nr_sects) { > __blkdev_put(whole, mode, 1); > + bdput(whole); > ret = -ENXIO; > goto out_clear; > } > @@ -1740,9 +1741,10 @@ static void __blkdev_put(struct block_device *bdev, > fmode_t mode, int for_part) > disk->fops->release(disk, mode); > } > mutex_unlock(&bdev->bd_mutex); > - bdput(bdev); > - if (victim) > + if (victim) { > __blkdev_put(victim, mode, 1); > + bdput(victim); > + } > } > > void blkdev_put(struct block_device *bdev, fmode_t mode) > @@ -1792,6 +1794,7 @@ void blkdev_put(struct block_device *bdev, fmode_t mode) > mutex_unlock(&bdev->bd_mutex); > > __blkdev_put(bdev, mode, 0); > + bdput(bdev); > put_disk_and_module(disk); > } > EXPORT_SYMBOL(blkdev_put); > -- > 2.29.2 > -- Jan Kara <[email protected]> SUSE Labs, CR -- dm-devel mailing list [email protected] https://www.redhat.com/mailman/listinfo/dm-devel
