-
Notifications
You must be signed in to change notification settings - Fork 235
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
[posix] set addr gen mode to none on linux #2419
[posix] set addr gen mode to none on linux #2419
Conversation
3e63d05
to
f9b4316
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2419 +/- ##
===========================================
- Coverage 55.77% 45.29% -10.49%
===========================================
Files 87 99 +12
Lines 6890 11171 +4281
Branches 0 805 +805
===========================================
+ Hits 3843 5060 +1217
- Misses 3047 5863 +2816
- Partials 0 248 +248 ☔ View full report in Codecov by Sentry. |
f9b4316
to
bafa257
Compare
bafa257
to
6fe9c37
Compare
6fe9c37
to
056e46c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
This kind of netlink messages requires specially NET_ADMIN CAPS I think, ot-daemon on Android doesn't have those CAPS so those configurations are delegated to Android system_server. You probably want to enable this in ot-br-posix for non-Android linux devices. (You can try your code in Android see what error it will be like :) |
Thanks! As discussed, let's add a macro for the entire netif module and not enable it on Android. On Android, we will use the specific implementation for netif. I'll add a seperate PR for it. |
This PR sets the addr gen mode of wpan interface to NONE on linux.
This PR depends on #2410. #2410 needs to be merged first.