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
Promiscuous mode off fixes #11303
Promiscuous mode off fixes #11303
Conversation
subsys/net/ip/net_if.c
Outdated
return; | ||
} | ||
|
||
#if defined(CONFIG_NET_L2_ETHERNET) |
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.
Could you try using IS_ENABLED instead?
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 is currently not possible as the ETHERNET symbol is only available if CONFIG_NET_L2_ETHERNET is set. But I can refactor the functions a bit more in order to avoid duplication. I will send a new version.
Codecov Report
@@ Coverage Diff @@
## master #11303 +/- ##
==========================================
+ Coverage 51.55% 51.56% +<.01%
==========================================
Files 216 216
Lines 27665 27671 +6
Branches 6937 6938 +1
==========================================
+ Hits 14263 14268 +5
- Misses 10653 10654 +1
Partials 2749 2749
Continue to review full report at Codecov.
|
New version:
|
Pass the promiscuous mode OFF event properly to device driver instead of just toggling the network interface flag. Signed-off-by: Jukka Rissanen <jukka.rissanen@linux.intel.com>
Add a shell that implements "promisc [on | off] <interface>" command to toggle promiscuous mode support. Signed-off-by: Jukka Rissanen <jukka.rissanen@linux.intel.com>
No description provided.