RE: [Suggestion] PowerPC: kernel: cross compiling issue withallmodconfig

From: Yoder Stuart-B08248
Date: Fri Mar 22 2013 - 15:17:38 EST




> -----Original Message-----
> From: Michael Neuling [mailto:mikey@xxxxxxxxxxx]
> Sent: Thursday, March 21, 2013 6:21 PM
> To: Chen Gang F T; Yoder Stuart-B08248; Kumar Gala
> Cc: Chen Gang; Benjamin Herrenschmidt; sfr@xxxxxxxxxxxxxxxx; paulus@xxxxxxxxx; matt@xxxxxxxxxx;
> imunsie@xxxxxxxxxxx; linuxppc-dev@xxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
> Subject: Re: [Suggestion] PowerPC: kernel: cross compiling issue with allmodconfig
>
> Chen Gang F T <chen.gang.flying.transformer@xxxxxxxxx> wrote:
>
> >
> > it seems:
> > only move slb_miss_realmode to the end, can fix this issue without negative effect.
>
> So this works but uncovers a new bug.
>
> Stuart: if you apply this patch allmodconfig now gets this build error
> on linux-next.
>
> arch/powerpc/kernel/built-in.o: In function `restore_pblist_ptr':
> ftrace.c:(.toc+0xd78): undefined reference to `epapr_ev_idle_start'
> ftrace.c:(.toc+0xd88): undefined reference to `epapr_ev_idle'
> make[1]: *** [vmlinux] Error 1
>
> Reverting your patch below fixes this:
>
> commit f070986a07e514e3b4fc4aef6551b8dffcb19287
> Author: Stuart Yoder <stuart.yoder@xxxxxxxxxxxxx>
> Date: Fri Feb 8 13:22:56 2013 -0600
>
> powerpc: Add paravirt idle loop for 64-bit Book-E
>
> Signed-off-by: Stuart Yoder <stuart.yoder@xxxxxxxxxxxxx>
> Signed-off-by: Kumar Gala <galak@xxxxxxxxxxxxxxxxxxx>
>
> Should we revert your patch or can you send a fix? This is currently
> broken in linux-next (at least next-20130321).

Sent a patch that resolves this. Not completely happy with it
because of the added #ifdefs, but not sure how else to resolve
this. allmodconfig is creating config combinations that don't
happen in a normal build (at least currently)-- 64-bit build
that enables EPAPR_PARAVIRT but not PPC_BOOK3E_64.

Stuart
N‹§²æìr¸›yúèšØb²X¬¶ÇvØ^–)Þ{.nÇ+‰·¥Š{±‘êçzX§¶›¡Ü}©ž²ÆzÚ&j:+v‰¨¾«‘êçzZ+€Ê+zf£¢·hšˆ§~†­†Ûiÿûàz¹®w¥¢¸?™¨è­Ú&¢)ßf”ù^jÇy§m…á@A«a¶Úÿ 0¶ìh®å’i