On Fri, 2018-02-23 at 10:06 +0100, Felix Fietkau wrote:
> Signed-off-by: Felix Fietkau <nbd@xxxxxxxx>
> ---
> net/mac80211/cfg.c | 2 --
> net/mac80211/rx.c | 11 +++++++----
> 2 files changed, 7 insertions(+), 6 deletions(-)
>
> diff --git a/net/mac80211/cfg.c b/net/mac80211/cfg.c
> index 6f9d5beec73e..72b5a2ec4e94 100644
> --- a/net/mac80211/cfg.c
> +++ b/net/mac80211/cfg.c
> @@ -160,7 +160,6 @@ static int ieee80211_change_iface(struct wiphy *wiphy,
> if (type == NL80211_IFTYPE_AP_VLAN &&
> params && params->use_4addr == 0) {
> RCU_INIT_POINTER(sdata->u.vlan.sta, NULL);
> - ieee80211_check_fast_rx_iface(sdata);
> } else if (type == NL80211_IFTYPE_STATION &&
> params && params->use_4addr >= 0) {
> sdata->u.mgd.use_4addr = params->use_4addr;
> @@ -1575,7 +1574,6 @@ static int ieee80211_change_station(struct wiphy *wiphy,
> }
>
> rcu_assign_pointer(vlansdata->u.vlan.sta, sta);
> - __ieee80211_check_fast_rx_iface(vlansdata);
> }
>
> if (sta->sdata->vif.type == NL80211_IFTYPE_AP_VLAN &&
I'm not convinced this is right - don't you have to recheck to modify
the station data? Otherwise you'd just drop all the frames to the
slowpath again due to the first patch in the series, no?
johannes
