On Thu, May 03, 2018 at 04:22:24PM -0500, Eric Sandeen wrote: > On 4/17/18 9:46 PM, Darrick J. Wong wrote: > > From: Darrick J. Wong <darrick.wong@xxxxxxxxxx> > > > > If the online filesystem geometry query doesn't work, try using xfs_db > > to see if we can grab the information offline. > > > > I'm slightly concerned that having all-singing, all-dancing tools > that operate on mounted, unmounted, devices, mountpoints, and images > will further encourage people to think that operations on mounted > block devices are Just Fine(tm). <nod> The current semantics of xfs_info are that one has to feed it a mountpoint, which means the arg is a direct... oh dangit. # touch mugga # mount /bin/true mugga --bind # xfs_info ./mugga meta-data=/dev/mapper/system-root isize=512 agcount=4, agsize=1953024 blks Uh, ok, so the new behavior of xfs_info is that if you pass it a block device or a file it'll try to call xfs_db (assuming it's a fs, fs image file, or I guess future-dchinner subvolume); and if it's any other kind of fs object it'll try spaceman (which requires a mountpoint arg). Hopefully that's ok with Dave's subvol plans? > But ... I suppose the convenience is worth it. > > ... > > > diff --git a/man/man8/xfs_info.8 b/man/man8/xfs_info.8 > > new file mode 100644 > > index 0000000..c4c470d > > --- /dev/null > > +++ b/man/man8/xfs_info.8 > > @@ -0,0 +1,95 @@ > > +.\" Verbatim blocks taken from openssl req manpage content > > +.de Vb \" Begin verbatim text > > +.ft CW > > +.nf > > +.ne \\$1 > > +.. > > +.de Ve \" End verbatim text > > +.ft R > > +.fi > > +.. > > + > > +.TH xfs_info 8 > > +.SH NAME > > +xfs_info, \- display XFS filesystem geometry information > > drop the comma pls, or I can do this. I'll drop it in my patches, though if you don't find any serious problems that warrant a reissue then you should do it. > ... > > > +.BR \-V . > > +.SH "EXAMPLES" > > + > > +Understanding xfs_info output. > > +.PP > > +Suppose one has the following "xfs_info /dev/sda" output: > > +.PP > > +.RS 2 > > +.Vb > > +\&meta-data=/dev/sda isize=256 agcount=32, agsize=16777184 blks > > +\& = sectsz=512 attr=2 > > +\&data = bsize=4096 blocks=536869888, imaxpct=5 > > +\& = sunit=32 swidth=128 blks > > +\&naming =version 2 bsize=4096 > > +\&log =internal log bsize=4096 blocks=32768, version=2 > > +\& = sectsz=512 sunit=32 blks, lazy-count=1 > > +\&realtime =none extsz=524288 blocks=0, rtextents=0 > > +.Ve > > Should we update this to a more modern version? *shrug* Yeah probably. --D > > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-xfs" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
