[PATCH 0/5] tx a-mpdu fixes

[Eugene Krasnikov]

Good job! There some minor things to be fixed but in general really good!

2014-03-03 20:45 GMT+00:00 Bob Copeland <me at bobcopeland.com>:
> This series includes some changes to fix ampdu operation from
> the wcn36xx STA.
>
> Patches 1 and 2 implement fixes for TX-ampdu operation.
>
> Patch 3 just takes care of a FIXME - also EDCA setup was not
> included in the start cmd patch due to not following the single
> u32 config parameter pattern.  In the future we should set this
> up with ops->conf_tx.
>
> Patches 4 and 5 were just some things spotted in review.
>
> Running three ten second iperf sessions in a row shows the difference
> in 2.4 GHz HT20 channel.
>
> Without patches:
>
> [  3]  0.0-10.0 sec  22.8 MBytes  19.0 Mbits/sec
> [  3]  0.0-10.1 sec  25.6 MBytes  21.4 Mbits/sec
> [  3]  0.0-10.1 sec  25.5 MBytes  21.3 Mbits/sec
>
> With patches:
>
> [  3]  0.0-10.0 sec  25.1 MBytes  21.0 Mbits/sec
> [  3]  0.0-10.0 sec  51.2 MBytes  42.9 Mbits/sec
> [  3]  0.0-10.0 sec  50.1 MBytes  42.0 Mbits/sec
>
> Note first trial is slower in each case due to device starting
> at lowest legacy rates and slowly ramping up; aggregation won't
> happen until MCS rates are in use.
>
> Bob Copeland (5):
>   wcn36xx: initialize device defaults on start
>   wcn36xx: let device generate qos seq numbers
>   wcn36xx: set initial edca queue parameters
>   wcn36xx: copy wmm_enabled into v1 config
>   wcn36xx: use !! when assigning int as a boolean
>
>  smd.c  |  131 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++--
>  txrx.c |    3 +-
>  txrx.h |    3 +-
>  3 files changed, 131 insertions(+), 6 deletions(-)
>
> --
> 1.7.10.4
>
>
> _______________________________________________
> wcn36xx mailing list
> wcn36xx at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/wcn36xx



-- 
Best regards,
Eugene