* [PATCH] libertas: Set/clear WPA keys before the WEP ones
@ 2010-01-18 23:19 Samuel Ortiz
2010-01-20 1:48 ` Dan Williams
2010-01-21 17:26 ` [PATCH v2] " Samuel Ortiz
0 siblings, 2 replies; 5+ messages in thread
From: Samuel Ortiz @ 2010-01-18 23:19 UTC (permalink / raw)
To: John W. Linville; +Cc: linux-wireless, Dan Williams
With the v10 firmware running on 8688 HW, clearing WPA keys after setting the
WEP key prevents us from being able to associate with WEP APs.
Swapping the calling order for assoc_helper_wpa_keys() and
assoc_helper_wep_keys fixes that issue.
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
---
drivers/net/wireless/libertas/assoc.c | 15 ++++++++-------
1 files changed, 8 insertions(+), 7 deletions(-)
diff --git a/drivers/net/wireless/libertas/assoc.c b/drivers/net/wireless/libertas/assoc.c
index 5e650f3..fb3dff0 100644
--- a/drivers/net/wireless/libertas/assoc.c
+++ b/drivers/net/wireless/libertas/assoc.c
@@ -2052,13 +2052,6 @@ void lbs_association_worker(struct work_struct *work)
goto out;
}
- if ( test_bit(ASSOC_FLAG_WEP_KEYS, &assoc_req->flags)
- || test_bit(ASSOC_FLAG_WEP_TX_KEYIDX, &assoc_req->flags)) {
- ret = assoc_helper_wep_keys(priv, assoc_req);
- if (ret)
- goto out;
- }
-
if (test_bit(ASSOC_FLAG_SECINFO, &assoc_req->flags)) {
ret = assoc_helper_secinfo(priv, assoc_req);
if (ret)
@@ -2078,6 +2071,14 @@ void lbs_association_worker(struct work_struct *work)
goto out;
}
+ if ( test_bit(ASSOC_FLAG_WEP_KEYS, &assoc_req->flags)
+ || test_bit(ASSOC_FLAG_WEP_TX_KEYIDX, &assoc_req->flags)) {
+ ret = assoc_helper_wep_keys(priv, assoc_req);
+ if (ret)
+ goto out;
+ }
+
+
/* SSID/BSSID should be the _last_ config option set, because they
* trigger the association attempt.
*/
--
1.6.3.3
--
Intel Open Source Technology Centre
http://oss.intel.com/
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH] libertas: Set/clear WPA keys before the WEP ones
2010-01-18 23:19 [PATCH] libertas: Set/clear WPA keys before the WEP ones Samuel Ortiz
@ 2010-01-20 1:48 ` Dan Williams
2010-01-21 17:23 ` Samuel Ortiz
2010-01-21 17:26 ` [PATCH v2] " Samuel Ortiz
1 sibling, 1 reply; 5+ messages in thread
From: Dan Williams @ 2010-01-20 1:48 UTC (permalink / raw)
To: Samuel Ortiz; +Cc: John W. Linville, linux-wireless
On Tue, 2010-01-19 at 00:19 +0100, Samuel Ortiz wrote:
> With the v10 firmware running on 8688 HW, clearing WPA keys after setting the
> WEP key prevents us from being able to associate with WEP APs.
> Swapping the calling order for assoc_helper_wpa_keys() and
> assoc_helper_wep_keys fixes that issue.
>
> Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
Acked-by: Dan Williams <dcbw@redhat.com>
Tested on both sd8686 (v9 fw) and usb8388 (v5 OLPC fw). Switching
between WEP and WPA-PSK on-the-fly works as expected with this patch.
Though maybe we want to add a comment about this specific issue that
says something like:
/* v10 FW wants WPA keys to be set/cleared after WEP key operations,
* otherwise it will fail to correctly associate to WEP networks.
* Other firmware versions don't appear to care.
*/
> ---
> drivers/net/wireless/libertas/assoc.c | 15 ++++++++-------
> 1 files changed, 8 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/net/wireless/libertas/assoc.c b/drivers/net/wireless/libertas/assoc.c
> index 5e650f3..fb3dff0 100644
> --- a/drivers/net/wireless/libertas/assoc.c
> +++ b/drivers/net/wireless/libertas/assoc.c
> @@ -2052,13 +2052,6 @@ void lbs_association_worker(struct work_struct *work)
> goto out;
> }
>
> - if ( test_bit(ASSOC_FLAG_WEP_KEYS, &assoc_req->flags)
> - || test_bit(ASSOC_FLAG_WEP_TX_KEYIDX, &assoc_req->flags)) {
> - ret = assoc_helper_wep_keys(priv, assoc_req);
> - if (ret)
> - goto out;
> - }
> -
> if (test_bit(ASSOC_FLAG_SECINFO, &assoc_req->flags)) {
> ret = assoc_helper_secinfo(priv, assoc_req);
> if (ret)
> @@ -2078,6 +2071,14 @@ void lbs_association_worker(struct work_struct *work)
> goto out;
> }
>
> + if ( test_bit(ASSOC_FLAG_WEP_KEYS, &assoc_req->flags)
> + || test_bit(ASSOC_FLAG_WEP_TX_KEYIDX, &assoc_req->flags)) {
> + ret = assoc_helper_wep_keys(priv, assoc_req);
> + if (ret)
> + goto out;
> + }
> +
> +
> /* SSID/BSSID should be the _last_ config option set, because they
> * trigger the association attempt.
> */
> --
> 1.6.3.3
>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] libertas: Set/clear WPA keys before the WEP ones
2010-01-20 1:48 ` Dan Williams
@ 2010-01-21 17:23 ` Samuel Ortiz
0 siblings, 0 replies; 5+ messages in thread
From: Samuel Ortiz @ 2010-01-21 17:23 UTC (permalink / raw)
To: Dan Williams; +Cc: John W. Linville, linux-wireless
On Tue, Jan 19, 2010 at 05:48:12PM -0800, Dan Williams wrote:
> On Tue, 2010-01-19 at 00:19 +0100, Samuel Ortiz wrote:
> > With the v10 firmware running on 8688 HW, clearing WPA keys after setting the
> > WEP key prevents us from being able to associate with WEP APs.
> > Swapping the calling order for assoc_helper_wpa_keys() and
> > assoc_helper_wep_keys fixes that issue.
> >
> > Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
>
> Acked-by: Dan Williams <dcbw@redhat.com>
>
> Tested on both sd8686 (v9 fw) and usb8388 (v5 OLPC fw). Switching
> between WEP and WPA-PSK on-the-fly works as expected with this patch.
>
> Though maybe we want to add a comment about this specific issue that
> says something like:
>
> /* v10 FW wants WPA keys to be set/cleared after WEP key operations,
> * otherwise it will fail to correctly associate to WEP networks.
> * Other firmware versions don't appear to care.
> */
Yes, I can add these comments. Thanks for testing it, I'll send a new version
of the patch.
Cheers,
Samuel.
> > ---
> > drivers/net/wireless/libertas/assoc.c | 15 ++++++++-------
> > 1 files changed, 8 insertions(+), 7 deletions(-)
> >
> > diff --git a/drivers/net/wireless/libertas/assoc.c b/drivers/net/wireless/libertas/assoc.c
> > index 5e650f3..fb3dff0 100644
> > --- a/drivers/net/wireless/libertas/assoc.c
> > +++ b/drivers/net/wireless/libertas/assoc.c
> > @@ -2052,13 +2052,6 @@ void lbs_association_worker(struct work_struct *work)
> > goto out;
> > }
> >
> > - if ( test_bit(ASSOC_FLAG_WEP_KEYS, &assoc_req->flags)
> > - || test_bit(ASSOC_FLAG_WEP_TX_KEYIDX, &assoc_req->flags)) {
> > - ret = assoc_helper_wep_keys(priv, assoc_req);
> > - if (ret)
> > - goto out;
> > - }
> > -
> > if (test_bit(ASSOC_FLAG_SECINFO, &assoc_req->flags)) {
> > ret = assoc_helper_secinfo(priv, assoc_req);
> > if (ret)
> > @@ -2078,6 +2071,14 @@ void lbs_association_worker(struct work_struct *work)
> > goto out;
> > }
> >
> > + if ( test_bit(ASSOC_FLAG_WEP_KEYS, &assoc_req->flags)
> > + || test_bit(ASSOC_FLAG_WEP_TX_KEYIDX, &assoc_req->flags)) {
> > + ret = assoc_helper_wep_keys(priv, assoc_req);
> > + if (ret)
> > + goto out;
> > + }
> > +
> > +
> > /* SSID/BSSID should be the _last_ config option set, because they
> > * trigger the association attempt.
> > */
> > --
> > 1.6.3.3
> >
>
--
Intel Open Source Technology Centre
http://oss.intel.com/
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v2] libertas: Set/clear WPA keys before the WEP ones
2010-01-18 23:19 [PATCH] libertas: Set/clear WPA keys before the WEP ones Samuel Ortiz
2010-01-20 1:48 ` Dan Williams
@ 2010-01-21 17:26 ` Samuel Ortiz
2010-01-21 17:59 ` John W. Linville
1 sibling, 1 reply; 5+ messages in thread
From: Samuel Ortiz @ 2010-01-21 17:26 UTC (permalink / raw)
To: John W. Linville; +Cc: linux-wireless, Dan Williams
With the v10 firmware running on 8688 HW, clearing WPA keys after setting the
WEP key prevents us from being able to associate with WEP APs.
Swapping the calling order for assoc_helper_wpa_keys() and
assoc_helper_wep_keys() fixes that issue.
Acked-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
---
drivers/net/wireless/libertas/assoc.c | 19 ++++++++++++-------
1 files changed, 12 insertions(+), 7 deletions(-)
diff --git a/drivers/net/wireless/libertas/assoc.c b/drivers/net/wireless/libertas/assoc.c
index 5e650f3..e15a83a 100644
--- a/drivers/net/wireless/libertas/assoc.c
+++ b/drivers/net/wireless/libertas/assoc.c
@@ -2052,13 +2052,6 @@ void lbs_association_worker(struct work_struct *work)
goto out;
}
- if ( test_bit(ASSOC_FLAG_WEP_KEYS, &assoc_req->flags)
- || test_bit(ASSOC_FLAG_WEP_TX_KEYIDX, &assoc_req->flags)) {
- ret = assoc_helper_wep_keys(priv, assoc_req);
- if (ret)
- goto out;
- }
-
if (test_bit(ASSOC_FLAG_SECINFO, &assoc_req->flags)) {
ret = assoc_helper_secinfo(priv, assoc_req);
if (ret)
@@ -2071,6 +2064,10 @@ void lbs_association_worker(struct work_struct *work)
goto out;
}
+ /* v10 FW wants WPA keys to be set/cleared before WEP key operations,
+ * otherwise it will fail to correctly associate to WEP networks.
+ * Other firmware versions don't appear to care.
+ */
if (test_bit(ASSOC_FLAG_WPA_MCAST_KEY, &assoc_req->flags)
|| test_bit(ASSOC_FLAG_WPA_UCAST_KEY, &assoc_req->flags)) {
ret = assoc_helper_wpa_keys(priv, assoc_req);
@@ -2078,6 +2075,14 @@ void lbs_association_worker(struct work_struct *work)
goto out;
}
+ if (test_bit(ASSOC_FLAG_WEP_KEYS, &assoc_req->flags)
+ || test_bit(ASSOC_FLAG_WEP_TX_KEYIDX, &assoc_req->flags)) {
+ ret = assoc_helper_wep_keys(priv, assoc_req);
+ if (ret)
+ goto out;
+ }
+
+
/* SSID/BSSID should be the _last_ config option set, because they
* trigger the association attempt.
*/
--
1.6.3.3
--
Intel Open Source Technology Centre
http://oss.intel.com/
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH v2] libertas: Set/clear WPA keys before the WEP ones
2010-01-21 17:26 ` [PATCH v2] " Samuel Ortiz
@ 2010-01-21 17:59 ` John W. Linville
0 siblings, 0 replies; 5+ messages in thread
From: John W. Linville @ 2010-01-21 17:59 UTC (permalink / raw)
To: Samuel Ortiz; +Cc: linux-wireless, Dan Williams
On Thu, Jan 21, 2010 at 06:26:19PM +0100, Samuel Ortiz wrote:
>
> With the v10 firmware running on 8688 HW, clearing WPA keys after setting the
> WEP key prevents us from being able to associate with WEP APs.
> Swapping the calling order for assoc_helper_wpa_keys() and
> assoc_helper_wep_keys() fixes that issue.
>
> Acked-by: Dan Williams <dcbw@redhat.com>
> Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
I already applied the first version of the patch. Could you send
one w/ just the new changes?
John
--
John W. Linville Someday the world will need a hero, and you
linville@tuxdriver.com might be all we have. Be ready.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2010-01-21 18:00 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-01-18 23:19 [PATCH] libertas: Set/clear WPA keys before the WEP ones Samuel Ortiz
2010-01-20 1:48 ` Dan Williams
2010-01-21 17:23 ` Samuel Ortiz
2010-01-21 17:26 ` [PATCH v2] " Samuel Ortiz
2010-01-21 17:59 ` John W. Linville
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).