Author: ae
Date: Wed Nov 14 17:23:48 2012
New Revision: 243029
URL: http://svnweb.freebsd.org/changeset/base/243029

Log:
  SCOPE6_LOCK protects V_sid_default, no need to acquire it without
  any access to V_sid_default.
  
  Sponsored by: Yandex LLC
  MFC after:    1 week

Modified:
  head/sys/netinet6/scope6.c

Modified: head/sys/netinet6/scope6.c
==============================================================================
--- head/sys/netinet6/scope6.c  Wed Nov 14 17:14:03 2012        (r243028)
+++ head/sys/netinet6/scope6.c  Wed Nov 14 17:23:48 2012        (r243029)
@@ -139,7 +139,6 @@ scope6_set(struct ifnet *ifp, struct sco
         * interface addresses, routing table entries, PCB entries...
         */
 
-       SCOPE6_LOCK();
        for (i = 0; i < 16; i++) {
                if (idlist->s6id_list[i] &&
                    idlist->s6id_list[i] != sid->s6id_list[i]) {
@@ -150,7 +149,6 @@ scope6_set(struct ifnet *ifp, struct sco
                        if (i == IPV6_ADDR_SCOPE_INTFACELOCAL &&
                            idlist->s6id_list[i] != ifp->if_index) {
                                IF_AFDATA_UNLOCK(ifp);
-                               SCOPE6_UNLOCK();
                                return (EINVAL);
                        }
 
@@ -163,7 +161,6 @@ scope6_set(struct ifnet *ifp, struct sco
                                 * safety in later use.
                                 */
                                IF_AFDATA_UNLOCK(ifp);
-                               SCOPE6_UNLOCK();
                                return (EINVAL);
                        }
 
@@ -175,7 +172,6 @@ scope6_set(struct ifnet *ifp, struct sco
                        sid->s6id_list[i] = idlist->s6id_list[i];
                }
        }
-       SCOPE6_UNLOCK();
        IF_AFDATA_UNLOCK(ifp);
 
        return (error);
@@ -193,9 +189,7 @@ scope6_get(struct ifnet *ifp, struct sco
                return (EINVAL);
        }
 
-       SCOPE6_LOCK();
        *idlist = *sid;
-       SCOPE6_UNLOCK();
 
        IF_AFDATA_UNLOCK(ifp);
        return (0);
@@ -442,8 +436,6 @@ in6_setscope(struct in6_addr *in6, struc
        }
 
        scope = in6_addrscope(in6);
-
-       SCOPE6_LOCK();
        switch (scope) {
        case IPV6_ADDR_SCOPE_INTFACELOCAL: /* should be interface index */
                zoneid = sid->s6id_list[IPV6_ADDR_SCOPE_INTFACELOCAL];
@@ -465,7 +457,6 @@ in6_setscope(struct in6_addr *in6, struc
                zoneid = 0;     /* XXX: treat as global. */
                break;
        }
-       SCOPE6_UNLOCK();
        IF_AFDATA_UNLOCK(ifp);
 
        if (ret_id != NULL)
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to