RE: [Suggestion] PowerPC: kernel: cross compiling issue with allmodconfig

Yoder Stuart-B08248
2013-03-22T19:17:34Z

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 1967 bytes --]

> -----Original Message-----
> From: Michael Neuling [mailto:mikey@neuling.org]
> Sent: Thursday, March 21, 2013 6:21 PM
> To: Chen Gang F T; Yoder Stuart-B08248; Kumar Gala
> Cc: Chen Gang; Benjamin Herrenschmidt; sfr@canb.auug.org.au; paulus@samba.org; matt@ozlabs.org;
> imunsie@au1.ibm.com; linuxppc-dev@lists.ozlabs.org; linux-kernel@vger.kernel.org
> Subject: Re: [Suggestion] PowerPC: kernel: cross compiling issue with allmodconfig
>
> Chen Gang F T <chen.gang.flying.transformer@gmail.com> 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@freescale.com>
> 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@freescale.com>
> Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
>
> 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Ç+‰·Ÿ®‰­†+%ŠËÿ±éÝ¶\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dÊ‡Ú™ë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥