Browse Source

dax: don't pass buffer_head to copy_user_dax

This way we can use this helper for the iomap based DAX implementation
as well.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Ross Zwisler <ross.zwisler@linux.intel.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>
Christoph Hellwig 9 years ago
parent
commit
b0d5e82fcf
1 changed files with 6 additions and 6 deletions
  1. 6 6
      fs/dax.c

+ 6 - 6
fs/dax.c

@@ -580,14 +580,13 @@ static int dax_load_hole(struct address_space *mapping, void *entry,
 	return VM_FAULT_LOCKED;
 	return VM_FAULT_LOCKED;
 }
 }
 
 
-static int copy_user_bh(struct page *to, struct inode *inode,
-		struct buffer_head *bh, unsigned long vaddr)
+static int copy_user_dax(struct block_device *bdev, sector_t sector, size_t size,
+		struct page *to, unsigned long vaddr)
 {
 {
 	struct blk_dax_ctl dax = {
 	struct blk_dax_ctl dax = {
-		.sector = to_sector(bh, inode),
-		.size = bh->b_size,
+		.sector = sector,
+		.size = size,
 	};
 	};
-	struct block_device *bdev = bh->b_bdev;
 	void *vto;
 	void *vto;
 
 
 	if (dax_map_atomic(bdev, &dax) < 0)
 	if (dax_map_atomic(bdev, &dax) < 0)
@@ -867,7 +866,8 @@ int dax_fault(struct vm_area_struct *vma, struct vm_fault *vmf,
 	if (vmf->cow_page) {
 	if (vmf->cow_page) {
 		struct page *new_page = vmf->cow_page;
 		struct page *new_page = vmf->cow_page;
 		if (buffer_written(&bh))
 		if (buffer_written(&bh))
-			error = copy_user_bh(new_page, inode, &bh, vaddr);
+			error = copy_user_dax(bh.b_bdev, to_sector(&bh, inode),
+					bh.b_size, new_page, vaddr);
 		else
 		else
 			clear_user_highpage(new_page, vaddr);
 			clear_user_highpage(new_page, vaddr);
 		if (error)
 		if (error)