Bz #248176: GFS2: invalid metadata block - RESEND
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(strgfs2_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 *ireturn ; 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 *ireturn ; 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 ggfs2_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);
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.
--Signed-off-by Bob Peterson <[email protected]>--
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(strgfs2_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 *ireturn ; 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 *ireturn ; 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 ggfs2_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);