On Wed 28-08-24 16:13:59, Julian Sun wrote:
> Currently, the show_inode_state() macro only prints
> part of the state of inode->i_state. Let’s improve it
> to display more of its state.
>
> Signed-off-by: Julian Sun <sunjunchao2870@xxxxxxxxx>
Looks good to me. Feel free to add:
Reviewed-by: Jan Kara <jack@xxxxxxx>
Honza
> ---
> include/trace/events/writeback.h | 10 +++++++++-
> 1 file changed, 9 insertions(+), 1 deletion(-)
>
> diff --git a/include/trace/events/writeback.h b/include/trace/events/writeback.h
> index 54e353c9f919..a261e86e61fa 100644
> --- a/include/trace/events/writeback.h
> +++ b/include/trace/events/writeback.h
> @@ -20,7 +20,15 @@
> {I_CLEAR, "I_CLEAR"}, \
> {I_SYNC, "I_SYNC"}, \
> {I_DIRTY_TIME, "I_DIRTY_TIME"}, \
> - {I_REFERENCED, "I_REFERENCED"} \
> + {I_REFERENCED, "I_REFERENCED"}, \
> + {I_LINKABLE, "I_LINKABLE"}, \
> + {I_WB_SWITCH, "I_WB_SWITCH"}, \
> + {I_OVL_INUSE, "I_OVL_INUSE"}, \
> + {I_CREATING, "I_CREATING"}, \
> + {I_DONTCACHE, "I_DONTCACHE"}, \
> + {I_SYNC_QUEUED, "I_SYNC_QUEUED"}, \
> + {I_PINNING_NETFS_WB, "I_PINNING_NETFS_WB"}, \
> + {I_LRU_ISOLATING, "I_LRU_ISOLATING"} \
> )
>
> /* enums need to be exported to user space */
> --
> 2.39.2
>
--
Jan Kara <jack@xxxxxxxx>
SUSE Labs, CR
