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

[sepherosa_gmail.com (Sepherosa Ziehau)]

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

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();
 }
 


_______________________________________________
freebsd-net@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-net
To unsubscribe, send any mail to "freebsd-net-unsubscr...@freebsd.org"