Date: Tue, 11 Oct 2016 02:30:10 +0000 From: "sepherosa_gmail.com (Sepherosa Ziehau)" <phabric-noreply@FreeBSD.org> To: freebsd-net@freebsd.org Subject: [Differential] D8214: ifnet: Use if_link_state snapshot to invoke ifnet_link_event Message-ID: <differential-rev-PHID-DREV-ezeyexpg6dawymus5pyz-req@FreeBSD.org>
next in thread | raw e-mail | index | archive | help
[-- Attachment #1 --] 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 [-- Attachment #2 --] 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(); }
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?differential-rev-PHID-DREV-ezeyexpg6dawymus5pyz-req>