diff options
| author | Bob Peterson <rpeterso@redhat.com> | 2007-07-25 11:06:22 -0400 |
|---|---|---|
| committer | Steven Whitehouse <swhiteho@redhat.com> | 2007-10-10 03:55:01 -0400 |
| commit | 0f8468c8bef3d04637c924e7bef20ca53018b319 (patch) | |
| tree | fc4fa6fedb9fe4e704c8bedc749a0b90a39d0218 /fs/gfs2 | |
| parent | cee23c79d08c57bbbb9923703409af3b17518c58 (diff) | |
[GFS2] Detach buf data during in-place writeback
This is patch 5 of 5 for bug #248176
Metadata corruption was occurring because page references weren't
being removed in all cases. I previously added a function called
detach_bufdata, but I discovered there already WAS a function out
there to do the job. It's called gfs2_meta_cache_flush. So I added
a call to that to remove the page references.
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2')
| -rw-r--r-- | fs/gfs2/log.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/fs/gfs2/log.c b/fs/gfs2/log.c index f7c0608332fb..00ab6c070a15 100644 --- a/fs/gfs2/log.c +++ b/fs/gfs2/log.c | |||
| @@ -219,6 +219,7 @@ static void gfs2_ail2_empty_one(struct gfs2_sbd *sdp, struct gfs2_ail *ai) | |||
| 219 | { | 219 | { |
| 220 | struct list_head *head = &ai->ai_ail2_list; | 220 | struct list_head *head = &ai->ai_ail2_list; |
| 221 | struct gfs2_bufdata *bd; | 221 | struct gfs2_bufdata *bd; |
| 222 | struct gfs2_inode *bh_ip; | ||
| 222 | 223 | ||
| 223 | while (!list_empty(head)) { | 224 | while (!list_empty(head)) { |
| 224 | bd = list_entry(head->prev, struct gfs2_bufdata, | 225 | bd = list_entry(head->prev, struct gfs2_bufdata, |
| @@ -228,6 +229,8 @@ static void gfs2_ail2_empty_one(struct gfs2_sbd *sdp, struct gfs2_ail *ai) | |||
| 228 | list_del(&bd->bd_ail_st_list); | 229 | list_del(&bd->bd_ail_st_list); |
| 229 | list_del(&bd->bd_ail_gl_list); | 230 | list_del(&bd->bd_ail_gl_list); |
| 230 | atomic_dec(&bd->bd_gl->gl_ail_count); | 231 | atomic_dec(&bd->bd_gl->gl_ail_count); |
| 232 | bh_ip = GFS2_I(bd->bd_bh->b_page->mapping->host); | ||
| 233 | gfs2_meta_cache_flush(bh_ip); | ||
| 231 | brelse(bd->bd_bh); | 234 | brelse(bd->bd_bh); |
| 232 | } | 235 | } |
| 233 | } | 236 | } |
