make sure xfs_repair doesn't restore corrupted primary superblock
PATCH 1/1 make sure xfs_repair doesn't restore corrupted primary superblock
Phase 2 in xfs_repair fixes corrupted superblock fields, but in phase 5, the originally read primary superblock before the phase 2 fix is written to disk with updated counters. The patch in scan.c makes sure the mount point superblock copy is updated if the primary superblock is modified.
Included is some fixes to libxfs when IO_DEBUG is enabled.
===========================================================================xfsprogs/libxfs/rdwr.c
===========================================================================
xfsprogs/libxfs/rdwr.c +8 -6
@@ -305,8 +305,9 @@ libxfs_getbuf(dev_t device, xfs_daddr_tif (cache_node_get (libxfs_bcache, &key, (struct cache_node **)&bp)) {#ifdef IO_DEBUG -fprintf (stderr,"%s: allocated buffer, key=%llu(%llu), %p\n" , - __FUNCTION__,BBTOB (len),LIBXFS_BBTOOFF64 (blkno), blkno, buf); +fprintf (stderr,"%s: allocated %ubytes buffer, key=%llu(%llu), %p\n" , + __FUNCTION__,BBTOB (len), + (long long )LIBXFS_BBTOOFF64 (blkno), (long long )blkno, bp);#endif libxfs_initbuf (bp, device, blkno, bytes); }
@@ -354,7 +355,7 @@ libxfs_readbufr(dev_t dev, xfs_daddr_t b }#ifdef IO_DEBUG fprintf (stderr,"readbufr read %ubytes, blkno=%llu(%llu), %p\n" , - bytes,LIBXFS_BBTOOFF64 (blkno), blkno, bp); + bytes, (long long )LIBXFS_BBTOOFF64 (blkno), (long long )blkno, bp);#endif if (bp->b_dev == dev && bp->b_blkno == blkno &&
@@ -403,7 +404,8 @@ libxfs_writebufr(xfs_buf_t *bp) }#ifdef IO_DEBUG fprintf (stderr,"writebufr wrote %ubytes, blkno=%llu(%llu), %p\n" , - bp->b_bcount,LIBXFS_BBTOOFF64 (bp->b_blkno), bp->b_blkno, bp); + bp->b_bcount, (long long )LIBXFS_BBTOOFF64 (bp->b_blkno), + (long long )bp->b_blkno, bp);#endif bp->b_flags |=LIBXFS_B_UPTODATE ; bp->b_flags &= ~(LIBXFS_B_DIRTY |LIBXFS_B_EXIT );
@@ -432,7 +434,7 @@ libxfs_iomove(xfs_buf_t *bp, uint boff,if (boff + len > bp->b_bcount) {fprintf (stderr,"Badness, iomove out of range!\n" "bp=(bno %llu, bytes %u) range=(boff %u, bytes %u)\n" , - bp->b_blkno, bp->b_bcount, boff, len); + (long long )bp->b_blkno, bp->b_bcount, boff, len);abort (); }#endif
@@ -464,7 +466,7 @@ libxfs_bcache_purge(void)cache_purge (libxfs_bcache); } -void +void libxfs_bcache_flush (void ) {cache_flush (libxfs_bcache);
===========================================================================xfsprogs/repair/scan.c
===========================================================================
xfsprogs/repair/scan.c +3 -1
@@ -431,7 +431,7 @@ _("out-of-order bmap key (file offset) i }/* -* If we're the last node at our level, check that the last child +* If we're the last node at our level, check that the last child * block's forward sibling pointer is NULL. */ if (check_dups == 0 &&
@@ -1295,6 +1295,8 @@ scan_ag(ASSERT (sb_dirty ==0 || (sb_dirty && !no_modify));if (sb_dirty && !no_modify) { +if (agno ==0 ) +memcpy (&mp->m_sb, sb,sizeof (xfs_sb_t));libxfs_xlate_sb (XFS_BUF_PTR (sbbuf), sb,-1 ,XFS_SB_ALL_BITS );libxfs_writebuf (sbbuf,0 ); }else
Quoted 6 lines · Barry Naujok
Phase 2 in
xfs_repairfixes corrupted superblock fields, but in phase 5, the originally read primary superblock before the phase 2 fix is written to disk with updated counters. The patch inscan.cmakes sure the mount point superblock copy is updated if the primary superblock is modified.Included is some fixes to libxfs when
IO_DEBUGis enabled.
Looks good to me.
Signature · 1 line
Nathan