2013-11-14 08:01:34

by Bartosz Markowski

Subject: [PATCH] ath10k: expand the wmi mgmt queue limit to 128

It was obseverd many times the short queue limit can be
easily exceeded in case of AP multiple BSSID scenarios.

"ath10k: wmi mgmt_tx queue limit reached."

This was leading to TX packets drops.

This patch expands the queue limit to 128. After many stress
experiments (high traffic and lot of management frames in
the air) the new value seems to fix the problem on platforms
with limited resources.

Reported-By: Tomasz Skapski <[email protected]>
Signed-off-by: Bartosz Markowski <[email protected]>
---
drivers/net/wireless/ath/ath10k/core.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/ath/ath10k/core.h b/drivers/net/wireless/ath/ath10k/core.h
index 0934f76..be35df2 100644
--- a/drivers/net/wireless/ath/ath10k/core.h
+++ b/drivers/net/wireless/ath/ath10k/core.h
@@ -43,7 +43,7 @@
/* Antenna noise floor */
#define ATH10K_DEFAULT_NOISE_FLOOR -95

-#define ATH10K_MAX_NUM_MGMT_PENDING 16
+#define ATH10K_MAX_NUM_MGMT_PENDING 128

struct ath10k;

--
1.7.10

2013-11-15 09:07:04

by Kalle Valo

Subject: Re: [PATCH] ath10k: expand the wmi mgmt queue limit to 128

Bartosz Markowski <[email protected]> writes:

> It was obseverd many times the short queue limit can be
> easily exceeded in case of AP multiple BSSID scenarios.
>
> "ath10k: wmi mgmt_tx queue limit reached."
>
> This was leading to TX packets drops.
>
> This patch expands the queue limit to 128. After many stress
> experiments (high traffic and lot of management frames in
> the air) the new value seems to fix the problem on platforms
> with limited resources.
>
> Reported-By: Tomasz Skapski <[email protected]>
> Signed-off-by: Bartosz Markowski <[email protected]>

Thanks, applied.

--
Kalle Valo