* [PATCH] ath9k: Don't delete key from hw if using sw-crypt.
@ 2012-04-02 20:41 greearb
2012-04-03 13:16 ` Felix Fietkau
0 siblings, 1 reply; 6+ messages in thread
From: greearb @ 2012-04-02 20:41 UTC (permalink / raw)
To: linux-wireless; +Cc: Ben Greear
From: Ben Greear <greearb@candelatech.com>
This spams logs if we are using large numbers of stations,
for instance:
ath: wiphy0: keyreset: keycache entry 228 out of range
ath: wiphy0: keyreset: keycache entry 228 out of range
Signed-off-by: Ben Greear <greearb@candelatech.com>
---
:100644 100644 917032a... 65e7281... M drivers/net/wireless/ath/ath9k/main.c
drivers/net/wireless/ath/ath9k/main.c | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/drivers/net/wireless/ath/ath9k/main.c b/drivers/net/wireless/ath/ath9k/main.c
index 917032a..65e7281 100644
--- a/drivers/net/wireless/ath/ath9k/main.c
+++ b/drivers/net/wireless/ath/ath9k/main.c
@@ -1771,6 +1771,9 @@ static void ath9k_del_ps_key(struct ath_softc *sc,
if (!an->ps_key)
return;
+ if (ath9k_modparam_nohwcrypt)
+ return;
+
ath_key_delete(common, &ps_key);
}
--
1.7.3.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] ath9k: Don't delete key from hw if using sw-crypt.
2012-04-02 20:41 [PATCH] ath9k: Don't delete key from hw if using sw-crypt greearb
@ 2012-04-03 13:16 ` Felix Fietkau
2012-04-03 14:44 ` Ben Greear
0 siblings, 1 reply; 6+ messages in thread
From: Felix Fietkau @ 2012-04-03 13:16 UTC (permalink / raw)
To: greearb; +Cc: linux-wireless
On 2012-04-02 10:41 PM, greearb@candelatech.com wrote:
> From: Ben Greear <greearb@candelatech.com>
>
> This spams logs if we are using large numbers of stations,
> for instance:
>
> ath: wiphy0: keyreset: keycache entry 228 out of range
> ath: wiphy0: keyreset: keycache entry 228 out of range
>
> Signed-off-by: Ben Greear <greearb@candelatech.com>
> ---
> :100644 100644 917032a... 65e7281... M drivers/net/wireless/ath/ath9k/main.c
> drivers/net/wireless/ath/ath9k/main.c | 3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/net/wireless/ath/ath9k/main.c b/drivers/net/wireless/ath/ath9k/main.c
> index 917032a..65e7281 100644
> --- a/drivers/net/wireless/ath/ath9k/main.c
> +++ b/drivers/net/wireless/ath/ath9k/main.c
> @@ -1771,6 +1771,9 @@ static void ath9k_del_ps_key(struct ath_softc *sc,
> if (!an->ps_key)
> return;
>
> + if (ath9k_modparam_nohwcrypt)
> + return;
> +
This looks wrong to me. ps_key is used for better powersave handling of
both unencrypted and encrypted stations (using the keycache only for tx
filtering on unencrypted stations).
- Felix
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ath9k: Don't delete key from hw if using sw-crypt.
2012-04-03 13:16 ` Felix Fietkau
@ 2012-04-03 14:44 ` Ben Greear
2012-04-03 16:05 ` Felix Fietkau
0 siblings, 1 reply; 6+ messages in thread
From: Ben Greear @ 2012-04-03 14:44 UTC (permalink / raw)
To: Felix Fietkau; +Cc: linux-wireless
On 04/03/2012 06:16 AM, Felix Fietkau wrote:
> On 2012-04-02 10:41 PM, greearb@candelatech.com wrote:
>> From: Ben Greear<greearb@candelatech.com>
>>
>> This spams logs if we are using large numbers of stations,
>> for instance:
>>
>> ath: wiphy0: keyreset: keycache entry 228 out of range
>> ath: wiphy0: keyreset: keycache entry 228 out of range
>>
>> Signed-off-by: Ben Greear<greearb@candelatech.com>
>> ---
>> :100644 100644 917032a... 65e7281... M drivers/net/wireless/ath/ath9k/main.c
>> drivers/net/wireless/ath/ath9k/main.c | 3 +++
>> 1 files changed, 3 insertions(+), 0 deletions(-)
>>
>> diff --git a/drivers/net/wireless/ath/ath9k/main.c b/drivers/net/wireless/ath/ath9k/main.c
>> index 917032a..65e7281 100644
>> --- a/drivers/net/wireless/ath/ath9k/main.c
>> +++ b/drivers/net/wireless/ath/ath9k/main.c
>> @@ -1771,6 +1771,9 @@ static void ath9k_del_ps_key(struct ath_softc *sc,
>> if (!an->ps_key)
>> return;
>>
>> + if (ath9k_modparam_nohwcrypt)
>> + return;
>> +
> This looks wrong to me. ps_key is used for better powersave handling of
> both unencrypted and encrypted stations (using the keycache only for tx
> filtering on unencrypted stations).
Ok..should we just remove the spammy printk when the keycache entry
is out of range then?
THanks,
Ben
>
> - Felix
--
Ben Greear <greearb@candelatech.com>
Candela Technologies Inc http://www.candelatech.com
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ath9k: Don't delete key from hw if using sw-crypt.
2012-04-03 14:44 ` Ben Greear
@ 2012-04-03 16:05 ` Felix Fietkau
2012-04-03 16:09 ` Ben Greear
0 siblings, 1 reply; 6+ messages in thread
From: Felix Fietkau @ 2012-04-03 16:05 UTC (permalink / raw)
To: Ben Greear; +Cc: Felix Fietkau, linux-wireless@vger.kernel.org
On 03.04.2012, at 16:44, Ben Greear <greearb@candelatech.com> wrote:
> On 04/03/2012 06:16 AM, Felix Fietkau wrote:
>> On 2012-04-02 10:41 PM, greearb@candelatech.com wrote:
>>> From: Ben Greear<greearb@candelatech.com>
>>>
>>> This spams logs if we are using large numbers of stations,
>>> for instance:
>>>
>>> ath: wiphy0: keyreset: keycache entry 228 out of range
>>> ath: wiphy0: keyreset: keycache entry 228 out of range
>>>
>>> Signed-off-by: Ben Greear<greearb@candelatech.com>
>>> ---
>>> :100644 100644 917032a... 65e7281... M drivers/net/wireless/ath/ath9k/main.c
>>> drivers/net/wireless/ath/ath9k/main.c | 3 +++
>>> 1 files changed, 3 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/drivers/net/wireless/ath/ath9k/main.c b/drivers/net/wireless/ath/ath9k/main.c
>>> index 917032a..65e7281 100644
>>> --- a/drivers/net/wireless/ath/ath9k/main.c
>>> +++ b/drivers/net/wireless/ath/ath9k/main.c
>>> @@ -1771,6 +1771,9 @@ static void ath9k_del_ps_key(struct ath_softc *sc,
>>> if (!an->ps_key)
>>> return;
>>>
>>> + if (ath9k_modparam_nohwcrypt)
>>> + return;
>>> +
>> This looks wrong to me. ps_key is used for better powersave handling of
>> both unencrypted and encrypted stations (using the keycache only for tx
>> filtering on unencrypted stations).
>
> Ok..should we just remove the spammy printk when the keycache entry
> is out of range then?
I'd say we should figure out why it triggers at all.
- Felix
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ath9k: Don't delete key from hw if using sw-crypt.
2012-04-03 16:05 ` Felix Fietkau
@ 2012-04-03 16:09 ` Ben Greear
2012-04-04 11:56 ` Felix Fietkau
0 siblings, 1 reply; 6+ messages in thread
From: Ben Greear @ 2012-04-03 16:09 UTC (permalink / raw)
To: Felix Fietkau; +Cc: Felix Fietkau, linux-wireless@vger.kernel.org
On 04/03/2012 09:05 AM, Felix Fietkau wrote:
>
> On 03.04.2012, at 16:44, Ben Greear<greearb@candelatech.com> wrote:
>
>> On 04/03/2012 06:16 AM, Felix Fietkau wrote:
>>> On 2012-04-02 10:41 PM, greearb@candelatech.com wrote:
>>>> From: Ben Greear<greearb@candelatech.com>
>>>>
>>>> This spams logs if we are using large numbers of stations,
>>>> for instance:
>>>>
>>>> ath: wiphy0: keyreset: keycache entry 228 out of range
>>>> ath: wiphy0: keyreset: keycache entry 228 out of range
>>>>
>>>> Signed-off-by: Ben Greear<greearb@candelatech.com>
>>>> ---
>>>> :100644 100644 917032a... 65e7281... M drivers/net/wireless/ath/ath9k/main.c
>>>> drivers/net/wireless/ath/ath9k/main.c | 3 +++
>>>> 1 files changed, 3 insertions(+), 0 deletions(-)
>>>>
>>>> diff --git a/drivers/net/wireless/ath/ath9k/main.c b/drivers/net/wireless/ath/ath9k/main.c
>>>> index 917032a..65e7281 100644
>>>> --- a/drivers/net/wireless/ath/ath9k/main.c
>>>> +++ b/drivers/net/wireless/ath/ath9k/main.c
>>>> @@ -1771,6 +1771,9 @@ static void ath9k_del_ps_key(struct ath_softc *sc,
>>>> if (!an->ps_key)
>>>> return;
>>>>
>>>> + if (ath9k_modparam_nohwcrypt)
>>>> + return;
>>>> +
>>> This looks wrong to me. ps_key is used for better powersave handling of
>>> both unencrypted and encrypted stations (using the keycache only for tx
>>> filtering on unencrypted stations).
>>
>> Ok..should we just remove the spammy printk when the keycache entry
>> is out of range then?
> I'd say we should figure out why it triggers at all.
I have more than 200 stations associating, and that is more than there
are key-indexes it appears. I can add a WARN_ON if you'd like to see
a backtrace or any other info...
Ben
--
Ben Greear <greearb@candelatech.com>
Candela Technologies Inc http://www.candelatech.com
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ath9k: Don't delete key from hw if using sw-crypt.
2012-04-03 16:09 ` Ben Greear
@ 2012-04-04 11:56 ` Felix Fietkau
0 siblings, 0 replies; 6+ messages in thread
From: Felix Fietkau @ 2012-04-04 11:56 UTC (permalink / raw)
To: Ben Greear; +Cc: Felix Fietkau, linux-wireless@vger.kernel.org
On 2012-04-03 6:09 PM, Ben Greear wrote:
> On 04/03/2012 09:05 AM, Felix Fietkau wrote:
>>
>> On 03.04.2012, at 16:44, Ben Greear<greearb@candelatech.com> wrote:
>>
>>> On 04/03/2012 06:16 AM, Felix Fietkau wrote:
>>>> On 2012-04-02 10:41 PM, greearb@candelatech.com wrote:
>>>>> From: Ben Greear<greearb@candelatech.com>
>>>>>
>>>>> This spams logs if we are using large numbers of stations,
>>>>> for instance:
>>>>>
>>>>> ath: wiphy0: keyreset: keycache entry 228 out of range
>>>>> ath: wiphy0: keyreset: keycache entry 228 out of range
>>>>>
>>>>> Signed-off-by: Ben Greear<greearb@candelatech.com>
>>>>> ---
>>>>> :100644 100644 917032a... 65e7281... M drivers/net/wireless/ath/ath9k/main.c
>>>>> drivers/net/wireless/ath/ath9k/main.c | 3 +++
>>>>> 1 files changed, 3 insertions(+), 0 deletions(-)
>>>>>
>>>>> diff --git a/drivers/net/wireless/ath/ath9k/main.c b/drivers/net/wireless/ath/ath9k/main.c
>>>>> index 917032a..65e7281 100644
>>>>> --- a/drivers/net/wireless/ath/ath9k/main.c
>>>>> +++ b/drivers/net/wireless/ath/ath9k/main.c
>>>>> @@ -1771,6 +1771,9 @@ static void ath9k_del_ps_key(struct ath_softc *sc,
>>>>> if (!an->ps_key)
>>>>> return;
>>>>>
>>>>> + if (ath9k_modparam_nohwcrypt)
>>>>> + return;
>>>>> +
>>>> This looks wrong to me. ps_key is used for better powersave handling of
>>>> both unencrypted and encrypted stations (using the keycache only for tx
>>>> filtering on unencrypted stations).
>>>
>>> Ok..should we just remove the spammy printk when the keycache entry
>>> is out of range then?
>> I'd say we should figure out why it triggers at all.
>
> I have more than 200 stations associating, and that is more than there
> are key-indexes it appears. I can add a WARN_ON if you'd like to see
> a backtrace or any other info...
In that case key index allocation should simply fail instead of
returning something that it refuses to delete later.
- Felix
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-04-04 11:55 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-04-02 20:41 [PATCH] ath9k: Don't delete key from hw if using sw-crypt greearb
2012-04-03 13:16 ` Felix Fietkau
2012-04-03 14:44 ` Ben Greear
2012-04-03 16:05 ` Felix Fietkau
2012-04-03 16:09 ` Ben Greear
2012-04-04 11:56 ` Felix Fietkau
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).