ralph-irving / squeezeos

Source code for the Logitech Controller, Radio and Touch Community Firmware
https://sourceforge.net/projects/lmsclients/files/squeezeos/
22 stars 7 forks source link

wpa_supplicant v2.9 - Fix "Wireless Event too big (33)" messages #8

Closed mw9 closed 3 years ago

mw9 commented 3 years ago

This proposed change eliminates the "Wireless Event too big (33)" kernel messages that are observed on a Squeezebox Radio each time it disconnects from an AP.

I don't think the message is of any significance, and it is also present in the Radio's "stock" firmware. However I think it may be a distraction when users are attempting to debug wi-fi connectivity issues, and there have been a number of reports of those over the last year or two. "Squashing" the message will remove one source of concern.

The issue has been discussed in forum thread "Community Build Radio Firmware". These two posts record some of the investigatory process that has been undertaken:

https://forums.slimdevices.com/showthread.php?111663-Community-Build-Radio-Firmware&p=1016905&viewfull=1#post1016905 https://forums.slimdevices.com/showthread.php?111663-Community-Build-Radio-Firmware&p=1017259&viewfull=1#post1017259

The change applies a patch to wpa_supplicant that causes it to reduce the size of a "bogus SSID" that it generates on disconnection to a length of 31 bytes, rather than the full 32 bytes possible. That eliminates the kernel message.

Further detail is provided in the commit message, and in the patch itself.

I have tested this on a Squeezebox Radio, and it appears to operate as intended, with no adverse side effects noted. It would be sensible to trial this on other Radios, in case an unexpectedissue should be revealed. And a second set of eyes applied to the proposed change would be welcome.