On Thu, Mar 07, 2013 at 04:46:30PM +0100, Wojciech Dubowik wrote: > On 03/07/2013 03:59 PM, Felix Fietkau wrote: > >On 2013-03-07 3:31 PM, Wojciech Dubowik wrote: > >>There is a regression introduced by this patch when power save is off on > >>the station for idle checks. > >>I have AR9590 station with rx and tx chain set to 0x1 connected > >>to legacy AP (based on Ar9390) with RF cable and 40dB attenuator. > >> > >>Before this patch in connection polling the station was properly sending > >>null function to check whether AP is still there. After this patch it sends > >>broadcast probe request which is anyway wrong or some 16 or so packets > >>of random data (rarely). It manifests itself in lost connection because > >>there > >>is no ack from AP which is expected for null function. > >> > >>I have been following skb's up to the descriptor setting in ath9k and it was > >>all ok i.e. proper null function with valid addresses. > >> > >>I have been bisecting it twice because it doesn't make much sense but maybe > >>it's a HW issue? > >You're right, it does not make much sense. I can't figure out how this > >patch could possibly change the runtime behavior with tx chainmask set > >to 0x1. Have you tried reverting this patch in a current build to see if > >that fixes the issue? > It does fix it. I will check tomorrow whether it's only AR9590 or also > previous revisions. I will also try with different chainmasks. I will have > to rescrew my setup... Do I need to revert this one? Or is there a new fix coming? -- John W. Linville Someday the world will need a hero, and you linville@xxxxxxxxxxxxx might be all we have. Be ready. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
