On 21.11.2017 12:15, Su Yue wrote:
> Previously, it was so useless to print message like
> "invalid location %d".
>
> Let it print objectid and offset of dir_item.
> Debug is easier now.
>
> Signed-off-by: Su Yue <suy.fnst@xxxxxxxxxxxxxx>
> ---
> cmds-check.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/cmds-check.c b/cmds-check.c
> index a93ac2c88a38..5c15dfb60b9a 100644
> --- a/cmds-check.c
> +++ b/cmds-check.c
> @@ -1635,8 +1635,8 @@ 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, "invalid location in dir item[%llu %llu]\n",
> + key->objectid, key->offset);
I think it will be good if in addition to the dir item's key you also
print the invalid type value. I.e. this check is triggered if
location.type is unrecognizable. So the error message could be something
like :
fprintf(stderr, "Unrecognised location.type (%u) in DIR_ITEM[%llu
%llu]", key->objectid, key->offset);
Let's not force the user to go and read the code to understand why this
error happened.
> 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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
