On 12/2/19 11:37 AM, Darrick J. Wong wrote: > From: Darrick J. Wong <darrick.wong@xxxxxxxxxx> > > Add to xfs_admin the ability to pass external log devices to xfs_db. > This is necessary to make changes on such filesystems. > > Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> > --- > db/xfs_admin.sh | 12 ++++++++++-- > man/man8/xfs_admin.8 | 3 +++ > 2 files changed, 13 insertions(+), 2 deletions(-) > > > diff --git a/db/xfs_admin.sh b/db/xfs_admin.sh > index 305ef2b7..bd325da2 100755 > --- a/db/xfs_admin.sh > +++ b/db/xfs_admin.sh > @@ -7,7 +7,7 @@ > status=0 > DB_OPTS="" > REPAIR_OPTS="" > -USAGE="Usage: xfs_admin [-efjlpuV] [-c 0|1] [-L label] [-U uuid] device" > +USAGE="Usage: xfs_admin [-efjlpuV] [-c 0|1] [-L label] [-U uuid] device [logdev]" > > while getopts "efjlpuc:L:U:V" c > do > @@ -33,7 +33,15 @@ done > set -- extra $@ > shift $OPTIND > case $# in > - 1) if [ -n "$DB_OPTS" ] > + 1|2) > + # Pick up the log device, if present > + if [ -n "$2" ]; then > + DB_OPTS=$DB_OPTS" -l '$2'" > + test -n "$REPAIR_OPTS" && \ > + REPAIR_OPTS=$REPAIR_OPTS" -l '$2'" > + fi > + > + if [ -n "$DB_OPTS" ] > then > eval xfs_db -x -p xfs_admin $DB_OPTS $1 > status=$? > diff --git a/man/man8/xfs_admin.8 b/man/man8/xfs_admin.8 > index 20a114f5..d7942418 100644 > --- a/man/man8/xfs_admin.8 > +++ b/man/man8/xfs_admin.8 > @@ -15,6 +15,9 @@ xfs_admin \- change parameters of an XFS filesystem > .I uuid > ] > .I device > +[ > +.I logdev > +] logdev should be marked as optional, right? [logdev] ? And documented as to when this arg should (or should not be) used? -Eric
