[Cluster-devel] [PATCH 4 of 5]Bz #248176: GFS2: invalid metadata block - RESEND
Steven Whitehouse
swhiteho at redhat.com
Wed Jul 25 09:49:02 UTC 2007
Hi,
I've pushed patches 1-3, patches 4 & 5 don't apply. It looks like a path
problem. You need to generate the patches so that they can be applied
with patch -p1 from the top level of the source tree (so the best way is
by being one level up from that when you run diff),
Steve.
On Tue, 2007-07-24 at 14:11 -0500, Bob Peterson wrote:
> This is patch 4 of 5 for bug #248176.
>
> There was metadata corruption caused because the clone bitmaps weren't
> being kept in sync with the regular bitmaps in some cases.
> Code was added to keep them in sync.
> --
> Signed-off-by: Bob Peterson <rpeterso at redhat.com>
> --
> --- rgrp.patch3.c 2007-07-24 12:08:36.000000000 -0500
> +++ rgrp.patch4.c 2007-07-24 12:29:07.000000000 -0500
> @@ -54,6 +54,7 @@ static const char valid_change[16] = {
>
> static u32 rgblk_search(struct gfs2_rgrpd *rgd, u32 goal,
> unsigned char old_state, unsigned char new_state);
> +void gfs2_rgrp_repolish_clones(struct gfs2_rgrpd *rgd);
>
> /**
> * gfs2_setbit - Set a bit in the bitmaps
> @@ -727,6 +728,9 @@ int gfs2_rgrp_bh_get(struct gfs2_rgrpd *
> if (rgd->rd_rg_vn != gl->gl_vn) {
> gfs2_rgrp_in(&rgd->rd_rg, (rgd->rd_bits[0].bi_bh)->b_data);
> rgd->rd_rg_vn = gl->gl_vn;
> + bi = rgd->rd_bits + x;
> + if (bi->bi_clone)
> + gfs2_rgrp_repolish_clones(rgd);
> }
>
> spin_lock(&sdp->sd_rindex_spin);
> @@ -1374,6 +1378,8 @@ static struct gfs2_rgrpd *rgblk_free(str
> gfs2_trans_add_bh(rgd->rd_gl, bi->bi_bh, 1);
> gfs2_setbit(rgd, bi->bi_bh->b_data + bi->bi_offset,
> bi->bi_len, buf_blk, new_state);
> + gfs2_setbit(rgd, bi->bi_clone + bi->bi_offset,
> + bi->bi_len, buf_blk, new_state);
> }
>
> return rgd;
> @@ -1530,6 +1536,9 @@ void gfs2_free_data(struct gfs2_inode *i
> return;
>
> rgd->rd_rg.rg_free += blen;
> + spin_lock(&sdp->sd_rindex_spin);
> + rgd->rd_free_clone += blen;
> + spin_unlock(&sdp->sd_rindex_spin);
>
> gfs2_trans_add_bh(rgd->rd_gl, rgd->rd_bits[0].bi_bh, 1);
> gfs2_rgrp_out(&rgd->rd_rg, rgd->rd_bits[0].bi_bh->b_data);
> @@ -1558,6 +1567,9 @@ void gfs2_free_meta(struct gfs2_inode *i
> return;
>
> rgd->rd_rg.rg_free += blen;
> + spin_lock(&sdp->sd_rindex_spin);
> + rgd->rd_free_clone += blen;
> + spin_unlock(&sdp->sd_rindex_spin);
>
> gfs2_trans_add_bh(rgd->rd_gl, rgd->rd_bits[0].bi_bh, 1);
> gfs2_rgrp_out(&rgd->rd_rg, rgd->rd_bits[0].bi_bh->b_data);
> @@ -1598,6 +1610,9 @@ static void gfs2_free_uninit_di(struct g
> gfs2_consist_rgrpd(rgd);
> rgd->rd_rg.rg_dinodes--;
> rgd->rd_rg.rg_free++;
> + spin_lock(&sdp->sd_rindex_spin);
> + rgd->rd_free_clone++;
> + spin_unlock(&sdp->sd_rindex_spin);
>
> gfs2_trans_add_bh(rgd->rd_gl, rgd->rd_bits[0].bi_bh, 1);
> gfs2_rgrp_out(&rgd->rd_rg, rgd->rd_bits[0].bi_bh->b_data);
>
>
More information about the Cluster-devel
mailing list