On Fri, Jul 16, 2010 at 2:16 AM, yogeshp <yogeshp@xxxxxxxxxxx> wrote: > Hi Luis, > Please see inline comments > > Luis R. Rodriguez wrote: >> On Thu, Jul 15, 2010 at 6:18 AM, yogeshp <yogeshp@xxxxxxxxxxx> wrote: >>> BA window size for a successful BA setup is not made available to the driver by >>> mac80211. The patch below gets the BA window size from addba response and >>> indicates it to driver through IEEE80211_AMPDU_TX_OPERATIONAL drv_ampdu_action. > >> But why do you want that? Your patch and commit log do not mention why >> you need this. > BA window size from ADDBA response defines how many outstanding MPDUs are allowed for the BA stream by recipient. Since ADDBA response is processed in mac80211 stack, mac80211 should communicate the BA window size to driver (and may be from driver it is further communicated to firmware/hardware) to control the number of outstanding MPDUs while transmitting MPDUs for the stream. What driver requires this? Your patch does all these changes but do not show any secondary patch of a user, so the patch is pointless right now. >> When I compile mac80211 with your patch it fails compilation > I do not see this compilation error, I think the patch did not apply cleanly. I will send out another patch without extra white spaces that should solve the compilation issue. If the change is fine, we can then send patches to fix the compilation issues for other drivers too. > .. Luis -- 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
