On Wed, Oct 29, 2008 at 11:16 AM, Sujith <Sujith.Manoharan@xxxxxxxxxxx> wrote: > Tomas Winkler wrote: >> On Wed, Oct 29, 2008 at 6:49 AM, Sujith <Sujith.Manoharan@xxxxxxxxxxx> wrote: >> > Send a notification to the driver on succesful >> > reception of an ADDBA response, add IEEE80211_AMPDU_TX_RESUME >> > for this purpose. >> >> What what is wrong with IEEE80211_AMPDU_TX_START ? I don' t understand >> why this is needed? > > On receiving an IEEE80211_AMPDU_TX_START notification, packets destined to > that TID are stored in a buffer, and to send them out as aggregates, ath9k > needs to know whether the ADDBA exchange has completed or not. > > If the ADDBA request has been declined, IEEE80211_AMPDU_TX_STOP is sent to > the driver, but no notification is sent if the ADDBA request has been accepted. Okay this means that we pushing this buffering to driver after all. Otherwise mac80211 would just trigger the TX on successful ADDBA Tomas -- 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