Re: [PATCH] xfs: trace ATTR flags in xattr tracepoints

["Darrick J. Wong" <darrick.wong@oracle.com>]

On Tue, Apr 17, 2018 at 03:03:32PM -0500, Eric Sandeen wrote:
> This will trace i.e. the ATTR_SECURE/ATTR_CREATE/ATTR_REPLACE
> flags as well as the OP_FLAGS.
> 
> Signed-off-by: Eric Sandeen <sandeen@redhat.com>

Looks ok, will pick up for 4.18.
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>

--D

> ---
> 
> diff --git a/fs/xfs/xfs_trace.h b/fs/xfs/xfs_trace.h
> index 8955254..b03a0f4 100644
> --- a/fs/xfs/xfs_trace.h
> +++ b/fs/xfs/xfs_trace.h
> @@ -1750,6 +1750,7 @@
>  		__field(int, namelen)
>  		__field(int, valuelen)
>  		__field(xfs_dahash_t, hashval)
> +		__field(int, flags)
>  		__field(int, op_flags)
>  	),
>  	TP_fast_assign(
> @@ -1760,10 +1761,11 @@
>  		__entry->namelen = args->namelen;
>  		__entry->valuelen = args->valuelen;
>  		__entry->hashval = args->hashval;
> +		__entry->flags = args->flags;
>  		__entry->op_flags = args->op_flags;
>  	),
>  	TP_printk("dev %d:%d ino 0x%llx name %.*s namelen %d valuelen %d "
> -		  "hashval 0x%x op_flags %s",
> +		  "hashval 0x%x flags %s op_flags %s",
>  		  MAJOR(__entry->dev), MINOR(__entry->dev),
>  		  __entry->ino,
>  		  __entry->namelen,
> @@ -1771,6 +1773,7 @@
>  		  __entry->namelen,
>  		  __entry->valuelen,
>  		  __entry->hashval,
> +		  __print_flags(__entry->flags, "|", XFS_ATTR_FLAGS),
>  		  __print_flags(__entry->op_flags, "|", XFS_DA_OP_FLAGS))
>  )
>  
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html