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 <[email protected]>
Looks good to me. Feel free to add: Reviewed-by: Jan Kara <[email protected]> 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 <[email protected]> SUSE Labs, CR
