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.

Brian
--
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

Reply via email to