2009-11-22 14:19:20

by Rafael J. Wysocki

[permalink] [raw]
Subject: Re: [Bug #14267] Disassociating atheros wlan

On Sunday 22 November 2009, Johannes Berg wrote:
> On Sat, 2009-11-21 at 09:38 -0800, Justin P. Mattock wrote:
>
> > I did a bisect yesterday on this, and the results
> > seemed to have worked over here by reverting:
> >
> > 75e6c3b72b3ab01c47629f3fbd0fed4e6550bf3a
> > cfg80211: lower dynamic PS timeout to 100ms
> >
> > if Kristoffer can try reverting this one
> > to see, then we can go from there.
>
> So that just means the driver is broken. Please run
>
> iwconfig wlan0 power off
>
> to fix it then. And the patch below will disable it by default since
> it's broken.
>
> johannes
>
> --- wireless-testing.orig/drivers/net/wireless/ath/ath9k/main.c 2009-11-22 11:44:41.000000000 +0100
> +++ wireless-testing/drivers/net/wireless/ath/ath9k/main.c 2009-11-22 11:45:26.000000000 +0100
> @@ -1893,6 +1893,8 @@ void ath_set_hw_capab(struct ath_softc *
> BIT(NL80211_IFTYPE_ADHOC) |
> BIT(NL80211_IFTYPE_MESH_POINT);
>
> + hw->wiphy->ps_default = false;
> +
> hw->queues = 4;
> hw->max_rates = 4;
> hw->channel_change_time = 5000;

OK, so perhaps this patch should be applied until th ath9k driver is fixed? Luis? John?


2009-11-22 15:21:23

by Kristoffer Ericson

[permalink] [raw]
Subject: Re: [Bug #14267] Disassociating atheros wlan

On Sun, 22 Nov 2009 15:19:54 +0100
"Rafael J. Wysocki" <[email protected]> wrote:

> On Sunday 22 November 2009, Johannes Berg wrote:
> > On Sat, 2009-11-21 at 09:38 -0800, Justin P. Mattock wrote:
> >
> > > I did a bisect yesterday on this, and the results
> > > seemed to have worked over here by reverting:
> > >
> > > 75e6c3b72b3ab01c47629f3fbd0fed4e6550bf3a
> > > cfg80211: lower dynamic PS timeout to 100ms
> > >
> > > if Kristoffer can try reverting this one
> > > to see, then we can go from there.
> >
> > So that just means the driver is broken. Please run
> >
> > iwconfig wlan0 power off
> >
> > to fix it then. And the patch below will disable it by default since
> > it's broken.
> >
> > johannes
> >
> > --- wireless-testing.orig/drivers/net/wireless/ath/ath9k/main.c 2009-11-22 11:44:41.000000000 +0100
> > +++ wireless-testing/drivers/net/wireless/ath/ath9k/main.c 2009-11-22 11:45:26.000000000 +0100
> > @@ -1893,6 +1893,8 @@ void ath_set_hw_capab(struct ath_softc *
> > BIT(NL80211_IFTYPE_ADHOC) |
> > BIT(NL80211_IFTYPE_MESH_POINT);
> >
> > + hw->wiphy->ps_default = false;
> > +
> > hw->queues = 4;
> > hw->max_rates = 4;
> > hw->channel_change_time = 5000;
>
> OK, so perhaps this patch should be applied until th ath9k driver is fixed? Luis? John?

I applied the patch from johannes and it ended my issues atleast. I ran my machine for approx
2 hours without seeing the disconnect issue (which I usually saw after 15-25min). Cant be 100% of course
but so far so good.

I agree with Justin that we could wait until he gets a chance to test it. It should be accepted
after initial merge (and stable trees) after that I think. Rafael thoughts?

Also, we should notify the atheros people so they could start fixing the driver (in a perfect world).

Best wishes
Kristoffer Ericson



--
Kristoffer Ericson <[email protected]>

2009-11-22 16:08:13

by Justin P. Mattock

[permalink] [raw]
Subject: Re: [Bug #14267] Disassociating atheros wlan

Kristoffer Ericson wrote:
> On Sun, 22 Nov 2009 15:19:54 +0100
> "Rafael J. Wysocki"<[email protected]> wrote:
>
>
>> On Sunday 22 November 2009, Johannes Berg wrote:
>>
>>> On Sat, 2009-11-21 at 09:38 -0800, Justin P. Mattock wrote:
>>>
>>>
>>>> I did a bisect yesterday on this, and the results
>>>> seemed to have worked over here by reverting:
>>>>
>>>> 75e6c3b72b3ab01c47629f3fbd0fed4e6550bf3a
>>>> cfg80211: lower dynamic PS timeout to 100ms
>>>>
>>>> if Kristoffer can try reverting this one
>>>> to see, then we can go from there.
>>>>
>>> So that just means the driver is broken. Please run
>>>
>>> iwconfig wlan0 power off
>>>
>>> to fix it then. And the patch below will disable it by default since
>>> it's broken.
>>>
>>> johannes
>>>
>>> --- wireless-testing.orig/drivers/net/wireless/ath/ath9k/main.c 2009-11-22 11:44:41.000000000 +0100
>>> +++ wireless-testing/drivers/net/wireless/ath/ath9k/main.c 2009-11-22 11:45:26.000000000 +0100
>>> @@ -1893,6 +1893,8 @@ void ath_set_hw_capab(struct ath_softc *
>>> BIT(NL80211_IFTYPE_ADHOC) |
>>> BIT(NL80211_IFTYPE_MESH_POINT);
>>>
>>> + hw->wiphy->ps_default = false;
>>> +
>>> hw->queues = 4;
>>> hw->max_rates = 4;
>>> hw->channel_change_time = 5000;
>>>
>> OK, so perhaps this patch should be applied until th ath9k driver is fixed? Luis? John?
>>
>
> I applied the patch from johannes and it ended my issues atleast. I ran my machine for approx
> 2 hours without seeing the disconnect issue (which I usually saw after 15-25min). Cant be 100% of course
> but so far so good.
>
> I agree with Justin that we could wait until he gets a chance to test it. It should be accepted
> after initial merge (and stable trees) after that I think. Rafael thoughts?
>
> Also, we should notify the atheros people so they could start fixing the driver (in a perfect world).
>
> Best wishes
> Kristoffer Ericson
>
>
>
>
I can probably test it out, main problem
is where I'm headed internet connection might
not be strong enough to run any kind of media
(hotel connections are only strong enough to retrieve e-mails
streaming music/movies can be a bit much). luckily
I'm away only for a few days.

Justin P. Mattock