From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com ([209.132.183.28]:46774 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754987AbeEAORy (ORCPT ); Tue, 1 May 2018 10:17:54 -0400 Date: Tue, 1 May 2018 10:17:52 -0400 From: Brian Foster Subject: Re: [PATCH 2/5] xfs: factor out secondary superblock updates Message-ID: <20180501141752.GF4525@bfoster.bfoster> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: linux-xfs-owner@vger.kernel.org List-ID: List-Id: xfs To: Eric Sandeen Cc: linux-xfs On Mon, Apr 30, 2018 at 10:43:55AM -0500, Eric Sandeen wrote: > growfs rewrites all secondary supers, and relabel should do so as well; > factor out the code which does this for use by both operations. > > Signed-off-by: Eric Sandeen > --- FYI I think Dave has some patches in progress to rework growfs. It might be useful to coordinate on the refactoring bits, at least.. Brian > fs/xfs/xfs_fsops.c | 130 +++++++++++++++++++++++++++++++---------------------- > fs/xfs/xfs_fsops.h | 2 + > 2 files changed, 79 insertions(+), 53 deletions(-) > > diff --git a/fs/xfs/xfs_fsops.c b/fs/xfs/xfs_fsops.c > index 5237927..8f2a73a 100644 > --- a/fs/xfs/xfs_fsops.c > +++ b/fs/xfs/xfs_fsops.c > @@ -71,6 +71,79 @@ > return bp; > } > > +/* > + * Copy the contents of the primary super to all backup supers. > + * %agcount is currently existing AGs > + * %new_ags is the number of new AGs to write out if we're growing the > + * filesystem. > + * We can't use mp->m_sb.sb_agcount here because growfs already updated it. > + */ > +int > +xfs_update_secondary_supers( > + xfs_mount_t *mp, > + xfs_agnumber_t agcount, > + xfs_agnumber_t new_ags) > +{ > + int error, saved_error; > + xfs_agnumber_t agno; > + xfs_buf_t *bp; > + > + error = saved_error = 0; > + > + /* update secondary superblocks. */ > + for (agno = 1; agno < agcount + new_ags; agno++) { > + error = 0; > + /* > + * new secondary superblocks need to be zeroed, not read from > + * disk as the contents of the new area we are growing into is > + * completely unknown. > + */ > + if (agno < agcount) { > + error = xfs_trans_read_buf(mp, NULL, mp->m_ddev_targp, > + XFS_AGB_TO_DADDR(mp, agno, XFS_SB_BLOCK(mp)), > + XFS_FSS_TO_BB(mp, 1), 0, &bp, > + &xfs_sb_buf_ops); > + } else { > + bp = xfs_trans_get_buf(NULL, mp->m_ddev_targp, > + XFS_AGB_TO_DADDR(mp, agno, XFS_SB_BLOCK(mp)), > + XFS_FSS_TO_BB(mp, 1), 0); > + if (bp) { > + bp->b_ops = &xfs_sb_buf_ops; > + xfs_buf_zero(bp, 0, BBTOB(bp->b_length)); > + } else > + error = -ENOMEM; > + } > + > + /* > + * If we get an error reading or writing alternate superblocks, > + * continue. xfs_repair chooses the "best" superblock based > + * on most matches; if we break early, we'll leave more > + * superblocks un-updated than updated, and xfs_repair may > + * pick them over the properly-updated primary. > + */ > + if (error) { > + xfs_warn(mp, > + "error %d reading secondary superblock for ag %d", > + error, agno); > + saved_error = error; > + continue; > + } > + xfs_sb_to_disk(XFS_BUF_TO_SBP(bp), &mp->m_sb); > + > + error = xfs_bwrite(bp); > + xfs_buf_relse(bp); > + if (error) { > + xfs_warn(mp, > + "write error %d updating secondary superblock for ag %d", > + error, agno); > + saved_error = error; > + continue; > + } > + } > + > + return saved_error ? saved_error : error; > +} > + > static int > xfs_growfs_data_private( > xfs_mount_t *mp, /* mount point for filesystem */ > @@ -86,7 +159,7 @@ > xfs_buf_t *bp; > int bucket; > int dpct; > - int error, saved_error = 0; > + int error; > xfs_agnumber_t nagcount; > xfs_agnumber_t nagimax = 0; > xfs_rfsblock_t nb, nb_mod; > @@ -557,59 +630,10 @@ > if (error && error != -ENOSPC) > goto out; > > - /* update secondary superblocks. */ > - for (agno = 1; agno < nagcount; agno++) { > - error = 0; > - /* > - * new secondary superblocks need to be zeroed, not read from > - * disk as the contents of the new area we are growing into is > - * completely unknown. > - */ > - if (agno < oagcount) { > - error = xfs_trans_read_buf(mp, NULL, mp->m_ddev_targp, > - XFS_AGB_TO_DADDR(mp, agno, XFS_SB_BLOCK(mp)), > - XFS_FSS_TO_BB(mp, 1), 0, &bp, > - &xfs_sb_buf_ops); > - } else { > - bp = xfs_trans_get_buf(NULL, mp->m_ddev_targp, > - XFS_AGB_TO_DADDR(mp, agno, XFS_SB_BLOCK(mp)), > - XFS_FSS_TO_BB(mp, 1), 0); > - if (bp) { > - bp->b_ops = &xfs_sb_buf_ops; > - xfs_buf_zero(bp, 0, BBTOB(bp->b_length)); > - } else > - error = -ENOMEM; > - } > - > - /* > - * If we get an error reading or writing alternate superblocks, > - * continue. xfs_repair chooses the "best" superblock based > - * on most matches; if we break early, we'll leave more > - * superblocks un-updated than updated, and xfs_repair may > - * pick them over the properly-updated primary. > - */ > - if (error) { > - xfs_warn(mp, > - "error %d reading secondary superblock for ag %d", > - error, agno); > - saved_error = error; > - continue; > - } > - xfs_sb_to_disk(XFS_BUF_TO_SBP(bp), &mp->m_sb); > - > - error = xfs_bwrite(bp); > - xfs_buf_relse(bp); > - if (error) { > - xfs_warn(mp, > - "write error %d updating secondary superblock for ag %d", > - error, agno); > - saved_error = error; > - continue; > - } > - } > - > + /* Copy new geometry to all backup superblocks, old & new */ > + error = xfs_update_secondary_supers(mp, oagcount, nagcount - oagcount); > out: > - return saved_error ? saved_error : error; > + return error; > > error0: > xfs_trans_cancel(tp); > diff --git a/fs/xfs/xfs_fsops.h b/fs/xfs/xfs_fsops.h > index 20484ed..ffc5377 100644 > --- a/fs/xfs/xfs_fsops.h > +++ b/fs/xfs/xfs_fsops.h > @@ -18,6 +18,8 @@ > #ifndef __XFS_FSOPS_H__ > #define __XFS_FSOPS_H__ > > +extern int xfs_update_secondary_supers(xfs_mount_t *mp, xfs_agnumber_t agcount, > + xfs_agnumber_t new_ags); > extern int xfs_growfs_data(xfs_mount_t *mp, xfs_growfs_data_t *in); > extern int xfs_growfs_log(xfs_mount_t *mp, xfs_growfs_log_t *in); > extern int xfs_fs_counts(xfs_mount_t *mp, xfs_fsop_counts_t *cnt); > -- > 1.8.3.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-xfs" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html