[ath5k-devel] [PATCH] ath5k: Fix TX/RX padding for all frames

2010-02-27 Thread Benoit Papillault
Currently, the padding position is based on ieee80211_get_hdrlen_from_skb(). This is not correct since the HW does padding on RX (and expect the same padding to be present on TX) at the following position : - management : 24 + 6 if 4-addr format - control: 24 + 6 if 4-addr format - data

Re: [ath5k-devel] [PATCH] ath5k: Fix TX/RX padding for all frames

2010-02-27 Thread Benoit PAPILLAULT
Bob Copeland a écrit : > On Sat, Feb 27, 2010 at 01:58:38PM +0100, Benoit Papillault wrote: > >> Currently, the padding position is based on >> ieee80211_get_hdrlen_from_skb(). This is not correct since the HW does >> padding on RX (and expect the same padding to be present on TX) at the >> foll

Re: [ath5k-devel] [PATCH] ath5k: Fix TX/RX padding for all frames

2010-02-27 Thread Bob Copeland
On Sat, Feb 27, 2010 at 01:58:38PM +0100, Benoit Papillault wrote: > Currently, the padding position is based on > ieee80211_get_hdrlen_from_skb(). This is not correct since the HW does > padding on RX (and expect the same padding to be present on TX) at the > following position : > > - management

[ath5k-devel] [PATCH] ath5k: Fix TX/RX padding for all frames

2010-02-27 Thread Benoit Papillault
Currently, the padding position is based on ieee80211_get_hdrlen_from_skb(). This is not correct since the HW does padding on RX (and expect the same padding to be present on TX) at the following position : - management : 24 + 6 if 4-addr format - control: 24 + 6 if 4-addr format - data

Re: [ath5k-devel] [PATCH] ath5k: Fix TX/RX padding for all frames

2010-02-16 Thread Jouni Malinen
On Tue, Feb 16, 2010 at 09:39:25PM +0100, Benoit PAPILLAULT wrote: > I don't have an ath5k based card at hand. I will try to grab one and > will give an example of said frame. Basically, I followed the same > strategy for ath9k already. Are you saying that this patch has not actually been tested a

Re: [ath5k-devel] [PATCH] ath5k: Fix TX/RX padding for all frames

2010-02-16 Thread Benoit PAPILLAULT
Jouni Malinen a écrit : > On Tue, Feb 16, 2010 at 09:39:25PM +0100, Benoit PAPILLAULT wrote: > >> I don't have an ath5k based card at hand. I will try to grab one and >> will give an example of said frame. Basically, I followed the same >> strategy for ath9k already. >> > > Are you saying t

Re: [ath5k-devel] [PATCH] ath5k: Fix TX/RX padding for all frames

2010-02-16 Thread Benoit PAPILLAULT
Bob Copeland a écrit : > On Mon, Feb 15, 2010 at 11:06:29PM +0100, Benoit PAPILLAULT wrote: > >>> Can you tell what the functional difference is between the old code and new >>> code? E.g. a padding that would be incorrectly computed from before? >>> >>> >>> >> Correct. On some frames

Re: [ath5k-devel] [PATCH] ath5k: Fix TX/RX padding for all frames

2010-02-15 Thread Bob Copeland
On Mon, Feb 15, 2010 at 11:06:29PM +0100, Benoit PAPILLAULT wrote: >> Can you tell what the functional difference is between the old code and new >> code? E.g. a padding that would be incorrectly computed from before? >> >> > Correct. On some frames padding is incorrect. This patch is more for

Re: [ath5k-devel] [PATCH] ath5k: Fix TX/RX padding for all frames

2010-02-15 Thread Benoit PAPILLAULT
Bob Copeland a écrit : > On Sun, Feb 14, 2010 at 6:36 PM, Benoit Papillault > wrote: > >> Instead of computing the padding size based on the IEEE 802.11 header length, >> we directly compute the padding position first and then the padding size >> next. >> We have changed some functions to pass

Re: [ath5k-devel] [PATCH] ath5k: Fix TX/RX padding for all frames

2010-02-15 Thread Bob Copeland
On Sun, Feb 14, 2010 at 6:36 PM, Benoit Papillault wrote: > Instead of computing the padding size based on the IEEE 802.11 header length, > we directly compute the padding position first and then the padding size next. > We have changed some functions to pass them the padding size directly. It has

[ath5k-devel] [PATCH] ath5k: Fix TX/RX padding for all frames

2010-02-14 Thread Benoit Papillault
Instead of computing the padding size based on the IEEE 802.11 header length, we directly compute the padding position first and then the padding size next. We have changed some functions to pass them the padding size directly. It has been tested using a monitor interface in TX and RX against a dif