Re: [PATCH] tracing: Add an option to show symbols in _text+offset for function profiler

[Google]

On Fri, 19 Sep 2025 13:16:21 -0400
Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:

> On Sat, 20 Sep 2025 01:09:52 +0800
> kernel test robot <lkp@xxxxxxxxx> wrote:
> 
> > Hi Masami,
> > 
> > kernel test robot noticed the following build errors:
> > 
> > [auto build test ERROR on trace/for-next]
> > [also build test ERROR on linus/master v6.17-rc6 next-20250918]
> > [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#_base_tree_information]
> > 
> > url:    https://github.com/intel-lab-lkp/linux/commits/Masami-Hiramatsu-Google/tracing-Add-an-option-to-show-symbols-in-_text-offset-for-function-profiler/20250919-135733
> > base:   https://git.kernel.org/pub/scm/linux/kernel/git/trace/linux-trace for-next
> > patch link:    https://lore.kernel.org/r/175826135058.101165.7219957344129610147.stgit%40devnote2
> > patch subject: [PATCH] tracing: Add an option to show symbols in _text+offset for function profiler
> > config: x86_64-buildonly-randconfig-002-20250919 (https://download.01.org/0day-ci/archive/20250920/202509200000.EJ4InVti-lkp@xxxxxxxxx/config)
> > compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
> > reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250920/202509200000.EJ4InVti-lkp@xxxxxxxxx/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@xxxxxxxxx>
> > | Closes: https://lore.kernel.org/oe-kbuild-all/202509200000.EJ4InVti-lkp@xxxxxxxxx/
> > 
> > All error/warnings (new ones prefixed by >>):
> > 
> >    In file included from kernel/trace/trace_sched_switch.c:15:
> > >> kernel/trace/trace.h:1422:37: warning: left shift count >= width of type [-Wshift-count-overflow]  
> >     1422 | #define C(a, b) TRACE_ITER_##a = (1 << TRACE_ITER_##a##_BIT)
> >          |                                     ^~
> 
> Ouch! We need to make the option enum64 now!

Ah, it is an enum, not macro.

Thanks!

> 
> -- Steve
> 


-- 
Masami Hiramatsu (Google) <mhiramat@xxxxxxxxxx>