[Differential] D8214: ifnet: Use if_link_state snapshot to invoke ifnet_link_event

sepherosa_gmail.com (Sepherosa Ziehau) phabric-noreply at FreeBSD.org
Tue Oct 11 02:30:11 UTC 2016


sepherosa_gmail.com created this revision.
sepherosa_gmail.com added reviewers: glebius, adrian, gnn, bz.
sepherosa_gmail.com added a subscriber: freebsd-net-list.

REVISION SUMMARY
  So that everyone in this task have consistent view of link state.

REVISION DETAIL
  https://reviews.freebsd.org/D8214

AFFECTED FILES
  sys/net/if.c

CHANGE DETAILS

diff --git a/sys/net/if.c b/sys/net/if.c
--- a/sys/net/if.c
+++ b/sys/net/if.c
@@ -2206,7 +2206,7 @@
 	if (log_link_state_change)
 		log(LOG_NOTICE, "%s: link state changed to %s\n", ifp->if_xname,
 		    (link_state == LINK_STATE_UP) ? "UP" : "DOWN" );
-	EVENTHANDLER_INVOKE(ifnet_link_event, ifp, ifp->if_link_state);
+	EVENTHANDLER_INVOKE(ifnet_link_event, ifp, link_state);
 	CURVNET_RESTORE();
 }
 



EMAIL PREFERENCES
  https://reviews.freebsd.org/settings/panel/emailpreferences/

To: sepherosa_gmail.com, glebius, adrian, gnn, bz
Cc: freebsd-net-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D8214.21239.patch
Type: text/x-patch
Size: 405 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/freebsd-net/attachments/20161011/7377c835/attachment.bin>


More information about the freebsd-net mailing list