-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
net: conn_mgr: Add IPv4 and IPv6 tracking #72393
Conversation
I haven't added unit tests for the new feature, nor have I updated old unit tests. I am going to put that off until after I transition my work laptop to linux, since it will be much easier to do local testing in that environment |
45c0833
to
fa2e3b9
Compare
@simensrostad I cannot add you as reviewer, but please feel free to review |
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.
Looked over new events. looks good
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.
Looks good overall, just one concern to discuss.
c43535e
to
efbc21f
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.
LGTM, small nit about the commit message which has old information.
ae44b2e
to
13957b9
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.
some minor doc issues and an actual comment typo
- :c:macro:`NET_EVENT_L4_IPV4_CONNECTED` | ||
- :c:macro:`NET_EVENT_L4_IPV4_DISCONNECTED` | ||
- :c:macro:`NET_EVENT_L4_IPV6_CONNECTED` | ||
- :c:macro:`NET_EVENT_L4_IPV6_DISCONNECTED` |
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.
- :c:macro:`NET_EVENT_L4_IPV4_CONNECTED` | |
- :c:macro:`NET_EVENT_L4_IPV4_DISCONNECTED` | |
- :c:macro:`NET_EVENT_L4_IPV6_CONNECTED` | |
- :c:macro:`NET_EVENT_L4_IPV6_DISCONNECTED` | |
- :c:enumerator:`NET_EVENT_L4_IPV4_CONNECTED` | |
- :c:enumerator:`NET_EVENT_L4_IPV4_DISCONNECTED` | |
- :c:enumerator:`NET_EVENT_L4_IPV6_CONNECTED` | |
- :c:enumerator:`NET_EVENT_L4_IPV6_DISCONNECTED` |
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.
Are you certain these should be changed to :c:enumerator:
? They are defined as macros whose values derive from the actual enumerator values
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.
Of course! Read too fast and missed the difference between the "CMD" enumerators ans the macros
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.
some minor doc issues and an actual comment typo
Instead of incrementing and decrementing global counter, just recompute the ready-count from scratch every time conn_mgr_mon_handle_update is called. This will simplify the introduction of additional ready count types. This should have no externally observable impact on the behavior of conn_mgr. Signed-off-by: Georges Oates_Larsen <[email protected]>
Don't track up/down blame separately. Instead, track a single last-blame iface. Don't track blame iface inside set_ready. Instead, track directly inside handle_update. These two changes will simplify the addition of blame for IPv4- and IPv6-specific events. Signed-off-by: Georges Oates_Larsen <[email protected]>
conn_mgr now fires: - NET_EVENT_L4_IPV4_CONNECTED - NET_EVENT_L4_IPV4_DISCONNECTED - NET_EVENT_L4_IPV6_CONNECTED - NET_EVENT_L4_IPV6_DISCONNECTED These events track whether there are any ready ifaces offering specifically IPv4 or specifically IPv6 connectivity. Signed-off-by: Georges Oates_Larsen <[email protected]>
Modify the test cycle routines to verify recently added IPv4- and IPv6-specific events. Signed-off-by: Georges Oates_Larsen <[email protected]>
Add descriptions for recently introduced IPv4 and IPv6 connectivity events to the net event monitor. Signed-off-by: Georges Oates_Larsen <[email protected]>
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.
Doc ok, thx!
No description provided.