RE: [PATCH] SN2 user-MMIO CPU migration

From: Luck, Tony <tony.luck_at_intel.com>
Date: 2006-01-26 12:06:39
Getting real close.  Now I'm down to nitpicking the format
of the e-mail :-)

Andrew Morton's perfect patch document:
 http://www.zipworld.com.au/~akpm/linux/patches/stuff/tpp.txt

says that the changelog comes first, and the random commentary
comes after the "---" line (see paragraph "g").  You have this
backwards, so git attempted to use the wrong bit of text as the
changelog.

> Signed-off-by: Brent Casavant <bcasavan@sgi.com>
> Signed-off-by: Ken Chen <kenneth.w.chen@intel.com>
> Signed-off-by: Brent Casavant <bcasavan@sgi.com>

Perhaps a little presumptious to assume that Ken is ok with
the combined version of his part of this patch (ok, I checked
with Ken, and as it happens he is ok with it).  One signoff from
you would be sufficient.

And now an actual (albeit trivial) problem.  I see a couple of
warnings when building sn2/generic kernels from the new code
that you added:

arch/ia64/sn/kernel/sn2/sn2_smp.c:184: warning: initialization discards qualifiers from pointer target type
arch/ia64/sn/kernel/sn2/sn2_smp.c:185: warning: ISO C90 forbids mixed declarations and code

(compiling with gcc 3.4.3)

-Tony
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Received on Thu Jan 26 12:07:17 2006

This archive was generated by hypermail 2.1.8 : 2006-01-26 12:07:25 EST