From: Tejun Heo <tj@kernel.org>
To: Lee Schermerhorn <Lee.Schermerhorn@hp.com>
Cc: torvalds@linux-foundation.org, akpm@linux-foundation.org,
linux-kernel@vger.kernel.org, cl@linux-foundation.org,
mingo@elte.hu, adobriyan@gmail.com, tglx@linutronix.de,
hpa@zytor.com, yi.zhu@intel.com, reinette.chatre@intel.com,
ilw@linux.intel.com, Ingo Molnar <mingo@redhat.com>
Subject: Re: [PATCH 3/4] include cleanup: Update gfp.h and slab.h includes to prepare for breaking implicit slab.h inclusion from percpu.h
Date: Wed, 31 Mar 2010 11:41:34 +0900 [thread overview]
Message-ID: <4BB2B65E.5030905@kernel.org> (raw)
In-Reply-To: <1269959410.16298.20.camel@useless.americas.hpqcorp.net>
Hello,
On 03/30/2010 11:30 PM, Lee Schermerhorn wrote:
> I rebased your series atop the mmotm dated 24march to test my "generic
> percpu numa_*_id()" series. This largish patch [3/4] threw a few [13]
> rejects. I'll list them below. You've said that you're going to submit
> this to linux-next where it might not see the same rejects. I built,
> booted and am using the resulting kernel on x86_64.
This series is already in linux-next from next-20100329. I forgot to
run slabh-sweep.py script on new code in linux-next which led to
various build breakages. Stephen fixed the ones which cause build
failures and I later ran the script on linux-next and sent out patches
to subsystem trees. The good news is there was only one build
breakage from mainline code. All else were new code in linux-next.
So, the conversion seems quite comprehensive.
> Building with allyesconfig, I found 15 additional files in the mmotm
> tree that will need an explicit include of slab.h added. I have
> attached a patch that addresses these.
>
> I have rebased my numa_*_id() series atop this mmotm, less the
> percpu.h/percpu-defs.h hack that led you down this path. That kernel
> builds and boots fine. I'm currently testing with and without
> memoryless nodes. So far, this series looks good.
Great, can you please retry mm once it pulls in linux-next from
20100401? That should have most of necessary fixes.
> drivers/hwmon/tmp102.c | 1 +
> drivers/input/misc/ad714x.c | 1 +
> drivers/input/misc/pcf8574_keypad.c | 1 +
> drivers/leds/leds-input.c | 1 +
> drivers/misc/hwlat_detector.c | 1 +
> drivers/mtd/sm_ftl.c | 1 +
> drivers/vbus/bus-proxy.c | 1 +
> kernel/debug/kdb/kdb_support.c | 1 +
> sound/soc/codecs/twl6040.c | 1 +
> sound/usb/endpoint.c | 1 +
> sound/usb/format.c | 1 +
> sound/usb/helper.c | 1 +
> sound/usb/mixer_quirks.c | 1 +
> sound/usb/pcm.c | 1 +
> sound/usb/quirks.c | 1 +
> 15 files changed, 15 insertions(+)
Most of the above are the ones already fixed in linux-next. Can you
please try out next-20100330?
Thanks.
--
tejun
next prev parent reply other threads:[~2010-03-31 2:42 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-03-24 8:48 [PATCHSET] remove implicit slab.h inclusion from from percpu.h Tejun Heo
2010-03-24 8:48 ` [PATCH 1/4] x86: don't include slab.h from arch/x86/include/asm/pgtable_32.h Tejun Heo
2010-03-24 10:58 ` Pekka Enberg
2010-03-24 19:43 ` Christoph Lameter
2010-03-24 8:48 ` [PATCH 2/4] iwlwifi: don't include iwl-dev.h from iwl-devtrace.h Tejun Heo
2010-03-24 17:55 ` reinette chatre
2010-03-24 8:48 ` [PATCH 3/4] include cleanup: Update gfp.h and slab.h includes to prepare for breaking implicit slab.h inclusion from percpu.h Tejun Heo
2010-03-24 11:00 ` Pekka Enberg
2010-03-24 11:57 ` Alexey Dobriyan
2010-03-24 13:33 ` Tejun Heo
2010-03-24 19:42 ` Christoph Lameter
2010-03-30 14:30 ` Lee Schermerhorn
2010-03-31 2:41 ` Tejun Heo [this message]
2010-03-24 8:48 ` [PATCH 4/4] percpu: don't implicitly include slab.h " Tejun Heo
2010-03-24 19:23 ` Christoph Lameter
2010-03-29 0:50 ` [PATCHSET] remove implicit slab.h inclusion from " Tejun Heo
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=4BB2B65E.5030905@kernel.org \
--to=tj@kernel.org \
--cc=Lee.Schermerhorn@hp.com \
--cc=adobriyan@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=cl@linux-foundation.org \
--cc=hpa@zytor.com \
--cc=ilw@linux.intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=mingo@redhat.com \
--cc=reinette.chatre@intel.com \
--cc=tglx@linutronix.de \
--cc=torvalds@linux-foundation.org \
--cc=yi.zhu@intel.com \
/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.