Michael Buesch wrote:
> On Thursday 02 August 2007, Larry Finger wrote:
>> Michael,
>>
>> I couldn't find any long/short indication in the header, so I added a bool
>> that
>> is set when the frame is sent.
>
> This is not going to work.
>
> But we can do this differently.
> You can do something like:
>
> if (!status->acked && !tx_control->noack)
> excessive_retries = 1;
>
> So we don't need to care about the retry count.
>
> Anyway. I don't know why we need excessive_retries _at_ _all_.
> The rc algorithm does already know if the frame succeed or failed
> anyway.
Is this what you had in mind?
Larry
@@ -1311,6 +1311,9 @@ void bcm43xx_dma_handle_txstatus(struct
*/
if (status->acked)
meta->txstat.flags |= IEEE80211_TX_STATUS_ACK;
+ else
+ if (!(meta->txstat.flags &
IEEE80211_TXCTL_NO_ACK))
+ meta->txstat.excessive_retries = 1;
meta->txstat.retry_count = status->frame_count - 1;
ieee80211_tx_status_irqsafe(dev->wl->hw, meta->skb,
&(meta->txstat));
/* skb is freed by ieee80211_tx_status_irqsafe() */
_______________________________________________
Bcm43xx-dev mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/bcm43xx-dev
