diff options
author | Steven Whitehouse <swhiteho@redhat.com> | 2007-08-16 11:03:57 -0400 |
---|---|---|
committer | Steven Whitehouse <swhiteho@redhat.com> | 2007-10-10 03:55:29 -0400 |
commit | bb3b0e3df5420fdf2c6bbb4417525c6d2ef55bbb (patch) | |
tree | 6eab0fa06b4dc9f9a8d62cedaa6879327966a20c /fs/gfs2/glops.c | |
parent | 2d9a4bbf6d28673f4057682cc02d16bf288b4a35 (diff) |
[GFS2] Clean up invalidatepage/releasepage
This patch fixes some bugs relating to journaled data files by cleaning
up the gfs2_invalidatepage() and gfs2_releasepage() functions. We now
never block during gfs2_releasepage(), instead we always either release
or refuse to release depending on the status of the buffers.
This fixes Red Hat bugzillas #248969 and #252392.
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Cc: Bob Peterson <rpeterso@redhat.com>
Diffstat (limited to 'fs/gfs2/glops.c')
-rw-r--r-- | fs/gfs2/glops.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/gfs2/glops.c b/fs/gfs2/glops.c index 777ca46010e8..88342e0b4bc7 100644 --- a/fs/gfs2/glops.c +++ b/fs/gfs2/glops.c | |||
@@ -156,9 +156,11 @@ static void inode_go_sync(struct gfs2_glock *gl) | |||
156 | ip = NULL; | 156 | ip = NULL; |
157 | 157 | ||
158 | if (test_bit(GLF_DIRTY, &gl->gl_flags)) { | 158 | if (test_bit(GLF_DIRTY, &gl->gl_flags)) { |
159 | if (ip) | 159 | if (ip && !gfs2_is_jdata(ip)) |
160 | filemap_fdatawrite(ip->i_inode.i_mapping); | 160 | filemap_fdatawrite(ip->i_inode.i_mapping); |
161 | gfs2_log_flush(gl->gl_sbd, gl); | 161 | gfs2_log_flush(gl->gl_sbd, gl); |
162 | if (ip && gfs2_is_jdata(ip)) | ||
163 | filemap_fdatawrite(ip->i_inode.i_mapping); | ||
162 | gfs2_meta_sync(gl); | 164 | gfs2_meta_sync(gl); |
163 | if (ip) { | 165 | if (ip) { |
164 | struct address_space *mapping = ip->i_inode.i_mapping; | 166 | struct address_space *mapping = ip->i_inode.i_mapping; |