Skip to contentSkip to conversation

Bz #248176: GFS2: invalid metadata block - RESEND

PATCH New Bob Peterson 2 participants 2 messages Jul 24, 2007 – Jul 25, 2007

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Bz #248176: GFS2: invalid metadata block - RESEND

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.
--

--

rgrp.patch4.c renamed from rgrp.patch3.c +15 -0
@@ -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);
#1 lore raw

PATCH 1/1 Bz #248176: GFS2: invalid metadata block - RESEND

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.

Quoted 69 lines · Bob Peterson

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.
--

--

rgrp.patch4.c renamed from rgrp.patch3.c +15 -0
@@ -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);