2011-05-04 19:55:57

by Larry Finger

[permalink] [raw]
Subject: WARNING from mac80211

Johannes and John,

A regression in recent versions of mac80211 are generating the WARNING at
net/mac80211/wep.c:101, because the skb headroom is less than WEP_IV_LEN or the
tailroom is less than WEP_ICV_LEN. My testing indicates that it is a headroom
problem as the following hack "fixes" it:

Index: wireless-testing-new/net/mac80211/tx.c
===================================================================
--- wireless-testing-new.orig/net/mac80211/tx.c
+++ wireless-testing-new/net/mac80211/tx.c
@@ -1935,6 +1935,8 @@ netdev_tx_t ieee80211_subif_start_xmit(s
head_need += IEEE80211_ENCRYPT_HEADROOM;
head_need += local->tx_headroom;
head_need = max_t(int, 0, head_need);
+ if (head_need < WEP_IV_LEN)
+ head_need = WEP_IV_LEN;
if (ieee80211_skb_resize(local, skb, head_need, true))
goto fail;
}

I'm certain that this is too simple minded, but it at least points to a place to
look. I have not bisected the problem to see which commit caused the regression,
but I can if you want.

Larry


2011-05-04 20:15:37

by Johannes Berg

[permalink] [raw]
Subject: Re: WARNING from mac80211

On Wed, 2011-05-04 at 15:11 -0500, Larry Finger wrote:

> > See also this thread:
> > http://thread.gmane.org/gmane.linux.kernel.wireless.general/69062/focus=69150
> >
> > and the resulting patch:
> > http://article.gmane.org/gmane.linux.kernel.wireless.general/69190
> > (not sure why it was RFC?)
>
> Thanks. I do remember the thread now that you refreshed my memory. That patch
> does fix my problem.

Good to know. John should probably merge that patch even though it was
RFC.

johannes


2011-05-04 20:02:45

by Johannes Berg

[permalink] [raw]
Subject: Re: WARNING from mac80211

On Wed, 2011-05-04 at 14:55 -0500, Larry Finger wrote:
> Johannes and John,
>
> A regression in recent versions of mac80211 are generating the WARNING at
> net/mac80211/wep.c:101, because the skb headroom is less than WEP_IV_LEN or the
> tailroom is less than WEP_ICV_LEN. My testing indicates that it is a headroom
> problem as the following hack "fixes" it:

See also this thread:
http://thread.gmane.org/gmane.linux.kernel.wireless.general/69062/focus=69150

and the resulting patch:
http://article.gmane.org/gmane.linux.kernel.wireless.general/69190
(not sure why it was RFC?)

johannes


2011-05-04 20:11:36

by Larry Finger

[permalink] [raw]
Subject: Re: WARNING from mac80211

On 05/04/2011 03:02 PM, Johannes Berg wrote:
> On Wed, 2011-05-04 at 14:55 -0500, Larry Finger wrote:
>> Johannes and John,
>>
>> A regression in recent versions of mac80211 are generating the WARNING at
>> net/mac80211/wep.c:101, because the skb headroom is less than WEP_IV_LEN or the
>> tailroom is less than WEP_ICV_LEN. My testing indicates that it is a headroom
>> problem as the following hack "fixes" it:
>
> See also this thread:
> http://thread.gmane.org/gmane.linux.kernel.wireless.general/69062/focus=69150
>
> and the resulting patch:
> http://article.gmane.org/gmane.linux.kernel.wireless.general/69190
> (not sure why it was RFC?)

Thanks. I do remember the thread now that you refreshed my memory. That patch
does fix my problem.

Larry

2011-05-05 04:47:15

by Mohammed Shafi

[permalink] [raw]
Subject: Re: WARNING from mac80211

On Thu, May 5, 2011 at 1:41 AM, Larry Finger <[email protected]> wrote:
> On 05/04/2011 03:02 PM, Johannes Berg wrote:
>>
>> On Wed, 2011-05-04 at 14:55 -0500, Larry Finger wrote:
>>>
>>> Johannes and John,
>>>
>>> A regression in recent versions of mac80211 are generating the WARNING at
>>> net/mac80211/wep.c:101, because the skb headroom is less than WEP_IV_LEN
>>> or the
>>> tailroom is less than WEP_ICV_LEN. My testing indicates that it is a
>>> headroom
>>> problem as the following hack "fixes" it:
>>
>> See also this thread:
>>
>> http://thread.gmane.org/gmane.linux.kernel.wireless.general/69062/focus=69150
>>
>> and the resulting patch:
>> http://article.gmane.org/gmane.linux.kernel.wireless.general/69190
>> (not sure why it was RFC?)

I will send the patch today :)

thanks,
shafi

>
> Thanks. I do remember the thread now that you refreshed my memory. That
> patch does fix my problem.
>
> Larry
> --
> To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
> the body of a message to [email protected]
> More majordomo info at ?http://vger.kernel.org/majordomo-info.html
>