On 2018-02-23 12:24, Johannes Berg wrote:
> On Fri, 2018-02-23 at 10:06 +0100, Felix Fietkau wrote:
>> Signed-off-by: Felix Fietkau <n...@nbd.name>
>> ---
>>  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?You're right and I 
> forgot to take those out before sending. Will
resubmit this one.

Thanks,

- Felix

Reply via email to