On 22.11.2017 04:17, Su Yue wrote: > Previously, it was so useless to print message like > "invalid location %d". > > Let it print objectid and offset of dir_item too. > Debug is easier now. > > Suggested-by: Nikolay Borisov <nbori...@suse.com> > Signed-off-by: Su Yue <suy.f...@cn.fujitsu.com>
Reviewed-by: Nikolay Borisov <nbori...@suse.com> > --- > changelog: > v2: > Print unknown location type. > --- > cmds-check.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/cmds-check.c b/cmds-check.c > index a93ac2c88a38..416a7601bce1 100644 > --- a/cmds-check.c > +++ b/cmds-check.c > @@ -1635,8 +1635,9 @@ static int process_dir_item(struct extent_buffer *eb, > namebuf, len, filetype, > key->type, error); > } else { > - fprintf(stderr, "invalid location in dir item %u\n", > - location.type); > + fprintf(stderr, > + "Unknown location type %d in DIR_ITEM[%llu > %llu]\n", > + location.type, key->objectid, key->offset); > add_inode_backref(inode_cache, BTRFS_MULTIPLE_OBJECTIDS, > key->objectid, key->offset, namebuf, > len, filetype, key->type, error); > -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html