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 kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 74988C43334 for ; Wed, 6 Jul 2022 09:09:28 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id DC8D76B0071; Wed, 6 Jul 2022 05:09:27 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id D50CC6B0073; Wed, 6 Jul 2022 05:09:27 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id BCB316B0074; Wed, 6 Jul 2022 05:09:27 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id A91BE6B0071 for ; Wed, 6 Jul 2022 05:09:27 -0400 (EDT) Received: from smtpin29.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay10.hostedemail.com (Postfix) with ESMTP id 6C61BE74 for ; Wed, 6 Jul 2022 09:09:27 +0000 (UTC) X-FDA: 79656101574.29.D8B2F01 Received: from mail-qk1-f174.google.com (mail-qk1-f174.google.com [209.85.222.174]) by imf18.hostedemail.com (Postfix) with ESMTP id E50AF1C002D for ; Wed, 6 Jul 2022 09:09:26 +0000 (UTC) Received: by mail-qk1-f174.google.com with SMTP id f14so10619804qkm.0 for ; Wed, 06 Jul 2022 02:09:26 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=message-id:date:mime-version:user-agent:subject:content-language:to :cc:references:from:in-reply-to:content-transfer-encoding; bh=vxiJQMa2a6ScrGhpzgzV+Tp/u0829jq0BqEczNesPFg=; b=o5doXy6hGI66V3VIS5tzxrBUc2XInriDflyet203f8a6lpypa36zQtQtV2M0B4mN+m LyUa6Hlr7zBWOwlrlfa/Vmk6gFSzsJnnGqRdjOURDPzefqTJzGgVi+M1iDn7QSNcAbwA ZxmYxzAO0vicXMCMAgZxEFcshx+08cRa+aeyZ76nqJw6mhaXWtXQ7BDS1XsPjnaO4Fi8 nGQl10+JvuZiY+LJv5uglQFkyzzUHGpxG0A4EE+T56VRjncNJDhhjrH8CdXyrUxJ2SMw eJvghUk/u1iSD923K7V3eWGwg+3PkhBgjFdskqMbDORqT/Y4TCKYYgW1IpAKEkWC3VON AfdA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:cc:references:from:in-reply-to :content-transfer-encoding; bh=vxiJQMa2a6ScrGhpzgzV+Tp/u0829jq0BqEczNesPFg=; b=LRLgidqIqVxTUgzQHcljbz9iclXVrLc6mXfpRn8tXiBEqeU3bU7REQSMl+0UbGp9S/ FrWbvOgkTM0x8zGmE5c28fg05bTCAh7lXt1OgdRGrLifTXHguAu3fTA+io2t0T4GXJ2N c1adSthVKcSCred/MEI+f1GmAKQY7gbJsQBUJ1ZtQ4PaC/fJc5RCazL5Js7j1mqIQtGr JCUTKGPJx0VYFRhhP4T30J46kV5jgEAP9jZ+dpXfcsih72+BIaSVbUaZaHX/5Fu3bKVk TAV0oP1f0UasYD/p4RtpplWdwCFF9ZTK5FDEp3njTr+cmccT0TlijzFQCR0BVavh7iEY gQVQ== X-Gm-Message-State: AJIora/lv94bI2CvNe4SkPNx8WGBFMwNIJS90ePQSmF9ygjpYEPTxTt3 /I6ThI/WcogZ/tb/mUI9p6o= X-Google-Smtp-Source: AGRyM1u05qeTfN+/HiKZW7k0MqRT9HazAjC+mvjUvYovZJgAZylA2VnE3JRJMa+53lspajs/Ln8Qjw== X-Received: by 2002:a05:620a:254d:b0:6ab:84b8:25eb with SMTP id s13-20020a05620a254d00b006ab84b825ebmr26293265qko.383.1657098566068; Wed, 06 Jul 2022 02:09:26 -0700 (PDT) Received: from [10.176.68.61] ([192.19.148.250]) by smtp.gmail.com with ESMTPSA id a21-20020a05620a16d500b006a7502d0070sm27504807qkn.21.2022.07.06.02.09.23 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 06 Jul 2022 02:09:25 -0700 (PDT) Message-ID: <0aa190fb-b761-6114-93c0-347aa5950a2e@gmail.com> Date: Wed, 6 Jul 2022 11:09:22 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.11.0 Subject: Re: [PATCH 04/13] tracing/brcm: Use the new __vstring() helper Content-Language: en-US To: Steven Rostedt , kernel test robot Cc: linux-kernel@vger.kernel.org, kbuild-all@lists.01.org, Ingo Molnar , Andrew Morton , Linux Memory Management List , Franky Lin , Hante Meuleman , Kalle Valo , Eric Dumazet , Jakub Kicinski , Paolo Abeni , linux-wireless@vger.kernel.org, brcm80211-dev-list.pdl@broadcom.com, SHA-cyfmac-dev-list@infineon.com, netdev@vger.kernel.org References: <20220705224749.622796175@goodmis.org> <202207061019.0zRrehFH-lkp@intel.com> <20220705225049.665db869@gandalf.local.home> From: Arend Van Spriel In-Reply-To: <20220705225049.665db869@gandalf.local.home> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1657098567; a=rsa-sha256; cv=none; b=514iGTU20MEpDx4CFx56dJGYqPnaN1xvvq1/Rw/Vi08aLGRiJKv3duLYgVOP0PyeoLxUkx MaKtcN5OFkIFv22TRqpx/u9PIwsw8XagnuZKGGK29ZuGEtBuJWT5id6wJafc/1OAgxhDMH hbZBCq1WaCscZ6Ucvd8wLZX1cLE+2KE= ARC-Authentication-Results: i=1; imf18.hostedemail.com; dkim=pass header.d=gmail.com header.s=20210112 header.b=o5doXy6h; dmarc=pass (policy=none) header.from=gmail.com; spf=pass (imf18.hostedemail.com: domain of aspriel@gmail.com designates 209.85.222.174 as permitted sender) smtp.mailfrom=aspriel@gmail.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1657098567; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=vxiJQMa2a6ScrGhpzgzV+Tp/u0829jq0BqEczNesPFg=; b=mLy/zhrn4m3+6xlhF4vol+GatsN9jk5cSSgz5PDNOhOnNMQGIXA8tHWyYCESog8/zk4xQ1 8qTM1vvcYVNpdOEEk2H2TM/ivBjeZtXhnru3UzUHL26QE2fi/jb/am8NpMIeaCFRLmUtxA br8C2W+IYzO8Y72cj+ka+aPY/DH+KhY= X-Stat-Signature: w3juxga7xy1hmtyacuqtkqafq5em8dpk X-Rspam-User: X-Rspamd-Server: rspam12 X-Rspamd-Queue-Id: E50AF1C002D Authentication-Results: imf18.hostedemail.com; dkim=pass header.d=gmail.com header.s=20210112 header.b=o5doXy6h; dmarc=pass (policy=none) header.from=gmail.com; spf=pass (imf18.hostedemail.com: domain of aspriel@gmail.com designates 209.85.222.174 as permitted sender) smtp.mailfrom=aspriel@gmail.com X-HE-Tag: 1657098566-880163 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On 7/6/2022 4:50 AM, Steven Rostedt wrote: > On Wed, 6 Jul 2022 10:35:50 +0800 > kernel test robot wrote: > >> Hi Steven, >> >> Thank you for the patch! Perhaps something to improve: >> >> [auto build test WARNING on rostedt-trace/for-next] >> [also build test WARNING on wireless-next/main wireless/main linus/master v5.19-rc5 next-20220705] >> [If your patch is applied to the wrong git tree, kindly drop us a note. >> And when submitting patch, we suggest to use '--base' as documented in >> https://git-scm.com/docs/git-format-patch] >> > > >> If you fix the issue, kindly add following tag where applicable >> Reported-by: kernel test robot >> >> All warnings (new ones prefixed by >>): > > OK, let's look at all the warnings. > >> >> In file included from include/trace/define_trace.h:102, >> from drivers/net/wireless/broadcom/brcm80211/brcmfmac/tracepoint.h:133, >> from drivers/net/wireless/broadcom/brcm80211/brcmfmac/tracepoint.c:12: >> drivers/net/wireless/broadcom/brcm80211/brcmfmac/./tracepoint.h: In function 'trace_event_get_offsets_brcmf_err': >>>> include/trace/trace_events.h:261:16: warning: function 'trace_event_get_offsets_brcmf_err' might be a candidate for 'gnu_printf' format attribute [-Wsuggest-attribute=format] > > 1. "might be a candidate for 'gnu_printf' format attribute" > >> 261 | struct trace_event_raw_##call __maybe_unused *entry; \ >> | ^~~~~~~~~~~~~~~~ >> include/trace/trace_events.h:40:9: note: in expansion of macro 'DECLARE_EVENT_CLASS' >> 40 | DECLARE_EVENT_CLASS(name, \ >> | ^~~~~~~~~~~~~~~~~~~ >> drivers/net/wireless/broadcom/brcm80211/brcmfmac/./tracepoint.h:31:1: note: in expansion of macro 'TRACE_EVENT' >> 31 | TRACE_EVENT(brcmf_err, >> | ^~~~~~~~~~~ >> drivers/net/wireless/broadcom/brcm80211/brcmfmac/./tracepoint.h: In function 'trace_event_get_offsets_brcmf_dbg': >>>> include/trace/trace_events.h:261:16: warning: function 'trace_event_get_offsets_brcmf_dbg' might be a candidate for 'gnu_printf' format attribute [-Wsuggest-attribute=format] > > 2. "might be a candidate for 'gnu_printf' format attribute" > >> 261 | struct trace_event_raw_##call __maybe_unused *entry; \ >> | ^~~~~~~~~~~~~~~~ >> include/trace/trace_events.h:40:9: note: in expansion of macro 'DECLARE_EVENT_CLASS' >> 40 | DECLARE_EVENT_CLASS(name, \ >> | ^~~~~~~~~~~~~~~~~~~ >> drivers/net/wireless/broadcom/brcm80211/brcmfmac/./tracepoint.h:45:1: note: in expansion of macro 'TRACE_EVENT' >> 45 | TRACE_EVENT(brcmf_dbg, >> | ^~~~~~~~~~~ >> In file included from include/trace/define_trace.h:102, >> from drivers/net/wireless/broadcom/brcm80211/brcmfmac/tracepoint.h:133, >> from drivers/net/wireless/broadcom/brcm80211/brcmfmac/tracepoint.c:12: >> drivers/net/wireless/broadcom/brcm80211/brcmfmac/./tracepoint.h: In function 'trace_event_raw_event_brcmf_err': >> include/trace/trace_events.h:386:16: warning: function 'trace_event_raw_event_brcmf_err' might be a candidate for 'gnu_printf' format attribute [-Wsuggest-attribute=format] > > 3. "might be a candidate for 'gnu_printf' format attribute" > >> 386 | struct trace_event_raw_##call *entry; \ >> | ^~~~~~~~~~~~~~~~ >> include/trace/trace_events.h:40:9: note: in expansion of macro 'DECLARE_EVENT_CLASS' >> 40 | DECLARE_EVENT_CLASS(name, \ >> | ^~~~~~~~~~~~~~~~~~~ >> drivers/net/wireless/broadcom/brcm80211/brcmfmac/./tracepoint.h:31:1: note: in expansion of macro 'TRACE_EVENT' >> 31 | TRACE_EVENT(brcmf_err, >> | ^~~~~~~~~~~ >> drivers/net/wireless/broadcom/brcm80211/brcmfmac/./tracepoint.h: In function 'trace_event_raw_event_brcmf_dbg': >> include/trace/trace_events.h:386:16: warning: function 'trace_event_raw_event_brcmf_dbg' might be a candidate for 'gnu_printf' format attribute [-Wsuggest-attribute=format] > > 4. "might be a candidate for 'gnu_printf' format attribute" > >> 386 | struct trace_event_raw_##call *entry; \ >> | ^~~~~~~~~~~~~~~~ >> include/trace/trace_events.h:40:9: note: in expansion of macro 'DECLARE_EVENT_CLASS' >> 40 | DECLARE_EVENT_CLASS(name, \ >> | ^~~~~~~~~~~~~~~~~~~ >> drivers/net/wireless/broadcom/brcm80211/brcmfmac/./tracepoint.h:45:1: note: in expansion of macro 'TRACE_EVENT' >> 45 | TRACE_EVENT(brcmf_dbg, >> | ^~~~~~~~~~~ >> In file included from include/trace/define_trace.h:103, >> from drivers/net/wireless/broadcom/brcm80211/brcmfmac/tracepoint.h:133, >> from drivers/net/wireless/broadcom/brcm80211/brcmfmac/tracepoint.c:12: >> drivers/net/wireless/broadcom/brcm80211/brcmfmac/./tracepoint.h: In function 'perf_trace_brcmf_err': >> include/trace/perf.h:64:16: warning: function 'perf_trace_brcmf_err' might be a candidate for 'gnu_printf' format attribute [-Wsuggest-attribute=format] > > 5. "might be a candidate for 'gnu_printf' format attribute" > >> 64 | struct hlist_head *head; \ >> | ^~~~~~~~~~ >> include/trace/trace_events.h:40:9: note: in expansion of macro 'DECLARE_EVENT_CLASS' >> 40 | DECLARE_EVENT_CLASS(name, \ >> | ^~~~~~~~~~~~~~~~~~~ >> drivers/net/wireless/broadcom/brcm80211/brcmfmac/./tracepoint.h:31:1: note: in expansion of macro 'TRACE_EVENT' >> 31 | TRACE_EVENT(brcmf_err, >> | ^~~~~~~~~~~ >> drivers/net/wireless/broadcom/brcm80211/brcmfmac/./tracepoint.h: In function 'perf_trace_brcmf_dbg': >> include/trace/perf.h:64:16: warning: function 'perf_trace_brcmf_dbg' might be a candidate for 'gnu_printf' format attribute [-Wsuggest-attribute=format] > > 6. "might be a candidate for 'gnu_printf' format attribute" > >> 64 | struct hlist_head *head; \ >> | ^~~~~~~~~~ >> include/trace/trace_events.h:40:9: note: in expansion of macro 'DECLARE_EVENT_CLASS' >> 40 | DECLARE_EVENT_CLASS(name, \ >> | ^~~~~~~~~~~~~~~~~~~ >> drivers/net/wireless/broadcom/brcm80211/brcmfmac/./tracepoint.h:45:1: note: in expansion of macro 'TRACE_EVENT' >> 45 | TRACE_EVENT(brcmf_dbg, >> | ^~~~~~~~~~~ >> -- >> In file included from include/trace/define_trace.h:102, >> from drivers/net/wireless/broadcom/brcm80211/brcmsmac/brcms_trace_brcmsmac_msg.h:82, >> from drivers/net/wireless/broadcom/brcm80211/brcmsmac/brcms_trace_events.h:38, >> from drivers/net/wireless/broadcom/brcm80211/brcmsmac/brcms_trace_events.c:22: >> drivers/net/wireless/broadcom/brcm80211/brcmsmac/./brcms_trace_brcmsmac_msg.h: In function 'trace_event_get_offsets_brcms_dbg': >>>> include/trace/trace_events.h:261:16: warning: function 'trace_event_get_offsets_brcms_dbg' might be a candidate for 'gnu_printf' format attribute [-Wsuggest-attribute=format] > > 7. "might be a candidate for 'gnu_printf' format attribute" > >> 261 | struct trace_event_raw_##call __maybe_unused *entry; \ >> | ^~~~~~~~~~~~~~~~ >> include/trace/trace_events.h:40:9: note: in expansion of macro 'DECLARE_EVENT_CLASS' >> 40 | DECLARE_EVENT_CLASS(name, \ >> | ^~~~~~~~~~~~~~~~~~~ >> drivers/net/wireless/broadcom/brcm80211/brcmsmac/./brcms_trace_brcmsmac_msg.h:59:1: note: in expansion of macro 'TRACE_EVENT' >> 59 | TRACE_EVENT(brcms_dbg, >> | ^~~~~~~~~~~ >> In file included from include/trace/define_trace.h:102, >> from drivers/net/wireless/broadcom/brcm80211/brcmsmac/brcms_trace_brcmsmac_msg.h:82, >> from drivers/net/wireless/broadcom/brcm80211/brcmsmac/brcms_trace_events.h:38, >> from drivers/net/wireless/broadcom/brcm80211/brcmsmac/brcms_trace_events.c:22: >> drivers/net/wireless/broadcom/brcm80211/brcmsmac/./brcms_trace_brcmsmac_msg.h: In function 'trace_event_raw_event_brcms_dbg': >> include/trace/trace_events.h:386:16: warning: function 'trace_event_raw_event_brcms_dbg' might be a candidate for 'gnu_printf' format attribute [-Wsuggest-attribute=format] > > 8. "might be a candidate for 'gnu_printf' format attribute" > > >> 386 | struct trace_event_raw_##call *entry; \ >> | ^~~~~~~~~~~~~~~~ >> include/trace/trace_events.h:40:9: note: in expansion of macro 'DECLARE_EVENT_CLASS' >> 40 | DECLARE_EVENT_CLASS(name, \ >> | ^~~~~~~~~~~~~~~~~~~ >> drivers/net/wireless/broadcom/brcm80211/brcmsmac/./brcms_trace_brcmsmac_msg.h:59:1: note: in expansion of macro 'TRACE_EVENT' >> 59 | TRACE_EVENT(brcms_dbg, >> | ^~~~~~~~~~~ >> In file included from include/trace/define_trace.h:103, >> from drivers/net/wireless/broadcom/brcm80211/brcmsmac/brcms_trace_brcmsmac_msg.h:82, >> from drivers/net/wireless/broadcom/brcm80211/brcmsmac/brcms_trace_events.h:38, >> from drivers/net/wireless/broadcom/brcm80211/brcmsmac/brcms_trace_events.c:22: >> drivers/net/wireless/broadcom/brcm80211/brcmsmac/./brcms_trace_brcmsmac_msg.h: In function 'perf_trace_brcms_dbg': >> include/trace/perf.h:64:16: warning: function 'perf_trace_brcms_dbg' might be a candidate for 'gnu_printf' format attribute [-Wsuggest-attribute=format] > > 9. "might be a candidate for 'gnu_printf' format attribute" > >> 64 | struct hlist_head *head; \ >> | ^~~~~~~~~~ >> include/trace/trace_events.h:40:9: note: in expansion of macro 'DECLARE_EVENT_CLASS' >> 40 | DECLARE_EVENT_CLASS(name, \ >> | ^~~~~~~~~~~~~~~~~~~ >> drivers/net/wireless/broadcom/brcm80211/brcmsmac/./brcms_trace_brcmsmac_msg.h:59:1: note: in expansion of macro 'TRACE_EVENT' >> 59 | TRACE_EVENT(brcms_dbg, >> | ^~~~~~~~~~~ >> >> >> vim +261 include/trace/trace_events.h >> >> 55de2c0b5610cb include/trace/trace_events.h Masami Hiramatsu 2021-11-22 253 >> 091ad3658e3c76 include/trace/ftrace.h Ingo Molnar 2009-11-26 254 #undef DECLARE_EVENT_CLASS >> 091ad3658e3c76 include/trace/ftrace.h Ingo Molnar 2009-11-26 255 #define DECLARE_EVENT_CLASS(call, proto, args, tstruct, assign, print) \ >> d0ee8f4a1f5f3d include/trace/trace_events.h Steven Rostedt (Red Hat 2015-05-13 256) static inline notrace int trace_event_get_offsets_##call( \ >> 62323a148fbeb0 include/trace/trace_events.h Steven Rostedt (Red Hat 2015-05-13 257) struct trace_event_data_offsets_##call *__data_offsets, proto) \ >> 7fcb7c472f455d include/trace/ftrace.h Li Zefan 2009-06-01 258 { \ >> 7fcb7c472f455d include/trace/ftrace.h Li Zefan 2009-06-01 259 int __data_size = 0; \ >> 114e7b52dee69c include/trace/ftrace.h Filipe Brandenburger 2014-02-28 260 int __maybe_unused __item_length; \ >> a7237765730a10 include/trace/trace_events.h Steven Rostedt (Red Hat 2015-05-13 @261) struct trace_event_raw_##call __maybe_unused *entry; \ >> 7fcb7c472f455d include/trace/ftrace.h Li Zefan 2009-06-01 262 \ >> 7fcb7c472f455d include/trace/ftrace.h Li Zefan 2009-06-01 263 tstruct; \ >> 7fcb7c472f455d include/trace/ftrace.h Li Zefan 2009-06-01 264 \ >> 7fcb7c472f455d include/trace/ftrace.h Li Zefan 2009-06-01 265 return __data_size; \ >> 7fcb7c472f455d include/trace/ftrace.h Li Zefan 2009-06-01 266 } >> 7fcb7c472f455d include/trace/ftrace.h Li Zefan 2009-06-01 267 >> > > Really? 9 warnings about something that *MIGHT* be a candidate for > gnu_printf format attribute? This is a macro that expanded into something > that could possibly use the printf format, but is nested deep in macro > magic. > > Can we please shut this up? Need a vote? Here it is: +1 Regards, Arend