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

[Geert Uytterhoeven]

On Wed, May 16, 2012 at 4:46 PM, James Bottomley
<James.Bottomley@xxxxxxxxxxxxxxxxxxxxx> wrote:
> On Wed, 2012-05-16 at 10:47 +0200, Geert Uytterhoeven wrote:
>> On Wed, May 16, 2012 at 10:30 AM, Geert Uytterhoeven
>> <geert@xxxxxxxxxxxxxx> wrote:
>> Â + lib/mpi/generic_mpih-mul1.c: error: inconsistent operand
>> constraints in an 'asm': Â=> 50:70
>> Â + lib/mpi/generic_mpih-mul2.c: error: inconsistent operand
>> constraints in an 'asm': Â=> 49:70
>> Â + lib/mpi/generic_mpih-mul3.c: error: inconsistent operand
>> constraints in an 'asm': Â=> 49:70
>> Â + lib/mpi/mpih-div.c: error: inconsistent operand constraints in an
>> 'asm': Â=> 135:122, 135:371, 97:122, 106:121, 106:370, 97:371
>>
>> parisc-allmodconfig
>
> 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 floating point disabled because we
> don't want to save and restore the floating point register file on each
> context switch, hence the operand constraints are unsatisfiable.

It also does fishy things on m68k, cfr. https://lkml.org/lkml/2012/3/11/22

> It appears to be completely untested on non-x86 and to have been
> imported via the security tree ... what are we supposed to do with this?
> I thought the general principle was that asm code was really supposed to
> be confined to the arch directories?

Kick it out again, or contain it in staging?

Gr{oetje,eeting}s,

            Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
             Â Â -- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/