2009-09-27 13:18:31

by Arjan van de Ven

[permalink] [raw]
Subject: [bisected] Wireless regression in 2.6.32-git

Hi,

With todays git my laptop fails to associate with my access point.
Bisection points to the commit below, and reverting this one commit on
the HEAD of tree also fixes the issue, so I'm pretty confident that this
commit is to blame.

I have a 4965 wifi card in my laptop, and the network I'm trying to
connect to has no encryption. I'm running Fedora 11 as OS.

I would like to kindly request for this commit to be reverted until a
more permanent solution is found (I'm happy to test any patches)..




94f85853324e02c3a32bc3101f090dc9a3f512b4 is first bad commit
commit 94f85853324e02c3a32bc3101f090dc9a3f512b4
Author: Johannes Berg <[email protected]>
Date: Thu Sep 17 17:15:31 2009 -0700

cfg80211: don't overwrite privacy setting

When cfg80211 is instructed to connect, it always
uses the default WEP key for the privacy setting,
which clearly is wrong when using wpa_supplicant.
Don't overwrite the setting, and rely on it being
false when wpa_supplicant is not running, instead
set it to true when we have keys.

Signed-off-by: Johannes Berg <[email protected]>
Signed-off-by: John W. Linville <[email protected]>

:040000 040000 27fb46273e88eefee373699eb7e3f2923ac0886b
9518ee3e52c8320613cc5eee5ac54aabf082432f M net

--
Arjan van de Ven Intel Open Source Technology Centre
For development, discussion and tips for power savings,
visit http://www.lesswatts.org


2009-09-27 18:46:38

by Johannes Berg

[permalink] [raw]
Subject: Re: [bisected] Wireless regression in 2.6.32-git

On Sun, 2009-09-27 at 20:45 +0200, Arjan van de Ven wrote:

> > Subject: cfg80211: don't set privacy w/o key
> >
> > When wpa_supplicant is used to connect to open networks,
> > it causes the wdev->wext.keys to point to key memory, but
> > that key memory is all empty. Only use privacy when there
> > is a default key to be used.
>
>
> indeed it does
>
> can we get this into mainline soon ?

John's on his way home I suppose.

johannes


Attachments:
signature.asc (801.00 B)
This is a digitally signed message part

2009-09-27 18:45:25

by Arjan van de Ven

[permalink] [raw]
Subject: Re: [bisected] Wireless regression in 2.6.32-git

On Sun, 27 Sep 2009 17:14:04 +0100 (BST)
Hugh Dickins <[email protected]> wrote:

> On Sun, 27 Sep 2009, Arjan van de Ven wrote:
> >
> > With todays git my laptop fails to associate with my access point.
> > Bisection points to the commit below, and reverting this one commit
> > on the HEAD of tree also fixes the issue, so I'm pretty confident
> > that this commit is to blame.
> >
> > I have a 4965 wifi card in my laptop, and the network I'm trying to
> > connect to has no encryption. I'm running Fedora 11 as OS.
> >
> > I would like to kindly request for this commit to be reverted until
> > a more permanent solution is found (I'm happy to test any patches)..
> >
> > 94f85853324e02c3a32bc3101f090dc9a3f512b4 is first bad commit
> > commit 94f85853324e02c3a32bc3101f090dc9a3f512b4
> > Author: Johannes Berg <[email protected]>
> > Date: Thu Sep 17 17:15:31 2009 -0700
> >
> > cfg80211: don't overwrite privacy setting
> >
> > When cfg80211 is instructed to connect, it always
> > uses the default WEP key for the privacy setting,
> > which clearly is wrong when using wpa_supplicant.
> > Don't overwrite the setting, and rely on it being
> > false when wpa_supplicant is not running, instead
> > set it to true when we have keys.
> >
> > Signed-off-by: Johannes Berg <[email protected]>
> > Signed-off-by: John W. Linville <[email protected]>
> >
> > :040000 040000 27fb46273e88eefee373699eb7e3f2923ac0886b
> > 9518ee3e52c8320613cc5eee5ac54aabf082432f M net
>
> I've a different problem with wireless that Johannes is investigating
> for me on linux-wireless; but here's a patch that he pointed me to
> along the way, didn't help my issue but I expect it will help yours...
>
>
> Subject: cfg80211: don't set privacy w/o key
>
> When wpa_supplicant is used to connect to open networks,
> it causes the wdev->wext.keys to point to key memory, but
> that key memory is all empty. Only use privacy when there
> is a default key to be used.


indeed it does

can we get this into mainline soon ?

2009-09-27 16:14:05

by Hugh Dickins

[permalink] [raw]
Subject: Re: [bisected] Wireless regression in 2.6.32-git

On Sun, 27 Sep 2009, Arjan van de Ven wrote:
>
> With todays git my laptop fails to associate with my access point.
> Bisection points to the commit below, and reverting this one commit on
> the HEAD of tree also fixes the issue, so I'm pretty confident that this
> commit is to blame.
>
> I have a 4965 wifi card in my laptop, and the network I'm trying to
> connect to has no encryption. I'm running Fedora 11 as OS.
>
> I would like to kindly request for this commit to be reverted until a
> more permanent solution is found (I'm happy to test any patches)..
>
> 94f85853324e02c3a32bc3101f090dc9a3f512b4 is first bad commit
> commit 94f85853324e02c3a32bc3101f090dc9a3f512b4
> Author: Johannes Berg <[email protected]>
> Date: Thu Sep 17 17:15:31 2009 -0700
>
> cfg80211: don't overwrite privacy setting
>
> When cfg80211 is instructed to connect, it always
> uses the default WEP key for the privacy setting,
> which clearly is wrong when using wpa_supplicant.
> Don't overwrite the setting, and rely on it being
> false when wpa_supplicant is not running, instead
> set it to true when we have keys.
>
> Signed-off-by: Johannes Berg <[email protected]>
> Signed-off-by: John W. Linville <[email protected]>
>
> :040000 040000 27fb46273e88eefee373699eb7e3f2923ac0886b
> 9518ee3e52c8320613cc5eee5ac54aabf082432f M net

I've a different problem with wireless that Johannes is investigating
for me on linux-wireless; but here's a patch that he pointed me to
along the way, didn't help my issue but I expect it will help yours...


Subject: cfg80211: don't set privacy w/o key

When wpa_supplicant is used to connect to open networks,
it causes the wdev->wext.keys to point to key memory, but
that key memory is all empty. Only use privacy when there
is a default key to be used.

Signed-off-by: Johannes Berg <[email protected]>
---
net/wireless/wext-sme.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)

--- wireless-testing.orig/net/wireless/wext-sme.c 2009-09-24 08:51:14.000000000 +0200
+++ wireless-testing/net/wireless/wext-sme.c 2009-09-24 08:57:01.000000000 +0200
@@ -30,7 +30,8 @@ int cfg80211_mgd_wext_connect(struct cfg
if (wdev->wext.keys) {
wdev->wext.keys->def = wdev->wext.default_key;
wdev->wext.keys->defmgmt = wdev->wext.default_mgmt_key;
- wdev->wext.connect.privacy = true;
+ if (wdev->wext.default_key != -1)
+ wdev->wext.connect.privacy = true;
}

if (!wdev->wext.connect.ssid_len)

2009-09-28 14:00:55

by John W. Linville

[permalink] [raw]
Subject: Re: [bisected] Wireless regression in 2.6.32-git

On Sun, Sep 27, 2009 at 08:46:39PM +0200, Johannes Berg wrote:
> On Sun, 2009-09-27 at 20:45 +0200, Arjan van de Ven wrote:
>
> > > Subject: cfg80211: don't set privacy w/o key
> > >
> > > When wpa_supplicant is used to connect to open networks,
> > > it causes the wdev->wext.keys to point to key memory, but
> > > that key memory is all empty. Only use privacy when there
> > > is a default key to be used.
> >
> >
> > indeed it does
> >
> > can we get this into mainline soon ?
>
> John's on his way home I suppose.

Yes, I'll get to it today -- sorry for the delays related to my travel!

John
--
John W. Linville Someday the world will need a hero, and you
[email protected] might be all we have. Be ready.

2009-09-27 13:31:42

by Arjan van de Ven

[permalink] [raw]
Subject: Re: [bisected] Wireless regression in 2.6.32-git

On Sun, 27 Sep 2009 15:24:10 +0200
Maciej Rutecki <[email protected]> wrote:

> Did You have similar messages in dmesg like this:
>
> http://bugzilla.intellinuxwireless.org/show_bug.cgi?id=2089
> (use WPA)

nope I don't get a Reason 6 disconnect...

(I get a reason 3 local choice, but that's true for working and
non-working both)


--
Arjan van de Ven Intel Open Source Technology Centre
For development, discussion and tips for power savings,
visit http://www.lesswatts.org

2009-09-27 13:24:08

by Maciej Rutecki

[permalink] [raw]
Subject: Re: [bisected] Wireless regression in 2.6.32-git

Did You have similar messages in dmesg like this:

http://bugzilla.intellinuxwireless.org/show_bug.cgi?id=2089
(use WPA)
?

I try make sure, that that result of my bisection is correct.
--
Maciej Rutecki
http://www.maciek.unixy.pl