On Thu, Dec 01, 2016 at 01:44:01PM -0800, Liu Bo wrote:
> TP_STRUCT__entry_btrfs(
> __field( u64, root_objectid )
> + __field( u64, ino )
> __field( u64, start )
> __field( u64, len )
> __field( u64, orig_start )
> @@ -204,7 +206,8 @@ TRACE_EVENT_CONDITION(btrfs_get_extent,
>
> TP_fast_assign_btrfs(root->fs_info,
> __entry->root_objectid = root->root_key.objectid;
> - __entry->start = map->start;
> + __entry->ino = btrfs_ino(inode);
> + __entry->start = map->start;
> __entry->len = map->len;
> __entry->orig_start = map->orig_start;
> __entry->block_start = map->block_start;
> @@ -214,12 +217,12 @@ TRACE_EVENT_CONDITION(btrfs_get_extent,
> __entry->compress_type = map->compress_type;
> ),
>
> - TP_printk_btrfs("root = %llu(%s), start = %llu, len = %llu, "
> + TP_printk_btrfs("root = %llu(%s), ino = %llu start = %llu, len = %llu, "
> "orig_start = %llu, block_start = %llu(%s), "
> "block_len = %llu, flags = %s, refs = %u, "
> "compress_type = %u",
> show_root_type(__entry->root_objectid),
> - (unsigned long long)__entry->start,
> + __entry->ino, (unsigned long long)__entry->start,
I think the (unsigned long long) cast should be used here as well, at
least for consistency with the other u64.
> (unsigned long long)__entry->len,
> (unsigned long long)__entry->orig_start,
> show_map_type(__entry->block_start),
--
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
