Skip to content
This repository was archived by the owner on Nov 8, 2023. It is now read-only.

Commit 01ed5e6

Browse files
committed
Revert "fs: Convert to bdev_open_by_dev()"
This reverts commit 4365d0d which is commit f4a48bc upstream. It breaks the Android kernel abi and can be brought back in the future in an abi-safe way if it is really needed. Bug: 161946584 Change-Id: I36bf7e4279a05eb0028ea4da17a2cb735de04bdc Signed-off-by: Greg Kroah-Hartman <[email protected]>
1 parent a8167db commit 01ed5e6

File tree

4 files changed

+8
-12
lines changed

4 files changed

+8
-12
lines changed

fs/cramfs/inode.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -495,7 +495,7 @@ static void cramfs_kill_sb(struct super_block *sb)
495495
sb->s_mtd = NULL;
496496
} else if (IS_ENABLED(CONFIG_CRAMFS_BLOCKDEV) && sb->s_bdev) {
497497
sync_blockdev(sb->s_bdev);
498-
bdev_release(sb->s_bdev_handle);
498+
blkdev_put(sb->s_bdev, sb);
499499
}
500500
kfree(sbi);
501501
}

fs/romfs/super.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -593,7 +593,7 @@ static void romfs_kill_sb(struct super_block *sb)
593593
#ifdef CONFIG_ROMFS_ON_BLOCK
594594
if (sb->s_bdev) {
595595
sync_blockdev(sb->s_bdev);
596-
bdev_release(sb->s_bdev_handle);
596+
blkdev_put(sb->s_bdev, sb);
597597
}
598598
#endif
599599
}

fs/super.c

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1493,24 +1493,22 @@ int setup_bdev_super(struct super_block *sb, int sb_flags,
14931493
struct fs_context *fc)
14941494
{
14951495
blk_mode_t mode = sb_open_mode(sb_flags);
1496-
struct bdev_handle *bdev_handle;
14971496
struct block_device *bdev;
14981497

1499-
bdev_handle = bdev_open_by_dev(sb->s_dev, mode, sb, &fs_holder_ops);
1500-
if (IS_ERR(bdev_handle)) {
1498+
bdev = blkdev_get_by_dev(sb->s_dev, mode, sb, &fs_holder_ops);
1499+
if (IS_ERR(bdev)) {
15011500
if (fc)
15021501
errorf(fc, "%s: Can't open blockdev", fc->source);
1503-
return PTR_ERR(bdev_handle);
1502+
return PTR_ERR(bdev);
15041503
}
1505-
bdev = bdev_handle->bdev;
15061504

15071505
/*
15081506
* This really should be in blkdev_get_by_dev, but right now can't due
15091507
* to legacy issues that require us to allow opening a block device node
15101508
* writable from userspace even for a read-only block device.
15111509
*/
15121510
if ((mode & BLK_OPEN_WRITE) && bdev_read_only(bdev)) {
1513-
bdev_release(bdev_handle);
1511+
blkdev_put(bdev, sb);
15141512
return -EACCES;
15151513
}
15161514

@@ -1526,11 +1524,10 @@ int setup_bdev_super(struct super_block *sb, int sb_flags,
15261524
mutex_unlock(&bdev->bd_fsfreeze_mutex);
15271525
if (fc)
15281526
warnf(fc, "%pg: Can't mount, blockdev is frozen", bdev);
1529-
bdev_release(bdev_handle);
1527+
blkdev_put(bdev, sb);
15301528
return -EBUSY;
15311529
}
15321530
spin_lock(&sb_lock);
1533-
sb->s_bdev_handle = bdev_handle;
15341531
sb->s_bdev = bdev;
15351532
sb->s_bdi = bdi_get(bdev->bd_disk->bdi);
15361533
if (bdev_stable_writes(bdev))
@@ -1663,7 +1660,7 @@ void kill_block_super(struct super_block *sb)
16631660
generic_shutdown_super(sb);
16641661
if (bdev) {
16651662
sync_blockdev(bdev);
1666-
bdev_release(sb->s_bdev_handle);
1663+
blkdev_put(bdev, sb);
16671664
}
16681665
}
16691666

include/linux/fs.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1214,7 +1214,6 @@ struct super_block {
12141214
struct hlist_bl_head s_roots; /* alternate root dentries for NFS */
12151215
struct list_head s_mounts; /* list of mounts; _not_ for fs use */
12161216
struct block_device *s_bdev;
1217-
struct bdev_handle *s_bdev_handle;
12181217
struct backing_dev_info *s_bdi;
12191218
struct mtd_info *s_mtd;
12201219
struct hlist_node s_instances;

0 commit comments

Comments
 (0)