diff options
author | Steven Whitehouse <swhiteho@redhat.com> | 2011-09-07 10:33:25 +0100 |
---|---|---|
committer | Steven Whitehouse <swhiteho@redhat.com> | 2011-10-21 12:39:41 +0100 |
commit | b5b24d7aeb9608935786369ac2d3e9f362877d55 (patch) | |
tree | 2043123df7a6e6ca367b24cb3442ee50314cdee0 /fs/gfs2/super.c | |
parent | 70b0c3656f12964a6dac104214c904c66e626058 (diff) | |
download | op-kernel-dev-b5b24d7aeb9608935786369ac2d3e9f362877d55.zip op-kernel-dev-b5b24d7aeb9608935786369ac2d3e9f362877d55.tar.gz |
GFS2: Fix AIL flush issue during fsync
Unfortunately, it is not enough to just ignore locked buffers during
the AIL flush from fsync. We need to be able to ignore all buffers
which are locked, dirty or pinned at this stage as they might have
been added subsequent to the log flush earlier in the fsync function.
In addition, this means that we no longer need to rely on i_mutex to
keep out writes during fsync, so we can, as a side-effect, remove
that protection too.
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Tested-By: Abhijith Das <adas@redhat.com>
Diffstat (limited to 'fs/gfs2/super.c')
-rw-r--r-- | fs/gfs2/super.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/gfs2/super.c b/fs/gfs2/super.c index 87e9141..71e4209 100644 --- a/fs/gfs2/super.c +++ b/fs/gfs2/super.c @@ -1533,7 +1533,7 @@ static void gfs2_evict_inode(struct inode *inode) out_truncate: gfs2_log_flush(sdp, ip->i_gl); write_inode_now(inode, 1); - gfs2_ail_flush(ip->i_gl); + gfs2_ail_flush(ip->i_gl, 0); /* Case 2 starts here */ error = gfs2_trans_begin(sdp, 0, sdp->sd_jdesc->jd_blocks); |