Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 20 Jun 2012 14:22:38 +0000 (UTC)
From:      Gleb Smirnoff <glebius@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-projects@freebsd.org
Subject:   svn commit: r237312 - projects/pf/head/sys/contrib/pf/net
Message-ID:  <201206201422.q5KEMc23047346@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: glebius
Date: Wed Jun 20 14:22:38 2012
New Revision: 237312
URL: http://svn.freebsd.org/changeset/base/237312

Log:
  Don't use locking in the clone destroy routine.

Modified:
  projects/pf/head/sys/contrib/pf/net/if_pfsync.c

Modified: projects/pf/head/sys/contrib/pf/net/if_pfsync.c
==============================================================================
--- projects/pf/head/sys/contrib/pf/net/if_pfsync.c	Wed Jun 20 14:19:55 2012	(r237311)
+++ projects/pf/head/sys/contrib/pf/net/if_pfsync.c	Wed Jun 20 14:22:38 2012	(r237312)
@@ -362,7 +362,6 @@ pfsync_clone_destroy(struct ifnet *ifp)
 	 * drain callouts.
 	 */
 relock:
-	PFSYNC_LOCK(sc);
 	while (sc->sc_deferred > 0) {
 		struct pfsync_deferral *pd = TAILQ_FIRST(&sc->sc_deferrals);
 	
@@ -374,7 +373,6 @@ relock:
 			free(pd, M_PFSYNC);
 		} else {
 			pd->pd_refs++;
-			PFSYNC_UNLOCK(sc);
 			callout_drain(&pd->pd_tmo);
 			free(pd, M_PFSYNC);
 			goto relock;



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201206201422.q5KEMc23047346>