Re: linux-next: Tree for Apr 19 (waketorture.c)

From: Paul E. McKenney
Date: Tue Apr 19 2016 - 16:01:33 EST


On Tue, Apr 19, 2016 at 12:34:10PM -0700, Randy Dunlap wrote:
> On 04/19/16 12:05, Paul E. McKenney wrote:
> > On Tue, Apr 19, 2016 at 10:55:21AM -0700, Randy Dunlap wrote:
> >> On 04/19/16 09:56, Paul E. McKenney wrote:
> >>> On Tue, Apr 19, 2016 at 09:20:24AM -0700, Randy Dunlap wrote:
> >>>> On 04/18/16 22:13, Stephen Rothwell wrote:
> >>>>> Hi all,
> >>>>>
> >>>>> Changes since 20160418:
> >>>>>
> >>>>
> >>>> on x86_64:
> >>>>
> >>>> kernel/built-in.o: In function `wake_torture_stats_print':
> >>>> waketorture.c:(.text+0x2f06d): undefined reference to `trace_clock_global'
> >>>> kernel/built-in.o: In function `wake_torture_waiter':
> >>>> waketorture.c:(.text+0x2f196): undefined reference to `trace_clock_local'
> >>>> waketorture.c:(.text+0x2f1bc): undefined reference to `trace_clock_local'
> >>>> waketorture.c:(.text+0x2f1d4): undefined reference to `trace_clock_local'
> >>>> waketorture.c:(.text+0x2f237): undefined reference to `trace_clock_local'
> >>>> waketorture.c:(.text+0x2f253): undefined reference to `trace_clock_local'
> >>>> kernel/built-in.o:waketorture.c:(.text+0x2f318): more undefined references to `trace_clock_local' follow
> >>>> kernel/built-in.o: In function `wake_torture_init':
> >>>> waketorture.c:(.init.text+0x2b3f): undefined reference to `trace_clock_global'
> >>>>
> >>>>
> >>>> Full randconfig file is attached.
> >>>
> >>> I don't see the .config, could you please send it to me?
> >>>
> >>> Looks like I need to add a few "depends" or "selects" clauses...
> >>
> >> Weird, mail client told me that it was attaching the config file.
> >> Trying again...
> >
> > I know that feeling!
> >
> > Does the following patch help? (The 64BIT is unrelated, feel free
> > to back that part out if you wish.)
>
> Yes, that works for me. Thanks.
>
> Acked-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>

Very good, thank you!

Thanx, Paul

> > ------------------------------------------------------------------------
> >
> > diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
> > index 8644658e75c0..a42570a0fb56 100644
> > --- a/lib/Kconfig.debug
> > +++ b/lib/Kconfig.debug
> > @@ -1434,7 +1434,8 @@ config RCU_TORTURE_TEST_SLOW_CLEANUP_DELAY
> > config WAKE_TORTURE_TEST
> > tristate "Torture test for wakeups and CPU hotplug"
> > depends on DEBUG_KERNEL
> > - depends on X86_64 || ARM64 || PPC64 || MIPS
> > + depends on 64BIT
> > + depends on TRACE_CLOCK
> > select TORTURE_TEST
> > default n
> > help
> >
>
>
>
> --
> ~Randy
>