diff options
author | Dave Chinner <david@fromorbit.com> | 2016-01-05 00:08:47 +0300 |
---|---|---|
committer | Dave Chinner <david@fromorbit.com> | 2016-01-05 00:08:47 +0300 |
commit | 4922be51ef1a95ca6a38694cf0cde5dd0308a24e (patch) | |
tree | 5a94d1298db141e63d43d9d6f42774be245d8221 /fs/xfs/xfs_iomap.c | |
parent | 7eeabbd4b6b69f3f6cb75730f17804b714bd853b (diff) | |
parent | a6d7636e8d0fd94fd1937db91d5b06a91fa85dde (diff) | |
download | linux-4922be51ef1a95ca6a38694cf0cde5dd0308a24e.tar.xz |
Merge branch 'xfs-dax-fixes-for-4.5' into for-next
Diffstat (limited to 'fs/xfs/xfs_iomap.c')
-rw-r--r-- | fs/xfs/xfs_iomap.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/fs/xfs/xfs_iomap.c b/fs/xfs/xfs_iomap.c index f4f5b43cf647..9ed146b96856 100644 --- a/fs/xfs/xfs_iomap.c +++ b/fs/xfs/xfs_iomap.c @@ -203,15 +203,20 @@ xfs_iomap_write_direct( * this outside the transaction context, but if we commit and then crash * we may not have zeroed the blocks and this will be exposed on * recovery of the allocation. Hence we must zero before commit. + * * Further, if we are mapping unwritten extents here, we need to zero * and convert them to written so that we don't need an unwritten extent * callback for DAX. This also means that we need to be able to dip into - * the reserve block pool if there is no space left but we need to do - * unwritten extent conversion. + * the reserve block pool for bmbt block allocation if there is no space + * left but we need to do unwritten extent conversion. */ + if (IS_DAX(VFS_I(ip))) { bmapi_flags = XFS_BMAPI_CONVERT | XFS_BMAPI_ZERO; - tp->t_flags |= XFS_TRANS_RESERVE; + if (ISUNWRITTEN(imap)) { + tp->t_flags |= XFS_TRANS_RESERVE; + resblks = XFS_DIOSTRAT_SPACE_RES(mp, 0) << 1; + } } error = xfs_trans_reserve(tp, &M_RES(mp)->tr_write, resblks, resrtextents); |