diff options
author | Stephen Hemminger <shemminger@osdl.org> | 2006-03-21 01:58:36 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2006-03-21 01:58:36 -0500 |
commit | f8ae737deea1be001d7d8ac11c382a7b73a1c98f (patch) | |
tree | fdf933b58a17e525e38683e1f8d5a52c4708964b /net/bridge/br_fdb.c | |
parent | fdeabdefb227be9aa932f59a23ddb47e003e643e (diff) |
[BRIDGE]: forwarding remove unneeded preempt and bh diasables
Optimize the forwarding and transmit paths. Both places are
called with bottom half/no preempt so there is no need to use
spin_lock_bh or rcu_read_lock.
Signed-off-by: Stephen Hemminger <shemminger@osdl.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/bridge/br_fdb.c')
-rw-r--r-- | net/bridge/br_fdb.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/net/bridge/br_fdb.c b/net/bridge/br_fdb.c index 1f08a59b51ea..3a73b8c94271 100644 --- a/net/bridge/br_fdb.c +++ b/net/bridge/br_fdb.c | |||
@@ -341,7 +341,6 @@ void br_fdb_update(struct net_bridge *br, struct net_bridge_port *source, | |||
341 | if (hold_time(br) == 0) | 341 | if (hold_time(br) == 0) |
342 | return; | 342 | return; |
343 | 343 | ||
344 | rcu_read_lock(); | ||
345 | fdb = fdb_find(head, addr); | 344 | fdb = fdb_find(head, addr); |
346 | if (likely(fdb)) { | 345 | if (likely(fdb)) { |
347 | /* attempt to update an entry for a local interface */ | 346 | /* attempt to update an entry for a local interface */ |
@@ -356,13 +355,12 @@ void br_fdb_update(struct net_bridge *br, struct net_bridge_port *source, | |||
356 | fdb->ageing_timer = jiffies; | 355 | fdb->ageing_timer = jiffies; |
357 | } | 356 | } |
358 | } else { | 357 | } else { |
359 | spin_lock_bh(&br->hash_lock); | 358 | spin_lock(&br->hash_lock); |
360 | if (!fdb_find(head, addr)) | 359 | if (!fdb_find(head, addr)) |
361 | fdb_create(head, source, addr, 0); | 360 | fdb_create(head, source, addr, 0); |
362 | /* else we lose race and someone else inserts | 361 | /* else we lose race and someone else inserts |
363 | * it first, don't bother updating | 362 | * it first, don't bother updating |
364 | */ | 363 | */ |
365 | spin_unlock_bh(&br->hash_lock); | 364 | spin_unlock(&br->hash_lock); |
366 | } | 365 | } |
367 | rcu_read_unlock(); | ||
368 | } | 366 | } |