On 3/14/19 4:05 PM, Darrick J. Wong wrote: > From: Darrick J. Wong <darrick.wong@xxxxxxxxxx> > > Use the sparse inobt record field decoder (inobt_spcrc_hfld) to decode > finobt records when sparse inodes are enabled. Otherwise, xfs_db > prints out bogus things like: > > recs[1] = [startino,freecount,free] > 1:[214720,16429,0xfffffffffff80000] > > There can never be 16429 records in an inode btree record; instead it > should print: > > recs[1] = [startino,holemask,count,freecount,free] > 1:[214720,0,64,45,0xfffffffffff80000] > > Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> Um... :) commit 38c599e7782ced7d2d1b41c3182daeaddad2cee4 Author: Darrick J. Wong <darrick.wong@xxxxxxxxxx> Date: Mon Jan 28 13:03:15 2019 -0600 xfs_db: fix finobt record decoding when sparse inodes enabled Use the sparse inobt record field decoder (inobt_spcrc_hfld) to decode finobt records when sparse inodes are enabled. Otherwise, xfs_db prints out bogus things like: recs[1] = [startino,freecount,free] 1:[214720,16429,0xfffffffffff80000] There can never be 16429 records in an inode btree record; instead it should print: recs[1] = [startino,holemask,count,freecount,free] 1:[214720,0,64,45,0xfffffffffff80000] Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx> Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxxx> > --- > db/type.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > > diff --git a/db/type.c b/db/type.c > index f5f65042..d42935a2 100644 > --- a/db/type.c > +++ b/db/type.c > @@ -107,7 +107,7 @@ static const typ_t __typtab_crc[] = { > { TYP_SYMLINK, "symlink", handle_struct, symlink_crc_hfld, > &xfs_symlink_buf_ops, XFS_SYMLINK_CRC_OFF }, > { TYP_TEXT, "text", handle_text, NULL, NULL, TYP_F_NO_CRC_OFF }, > - { TYP_FINOBT, "finobt", handle_struct, inobt_crc_hfld, > + { TYP_FINOBT, "finobt", handle_struct, inobt_spcrc_hfld, > &xfs_inobt_buf_ops, XFS_BTREE_SBLOCK_CRC_OFF }, > { TYP_NONE, NULL } > }; >