Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

iw_nl80211: don't block when receiving msgs #123

Merged
merged 1 commit into from
Oct 31, 2023

Conversation

wsakernel
Copy link
Contributor

Info screen waits for the first data to arrive. This stalls sometimes on my machine because the netlink command does not complete for some reason. So, set the socket to non-blocking and try again next cycle if no data is available. This needs a small version bump for libnl from 3.2 to 3.2.22 because only since then the call to nl_recvmsgs() returns -NLE_AGAIN. Fixes #120.

I have not included the autogenerated changes to configure because I have autotools 2.71 which modifies a lot.

Further testing is appreciated.

Info screen waits for the first data to arrive. This stalls sometimes on
my machine because the netlink command does not complete for some
reason. So, set the socket to non-blocking and try again next cycle if
no data is available. This needs a small version bump for libnl from
3.2 to 3.2.22 because only since then the call to nl_recvmsgs() returns
-NLE_AGAIN. Fixes uoaerg#120.
@grrtrr
Copy link
Contributor

grrtrr commented Oct 31, 2023

Thank you for the fix. I will update configure.ac, test this a bit and then release 0.9.6 if it works out.

@grrtrr grrtrr merged commit 9538e06 into uoaerg:master Oct 31, 2023
@wsakernel
Copy link
Contributor Author

It seems it did work out? A version 0.9.6 would be great. Thanks!

@grrtrr
Copy link
Contributor

grrtrr commented Jul 12, 2024

Apologies for the delay, I meant to clean up configure.ac and related, but got busy with other things.

Will look into this within the next week.

@grrtrr
Copy link
Contributor

grrtrr commented Jul 13, 2024

Released v0.9.6.

@wsakernel
Copy link
Contributor Author

Awesome, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Switching to info screen sometimes hangs
2 participants