Re: [stable] [patch 00/17] 2.6.27-stable review

On Thu, Oct 23, 2008 at 06:33:39AM -0400, Josh Boyer wrote:
On Wed, 22 Oct 2008 21:53:45 -0700
Greg KH <greg@xxxxxxxxx> wrote:

On Wed, Oct 22, 2008 at 09:01:26PM -0400, Josh Boyer wrote:
On Sat, Oct 18, 2008 at 11:33:34AM -0700, Greg KH wrote:
This is the start of the stable review cycle for the 2.6.27.3 release.
There are 17 patches in this series, all will be posted as a response
to this one. If anyone has any issues with these being applied, please
let us know. If anyone is a maintainer of the proper subsystem, and
wants to add a Signed-off-by: line to the patch, please respond with it.

These patches are sent out with a number of different people on the
Cc: line. If you wish to be a reviewer, please email stable@xxxxxxxxxx
to add your name to the list. If you want to be off the reviewer list,
also email us.

Responses should be made by Wed, October 22, 2008 19:00:00 UTC.
Anything received after that time might be too late.

OK, I realize I'm late. Apologies in advance for that.

I don't see how patches 3, 16, and 17 really fit into the "stable"
rules. None of them:

"... fix a problem that causes a build error (but not for things
marked CONFIG_BROKEN), an oops, a hang, data corruption, a real
security issue, or some "oh, that's not good" issue. In short,
something critical."

So, are we being a bit more lax on the requirements for the
-stable kernels and I missed the memo, or?

Huh?

Patch 3:
Driver core: Fix cleanup in device_create_vargs().
solves a memory leak on an error path that has every opportunity to
happen in the driver core. Do you think this is not a real bug?

Grr.. Typo on my part. Patch 4 is the one I originally meant:
"Driver Core: Clarify device cleanup." It changes nothing but
comments. I don't think it's a big deal at all, but are documentation
changes also allowed now?

It was a documentation change, fixing the information for a core API
call to be correct and match what the code really does.

It also carried no risk of a regression, and as such, I decided to take
it. If you note, we have also taken other patches that fix up
documentation issues like this in the past, so it was not the first
time.

Was this that big of a deal?

thanks,

greg k-h
--
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/

Relevant Pages

  • Re: [stable] [patch 00/17] 2.6.27-stable review
    ... There are 17 patches in this series, all will be posted as a response ... Driver core: Fix cleanup in device_create_vargs. ... It was a documentation change, fixing the information for a core API ...
    (Linux-Kernel)
  • Re: [stable] [patch 00/17] 2.6.27-stable review
    ... There are 17 patches in this series, all will be posted as a response ... line to the patch, ... Driver core: Fix cleanup in device_create_vargs. ...
    (Linux-Kernel)
  • Re: [PATCH] fix bad locking in drivers/base/driver.c
    ... On Tue, 2005-01-25 at 11:19 -0800, Greg KH wrote: ... > I will add it to my queue of patches for the driver core, ... Also Ingo's RT patches have it since October. ... send the line "unsubscribe linux-kernel" in ...
    (Linux-Kernel)
  • Re: [patch 00/83] 2.6.27.9-stable review
    ... Greg KH a écrit: ... There are 83 patches in this series, all will be posted as a response to ... line to the patch, ... These patches are sent out with a number of different people on the Cc: ...
    (Linux-Kernel)
  • Re: [stable] [patch 00/17] 2.6.27-stable review
    ... There are 17 patches in this series, all will be posted as a response ... line to the patch, ... Driver core: Fix cleanup in device_create_vargs. ...
    (Linux-Kernel)