On Mon, Apr 18, 2011 at 15:26, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Mon, 2011-04-18 at 14:22 +0300, Arik Nemtsov wrote: >> Add API that allows low level drivers to notify mac80211 about TX >> packet loss. This is useful when there are FW triggers to notify the >> low level driver about these events. > >> +void ieee80211_report_low_ack(struct ieee80211_sta *pubsta, u32 num_packets) >> +{ >> + struct sta_info *sta = container_of(pubsta, struct sta_info, sta); >> + cfg80211_cqm_pktloss_notify(sta->sdata->dev, sta->sta.addr, >> + num_packets, GFP_ATOMIC); > > I think it would make more sense to add num_packets to sta->lost_packets > and invoke it only when over the threshold as done in > ieee80211_tx_status(), as then we can share any threshold calculation > improvements. Well in a wl12xx chip we just configure some thresholds to the FW and it triggers the event for us. Using TX status doesn't really make sense since the FW has automatic rate control (so 50 tries are more like 50 * 8). > > Also, are you sure? Do you really not get _any_ TX status? Currently we do get TX status, but there are plans to remove it later on. Regards, Arik -- 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