linux-kernel - Re: [kbuild-all] Re: include/trace/trace_events.h:261:16: error: function 'trace_event_get_offsets_iwlwifi_dbg' might be a candidate for 'gnu

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20221024103524.43e13ab3@gandalf.local.home>
Date:   Mon, 24 Oct 2022 10:35:24 -0400
From:   Steven Rostedt <rostedt@...dmis.org>
To:     "Chen, Rong A" <rong.a.chen@...el.com>
Cc:     kernel test robot <lkp@...el.com>, kbuild-all@...ts.01.org,
        linux-kernel@...r.kernel.org,
        Gregory Greenman <gregory.greenman@...el.com>,
        linux-wireless@...r.kernel.org
Subject: Re: [kbuild-all] Re: include/trace/trace_events.h:261:16: error:
 function 'trace_event_get_offsets_iwlwifi_dbg' might be a candidate for
 'gnu_printf' format attribute

On Mon, 24 Oct 2022 15:26:00 +0800
"Chen, Rong A" <rong.a.chen@...el.com> wrote:

> On 10/21/2022 10:10 PM, Steven Rostedt wrote:
> > On Fri, 21 Oct 2022 22:07:17 +0800
> > kernel test robot <lkp@...el.com> wrote:
> >   
> >> Hi Steven,
> >>
> >> FYI, the error/warning still remains.  
> > 
> >    WONT FIX
> > 
> > I thought we already discussed getting rid of this warning?  
> 
> Hi Steve,
> 
> Sorry for the noise, the bot doesn't ignore this warning with prefix 
> 'error:', we will fix it.

Ug. Let me guess, is this compiled with -Werror?

Because a warning about a macro that may or may not create printf
formatting, where gcc thinks it needs a gcc printf attribute and warns
about it, is not something to fail the build on.

For any build that fails on warnings, that specific warning needs to be
silenced.

I mean, "-Werror=suggest-attribute=format", really?

-- Steve

> >   
> >>
> >> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> >> head:   e35184f321518acadb681928a016da21a9a20c13
> >> commit: c7c37bb87590886e08d24dec53089f74b89f5fbb tracing/iwlwifi: Use the new __vstring() helper
> >> date:   3 months ago
> >> config: x86_64-sof-customedconfig-edison-defconfig
> >> compiler: gcc-11 (Debian 11.3.0-8) 11.3.0
> >> reproduce (this is a W=1 build):
> >>          # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=c7c37bb87590886e08d24dec53089f74b89f5fbb
> >>          git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
> >>          git fetch --no-tags linus master
> >>          git checkout c7c37bb87590886e08d24dec53089f74b89f5fbb
> >>          # save the config file
> >>          mkdir build_dir && cp config build_dir/.config
> >>          make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/net/wireless/intel/iwlwifi/
> >>
> >> If you fix the issue, kindly add following tag where applicable
> >> | Reported-by: kernel test robot <lkp@...el.com>
> >>
> >> All errors (new ones prefixed by >>):
> >>
> >>     In file included from include/trace/define_trace.h:102,
> >>                      from drivers/net/wireless/intel/iwlwifi/iwl-devtrace-msg.h:71,
> >>                      from drivers/net/wireless/intel/iwlwifi/iwl-devtrace.h:91,
> >>                      from drivers/net/wireless/intel/iwlwifi/iwl-devtrace.c:15:
> >>     drivers/net/wireless/intel/iwlwifi/./iwl-devtrace-msg.h: In function 'trace_event_get_offsets_iwlwifi_dbg':  
> >>>> include/trace/trace_events.h:261:16: error: function 'trace_event_get_offsets_iwlwifi_dbg' might be a candidate for 'gnu_printf' format attribute [-Werror=suggest-attribute=format]  
> >>       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/intel/iwlwifi/./iwl-devtrace-msg.h:49:1: note: in expansion of macro 'TRACE_EVENT'
> >>        49 | TRACE_EVENT(iwlwifi_dbg,
> >>           | ^~~~~~~~~~~