On 2013-11-20 08:32, Felix Fietkau wrote: > On 2013-11-20 01:51, Karl Beldan wrote: >> From: Karl Beldan <karl.beldan@xxxxxxxxxxxxxxxx> >> >> Commit 3e8b1eb "mac80211/minstrel_ht: improve rate selection stability" >> introduced a local capped prob in minstrel_ht_calc_tp but omitted to use >> it to compute the rate throughput. >> >> Signed-off-by: Karl Beldan <karl.beldan@xxxxxxxxxxxxxxxx> >> CC: Felix Fietkau <nbd@xxxxxxxxxxx> > Nice catch! > Acked-by: Felix Fietkau <nbd@xxxxxxxxxxx> Sorry, I need to revoke that ACK - there's a bug in this patch: > - tp = 1000000 * ((mr->probability * 1000) / nsecs); > + tp = 1000000 * (prob * 1000) / nsecs; Removing the outer () is wrong, it leads to overflow that makes all throughput values 0. - Felix -- 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
