On Monday 17 November 2008 17:47:07 Johannes Berg wrote:
> 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.
Ah nice, I didn't saw it, I'm usually lagging reading mail... I saw it now.
Acked-by: Herton Ronaldo Krzesinski <herton@xxxxxxxxxxxxxxx>
>
> johannes
--
[]'s
Herton
--
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
