diff options
| author | Christoph Hellwig | 2018-07-12 07:26:01 +0200 |
|---|---|---|
| committer | Darrick J. Wong | 2018-07-12 07:26:01 +0200 |
| commit | 060d4eaa0bf30a8fc2d189e4d4922f6e9027857b (patch) | |
| tree | 2b79fd3d6c17ed96dd4017b1a6fd8df1f9f5d36b /fs/xfs/xfs_reflink.c | |
| parent | xfs: remove the now unused XFS_BMAPI_IGSTATE flag (diff) | |
| download | kernel-qcow2-linux-060d4eaa0bf30a8fc2d189e4d4922f6e9027857b.tar.gz kernel-qcow2-linux-060d4eaa0bf30a8fc2d189e4d4922f6e9027857b.tar.xz kernel-qcow2-linux-060d4eaa0bf30a8fc2d189e4d4922f6e9027857b.zip | |
xfs: remove xfs_reflink_find_cow_mapping
We only have one caller left, and open coding the simple extent list
lookup in it allows us to make the code both more understandable and
reuse calculations and variables already present.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Brian Foster <bfoster@redhat.com>
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Diffstat (limited to 'fs/xfs/xfs_reflink.c')
| -rw-r--r-- | fs/xfs/xfs_reflink.c | 30 |
1 files changed, 0 insertions, 30 deletions
diff --git a/fs/xfs/xfs_reflink.c b/fs/xfs/xfs_reflink.c index 22c11b98ab26..49e4913fa779 100644 --- a/fs/xfs/xfs_reflink.c +++ b/fs/xfs/xfs_reflink.c @@ -471,36 +471,6 @@ out: } /* - * Find the CoW reservation for a given byte offset of a file. - */ -bool -xfs_reflink_find_cow_mapping( - struct xfs_inode *ip, - xfs_off_t offset, - struct xfs_bmbt_irec *imap) -{ - struct xfs_ifork *ifp = XFS_IFORK_PTR(ip, XFS_COW_FORK); - xfs_fileoff_t offset_fsb; - struct xfs_bmbt_irec got; - struct xfs_iext_cursor icur; - - ASSERT(xfs_isilocked(ip, XFS_ILOCK_EXCL | XFS_ILOCK_SHARED)); - - if (!xfs_is_reflink_inode(ip)) - return false; - offset_fsb = XFS_B_TO_FSBT(ip->i_mount, offset); - if (!xfs_iext_lookup_extent(ip, ifp, offset_fsb, &icur, &got)) - return false; - if (got.br_startoff > offset_fsb) - return false; - - trace_xfs_reflink_find_cow_mapping(ip, offset, 1, XFS_IO_OVERWRITE, - &got); - *imap = got; - return true; -} - -/* * Cancel CoW reservations for some block range of an inode. * * If cancel_real is true this function cancels all COW fork extents for the |