Re: [PATCH] rtl8187: Fix transmission count sent to mac80211

[Herton Ronaldo Krzesinski <herton@mandriva.com.br>]

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@mandriva.com.br>

>
> johannes

-- 
[]'s
Herton