Re: Stable update of linux-2.6
- To: Kamalesh Babulal <kamalesh@linux.vnet.ibm.com>
- Cc: Ben Hutchings <ben@decadent.org.uk>, stable@kernel.org, Debian kernel maintainers <debian-kernel@lists.debian.org>, debian-release@lists.debian.org, "Adam D. Barratt" <adam@adam-barratt.org.uk>, paulmck@linux.vnet.ibm.com, mikey@neuling.org, benh@kernel.crashing.org, anton@samba.org, dann frazier <dannf@dannf.org>
- Subject: Re: Stable update of linux-2.6
- From: Greg KH <greg@kroah.com>
- Date: Tue, 5 Apr 2011 09:46:48 -0700
- Message-id: <[🔎] 20110405164648.GA15097@kroah.com>
- In-reply-to: <[🔎] 20110405162652.GG28246@linux.vnet.ibm.com>
- References: <1301440319.10056.13.camel@localhost> <20110330084109.GA8255@linux.vnet.ibm.com> <[🔎] 20110402172303.GB2611@dannf.org> <[🔎] 20110403075103.GB4822@linux.vnet.ibm.com> <[🔎] 20110404215755.GE3720@dannf.org> <[🔎] 20110405105133.GE28246@linux.vnet.ibm.com> <[🔎] 20110405140830.GA25671@kroah.com> <[🔎] 20110405151150.GF28246@linux.vnet.ibm.com> <[🔎] 20110405154737.GA31285@kroah.com> <[🔎] 20110405162652.GG28246@linux.vnet.ibm.com>
On Tue, Apr 05, 2011 at 09:56:52PM +0530, Kamalesh Babulal wrote:
> Resending the patch with commit id and message included. Please let
> me know, incase the patch needs any other changes to be made.
What is the proper Subject: of this patch?
>
> powerpc: Fix default_machine_crash_shutdown #ifdef build failure
>
> This patch introduces #ifdef bits from below commits, to fix the
> build failure on powerpc caused by crash_kexec_wait_realmode()
> when compiled with !SMP.
>
> Commit: c2be05481f6125254c45b78f334d4dd09c701c82 upstream
>
> powerpc: Fix default_machine_crash_shutdown #ifdef botch
2 subjects?
>
> crash_kexec_wait_realmode() is defined only if CONFIG_PPC_STD_MMU_64
> and CONFIG_SMP, but is called if CONFIG_PPC_STD_MMU_64 even if !CONFIG_SMP.
> Fix the conditional compilation around the invocation.
>
> Commit: b3df895aebe091b1657a42a8c859bd49fc96646b upstream
Wait, have you combined 2 patches upstream into one? Why not just use
the 2?
Also, what's with the indentation, I'm tired of having to edit
changelogs and should NEVER have to do that for stable patches as the
log should be identical to what they are upstream (which doesn't have
indentation if you export them properly, please do so...)
So, care to try again, with proper subject: lines and multiple patches?
thanks,
greg k-h
Reply to: