Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1258397

Re: [PATCH v2 4/5] block: introduce bdev_file_inode()

From Jeff Moyer <jmoyer@redhat.com>
Newsgroups linux.kernel
Subject Re: [PATCH v2 4/5] block: introduce bdev_file_inode()
Date 2015-10-28 21:20 +0100
Message-ID <qoFmp-7nd-1@gated-at.bofh.it> (permalink)
References <qmrGW-BA-3@gated-at.bofh.it> <qmrGX-BA-11@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Dan Williams <dan.j.williams@intel.com> writes:

> Similar to the file_inode() helper, provide a helper to lookup the inode for a
> raw block device itself.
>
> Cc: Al Viro <viro@zeniv.linux.org.uk>
> Suggested-by: Jan Kara <jack@suse.cz>
> Signed-off-by: Dan Williams <dan.j.williams@intel.com>

Reviewed-by: Jeff Moyer <jmoyer@redhat.com>


> ---
>  fs/block_dev.c |   19 ++++++++++++-------
>  1 file changed, 12 insertions(+), 7 deletions(-)
>
> diff --git a/fs/block_dev.c b/fs/block_dev.c
> index 0a793c7930eb..c1f691859a56 100644
> --- a/fs/block_dev.c
> +++ b/fs/block_dev.c
> @@ -147,11 +147,16 @@ blkdev_get_block(struct inode *inode, sector_t iblock,
>  	return 0;
>  }
>  
> +static struct inode *bdev_file_inode(struct file *file)
> +{
> +	return file->f_mapping->host;
> +}
> +
>  static ssize_t
>  blkdev_direct_IO(struct kiocb *iocb, struct iov_iter *iter, loff_t offset)
>  {
>  	struct file *file = iocb->ki_filp;
> -	struct inode *inode = file->f_mapping->host;
> +	struct inode *inode = bdev_file_inode(file);
>  
>  	if (IS_DAX(inode))
>  		return dax_do_io(iocb, inode, iter, offset, blkdev_get_block,
> @@ -329,7 +334,7 @@ static int blkdev_write_end(struct file *file, struct address_space *mapping,
>   */
>  static loff_t block_llseek(struct file *file, loff_t offset, int whence)
>  {
> -	struct inode *bd_inode = file->f_mapping->host;
> +	struct inode *bd_inode = bdev_file_inode(file);
>  	loff_t retval;
>  
>  	mutex_lock(&bd_inode->i_mutex);
> @@ -340,7 +345,7 @@ static loff_t block_llseek(struct file *file, loff_t offset, int whence)
>  	
>  int blkdev_fsync(struct file *filp, loff_t start, loff_t end, int datasync)
>  {
> -	struct inode *bd_inode = filp->f_mapping->host;
> +	struct inode *bd_inode = bdev_file_inode(filp);
>  	struct block_device *bdev = I_BDEV(bd_inode);
>  	int error;
>  	
> @@ -1579,14 +1584,14 @@ EXPORT_SYMBOL(blkdev_put);
>  
>  static int blkdev_close(struct inode * inode, struct file * filp)
>  {
> -	struct block_device *bdev = I_BDEV(filp->f_mapping->host);
> +	struct block_device *bdev = I_BDEV(bdev_file_inode(filp));
>  	blkdev_put(bdev, filp->f_mode);
>  	return 0;
>  }
>  
>  static long block_ioctl(struct file *file, unsigned cmd, unsigned long arg)
>  {
> -	struct block_device *bdev = I_BDEV(file->f_mapping->host);
> +	struct block_device *bdev = I_BDEV(bdev_file_inode(file));
>  	fmode_t mode = file->f_mode;
>  
>  	/*
> @@ -1611,7 +1616,7 @@ static long block_ioctl(struct file *file, unsigned cmd, unsigned long arg)
>  ssize_t blkdev_write_iter(struct kiocb *iocb, struct iov_iter *from)
>  {
>  	struct file *file = iocb->ki_filp;
> -	struct inode *bd_inode = file->f_mapping->host;
> +	struct inode *bd_inode = bdev_file_inode(file);
>  	loff_t size = i_size_read(bd_inode);
>  	struct blk_plug plug;
>  	ssize_t ret;
> @@ -1643,7 +1648,7 @@ EXPORT_SYMBOL_GPL(blkdev_write_iter);
>  ssize_t blkdev_read_iter(struct kiocb *iocb, struct iov_iter *to)
>  {
>  	struct file *file = iocb->ki_filp;
> -	struct inode *bd_inode = file->f_mapping->host;
> +	struct inode *bd_inode = bdev_file_inode(file);
>  	loff_t size = i_size_read(bd_inode);
>  	loff_t pos = iocb->ki_pos;
>  
>
> _______________________________________________
> Linux-nvdimm mailing list
> Linux-nvdimm@lists.01.org
> https://lists.01.org/mailman/listinfo/linux-nvdimm
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH v2 0/5] block, dax: updates for 4.4 Dan Williams <dan.j.williams@intel.com> - 2015-10-22 19:20 +0200
  [PATCH v2 5/5] block: enable dax for raw block devices Dan Williams <dan.j.williams@intel.com> - 2015-10-22 19:20 +0200
    Re: [PATCH v2 5/5] block: enable dax for raw block devices Jeff Moyer <jmoyer@redhat.com> - 2015-10-28 22:00 +0100
      Re: [PATCH v2 5/5] block: enable dax for raw block devices Dan Williams <dan.j.williams@intel.com> - 2015-10-29 08:20 +0100
  [PATCH v2 4/5] block: introduce bdev_file_inode() Dan Williams <dan.j.williams@intel.com> - 2015-10-22 19:20 +0200
    Re: [PATCH v2 4/5] block: introduce bdev_file_inode() Jan Kara <jack@suse.cz> - 2015-10-22 22:40 +0200
    Re: [PATCH v2 4/5] block: introduce bdev_file_inode() Jeff Moyer <jmoyer@redhat.com> - 2015-10-28 21:20 +0100
  [PATCH v2 3/5] block,  dax: fix lifetime of in-kernel dax mappings with dax_map_atomic() Dan Williams <dan.j.williams@intel.com> - 2015-10-22 19:20 +0200
    Re: [PATCH v2 3/5] block, dax: fix lifetime of in-kernel dax mappings with dax_map_atomic() Jeff Moyer <jmoyer@redhat.com> - 2015-10-28 21:20 +0100
  [PATCH v2 2/5] dax: increase granularity of dax_clear_blocks()  operations Dan Williams <dan.j.williams@intel.com> - 2015-10-22 19:20 +0200
    Re: [PATCH v2 2/5] dax: increase granularity of dax_clear_blocks() operations Jeff Moyer <jmoyer@redhat.com> - 2015-10-22 23:10 +0200
      Re: [PATCH v2 2/5] dax: increase granularity of dax_clear_blocks()  operations "Williams, Dan J" <dan.j.williams@intel.com> - 2015-10-23 01:00 +0200
        Re: [PATCH v2 2/5] dax: increase granularity of dax_clear_blocks() operations Jeff Moyer <jmoyer@redhat.com> - 2015-10-28 22:10 +0100
  [PATCH v2 1/5] pmem, dax: clean up clear_pmem() Dan Williams <dan.j.williams@intel.com> - 2015-10-22 19:20 +0200
    Re: [PATCH v2 1/5] pmem, dax: clean up clear_pmem() Jeff Moyer <jmoyer@redhat.com> - 2015-10-22 22:50 +0200
      Re: [PATCH v2 1/5] pmem, dax: clean up clear_pmem() Dan Williams <dan.j.williams@intel.com> - 2015-10-23 00:30 +0200
        Re: [PATCH v2 1/5] pmem, dax: clean up clear_pmem() Jeff Moyer <jmoyer@redhat.com> - 2015-10-28 22:10 +0100
    Re: [PATCH v2 1/5] pmem, dax: clean up clear_pmem() Ross Zwisler <ross.zwisler@linux.intel.com> - 2015-10-27 18:40 +0100

csiph-web