Re: [RFC/PULL 00/11] introduce export.h; reduce module.h usage

[Geert Uytterhoeven]

On Tue, Aug 2, 2011 at 07:02, Paul Gortmaker
<paul.gortmaker@xxxxxxxxxxxxx> wrote:
> [Re: [RFC/PULL 00/11] introduce export.h; reduce module.h usage] On 01/08/2011 (Mon 13:49) Geert Uytterhoeven wrote:
>> On Fri, Jul 29, 2011 at 03:20, Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> wrote:
>> > On Thu, 28 Jul 2011 11:42:29 -0400 Paul Gortmaker <paul.gortmaker@xxxxxxxxxxxxx> wrote:
>> >> [Re: [RFC/PULL 00/11] introduce export.h; reduce module.h usage] On 28/07/2011 (Thu 09:15) Ingo Molnar wrote:
>> >>
>> >> > * Paul Gortmaker <paul.gortmaker@xxxxxxxxxxxxx> wrote:
>> >> >
>> >> > > I don't think there really is any rocket science or contentious
>> >> > > stuff here. It is a sensible cleanup that adds organization and
>> >> > > speeds up compiles. The RFC I'm hoping for is more about how/when
>> >> > > we want to get this in tree.
>> >> >
>> >> > I'd suggest to stick it into linux-next ASAP, leave there for a few
>> >> > days and after fixing any potential fallout send it Linuswards.
>> >>
>> >> Hi Stephen,
>> >>
>> >> Would you please add this to linux-next?
>> >>
>> >> git://git.kernel.org/pub/scm/linux/kernel/git/paulg/linux.git#module.h-split
>> >
>> > OK, I will add it today.
>>
>> This causes several failures in m68k defconfig (and allmodconfig, FWIW):
>>
>> http://kisskb.ellerman.id.au/kisskb/buildresult/4428037/
>> (http://kisskb.ellerman.id.au/kisskb/buildresult/4414683/)
>>
>> Still recovering from holidays, so no patches attached.
>
> I grabbed a m68k toolchain (and several others I didn't have)
> and expanded the arch support outside of the original testing
> I did on x86/x86_64/mips/arm/powerpc/sparc to include m68k.
>
> So your defconfig is restored green again, once it comes around in
> the build (it appears down in the order) -- and for the modconfig,
> it hasn't been building green for a while, but I still went over
> the log looking for and fixing any module.h related fallout.

Thanks a lot!

> /home/paul/git/module.h/linux-2.6.git/drivers/scsi/libsas/sas_scsi_host.c:117:3: warning: case value '2' not in enumerated type 'enum exec_status'

That's a fishy one for the SCSI people...

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/