git: 9674c2e68c61 - main - netgraph/ng_bridge: become SMP aware

Lutz Donnerhacke donner at FreeBSD.org
Thu May 13 15:54:07 UTC 2021


The branch main has been updated by donner:

URL: https://cgit.FreeBSD.org/src/commit/?id=9674c2e68c613757ddf5a8780743842b5ac4d74f

commit 9674c2e68c613757ddf5a8780743842b5ac4d74f
Author:     Lutz Donnerhacke <donner at FreeBSD.org>
AuthorDate: 2021-01-12 21:28:29 +0000
Commit:     Lutz Donnerhacke <donner at FreeBSD.org>
CommitDate: 2021-05-13 15:53:07 +0000

    netgraph/ng_bridge: become SMP aware
    
    The node ng_bridge underwent a lot of changes in the last few months.
    All those steps were necessary to distinguish between structure
    modifying and read-only data transport paths.  Now it's done, the node
    can perform frame forwarding on multiple cores in parallel.
    
    MFC after:      2 weeks
    Differential Revision:  https://reviews.freebsd.org/D28123
---
 sys/netgraph/ng_bridge.c | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/sys/netgraph/ng_bridge.c b/sys/netgraph/ng_bridge.c
index ecb00f9ba29f..470c0f05f18d 100644
--- a/sys/netgraph/ng_bridge.c
+++ b/sys/netgraph/ng_bridge.c
@@ -351,15 +351,6 @@ ng_bridge_constructor(node_p node)
 	priv->conf.minStableAge = DEFAULT_MIN_STABLE_AGE;
 	priv->sendUnknown = 1;	       /* classic bridge */
 
-	/*
-	 * This node has all kinds of stuff that could be screwed by SMP.
-	 * Until it gets it's own internal protection, we go through in 
-	 * single file. This could hurt a machine bridging between two 
-	 * GB ethernets so it should be fixed. 
-	 * When it's fixed the process SHOULD NOT SLEEP, spinlocks please!
-	 * (and atomic ops )
-	 */
-	NG_NODE_FORCE_WRITER(node);
 	NG_NODE_SET_PRIVATE(node, priv);
 	priv->node = node;
 


More information about the dev-commits-src-all mailing list