On Monday 08 August 2011 08:47 PM, Mohammed Shafi Shajakhan wrote:
From: Mohammed Shafi Shajakhan<mohammed@xxxxxxxxxxxxxxxx>
for ease of debugging, the optimization is done such that the rate
control stats are shown only for rate's that might be possibly used.
for instance if static HT20 is configured, showing HT40 rate control
stats is unnecessary. for dynamic HT40/HT20 i had made it such that the
stats will be displayed till HT40. double streams/ triple usage was
also considered. did some basic testing.
I just did further more testing with this patch and found that its
better to simply use 'rate_table_size' parameter to determine the rate
table size. with this patch, few times some unnecessary stats were
showed, when the Access point is configured in legacy mode. will send a
proper patch for this.
Cc: Vasanthakumar Thiagarajan<vthiagar@xxxxxxxxxxxxxxxx>
Cc: "Balasubramanian, senthilkumar"<senthilb@xxxxxxxxxxxxxxxx>
Signed-off-by: Mohammed Shafi Shajakhan<mohammed@xxxxxxxxxxxxxxxx>
---
drivers/net/wireless/ath/ath9k/rc.c | 8 +++++---
1 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/drivers/net/wireless/ath/ath9k/rc.c b/drivers/net/wireless/ath/ath9k/rc.c
index c04a6c3..f964473 100644
--- a/drivers/net/wireless/ath/ath9k/rc.c
+++ b/drivers/net/wireless/ath/ath9k/rc.c
@@ -1478,13 +1478,15 @@ static ssize_t read_file_rcstat(struct file *file, char __user *user_buf,
struct ath_rate_priv *rc = file->private_data;
char *buf;
unsigned int len = 0, max;
- int i = 0;
+ u8 i = 0;
+ static u8 max_rate_table_size;
ssize_t retval;
if (rc->rate_table == NULL)
return 0;
+ max_rate_table_size = max(max_rate_table_size, rc->rate_table_size);
- max = 80 + rc->rate_table->rate_cnt * 1024 + 1;
+ max = 80 + max_rate_table_size * 1024 + 1;
buf = kmalloc(max, GFP_KERNEL);
if (buf == NULL)
return -ENOMEM;
@@ -1494,7 +1496,7 @@ static ssize_t read_file_rcstat(struct file *file, char __user *user_buf,
"HT", "MCS", "Rate",
"Success", "Retries", "XRetries", "PER");
- for (i = 0; i< rc->rate_table->rate_cnt; i++) {
+ for (i = 0; i< max_rate_table_size; i++) {
u32 ratekbps = rc->rate_table->info[i].ratekbps;
struct ath_rc_stats *stats =&rc->rcstats[i];
char mcs[5];
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
