On Fri, 2018-05-18 at 11:37 +0200, Arend van Spriel wrote: > On 5/18/2018 11:25 AM, Johannes Berg wrote: > > On Thu, 2018-05-10 at 13:50 +0200, Arend van Spriel wrote: > > > > > > Here an alternative approach. Currently the only cfg80211-based driver > > > providing per-tid stats is mac80211. This patch only changes mac80211 > > > and the other driver can keep using stack allocation. Even mac80211 could > > > if wanted, but I left that part as is. > > > > I decided to take this and remove the BIT() thing entirely (you had a > > bug there anyway) > > I figured you would drop this one. The error paths may need to do kfree > here and there. Kalle took dynamic alloc fixes in driver already so my > patch is not really needed right now. Heh. I asked Kalle if he wanted to revert, but I decided that for the single user in mac80211 it wasn't worth the hassle of doing the dynamic allocations all over the place. I'll look at the error paths I guess. johannes