On Mon, 2008-11-17 at 17:42 -0200, Herton Ronaldo Krzesinski wrote:
> On Monday 17 November 2008 13:08:21 Larry Finger wrote:
> > In the commit entitled "mac80211/drivers: rewrite the rate control API"
> > (commit 9ea2c74de0ec971e8ec9fc5aaea9cd5b4fec95b6), the meaning of the
> > packet transmit count was changed from the number of retries to the total
> > number. In driver rtl8187, this change was missed.
>
> With this change as before, when using pid, rate never go above 1M in testing
> here (minstrel doesn't have the same issue). May be because this in
> rate_control_pid_tx_status (inside rc80211_pid_algo.c)?
>
> if (!(info->flags & IEEE80211_TX_STAT_ACK)) {
> spinfo->tx_num_failed += 2;
> spinfo->tx_num_xmit++;
> } else if (info->status.rates[0].count) {
> spinfo->tx_num_failed++;
> spinfo->tx_num_xmit++;
> }
>
> as pkt_rc + 1, looks like we always will increment tx_num_failed, and may be
> it's causing this. Well this is only judging by a quick look, I may be wrong.
Larry recently fixed the above code too.
johannes
Attachment:
signature.asc
Description: This is a digitally signed message part
