libremesh / lime-packages

LibreMesh packages configuring OpenWrt for wireless mesh networking
https://libremesh.org/
GNU Affero General Public License v3.0
276 stars 96 forks source link

Fix 11s support on Atheros chips #102

Closed nicopace closed 7 years ago

nicopace commented 7 years ago

In the current implementation of Atheros' 802.11s there seems to be a bug that hangs the kernel. @p4u and @altergui may be able to add more detail to this issue.

dangowrt commented 7 years ago

/me is going to pick up a serious amount of ath9k test gear from @FreifunkUFO at some point during the weekend

nicopace commented 7 years ago

@dangowrt awesome!!! Let us know what you find out :)

FreifunkUFO commented 7 years ago

which firmware? kernel-hangs? so we might need serial console?

FreifunkUFO commented 7 years ago

some freifunk-gluon users also uses 11s.. we have to ask them about their experiences and wifi-config.

https://github.com/freifunk-gluon/gluon

see f.i. gluon-patch here (maybe uploading to lede-dev?) https://github.com/freifunk-gluon/gluon/pull/1078

dangowrt commented 7 years ago

Looks like @nbd168 just fixed 802.11s in LEDE, at least that kernel bug does't get triggered any more. See commit cffeb311 mac80211: backport an upstream fix for queue start/stop handling

nicopace commented 7 years ago

Are you sure @dangowrt? it doesn't seem to mention much more than 'Fixes issues with 802.11s'... we should test

dangowrt commented 7 years ago

It looks like it, but I'm about to test this excessively just now...

FreifunkUFO commented 7 years ago

ok, i found a ticket about this stuff: https://bugs.lede-project.org/index.php?do=details&task_id=644&order=dateopened&sort=desc

FreifunkUFO commented 7 years ago

and i found another ticket: https://bugs.lede-project.org/index.php?do=details&task_id=619&order=dateopened&sort=desc

dangowrt commented 7 years ago

619 is the one which got fixed by the patch @nbd168 imported from

upstream (I referenced previously).\

644 got an obviously wrong wifi config (option ifname is the most

likely cause of the probolem, coverage '5' seems wrong, too)

p4u commented 7 years ago

Fixed in the last release

FreifunkUFO commented 7 years ago

so why there is a need for an extra (gluon?-)snippet/patch? https://github.com/libremesh/lime-sdk/blob/master/snippets/patch_sdk_11s.sh its uploaded by anonymious!? :-o

ilario commented 7 years ago

anonymous = p4u, as he confirmed in lime-users mailing list

p4u commented 7 years ago

That was a mistake. I commited from another computer where my name was not configured properly on git.

On 12/05/17 16:41, Ilario Gelmetti wrote:

anonymous = p4u, as he confirmed in lime-users mailing list

-- ./p4u

dangowrt commented 7 years ago

See also https://github.com/libremesh/lime-packages/issues/123 which seems to be a duplicate of this bug for more details