All of lore.kernel.org
 help / color / mirror / Atom feed
* include/trace/stages/init.h:2:23: warning: 'str__irq_vectors__trace_system_name' defined but not used
@ 2022-05-09 18:08 kernel test robot
  2022-05-10 13:44   ` Steven Rostedt
  0 siblings, 1 reply; 7+ messages in thread
From: kernel test robot @ 2022-05-09 18:08 UTC (permalink / raw)
  To: Steven Rostedt (Google); +Cc: kbuild-all, linux-kernel

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   9be9ed2612b5aedb52a2c240edb1630b6b743cb6
commit: af6b9668e85ffd1502aada8036ccbf4dbd481708 tracing: Move the defines to create TRACE_EVENTS into their own files
date:   8 weeks ago
config: i386-randconfig-s002-20220509 (https://download.01.org/0day-ci/archive/20220510/202205100202.Ag0S76wP-lkp@intel.com/config)
compiler: gcc-11 (Debian 11.2.0-20) 11.2.0
reproduce:
        # apt-get install sparse
        # sparse version: v0.6.4-dirty
        # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=af6b9668e85ffd1502aada8036ccbf4dbd481708
        git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
        git fetch --no-tags linus master
        git checkout af6b9668e85ffd1502aada8036ccbf4dbd481708
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=i386 SHELL=/bin/bash arch/x86/kernel/

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

   In file included from include/trace/trace_events.h:27,
                    from include/trace/define_trace.h:102,
                    from arch/x86/include/asm/trace/irq_vectors.h:383,
                    from arch/x86/kernel/irq.c:27:
>> include/trace/stages/init.h:2:23: warning: 'str__irq_vectors__trace_system_name' defined but not used [-Wunused-const-variable=]
       2 | #define __app__(x, y) str__##x##y
         |                       ^~~~~
   include/trace/stages/init.h:3:21: note: in expansion of macro '__app__'
       3 | #define __app(x, y) __app__(x, y)
         |                     ^~~~~~~
   include/trace/stages/init.h:5:29: note: in expansion of macro '__app'
       5 | #define TRACE_SYSTEM_STRING __app(TRACE_SYSTEM_VAR,__trace_system_name)
         |                             ^~~~~
   include/trace/stages/init.h:8:27: note: in expansion of macro 'TRACE_SYSTEM_STRING'
       8 |         static const char TRACE_SYSTEM_STRING[] =       \
         |                           ^~~~~~~~~~~~~~~~~~~
   include/trace/stages/init.h:11:1: note: in expansion of macro 'TRACE_MAKE_SYSTEM_STR'
      11 | TRACE_MAKE_SYSTEM_STR();
         | ^~~~~~~~~~~~~~~~~~~~~


vim +/str__irq_vectors__trace_system_name +2 include/trace/stages/init.h

   > 2	#define __app__(x, y) str__##x##y
     3	#define __app(x, y) __app__(x, y)
     4	

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: include/trace/stages/init.h:2:23: warning: 'str__irq_vectors__trace_system_name' defined but not used
  2022-05-09 18:08 include/trace/stages/init.h:2:23: warning: 'str__irq_vectors__trace_system_name' defined but not used kernel test robot
@ 2022-05-10 13:44   ` Steven Rostedt
  0 siblings, 0 replies; 7+ messages in thread
From: Steven Rostedt @ 2022-05-10 13:44 UTC (permalink / raw)
  To: kernel test robot; +Cc: kbuild-all, linux-kernel

On Tue, 10 May 2022 02:08:23 +0800
kernel test robot <lkp@intel.com> wrote:

> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head:   9be9ed2612b5aedb52a2c240edb1630b6b743cb6
> commit: af6b9668e85ffd1502aada8036ccbf4dbd481708 tracing: Move the defines to create TRACE_EVENTS into their own files
> date:   8 weeks ago
> config: i386-randconfig-s002-20220509 (https://download.01.org/0day-ci/archive/20220510/202205100202.Ag0S76wP-lkp@intel.com/config)
> compiler: gcc-11 (Debian 11.2.0-20) 11.2.0
> reproduce:
>         # apt-get install sparse
>         # sparse version: v0.6.4-dirty
>         # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=af6b9668e85ffd1502aada8036ccbf4dbd481708
>         git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
>         git fetch --no-tags linus master
>         git checkout af6b9668e85ffd1502aada8036ccbf4dbd481708
>         # save the config file
>         mkdir build_dir && cp config build_dir/.config
>         make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=i386 SHELL=/bin/bash arch/x86/kernel/
> 
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
> 
> All warnings (new ones prefixed by >>):
> 
>    In file included from include/trace/trace_events.h:27,
>                     from include/trace/define_trace.h:102,
>                     from arch/x86/include/asm/trace/irq_vectors.h:383,
>                     from arch/x86/kernel/irq.c:27:

Hmm, when I do:

	$ make arch/x86/kernel/irq.i
	$ astyle arch/x86/kernel/irq.i
	$ grep str__irq_irq_vectors__trace_system_name

  static const char str__irq_vectors__trace_system_name[] = "irq_vectors";
  static struct trace_event_class __attribute__((__used__)) __attribute__((__section__(".ref.data"))) event_class_x86_irq_vector = { .system = str__irq_vectors__trace_system_name, .fields_array = trace_event_fields_x86_irq_vector, .fields = { &(event_class_x86_irq_vector.fields),  &(event_class_x86_irq_vector.fields) }, .raw_init = trace_event_raw_init, .probe = trace_event_raw_event_x86_irq_vector, .reg = trace_event_reg, .perf_probe = perf_trace_x86_irq_vector, };
   static struct trace_event_class __attribute__((__used__)) __attribute__((__section__(".ref.data"))) event_class_vector_config = { .system = str__irq_vectors__trace_system_name, .fields_array = trace_event_fields_vector_config, .fields = { &(event_class_vector_config.fields), &(event_class_vector_config.fields) }, .raw_init = trace_event_raw_init, .probe = trace_event_raw_event_vector_config, .reg = trace_event_reg, .perf_probe = perf_trace_vector_config, };;
[..]

How is it not used?

-- Steve


> >> include/trace/stages/init.h:2:23: warning: 'str__irq_vectors__trace_system_name' defined but not used [-Wunused-const-variable=]  
>        2 | #define __app__(x, y) str__##x##y
>          |                       ^~~~~
>    include/trace/stages/init.h:3:21: note: in expansion of macro '__app__'
>        3 | #define __app(x, y) __app__(x, y)
>          |                     ^~~~~~~
>    include/trace/stages/init.h:5:29: note: in expansion of macro '__app'
>        5 | #define TRACE_SYSTEM_STRING __app(TRACE_SYSTEM_VAR,__trace_system_name)
>          |                             ^~~~~
>    include/trace/stages/init.h:8:27: note: in expansion of macro 'TRACE_SYSTEM_STRING'
>        8 |         static const char TRACE_SYSTEM_STRING[] =       \
>          |                           ^~~~~~~~~~~~~~~~~~~
>    include/trace/stages/init.h:11:1: note: in expansion of macro 'TRACE_MAKE_SYSTEM_STR'
>       11 | TRACE_MAKE_SYSTEM_STR();
>          | ^~~~~~~~~~~~~~~~~~~~~
> 
> 
> vim +/str__irq_vectors__trace_system_name +2 include/trace/stages/init.h
> 
>    > 2	#define __app__(x, y) str__##x##y  
>      3	#define __app(x, y) __app__(x, y)
>      4	
> 


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: include/trace/stages/init.h:2:23: warning: 'str__irq_vectors__trace_system_name' defined but not used
@ 2022-05-10 13:44   ` Steven Rostedt
  0 siblings, 0 replies; 7+ messages in thread
From: Steven Rostedt @ 2022-05-10 13:44 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 3996 bytes --]

On Tue, 10 May 2022 02:08:23 +0800
kernel test robot <lkp@intel.com> wrote:

> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head:   9be9ed2612b5aedb52a2c240edb1630b6b743cb6
> commit: af6b9668e85ffd1502aada8036ccbf4dbd481708 tracing: Move the defines to create TRACE_EVENTS into their own files
> date:   8 weeks ago
> config: i386-randconfig-s002-20220509 (https://download.01.org/0day-ci/archive/20220510/202205100202.Ag0S76wP-lkp(a)intel.com/config)
> compiler: gcc-11 (Debian 11.2.0-20) 11.2.0
> reproduce:
>         # apt-get install sparse
>         # sparse version: v0.6.4-dirty
>         # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=af6b9668e85ffd1502aada8036ccbf4dbd481708
>         git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
>         git fetch --no-tags linus master
>         git checkout af6b9668e85ffd1502aada8036ccbf4dbd481708
>         # save the config file
>         mkdir build_dir && cp config build_dir/.config
>         make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=i386 SHELL=/bin/bash arch/x86/kernel/
> 
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
> 
> All warnings (new ones prefixed by >>):
> 
>    In file included from include/trace/trace_events.h:27,
>                     from include/trace/define_trace.h:102,
>                     from arch/x86/include/asm/trace/irq_vectors.h:383,
>                     from arch/x86/kernel/irq.c:27:

Hmm, when I do:

	$ make arch/x86/kernel/irq.i
	$ astyle arch/x86/kernel/irq.i
	$ grep str__irq_irq_vectors__trace_system_name

  static const char str__irq_vectors__trace_system_name[] = "irq_vectors";
  static struct trace_event_class __attribute__((__used__)) __attribute__((__section__(".ref.data"))) event_class_x86_irq_vector = { .system = str__irq_vectors__trace_system_name, .fields_array = trace_event_fields_x86_irq_vector, .fields = { &(event_class_x86_irq_vector.fields),  &(event_class_x86_irq_vector.fields) }, .raw_init = trace_event_raw_init, .probe = trace_event_raw_event_x86_irq_vector, .reg = trace_event_reg, .perf_probe = perf_trace_x86_irq_vector, };
   static struct trace_event_class __attribute__((__used__)) __attribute__((__section__(".ref.data"))) event_class_vector_config = { .system = str__irq_vectors__trace_system_name, .fields_array = trace_event_fields_vector_config, .fields = { &(event_class_vector_config.fields), &(event_class_vector_config.fields) }, .raw_init = trace_event_raw_init, .probe = trace_event_raw_event_vector_config, .reg = trace_event_reg, .perf_probe = perf_trace_vector_config, };;
[..]

How is it not used?

-- Steve


> >> include/trace/stages/init.h:2:23: warning: 'str__irq_vectors__trace_system_name' defined but not used [-Wunused-const-variable=]  
>        2 | #define __app__(x, y) str__##x##y
>          |                       ^~~~~
>    include/trace/stages/init.h:3:21: note: in expansion of macro '__app__'
>        3 | #define __app(x, y) __app__(x, y)
>          |                     ^~~~~~~
>    include/trace/stages/init.h:5:29: note: in expansion of macro '__app'
>        5 | #define TRACE_SYSTEM_STRING __app(TRACE_SYSTEM_VAR,__trace_system_name)
>          |                             ^~~~~
>    include/trace/stages/init.h:8:27: note: in expansion of macro 'TRACE_SYSTEM_STRING'
>        8 |         static const char TRACE_SYSTEM_STRING[] =       \
>          |                           ^~~~~~~~~~~~~~~~~~~
>    include/trace/stages/init.h:11:1: note: in expansion of macro 'TRACE_MAKE_SYSTEM_STR'
>       11 | TRACE_MAKE_SYSTEM_STR();
>          | ^~~~~~~~~~~~~~~~~~~~~
> 
> 
> vim +/str__irq_vectors__trace_system_name +2 include/trace/stages/init.h
> 
>    > 2	#define __app__(x, y) str__##x##y  
>      3	#define __app(x, y) __app__(x, y)
>      4	
> 

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: include/trace/stages/init.h:2:23: warning: 'str__irq_vectors__trace_system_name' defined but not used
  2022-05-10 13:44   ` Steven Rostedt
@ 2022-05-10 13:48     ` Steven Rostedt
  -1 siblings, 0 replies; 7+ messages in thread
From: Steven Rostedt @ 2022-05-10 13:48 UTC (permalink / raw)
  To: kernel test robot; +Cc: kbuild-all, linux-kernel

On Tue, 10 May 2022 09:44:59 -0400
Steven Rostedt <rostedt@goodmis.org> wrote:

> Hmm, when I do:
> 
> 	$ make arch/x86/kernel/irq.i
> 	$ astyle arch/x86/kernel/irq.i
> 	$ grep str__irq_irq_vectors__trace_system_name
> 
>   static const char str__irq_vectors__trace_system_name[] = "irq_vectors";
>   static struct trace_event_class __attribute__((__used__)) __attribute__((__section__(".ref.data"))) event_class_x86_irq_vector = { .system = str__irq_vectors__trace_system_name, .fields_array = trace_event_fields_x86_irq_vector, .fields = { &(event_class_x86_irq_vector.fields),  &(event_class_x86_irq_vector.fields) }, .raw_init = trace_event_raw_init, .probe = trace_event_raw_event_x86_irq_vector, .reg = trace_event_reg, .perf_probe = perf_trace_x86_irq_vector, };

Or is it because it is used only in initdata?

I guess I can check if it is referenced afterward anywhere, and set it to
initdata as well, if that's the case.

-- Steve

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: include/trace/stages/init.h:2:23: warning: 'str__irq_vectors__trace_system_name' defined but not used
@ 2022-05-10 13:48     ` Steven Rostedt
  0 siblings, 0 replies; 7+ messages in thread
From: Steven Rostedt @ 2022-05-10 13:48 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 956 bytes --]

On Tue, 10 May 2022 09:44:59 -0400
Steven Rostedt <rostedt@goodmis.org> wrote:

> Hmm, when I do:
> 
> 	$ make arch/x86/kernel/irq.i
> 	$ astyle arch/x86/kernel/irq.i
> 	$ grep str__irq_irq_vectors__trace_system_name
> 
>   static const char str__irq_vectors__trace_system_name[] = "irq_vectors";
>   static struct trace_event_class __attribute__((__used__)) __attribute__((__section__(".ref.data"))) event_class_x86_irq_vector = { .system = str__irq_vectors__trace_system_name, .fields_array = trace_event_fields_x86_irq_vector, .fields = { &(event_class_x86_irq_vector.fields),  &(event_class_x86_irq_vector.fields) }, .raw_init = trace_event_raw_init, .probe = trace_event_raw_event_x86_irq_vector, .reg = trace_event_reg, .perf_probe = perf_trace_x86_irq_vector, };

Or is it because it is used only in initdata?

I guess I can check if it is referenced afterward anywhere, and set it to
initdata as well, if that's the case.

-- Steve

^ permalink raw reply	[flat|nested] 7+ messages in thread

* include/trace/stages/init.h:2:23: warning: 'str__irq_vectors__trace_system_name' defined but not used
@ 2024-01-10 17:20 kernel test robot
  0 siblings, 0 replies; 7+ messages in thread
From: kernel test robot @ 2024-01-10 17:20 UTC (permalink / raw)
  To: Steven Rostedt (Google); +Cc: oe-kbuild-all, linux-kernel

Hi Steven,

FYI, the error/warning still remains.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   ab27740f76654ed58dd32ac0ba0031c18a6dea3b
commit: af6b9668e85ffd1502aada8036ccbf4dbd481708 tracing: Move the defines to create TRACE_EVENTS into their own files
date:   1 year, 10 months ago
config: i386-buildonly-randconfig-003-20240106 (https://download.01.org/0day-ci/archive/20240111/202401110124.FXHxWlN9-lkp@intel.com/config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240111/202401110124.FXHxWlN9-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202401110124.FXHxWlN9-lkp@intel.com/

All warnings (new ones prefixed by >>):

   In file included from include/trace/trace_events.h:27,
                    from include/trace/define_trace.h:102,
                    from arch/x86/include/asm/trace/irq_vectors.h:383,
                    from arch/x86/kernel/irq.c:27:
>> include/trace/stages/init.h:2:23: warning: 'str__irq_vectors__trace_system_name' defined but not used [-Wunused-const-variable=]
       2 | #define __app__(x, y) str__##x##y
         |                       ^~~~~
   include/trace/stages/init.h:3:21: note: in expansion of macro '__app__'
       3 | #define __app(x, y) __app__(x, y)
         |                     ^~~~~~~
   include/trace/stages/init.h:5:29: note: in expansion of macro '__app'
       5 | #define TRACE_SYSTEM_STRING __app(TRACE_SYSTEM_VAR,__trace_system_name)
         |                             ^~~~~
   include/trace/stages/init.h:8:20: note: in expansion of macro 'TRACE_SYSTEM_STRING'
       8 |  static const char TRACE_SYSTEM_STRING[] = \
         |                    ^~~~~~~~~~~~~~~~~~~
   include/trace/stages/init.h:11:1: note: in expansion of macro 'TRACE_MAKE_SYSTEM_STR'
      11 | TRACE_MAKE_SYSTEM_STR();
         | ^~~~~~~~~~~~~~~~~~~~~


vim +/str__irq_vectors__trace_system_name +2 include/trace/stages/init.h

   > 2	#define __app__(x, y) str__##x##y
     3	#define __app(x, y) __app__(x, y)
     4	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

^ permalink raw reply	[flat|nested] 7+ messages in thread

* include/trace/stages/init.h:2:23: warning: 'str__irq_vectors__trace_system_name' defined but not used
@ 2022-05-09 18:09 kernel test robot
  0 siblings, 0 replies; 7+ messages in thread
From: kernel test robot @ 2022-05-09 18:09 UTC (permalink / raw)
  To: Steven Rostedt (Google); +Cc: kbuild-all, linux-kernel

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   9be9ed2612b5aedb52a2c240edb1630b6b743cb6
commit: af6b9668e85ffd1502aada8036ccbf4dbd481708 tracing: Move the defines to create TRACE_EVENTS into their own files
date:   8 weeks ago
config: i386-randconfig-s002-20220509 (https://download.01.org/0day-ci/archive/20220510/202205100242.TCazrhkS-lkp@intel.com/config)
compiler: gcc-11 (Debian 11.2.0-20) 11.2.0
reproduce:
        # apt-get install sparse
        # sparse version: v0.6.4-dirty
        # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=af6b9668e85ffd1502aada8036ccbf4dbd481708
        git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
        git fetch --no-tags linus master
        git checkout af6b9668e85ffd1502aada8036ccbf4dbd481708
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=i386 SHELL=/bin/bash arch/x86/kernel/

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

   In file included from include/trace/trace_events.h:27,
                    from include/trace/define_trace.h:102,
                    from arch/x86/include/asm/trace/irq_vectors.h:383,
                    from arch/x86/kernel/irq.c:27:
>> include/trace/stages/init.h:2:23: warning: 'str__irq_vectors__trace_system_name' defined but not used [-Wunused-const-variable=]
       2 | #define __app__(x, y) str__##x##y
         |                       ^~~~~
   include/trace/stages/init.h:3:21: note: in expansion of macro '__app__'
       3 | #define __app(x, y) __app__(x, y)
         |                     ^~~~~~~
   include/trace/stages/init.h:5:29: note: in expansion of macro '__app'
       5 | #define TRACE_SYSTEM_STRING __app(TRACE_SYSTEM_VAR,__trace_system_name)
         |                             ^~~~~
   include/trace/stages/init.h:8:27: note: in expansion of macro 'TRACE_SYSTEM_STRING'
       8 |         static const char TRACE_SYSTEM_STRING[] =       \
         |                           ^~~~~~~~~~~~~~~~~~~
   include/trace/stages/init.h:11:1: note: in expansion of macro 'TRACE_MAKE_SYSTEM_STR'
      11 | TRACE_MAKE_SYSTEM_STR();
         | ^~~~~~~~~~~~~~~~~~~~~


vim +/str__irq_vectors__trace_system_name +2 include/trace/stages/init.h

   > 2	#define __app__(x, y) str__##x##y
     3	#define __app(x, y) __app__(x, y)
     4	

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2024-01-10 17:21 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-09 18:08 include/trace/stages/init.h:2:23: warning: 'str__irq_vectors__trace_system_name' defined but not used kernel test robot
2022-05-10 13:44 ` Steven Rostedt
2022-05-10 13:44   ` Steven Rostedt
2022-05-10 13:48   ` Steven Rostedt
2022-05-10 13:48     ` Steven Rostedt
2022-05-09 18:09 kernel test robot
2024-01-10 17:20 kernel test robot

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.