xfs
[Top] [All Lists]

[PATCH 079/102] xfs: limit specualtive delalloc to maxioffset

To: xfs@xxxxxxxxxxx
Subject: [PATCH 079/102] xfs: limit specualtive delalloc to maxioffset
From: Dave Chinner <david@xxxxxxxxxxxxx>
Date: Thu, 23 Aug 2012 15:02:37 +1000
In-reply-to: <1345698180-13612-1-git-send-email-david@xxxxxxxxxxxxx>
References: <1345698180-13612-1-git-send-email-david@xxxxxxxxxxxxx>
From: Dave Chinner <dchinner@xxxxxxxxxx>

Upstream commit: 3ed9116e8a3e9c0870b2076340b3da9b8f900f3b

Speculative delayed allocation beyond EOF near the maximum supported
file offset can result in creating delalloc extents beyond
mp->m_maxioffset (8EB). These can never be trimmed during
xfs_free_eof_blocks() because they are beyond mp->m_maxioffset, and
that results in assert failures in xfs_fs_destroy_inode() due to
delalloc blocks still being present. xfstests 071 exposes this
problem.

Limit speculative delalloc to mp->m_maxioffset to avoid this
problem.

Signed-off-by: Dave Chinner <dchinner@xxxxxxxxxx>
Signed-off-by: Ben Myers <bpm@xxxxxxx>
---
 fs/xfs/xfs_iomap.c |    9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/fs/xfs/xfs_iomap.c b/fs/xfs/xfs_iomap.c
index 29e2d85..121309f 100644
--- a/fs/xfs/xfs_iomap.c
+++ b/fs/xfs/xfs_iomap.c
@@ -418,6 +418,15 @@ retry:
                        return error;
        }
 
+       /*
+        * Make sure preallocation does not create extents beyond the range we
+        * actually support in this filesystem.
+        */
+       if (last_fsb > XFS_B_TO_FSB(mp, mp->m_maxioffset))
+               last_fsb = XFS_B_TO_FSB(mp, mp->m_maxioffset);
+
+       ASSERT(last_fsb > offset_fsb);
+
        nimaps = XFS_WRITE_IMAPS;
        firstblock = NULLFSBLOCK;
        error = xfs_bmapi(NULL, ip, offset_fsb,
-- 
1.7.10

<Prev in Thread] Current Thread [Next in Thread>