Hi Mikolaj, Stefan,

Am 08.05.21 um 16:16 schrieb Mikolaj Kucharski:
> On Sat, May 08, 2021 at 10:06:08AM +0200, Stefan Sperling wrote:
>> On Fri, May 07, 2021 at 10:27:36PM +0000, Mikolaj Kucharski wrote:
>>> Yes, I see ieee80211_encap: data frame for node... messages in
>>> dmesg. Two dmesgs, from two PC Engines machines, in two different
>>> locations, 150+km apart, below.
>>>
>>> Uptime of 12 minutes as hostap from pce-0041:
>>
>>> athn0: AR9280 rev 2 (2T2R), ROM rev 22, address 04:f0:21:34:e4:23
>>
>>> ieee80211_encap: data frame for node 04:f0:21:34:e4:23 in state 0
>>
>> Ooops, sorry I made a mistake. This is the MAC address of the AP itself.
>> The new check I am adding here should obviously exclude the AP itself.
>> Otherwise your AP won't work very well because it blocks its own broadcasts.
>>
>> Fixed patch, with the condition ni != ic->ic_bss added:
>>
>> diff b412d6f4250f42e50e0333dbcedab9189b4b19e2 /usr/src
>> blob - 58f654273f7ea0cc3c04e0a60d5e5e6a33296dc0
>> file + sys/net80211/ieee80211_output.c
>> --- sys/net80211/ieee80211_output.c
>> +++ sys/net80211/ieee80211_output.c
>> @@ -556,6 +556,16 @@ ieee80211_encap(struct ifnet *ifp, struct mbuf *m, str
>>              goto bad;
>>      }
>>  
>> +#ifndef IEEE80211_STA_ONLY
>> +    if (ic->ic_opmode == IEEE80211_M_HOSTAP && ni != ic->ic_bss &&
>> +        ni->ni_state != IEEE80211_STA_ASSOC) {
>> +            printf("%s: data frame for node %s in state %d\n",
>> +                __func__, ether_sprintf(ni->ni_macaddr), ni->ni_state);
>> +            ic->ic_stats.is_tx_nonode++;
>> +            goto bad;
>> +    }
>> +#endif
>> +
>>      if ((ic->ic_flags & IEEE80211_F_RSNON) &&
>>          !ni->ni_port_valid &&
>>          eh.ether_type != htons(ETHERTYPE_PAE)) {
> 
> So far nothing, 10 min of uptime for both machines:
> 
> pce-0035# dmesg | grep -cF ieee80211_encap
> 0
> 
> # dmesg | grep -cF ieee80211_encap
> 0
> 

Sorry if this should be obvious - How can I test this myself?

Thanks

Stephen

Attachment: OpenPGP_signature
Description: OpenPGP digital signature

Reply via email to