Re: [RESEND PATCH] cfg80211: dynamically allocate per-tid stats for station info

2018-05-18 Thread Johannes Berg
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.

Re: [RESEND PATCH] cfg80211: dynamically allocate per-tid stats for station info

2018-05-18 Thread Arend van Spriel
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

Re: [RESEND PATCH] cfg80211: dynamically allocate per-tid stats for station info

2018-05-18 Thread Johannes Berg
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,

Re: [RESEND PATCH] cfg80211: dynamically allocate per-tid stats for station info

2018-05-10 Thread Toke Høiland-Jørgensen
Arend van Spriel writes: > From: Arend van Spriel > > With the addition of TXQ stats in the per-tid statistics the struct > station_info grew significantly. This resulted in stack limit warnings > like below: > >CC [M]

[RESEND PATCH] cfg80211: dynamically allocate per-tid stats for station info

2018-05-10 Thread Arend van Spriel
From: Arend van Spriel With the addition of TXQ stats in the per-tid statistics the struct station_info grew significantly. This resulted in stack limit warnings like below: CC [M] drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.o