On Fri, Mar 23, 2012 at 11:32:00PM -0700, Brian Norris wrote: > Hi Greg, > > On Fri, Mar 23, 2012 at 10:10 AM, <[email protected]> wrote: > > If you, or anyone else, feels it should not be added to the stable tree, > > please let <[email protected]> know about it. > > Please, do *not* send this patch (or the rest of the series) to > anything pre-3.3. It's 3.3 only, since people were seeing regressions > due to a 3.3-rc patch. This went to stable because we were to late for > the 3.3 release. > > > --- a/drivers/ata/libahci.c > > +++ b/drivers/ata/libahci.c > > @@ -748,6 +749,10 @@ static void ahci_start_port(struct ata_p > > /* enable DMA */ > > ahci_start_engine(ap); > > > > + /* enable DMA */ > > + if (!(hpriv->flags & AHCI_HFLAG_DELAY_ENGINE)) > > + ahci_start_engine(ap); > > (BTW, this is a bad backporting of the patch; you can see that this > might call start_engine() twice, whereas the original patch called it > at most once) > > So to reiterate, do not apply this patch or the rest of its series to > 3.0 or 3.2. It should have been tagged 'Cc: <[email protected]> > [3.3]' or something like that. I'll post a quick reply to the other > patches to make sure you can catch all the right ones.
Now dropped, thanks for letting me know. greg k-h -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
