From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id C72E7C433F5 for ; Sat, 12 Mar 2022 15:14:13 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231425AbiCLPPP (ORCPT ); Sat, 12 Mar 2022 10:15:15 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:43374 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230322AbiCLPPL (ORCPT ); Sat, 12 Mar 2022 10:15:11 -0500 Received: from ams.source.kernel.org (ams.source.kernel.org [145.40.68.75]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5AA0F14076D; Sat, 12 Mar 2022 07:14:05 -0800 (PST) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id C07DAB80184; Sat, 12 Mar 2022 15:14:03 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 86FF8C340EC; Sat, 12 Mar 2022 15:14:01 +0000 (UTC) Date: Sat, 12 Mar 2022 10:13:59 -0500 From: Steven Rostedt To: Ritesh Harjani Cc: linux-ext4@vger.kernel.org, Jan Kara , "Theodore Ts'o" , Harshad Shirwadkar , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, stable@kernel.org Subject: Re: [PATCHv3 02/10] ext4: Fix ext4_fc_stats trace point Message-ID: <20220312101359.27b713b8@rorschach.local.home> In-Reply-To: References: X-Mailer: Claws Mail 3.17.8 (GTK+ 2.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, 12 Mar 2022 11:09:47 +0530 Ritesh Harjani wrote: > ftrace's __print_symbolic() requires that any enum values used in the > symbol to string translation table be wrapped in a TRACE_DEFINE_ENUM > so that the enum value can be decoded from the ftrace ring buffer by > user space tooling. > > This patch also fixes few other problems found in this trace point. > e.g. dereferencing structures in TP_printk which should not be done > at any cost. > > Also to avoid checkpatch warnings, this patch removes those > whitespaces/tab stops issues. > It is recommend now that when there's a fixes and a reported-by tag, you should include the link to the report when available. Link: https://lore.kernel.org/all/20220221160916.333e6491@rorschach.local.home/ -- Steve > Cc: stable@kernel.org > Fixes: commit aa75f4d3daae ("ext4: main fast-commit commit path") > Reported-by: Steven Rostedt > Signed-off-by: Ritesh Harjani > Reviewed-by: Jan Kara > Reviewed-by: Steven Rostedt (Google) > Reviewed-by: Harshad Shirwadkar > --- > include/trace/events/ext4.h | 78 +++++++++++++++++++++++-------------- > 1 file changed, 49 insertions(+), 29 deletions(-) > > diff --git a/include/trace/events/ext4.h b/include/trace/events/ext4.h > index 19e957b7f941..1a0b7030f72a 100644 > --- a/include/trace/events/ext4.h > +++ b/include/trace/events/ext4.h > @@ -95,6 +95,17 @@ TRACE_DEFINE_ENUM(ES_REFERENCED_B); > { FALLOC_FL_COLLAPSE_RANGE, "COLLAPSE_RANGE"}, \ > { FALLOC_FL_ZERO_RANGE, "ZERO_RANGE"}) > > +TRACE_DEFINE_ENUM(EXT4_FC_REASON_XATTR); > +TRACE_DEFINE_ENUM(EXT4_FC_REASON_CROSS_RENAME); > +TRACE_DEFINE_ENUM(EXT4_FC_REASON_JOURNAL_FLAG_CHANGE); > +TRACE_DEFINE_ENUM(EXT4_FC_REASON_NOMEM); > +TRACE_DEFINE_ENUM(EXT4_FC_REASON_SWAP_BOOT); > +TRACE_DEFINE_ENUM(EXT4_FC_REASON_RESIZE); > +TRACE_DEFINE_ENUM(EXT4_FC_REASON_RENAME_DIR); > +TRACE_DEFINE_ENUM(EXT4_FC_REASON_FALLOC_RANGE); > +TRACE_DEFINE_ENUM(EXT4_FC_REASON_INODE_JOURNAL_DATA); > +TRACE_DEFINE_ENUM(EXT4_FC_REASON_MAX); > + > #define show_fc_reason(reason) \ > __print_symbolic(reason, \ > { EXT4_FC_REASON_XATTR, "XATTR"}, \ > @@ -2723,41 +2734,50 @@ TRACE_EVENT(ext4_fc_commit_stop, > > #define FC_REASON_NAME_STAT(reason) \ > show_fc_reason(reason), \ > - __entry->sbi->s_fc_stats.fc_ineligible_reason_count[reason] > + __entry->fc_ineligible_rc[reason] > > TRACE_EVENT(ext4_fc_stats, > - TP_PROTO(struct super_block *sb), > - > - TP_ARGS(sb), > + TP_PROTO(struct super_block *sb), > > - TP_STRUCT__entry( > - __field(dev_t, dev) > - __field(struct ext4_sb_info *, sbi) > - __field(int, count) > - ), > + TP_ARGS(sb), > > - TP_fast_assign( > - __entry->dev = sb->s_dev; > - __entry->sbi = EXT4_SB(sb); > - ), > + TP_STRUCT__entry( > + __field(dev_t, dev) > + __array(unsigned int, fc_ineligible_rc, EXT4_FC_REASON_MAX) > + __field(unsigned long, fc_commits) > + __field(unsigned long, fc_ineligible_commits) > + __field(unsigned long, fc_numblks) > + ), > > - TP_printk("dev %d:%d fc ineligible reasons:\n" > - "%s:%d, %s:%d, %s:%d, %s:%d, %s:%d, %s:%d, %s:%d, %s:%d, %s:%d; " > - "num_commits:%ld, ineligible: %ld, numblks: %ld", > - MAJOR(__entry->dev), MINOR(__entry->dev), > - FC_REASON_NAME_STAT(EXT4_FC_REASON_XATTR), > - FC_REASON_NAME_STAT(EXT4_FC_REASON_CROSS_RENAME), > - FC_REASON_NAME_STAT(EXT4_FC_REASON_JOURNAL_FLAG_CHANGE), > - FC_REASON_NAME_STAT(EXT4_FC_REASON_NOMEM), > - FC_REASON_NAME_STAT(EXT4_FC_REASON_SWAP_BOOT), > - FC_REASON_NAME_STAT(EXT4_FC_REASON_RESIZE), > - FC_REASON_NAME_STAT(EXT4_FC_REASON_RENAME_DIR), > - FC_REASON_NAME_STAT(EXT4_FC_REASON_FALLOC_RANGE), > - FC_REASON_NAME_STAT(EXT4_FC_REASON_INODE_JOURNAL_DATA), > - __entry->sbi->s_fc_stats.fc_num_commits, > - __entry->sbi->s_fc_stats.fc_ineligible_commits, > - __entry->sbi->s_fc_stats.fc_numblks) > + TP_fast_assign( > + int i; > > + __entry->dev = sb->s_dev; > + for (i = 0; i < EXT4_FC_REASON_MAX; i++) { > + __entry->fc_ineligible_rc[i] = > + EXT4_SB(sb)->s_fc_stats.fc_ineligible_reason_count[i]; > + } > + __entry->fc_commits = EXT4_SB(sb)->s_fc_stats.fc_num_commits; > + __entry->fc_ineligible_commits = > + EXT4_SB(sb)->s_fc_stats.fc_ineligible_commits; > + __entry->fc_numblks = EXT4_SB(sb)->s_fc_stats.fc_numblks; > + ), > + > + TP_printk("dev %d,%d fc ineligible reasons:\n" > + "%s:%u, %s:%u, %s:%u, %s:%u, %s:%u, %s:%u, %s:%u, %s:%u, %s:%u " > + "num_commits:%lu, ineligible: %lu, numblks: %lu", > + MAJOR(__entry->dev), MINOR(__entry->dev), > + FC_REASON_NAME_STAT(EXT4_FC_REASON_XATTR), > + FC_REASON_NAME_STAT(EXT4_FC_REASON_CROSS_RENAME), > + FC_REASON_NAME_STAT(EXT4_FC_REASON_JOURNAL_FLAG_CHANGE), > + FC_REASON_NAME_STAT(EXT4_FC_REASON_NOMEM), > + FC_REASON_NAME_STAT(EXT4_FC_REASON_SWAP_BOOT), > + FC_REASON_NAME_STAT(EXT4_FC_REASON_RESIZE), > + FC_REASON_NAME_STAT(EXT4_FC_REASON_RENAME_DIR), > + FC_REASON_NAME_STAT(EXT4_FC_REASON_FALLOC_RANGE), > + FC_REASON_NAME_STAT(EXT4_FC_REASON_INODE_JOURNAL_DATA), > + __entry->fc_commits, __entry->fc_ineligible_commits, > + __entry->fc_numblks) > ); > > #define DEFINE_TRACE_DENTRY_EVENT(__type) \