Re: [PATCH 05/10] rt2800: make ba_size depend on ampdu_factor

2016-11-16 Thread Mathias Kresin
2016-11-16 13:02 GMT+01:00 Stanislaw Gruszka : > On Wed, Nov 16, 2016 at 09:07:00AM +0100, Mathias Kresin wrote: >> Here are the results of the requested tests. Please keep in mind, I'm not in >> a lab environment: >> >> LEDE head >> connect >> action 0 sta 9c:f3:87:bc:AA:BB tid 6 buf_size 64

Re: [PATCH 05/10] rt2800: make ba_size depend on ampdu_factor

2016-11-16 Thread Stanislaw Gruszka
On Wed, Nov 16, 2016 at 09:07:00AM +0100, Mathias Kresin wrote: > Here are the results of the requested tests. Please keep in mind, I'm not in > a lab environment: > > LEDE head > connect > action 0 sta 9c:f3:87:bc:AA:BB tid 6 buf_size 64 ampdu_factor 3 > action 2 sta 9c:f3:87:bc:AA:BB t

Re: [PATCH 05/10] rt2800: make ba_size depend on ampdu_factor

2016-11-16 Thread Mathias Kresin
14.11.2016 13:43, Stanislaw Gruszka: On Mon, Nov 14, 2016 at 09:45:36AM +0100, Stanislaw Gruszka wrote: Could you check below patch and see if it helps? If it does not, could you printk sta->ht_cap.ampdu_density and ba_size values and provide them here. Actually please print parameters from be

Re: [PATCH 05/10] rt2800: make ba_size depend on ampdu_factor

2016-11-14 Thread Stanislaw Gruszka
On Wed, Nov 02, 2016 at 03:11:00PM +0100, Stanislaw Gruszka wrote: > We can calculate BA window size (max number of pending frames not > yet block acked) of remote station using Maximum A-MPDU length factor > for that station. > > Signed-off-by: Stanislaw Gruszka Kalle, please drop this patch fr

Re: [PATCH 05/10] rt2800: make ba_size depend on ampdu_factor

2016-11-14 Thread Stanislaw Gruszka
On Mon, Nov 14, 2016 at 09:45:36AM +0100, Stanislaw Gruszka wrote: > Could you check below patch and see if it helps? If it does not, > could you printk sta->ht_cap.ampdu_density and ba_size values > and provide them here. Actually please print parameters from below patch. I think ba_size should b

Re: [PATCH 05/10] rt2800: make ba_size depend on ampdu_factor

2016-11-14 Thread Stanislaw Gruszka
Hi On Mon, Nov 14, 2016 at 08:41:57PM +1100, Julian Calaby wrote: > > - rt2x00_set_field32(®, MAX_LEN_CFG_MAX_PSDU, 1); > > + rt2x00_set_field32(®, MAX_LEN_CFG_MAX_PSDU, 3; > > You're missing a closing parenthesis, so it isn't going to work unless > it's added back in.

Re: [PATCH 05/10] rt2800: make ba_size depend on ampdu_factor

2016-11-14 Thread Julian Calaby
Hi Stainslaw, On Mon, Nov 14, 2016 at 7:45 PM, Stanislaw Gruszka wrote: > On Sat, Nov 05, 2016 at 01:56:58PM +0100, Mathias Kresin wrote: >> 02.11.2016 15:11, Stanislaw Gruszka: >> > >> >-sta_priv = sta_to_rt2x00_sta(sta); >> >+txdesc->u.ht.mpdu_density = sta->ht_cap.ampdu

Re: [PATCH 05/10] rt2800: make ba_size depend on ampdu_factor

2016-11-14 Thread Stanislaw Gruszka
On Sat, Nov 05, 2016 at 01:56:58PM +0100, Mathias Kresin wrote: > 02.11.2016 15:11, Stanislaw Gruszka: > > > >-sta_priv = sta_to_rt2x00_sta(sta); > >+txdesc->u.ht.mpdu_density = sta->ht_cap.ampdu_density; > > txdesc->u.ht.wcid = sta_priv->wcid; > >+ > >+

Re: [PATCH 05/10] rt2800: make ba_size depend on ampdu_factor

2016-11-05 Thread Mathias Kresin
02.11.2016 15:11, Stanislaw Gruszka: We can calculate BA window size (max number of pending frames not yet block acked) of remote station using Maximum A-MPDU length factor for that station. Signed-off-by: Stanislaw Gruszka --- drivers/net/wireless/ralink/rt2x00/rt2x00queue.c | 15 ++

[PATCH 05/10] rt2800: make ba_size depend on ampdu_factor

2016-11-02 Thread Stanislaw Gruszka
We can calculate BA window size (max number of pending frames not yet block acked) of remote station using Maximum A-MPDU length factor for that station. Signed-off-by: Stanislaw Gruszka --- drivers/net/wireless/ralink/rt2x00/rt2x00queue.c | 15 ++- 1 files changed, 10 insertions(+