diff options
author | David S. Miller <davem@davemloft.net> | 2008-07-15 03:15:08 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-07-15 03:15:08 -0400 |
commit | b9e40857682ecfc5bcd0356a23ff409883ffb982 (patch) | |
tree | d241fd289bed6d16f36f6d26815c1e78e212c89a /drivers/media | |
parent | e308a5d806c852f56590ffdd3834d0df0cbed8d7 (diff) |
netdev: Do not use TX lock to protect address lists.
Now that we have a specific lock to protect the network
device unicast and multicast lists, remove extraneous
grabs of the TX lock in cases where the code only needs
address list protection.
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/media')
-rw-r--r-- | drivers/media/dvb/dvb-core/dvb_net.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/media/dvb/dvb-core/dvb_net.c b/drivers/media/dvb/dvb-core/dvb_net.c index 809d18c663bc..c2c033722a93 100644 --- a/drivers/media/dvb/dvb-core/dvb_net.c +++ b/drivers/media/dvb/dvb-core/dvb_net.c | |||
@@ -1133,8 +1133,7 @@ static void wq_set_multicast_list (struct work_struct *work) | |||
1133 | 1133 | ||
1134 | dvb_net_feed_stop(dev); | 1134 | dvb_net_feed_stop(dev); |
1135 | priv->rx_mode = RX_MODE_UNI; | 1135 | priv->rx_mode = RX_MODE_UNI; |
1136 | netif_tx_lock_bh(dev); | 1136 | netif_addr_lock_bh(dev); |
1137 | netif_addr_lock(dev); | ||
1138 | 1137 | ||
1139 | if (dev->flags & IFF_PROMISC) { | 1138 | if (dev->flags & IFF_PROMISC) { |
1140 | dprintk("%s: promiscuous mode\n", dev->name); | 1139 | dprintk("%s: promiscuous mode\n", dev->name); |
@@ -1159,8 +1158,7 @@ static void wq_set_multicast_list (struct work_struct *work) | |||
1159 | } | 1158 | } |
1160 | } | 1159 | } |
1161 | 1160 | ||
1162 | netif_addr_unlock(dev); | 1161 | netif_addr_unlock_bh(dev); |
1163 | netif_tx_unlock_bh(dev); | ||
1164 | dvb_net_feed_start(dev); | 1162 | dvb_net_feed_start(dev); |
1165 | } | 1163 | } |
1166 | 1164 | ||