dax: return the partition offset from fs_dax_get_by_bdev
Prepare for the removal of the block_device from the DAX I/O path by returning the partition offset from fs_dax_get_by_bdev so that the file systems have it at hand for use during I/O. Signed-off-by: Christoph Hellwig <hch@lst.de> Reviewed-by: Dan Williams <dan.j.williams@intel.com> Reviewed-by: Darrick J. Wong <djwong@kernel.org> Link: https://lore.kernel.org/r/20211129102203.2243509-26-hch@lst.de Signed-off-by: Dan Williams <dan.j.williams@intel.com>
This commit is contained in:
committed by
Dan Williams
parent
952da06375
commit
cd913c76f4
@@ -69,17 +69,20 @@ EXPORT_SYMBOL_GPL(dax_remove_host);
|
|||||||
/**
|
/**
|
||||||
* fs_dax_get_by_bdev() - temporary lookup mechanism for filesystem-dax
|
* fs_dax_get_by_bdev() - temporary lookup mechanism for filesystem-dax
|
||||||
* @bdev: block device to find a dax_device for
|
* @bdev: block device to find a dax_device for
|
||||||
|
* @start_off: returns the byte offset into the dax_device that @bdev starts
|
||||||
*/
|
*/
|
||||||
struct dax_device *fs_dax_get_by_bdev(struct block_device *bdev)
|
struct dax_device *fs_dax_get_by_bdev(struct block_device *bdev, u64 *start_off)
|
||||||
{
|
{
|
||||||
struct dax_device *dax_dev;
|
struct dax_device *dax_dev;
|
||||||
|
u64 part_size;
|
||||||
int id;
|
int id;
|
||||||
|
|
||||||
if (!blk_queue_dax(bdev->bd_disk->queue))
|
if (!blk_queue_dax(bdev->bd_disk->queue))
|
||||||
return NULL;
|
return NULL;
|
||||||
|
|
||||||
if ((get_start_sect(bdev) * SECTOR_SIZE) % PAGE_SIZE ||
|
*start_off = get_start_sect(bdev) * SECTOR_SIZE;
|
||||||
(bdev_nr_sectors(bdev) * SECTOR_SIZE) % PAGE_SIZE) {
|
part_size = bdev_nr_sectors(bdev) * SECTOR_SIZE;
|
||||||
|
if (*start_off % PAGE_SIZE || part_size % PAGE_SIZE) {
|
||||||
pr_info("%pg: error: unaligned partition for dax\n", bdev);
|
pr_info("%pg: error: unaligned partition for dax\n", bdev);
|
||||||
return NULL;
|
return NULL;
|
||||||
}
|
}
|
||||||
|
@@ -637,7 +637,7 @@ static int open_table_device(struct table_device *td, dev_t dev,
|
|||||||
struct mapped_device *md)
|
struct mapped_device *md)
|
||||||
{
|
{
|
||||||
struct block_device *bdev;
|
struct block_device *bdev;
|
||||||
|
u64 part_off;
|
||||||
int r;
|
int r;
|
||||||
|
|
||||||
BUG_ON(td->dm_dev.bdev);
|
BUG_ON(td->dm_dev.bdev);
|
||||||
@@ -653,7 +653,7 @@ static int open_table_device(struct table_device *td, dev_t dev,
|
|||||||
}
|
}
|
||||||
|
|
||||||
td->dm_dev.bdev = bdev;
|
td->dm_dev.bdev = bdev;
|
||||||
td->dm_dev.dax_dev = fs_dax_get_by_bdev(bdev);
|
td->dm_dev.dax_dev = fs_dax_get_by_bdev(bdev, &part_off);
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@@ -51,6 +51,7 @@ struct erofs_device_info {
|
|||||||
char *path;
|
char *path;
|
||||||
struct block_device *bdev;
|
struct block_device *bdev;
|
||||||
struct dax_device *dax_dev;
|
struct dax_device *dax_dev;
|
||||||
|
u64 dax_part_off;
|
||||||
|
|
||||||
u32 blocks;
|
u32 blocks;
|
||||||
u32 mapped_blkaddr;
|
u32 mapped_blkaddr;
|
||||||
@@ -109,6 +110,7 @@ struct erofs_sb_info {
|
|||||||
#endif /* CONFIG_EROFS_FS_ZIP */
|
#endif /* CONFIG_EROFS_FS_ZIP */
|
||||||
struct erofs_dev_context *devs;
|
struct erofs_dev_context *devs;
|
||||||
struct dax_device *dax_dev;
|
struct dax_device *dax_dev;
|
||||||
|
u64 dax_part_off;
|
||||||
u64 total_blocks;
|
u64 total_blocks;
|
||||||
u32 primarydevice_blocks;
|
u32 primarydevice_blocks;
|
||||||
|
|
||||||
|
@@ -312,7 +312,7 @@ static int erofs_init_devices(struct super_block *sb,
|
|||||||
goto err_out;
|
goto err_out;
|
||||||
}
|
}
|
||||||
dif->bdev = bdev;
|
dif->bdev = bdev;
|
||||||
dif->dax_dev = fs_dax_get_by_bdev(bdev);
|
dif->dax_dev = fs_dax_get_by_bdev(bdev, &dif->dax_part_off);
|
||||||
dif->blocks = le32_to_cpu(dis->blocks);
|
dif->blocks = le32_to_cpu(dis->blocks);
|
||||||
dif->mapped_blkaddr = le32_to_cpu(dis->mapped_blkaddr);
|
dif->mapped_blkaddr = le32_to_cpu(dis->mapped_blkaddr);
|
||||||
sbi->total_blocks += dif->blocks;
|
sbi->total_blocks += dif->blocks;
|
||||||
@@ -644,7 +644,7 @@ static int erofs_fc_fill_super(struct super_block *sb, struct fs_context *fc)
|
|||||||
|
|
||||||
sb->s_fs_info = sbi;
|
sb->s_fs_info = sbi;
|
||||||
sbi->opt = ctx->opt;
|
sbi->opt = ctx->opt;
|
||||||
sbi->dax_dev = fs_dax_get_by_bdev(sb->s_bdev);
|
sbi->dax_dev = fs_dax_get_by_bdev(sb->s_bdev, &sbi->dax_part_off);
|
||||||
sbi->devs = ctx->devs;
|
sbi->devs = ctx->devs;
|
||||||
ctx->devs = NULL;
|
ctx->devs = NULL;
|
||||||
|
|
||||||
|
@@ -118,6 +118,7 @@ struct ext2_sb_info {
|
|||||||
spinlock_t s_lock;
|
spinlock_t s_lock;
|
||||||
struct mb_cache *s_ea_block_cache;
|
struct mb_cache *s_ea_block_cache;
|
||||||
struct dax_device *s_daxdev;
|
struct dax_device *s_daxdev;
|
||||||
|
u64 s_dax_part_off;
|
||||||
};
|
};
|
||||||
|
|
||||||
static inline spinlock_t *
|
static inline spinlock_t *
|
||||||
|
@@ -831,7 +831,7 @@ static int ext2_fill_super(struct super_block *sb, void *data, int silent)
|
|||||||
}
|
}
|
||||||
sb->s_fs_info = sbi;
|
sb->s_fs_info = sbi;
|
||||||
sbi->s_sb_block = sb_block;
|
sbi->s_sb_block = sb_block;
|
||||||
sbi->s_daxdev = fs_dax_get_by_bdev(sb->s_bdev);
|
sbi->s_daxdev = fs_dax_get_by_bdev(sb->s_bdev, &sbi->s_dax_part_off);
|
||||||
|
|
||||||
spin_lock_init(&sbi->s_lock);
|
spin_lock_init(&sbi->s_lock);
|
||||||
ret = -EINVAL;
|
ret = -EINVAL;
|
||||||
|
@@ -1697,6 +1697,7 @@ struct ext4_sb_info {
|
|||||||
*/
|
*/
|
||||||
struct percpu_rw_semaphore s_writepages_rwsem;
|
struct percpu_rw_semaphore s_writepages_rwsem;
|
||||||
struct dax_device *s_daxdev;
|
struct dax_device *s_daxdev;
|
||||||
|
u64 s_dax_part_off;
|
||||||
#ifdef CONFIG_EXT4_DEBUG
|
#ifdef CONFIG_EXT4_DEBUG
|
||||||
unsigned long s_simulate_fail;
|
unsigned long s_simulate_fail;
|
||||||
#endif
|
#endif
|
||||||
|
@@ -3913,7 +3913,7 @@ static int ext4_fill_super(struct super_block *sb, void *data, int silent)
|
|||||||
if (!sbi->s_blockgroup_lock)
|
if (!sbi->s_blockgroup_lock)
|
||||||
goto out_free_base;
|
goto out_free_base;
|
||||||
|
|
||||||
sbi->s_daxdev = fs_dax_get_by_bdev(sb->s_bdev);
|
sbi->s_daxdev = fs_dax_get_by_bdev(sb->s_bdev, &sbi->s_dax_part_off);
|
||||||
sb->s_fs_info = sbi;
|
sb->s_fs_info = sbi;
|
||||||
sbi->s_sb = sb;
|
sbi->s_sb = sb;
|
||||||
sbi->s_inode_readahead_blks = EXT4_DEF_INODE_READAHEAD_BLKS;
|
sbi->s_inode_readahead_blks = EXT4_DEF_INODE_READAHEAD_BLKS;
|
||||||
|
@@ -1945,7 +1945,7 @@ xfs_alloc_buftarg(
|
|||||||
btp->bt_mount = mp;
|
btp->bt_mount = mp;
|
||||||
btp->bt_dev = bdev->bd_dev;
|
btp->bt_dev = bdev->bd_dev;
|
||||||
btp->bt_bdev = bdev;
|
btp->bt_bdev = bdev;
|
||||||
btp->bt_daxdev = fs_dax_get_by_bdev(bdev);
|
btp->bt_daxdev = fs_dax_get_by_bdev(bdev, &btp->bt_dax_part_off);
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Buffer IO error rate limiting. Limit it to no more than 10 messages
|
* Buffer IO error rate limiting. Limit it to no more than 10 messages
|
||||||
|
@@ -89,6 +89,7 @@ typedef struct xfs_buftarg {
|
|||||||
dev_t bt_dev;
|
dev_t bt_dev;
|
||||||
struct block_device *bt_bdev;
|
struct block_device *bt_bdev;
|
||||||
struct dax_device *bt_daxdev;
|
struct dax_device *bt_daxdev;
|
||||||
|
u64 bt_dax_part_off;
|
||||||
struct xfs_mount *bt_mount;
|
struct xfs_mount *bt_mount;
|
||||||
unsigned int bt_meta_sectorsize;
|
unsigned int bt_meta_sectorsize;
|
||||||
size_t bt_meta_sectormask;
|
size_t bt_meta_sectormask;
|
||||||
|
@@ -117,7 +117,8 @@ static inline void fs_put_dax(struct dax_device *dax_dev)
|
|||||||
put_dax(dax_dev);
|
put_dax(dax_dev);
|
||||||
}
|
}
|
||||||
|
|
||||||
struct dax_device *fs_dax_get_by_bdev(struct block_device *bdev);
|
struct dax_device *fs_dax_get_by_bdev(struct block_device *bdev,
|
||||||
|
u64 *start_off);
|
||||||
int dax_writeback_mapping_range(struct address_space *mapping,
|
int dax_writeback_mapping_range(struct address_space *mapping,
|
||||||
struct dax_device *dax_dev, struct writeback_control *wbc);
|
struct dax_device *dax_dev, struct writeback_control *wbc);
|
||||||
|
|
||||||
@@ -138,7 +139,8 @@ static inline void fs_put_dax(struct dax_device *dax_dev)
|
|||||||
{
|
{
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline struct dax_device *fs_dax_get_by_bdev(struct block_device *bdev)
|
static inline struct dax_device *fs_dax_get_by_bdev(struct block_device *bdev,
|
||||||
|
u64 *start_off)
|
||||||
{
|
{
|
||||||
return NULL;
|
return NULL;
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user