All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Gortmaker <paul.gortmaker@windriver.com>
To: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Stephen Rothwell <sfr@canb.auug.org.au>,
	Ingo Molnar <mingo@elte.hu>,
	torvalds@linux-foundation.org, linux-kernel@vger.kernel.org,
	akpm@linux-foundation.org, linux-arch@vger.kernel.org
Subject: Re: [RFC/PULL 00/11] introduce export.h; reduce module.h usage
Date: Tue, 2 Aug 2011 01:02:54 -0400	[thread overview]
Message-ID: <20110802050254.GI11811@windriver.com> (raw)
In-Reply-To: <CAMuHMdUJbZsJ1Q0DM4h3uGVCpe+q=28oxphbLzQ6kW4bn9ZoHA@mail.gmail.com>

[Re: [RFC/PULL 00/11] introduce export.h; reduce module.h usage] On 01/08/2011 (Mon 13:49) Geert Uytterhoeven wrote:

> Hi Paul,
> 
> On Fri, Jul 29, 2011 at 03:20, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> > On Thu, 28 Jul 2011 11:42:29 -0400 Paul Gortmaker <paul.gortmaker@windriver.com> 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@windriver.com> 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.

I've just double checked it again and attached a local defconfig build
log below.

Thanks,
Paul.

paul@yow-lpgnfs-02:~/git/module.h/linux-2.6.git$ make O=../m68k-def -j50 2>&1 |grep 'warning:\|error:'
<stdin>:1522:2: warning: #warning syscall recvmmsg not implemented
<stdin>:1554:2: warning: #warning syscall sendmmsg not implemented
/home/paul/git/module.h/linux-2.6.git/arch/m68k/mvme147/config.c:152:2: warning: #warning check me!
/home/paul/git/module.h/linux-2.6.git/arch/m68k/mvme16x/config.c:261:2: warning: #warning check me!
/home/paul/git/module.h/linux-2.6.git/security/keys/keyctl.c:1521:2: warning: #warning TIF_NOTIFY_RESUME not implemented
/home/paul/git/module.h/linux-2.6.git/kernel/async.c:272:10: warning: 'starttime.tv64' may be used uninitialized in this function
/home/paul/git/module.h/linux-2.6.git/kernel/async.c:123:10: warning: 'calltime.tv64' may be used uninitialized in this function
/home/paul/git/module.h/linux-2.6.git/drivers/md/dm.c:1052:3: warning: statement with no effect
/home/paul/git/module.h/linux-2.6.git/drivers/md/dm.c:1079:3: warning: statement with no effect
/home/paul/git/module.h/linux-2.6.git/drivers/md/dm-table.c:1193:2: warning: statement with no effect
/home/paul/git/module.h/linux-2.6.git/drivers/input/misc/hp_sdc_rtc.c:280:12: warning: 'hp_sdc_rtc_set_rt' defined but not used
/home/paul/git/module.h/linux-2.6.git/drivers/input/misc/hp_sdc_rtc.c:320:12: warning: 'hp_sdc_rtc_set_fhs' defined but not used
/home/paul/git/module.h/linux-2.6.git/drivers/input/misc/hp_sdc_rtc.c:360:12: warning: 'hp_sdc_rtc_set_i8042timer' defined but not used
/home/paul/git/module.h/linux-2.6.git/drivers/input/misc/hp_sdc_rtc.c:112:20: warning: ignoring return value of 'down_interruptible', declared with attribute warn_unused_result
/home/paul/git/module.h/linux-2.6.git/drivers/input/misc/hp_sdc_rtc.c:179:20: warning: ignoring return value of 'down_interruptible', declared with attribute warn_unused_result
/home/paul/git/module.h/linux-2.6.git/drivers/input/misc/hp_sdc_rtc.c:183:20: warning: ignoring return value of 'down_interruptible', declared with attribute warn_unused_result
/home/paul/git/module.h/linux-2.6.git/include/net/netfilter/xt_log.h:50:3: warning: value computed is not used
/home/paul/git/module.h/linux-2.6.git/include/net/netfilter/xt_log.h:50:3: warning: value computed is not used
/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'
paul@yow-lpgnfs-02:~/git/module.h/linux-2.6.git$ ls -l ../m68k-def/vmlinux
-rwxr-xr-x 1 paul paul 4951289 2011-08-02 00:59 ../m68k-def/vmlinux
paul@yow-lpgnfs-02:~/git/module.h/linux-2.6.git$ 

WARNING: multiple messages have this Message-ID (diff)
From: Paul Gortmaker <paul.gortmaker@windriver.com>
To: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Stephen Rothwell <sfr@canb.auug.org.au>,
	Ingo Molnar <mingo@elte.hu>, <torvalds@linux-foundation.org>,
	<linux-kernel@vger.kernel.org>, <akpm@linux-foundation.org>,
	<linux-arch@vger.kernel.org>
Subject: Re: [RFC/PULL 00/11] introduce export.h; reduce module.h usage
Date: Tue, 2 Aug 2011 01:02:54 -0400	[thread overview]
Message-ID: <20110802050254.GI11811@windriver.com> (raw)
In-Reply-To: <CAMuHMdUJbZsJ1Q0DM4h3uGVCpe+q=28oxphbLzQ6kW4bn9ZoHA@mail.gmail.com>

[Re: [RFC/PULL 00/11] introduce export.h; reduce module.h usage] On 01/08/2011 (Mon 13:49) Geert Uytterhoeven wrote:

> Hi Paul,
> 
> On Fri, Jul 29, 2011 at 03:20, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> > On Thu, 28 Jul 2011 11:42:29 -0400 Paul Gortmaker <paul.gortmaker@windriver.com> 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@windriver.com> 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.

I've just double checked it again and attached a local defconfig build
log below.

Thanks,
Paul.

paul@yow-lpgnfs-02:~/git/module.h/linux-2.6.git$ make O=../m68k-def -j50 2>&1 |grep 'warning:\|error:'
<stdin>:1522:2: warning: #warning syscall recvmmsg not implemented
<stdin>:1554:2: warning: #warning syscall sendmmsg not implemented
/home/paul/git/module.h/linux-2.6.git/arch/m68k/mvme147/config.c:152:2: warning: #warning check me!
/home/paul/git/module.h/linux-2.6.git/arch/m68k/mvme16x/config.c:261:2: warning: #warning check me!
/home/paul/git/module.h/linux-2.6.git/security/keys/keyctl.c:1521:2: warning: #warning TIF_NOTIFY_RESUME not implemented
/home/paul/git/module.h/linux-2.6.git/kernel/async.c:272:10: warning: 'starttime.tv64' may be used uninitialized in this function
/home/paul/git/module.h/linux-2.6.git/kernel/async.c:123:10: warning: 'calltime.tv64' may be used uninitialized in this function
/home/paul/git/module.h/linux-2.6.git/drivers/md/dm.c:1052:3: warning: statement with no effect
/home/paul/git/module.h/linux-2.6.git/drivers/md/dm.c:1079:3: warning: statement with no effect
/home/paul/git/module.h/linux-2.6.git/drivers/md/dm-table.c:1193:2: warning: statement with no effect
/home/paul/git/module.h/linux-2.6.git/drivers/input/misc/hp_sdc_rtc.c:280:12: warning: 'hp_sdc_rtc_set_rt' defined but not used
/home/paul/git/module.h/linux-2.6.git/drivers/input/misc/hp_sdc_rtc.c:320:12: warning: 'hp_sdc_rtc_set_fhs' defined but not used
/home/paul/git/module.h/linux-2.6.git/drivers/input/misc/hp_sdc_rtc.c:360:12: warning: 'hp_sdc_rtc_set_i8042timer' defined but not used
/home/paul/git/module.h/linux-2.6.git/drivers/input/misc/hp_sdc_rtc.c:112:20: warning: ignoring return value of 'down_interruptible', declared with attribute warn_unused_result
/home/paul/git/module.h/linux-2.6.git/drivers/input/misc/hp_sdc_rtc.c:179:20: warning: ignoring return value of 'down_interruptible', declared with attribute warn_unused_result
/home/paul/git/module.h/linux-2.6.git/drivers/input/misc/hp_sdc_rtc.c:183:20: warning: ignoring return value of 'down_interruptible', declared with attribute warn_unused_result
/home/paul/git/module.h/linux-2.6.git/include/net/netfilter/xt_log.h:50:3: warning: value computed is not used
/home/paul/git/module.h/linux-2.6.git/include/net/netfilter/xt_log.h:50:3: warning: value computed is not used
/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'
paul@yow-lpgnfs-02:~/git/module.h/linux-2.6.git$ ls -l ../m68k-def/vmlinux
-rwxr-xr-x 1 paul paul 4951289 2011-08-02 00:59 ../m68k-def/vmlinux
paul@yow-lpgnfs-02:~/git/module.h/linux-2.6.git$ 


  reply	other threads:[~2011-08-02  5:03 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-28  5:16 [RFC/PULL 00/11] introduce export.h; reduce module.h usage Paul Gortmaker
2011-07-28  5:16 ` [PATCH 01/11] module.h: split out the EXPORT_SYMBOL for faster compiles Paul Gortmaker
2011-07-28  5:16 ` [PATCH 02/11] sysdev.h: dont include <linux/module.h> for no reason Paul Gortmaker
2011-07-28  5:16 ` [PATCH 03/11] net: inet_timewait_sock doesnt need <linux/module.h> Paul Gortmaker
2011-07-28  5:16 ` [PATCH 04/11] device_cgroup.h: delete needless include <linux/module.h> Paul Gortmaker
2011-07-28  5:16 ` [PATCH 05/11] crypto.h: remove unused crypto_tfm_alg_modname() inline Paul Gortmaker
2011-07-28  5:16 ` [PATCH 06/11] linux/stop_machine.h: fix implicit use of smp.h for smp_processor_id Paul Gortmaker
2011-07-28  5:16 ` [PATCH 07/11] pm_runtime.h: explicitly requires notifier.h Paul Gortmaker
2011-07-28  5:16 ` [PATCH 08/11] uwb.h: fix implicit use of asm/page.h for PAGE_SIZE Paul Gortmaker
2011-07-28  5:16 ` [PATCH 09/11] net: sch_generic remove redundant use of <linux/module.h> Paul Gortmaker
2011-07-28  5:16 ` [PATCH 10/11] include: convert various register fcns to macros to avoid include chaining Paul Gortmaker
2011-07-28  5:16 ` [PATCH 11/11] include: replace linux/module.h with "struct module" wherever possible Paul Gortmaker
2011-07-28  7:15 ` [RFC/PULL 00/11] introduce export.h; reduce module.h usage Ingo Molnar
2011-07-28 15:42   ` Paul Gortmaker
2011-07-28 15:42     ` Paul Gortmaker
2011-07-29  1:20     ` Stephen Rothwell
2011-07-29  1:20       ` Stephen Rothwell
2011-08-01 11:49       ` Geert Uytterhoeven
2011-08-02  5:02         ` Paul Gortmaker [this message]
2011-08-02  5:02           ` Paul Gortmaker
2011-08-02  6:54           ` Geert Uytterhoeven
2011-07-28  9:40 ` Anca Emanuel
2011-07-28 13:19 ` Linus Torvalds
2011-07-28 15:27   ` Paul Gortmaker
2011-07-28 15:27     ` Paul Gortmaker
2011-07-28 19:18     ` Ingo Molnar
2011-08-02 18:30   ` Paul Gortmaker
2011-08-02 18:30     ` Paul Gortmaker
2011-08-08  2:42     ` Mike Frysinger
2011-08-10 19:09       ` Paul Gortmaker
2011-08-10 19:09         ` Paul Gortmaker
2011-08-11  4:34         ` Mike Frysinger
2011-08-11 13:10           ` Paul Gortmaker
2011-08-11 13:10             ` Paul Gortmaker
2011-08-11 17:15             ` Mike Frysinger
2011-08-11 17:20               ` Paul Gortmaker
2011-08-11 17:20                 ` Paul Gortmaker
2011-10-13  2:07 ` Rusty Russell
2011-10-13 21:51   ` Paul Gortmaker
2011-10-13 21:51     ` Paul Gortmaker

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20110802050254.GI11811@windriver.com \
    --to=paul.gortmaker@windriver.com \
    --cc=akpm@linux-foundation.org \
    --cc=geert@linux-m68k.org \
    --cc=linux-arch@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=sfr@canb.auug.org.au \
    --cc=torvalds@linux-foundation.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.