RE: [PATCH 1/1] atl1c: fix issue of transmit queue 0 timed out

["Huang, Xiong" <xiong@xxxxxxxxxxxxxxxx>]

Understand, thank you !

> -----Original Message-----
> From: Rodriguez, Luis
> Sent: Wednesday, June 27, 2012 4:55
> To: Huang, Xiong
> Cc: Ren, Cloud; davem@xxxxxxxxxxxxx; netdev@xxxxxxxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx; qca-linux-team; nic-devel
> Subject: Re: [PATCH 1/1] atl1c: fix issue of transmit queue 0 timed out
> 
> On Tue, Jun 26, 2012 at 01:41:11PM -0700, Huang, Xiong wrote:
> > Luis
> >    It should be a stable fix, but as Ben Hutchings mentioned in
> > another mail, Maybe, removing netif_stop_queue when cable link down is a
> better choice.
> >
> >    Do you mean we need add 'cc:stable@xxxxxxxxxxxxxxx'  just before 'some
> people report  ...' ?
> 
> Nope, see commit 4f7a67e2dd49fbfba002c453bc24bf00e701cc71
> as an example of how to do this. This is a random commit that has been
> marked as stable.
> 
> commit 4f7a67e2dd49fbfba002c453bc24bf00e701cc71
> Author: Ricardo Martins <rasm@xxxxxxxx>
> Date:   Tue May 22 18:02:03 2012 +0100
> 
>     USB: fix PS3 EHCI systems
> 
>     After commit aaa0ef289afe9186f81e2340114ea413eef0492a "PS3 EHCI
> QH
>     read work-around", Terratec Grabby (em28xx) stopped working with AMD
>     Geode LX 800 (USB controller AMD CS5536). Since this is a PS3 only
>     fix, the following patch adds a conditional block around it.
> 
>     Signed-off-by: Ricardo Martins <rasm@xxxxxxxx>
>     Acked-by: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>
>     Cc: stable <stable@xxxxxxxxxxxxxxx>
>     Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> 
> Sometimes it helps if you specify the oldest stable kernel to apply patches to,
> so for example:
> 
> commit 80b08a8d8829a58b5db14b1417151094cc28face
> Author: Felix Fietkau <nbd@xxxxxxxxxxx>
> Date:   Fri Jun 15 03:04:53 2012 +0200
> 
>     ath9k: fix invalid pointer access in the tx path
> 
>     After setup_frame_info has been called, only info->control.rates is still
>     valid, other control fields have been overwritten by the ath_frame_info
>     data. Move the access to info->control.vif for checking short preamble
>     to setup_frame_info before it gets overwritten.
> 
>     This regression was introduced in commit d47a61aa
>     "ath9k: Fix multi-VIF BSS handling"
> 
>     Signed-off-by: Felix Fietkau <nbd@xxxxxxxxxxx>
>     Reported-by: Thomas Hühn <thomas@xxxxxxxxxxxxxxxxxxxxxxx>
>     Acked-by: Sujith Manoharan <c_manoha@xxxxxxxxxxxxxxxx>
>     Cc: stable@xxxxxxxxxxxxxxx [3.4]
>     Signed-off-by: John W. Linville <linville@xxxxxxxxxxxxx>
> 
> To be clear, this is not a Cc: in the e-mail but instead a Cc line in the commit
> log entry.
> 
>   Luis
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html