Re: Re: Build regressions/improvements in v3.4-rc7

2012-05-18 Thread Michael Ellerman
On Thu, 2012-05-17 at 13:42 -0500, Timur Tabi wrote: Benjamin Herrenschmidt wrote: + arch/powerpc/platforms/512x/mpc512x_shared.c: error: 'enum fsl_diu_monitor_port' declared inside parameter list [-Werror]: = 70:9, 84:9, 88:36 + arch/powerpc/platforms/512x/mpc512x_shared.c: error:

Re: Re: Build regressions/improvements in v3.4-rc7

2012-05-17 Thread Timur Tabi
Benjamin Herrenschmidt wrote: Thanks. These are all freescale platforms, the relevant people should be already but I've added Scott and Kumar to the CC list just in case. Seeing how this is my code, I'd say the relevant people were not CC'd. :-) Anyway, I think I see what the problem is, but

Re: Re: Build regressions/improvements in v3.4-rc7

2012-05-17 Thread Tabi Timur-B04825
On Thu, May 17, 2012 at 12:30 PM, Timur Tabi ti...@freescale.com wrote: Anyway, I think I see what the problem is, but it does appear when I use the normal defconfigs.  What .config was being used? I meant to say that it does NOT appear. -- Timur Tabi Linux kernel developer at Freescale

Re: Re: Build regressions/improvements in v3.4-rc7

2012-05-17 Thread Geert Uytterhoeven
On Thu, May 17, 2012 at 7:52 PM, Tabi Timur-B04825 b04...@freescale.com wrote: On Thu, May 17, 2012 at 12:30 PM, Timur Tabi ti...@freescale.com wrote: Anyway, I think I see what the problem is, but it does appear when I use the normal defconfigs.  What .config was being used? I meant to say

Re: Re: Build regressions/improvements in v3.4-rc7

2012-05-17 Thread Timur Tabi
Benjamin Herrenschmidt wrote: + arch/powerpc/platforms/512x/mpc512x_shared.c: error: 'enum fsl_diu_monitor_port' declared inside parameter list [-Werror]: = 70:9, 84:9, 88:36 + arch/powerpc/platforms/512x/mpc512x_shared.c: error: 'return' with a value, in function returning void

Re: Build regressions/improvements in v3.4-rc7

2012-05-17 Thread Timur Tabi
Geert Uytterhoeven wrote: powerpc randconfig Doesn't this generate a different .config every time it's run? Do I keep generating random .config files until the problem shows up? -- Timur Tabi Linux kernel developer at Freescale ___ Linuxppc-dev

Re: Build regressions/improvements in v3.4-rc7

2012-05-17 Thread Scott Wood
On 05/17/2012 01:44 PM, Timur Tabi wrote: Geert Uytterhoeven wrote: powerpc randconfig Doesn't this generate a different .config every time it's run? Do I keep generating random .config files until the problem shows up? Regardless of the config that shows this, the MSR[GS] check should

Re: Build regressions/improvements in v3.4-rc7

2012-05-17 Thread Timur Tabi
Scott Wood wrote: Regardless of the config that shows this, the MSR[GS] check should go away. If you have a /hypervisor node, you already know you're a guest. Fair enough. -- Timur Tabi Linux kernel developer at Freescale ___ Linuxppc-dev mailing

Re: Build regressions/improvements in v3.4-rc7

2012-05-16 Thread Geert Uytterhoeven
On Wed, May 16, 2012 at 10:30 AM, Geert Uytterhoeven ge...@linux-m68k.org wrote: JFYI, when comparing v3.4-rc7 to v3.4-rc6[3], the summaries are:  - build errors: +15/-14 15 regressions: + arch/powerpc/platforms/512x/mpc512x_shared.c: error: 'enum fsl_diu_monitor_port' declared inside

Re: Build regressions/improvements in v3.4-rc7

2012-05-16 Thread James Bottomley
On Wed, 2012-05-16 at 10:47 +0200, Geert Uytterhoeven wrote: On Wed, May 16, 2012 at 10:30 AM, Geert Uytterhoeven ge...@linux-m68k.org wrote: + lib/mpi/generic_mpih-mul1.c: error: inconsistent operand constraints in an 'asm': = 50:70 + lib/mpi/generic_mpih-mul2.c: error: inconsistent

Re: Build regressions/improvements in v3.4-rc7

2012-05-16 Thread Geert Uytterhoeven
On Wed, May 16, 2012 at 4:46 PM, James Bottomley james.bottom...@hansenpartnership.com wrote: On Wed, 2012-05-16 at 10:47 +0200, Geert Uytterhoeven wrote: On Wed, May 16, 2012 at 10:30 AM, Geert Uytterhoeven ge...@linux-m68k.org wrote:   + lib/mpi/generic_mpih-mul1.c: error: inconsistent

Re: Build regressions/improvements in v3.4-rc7

2012-05-16 Thread Kasatkin, Dmitry
On Wed, May 16, 2012 at 5:46 PM, James Bottomley james.bottom...@hansenpartnership.com wrote: On Wed, 2012-05-16 at 10:47 +0200, Geert Uytterhoeven wrote: On Wed, May 16, 2012 at 10:30 AM, Geert Uytterhoeven ge...@linux-m68k.org wrote:   + lib/mpi/generic_mpih-mul1.c: error: inconsistent

Re: Build regressions/improvements in v3.4-rc7

2012-05-16 Thread Benjamin Herrenschmidt
On Wed, 2012-05-16 at 10:47 +0200, Geert Uytterhoeven wrote: On Wed, May 16, 2012 at 10:30 AM, Geert Uytterhoeven ge...@linux-m68k.org wrote: JFYI, when comparing v3.4-rc7 to v3.4-rc6[3], the summaries are: - build errors: +15/-14 15 regressions: +

Re: Build regressions/improvements in v3.4-rc7

2012-05-16 Thread John David Anglin
On 16-May-12, at 10:46 AM, James Bottomley wrote: Wow, lib/mpi/ is a complete horror: it's full of hand crafted asm code. The error in this case appears to be that umul_ppm() is implemented as an xmpyu instruction. That's a floating point instruction. We deliberately compile the kernel with