lkml.org 
[lkml]   [2017]   [Aug]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [tip:WIP.x86/apic 40/43] arch/x86/kernel/irq.o:(__tracepoints+0x170): undefined reference to `trace_resched_ipi_reg'
On Fri, 25 Aug 2017, Thomas Gleixner wrote:

CC + Rostedt

> On Fri, 25 Aug 2017, kbuild test robot wrote:
>
> > Hi Thomas,
> >
> > It's probably a bug fix that unveils the link errors.
> >
> > tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git WIP.x86/apic
> > head: 395b2c62edea5a2bb8a8786392feb12fae5917a7
> > commit: 902f1bb88397dd6769c98b2177bbda134b3a1ef2 [40/43] x86/idt: Deinline setup functions
> > config: i386-randconfig-s0-201734 (attached as .config)
> > compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
> > reproduce:
> > git checkout 902f1bb88397dd6769c98b2177bbda134b3a1ef2
> > # save the attached .config to linux build tree
> > make ARCH=i386
> >
> > All errors (new ones prefixed by >>):
> >
> > >> arch/x86/kernel/irq.o:(__tracepoints+0x170): undefined reference to `trace_resched_ipi_reg'
> > >> arch/x86/kernel/irq.o:(__tracepoints+0x174): undefined reference to `trace_resched_ipi_unreg'
> > arch/x86/kernel/irq.o:(__tracepoints+0x184): undefined reference to `trace_resched_ipi_reg'
> > arch/x86/kernel/irq.o:(__tracepoints+0x188): undefined reference to `trace_resched_ipi_unreg'
> >
>
> That link error is caused by the tracepoint code being stupid enough to
> generate tracepoint stuff in an object file whether they are used or not.
>
> The issue here is that the reschedule ipi tracepoint has an initializer
> function which is only compiled in when CONFIG_SMP=y.
>
> That tracepoint is not used in irq.o at all, but still the tracepoint macro
> hell generates something for it:
>
> objdump -t ../build-386/arch/x86/kernel/irq.o | grep resched
> 00000048 l O _ftrace_events 00000004 __event_reschedule_exit
> 00000a00 l O .data 0000004c event_reschedule_exit
> 0000004c l O _ftrace_events 00000004 __event_reschedule_entry
> 00000a60 l O .data 0000004c event_reschedule_entry
> 00000048 l O __tracepoints_ptrs 00000004 __tracepoint_ptr_reschedule_exit
> 00000188 l O __tracepoints_strings 00000010 __tpstrtab_reschedule_exit
> 0000004c l O __tracepoints_ptrs 00000004 __tracepoint_ptr_reschedule_entry
> 00000198 l O __tracepoints_strings 00000011 __tpstrtab_reschedule_entry
> 0000017c g O __tracepoints 00000014 __tracepoint_reschedule_entry
> 00000000 *UND* 00000000 trace_resched_ipi_reg
> 00000168 g O __tracepoints 00000014 __tracepoint_reschedule_exit
> 00000000 *UND* 00000000 trace_resched_ipi_unreg
>
> Brilliant stuff that. I'll hide the tracepoint behind a CONFIG_SMP ifdef
> for now.
>
> Thanks,
>
> tglx
>

\
 
 \ /
  Last update: 2017-08-25 14:27    [W:0.024 / U:1.380 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site