Commit 48af05ed authored by Paolo 'Blaisorblade' Giarrusso's avatar Paolo 'Blaisorblade' Giarrusso Committed by Linus Torvalds

[PATCH] uml: fix proc-vs-interrupt context spinlock deadlock

This spinlock can be taken on interrupt too, so spin_lock_irq[save] must be
used.

However, Documentation/networking/netdevices.txt explains we are called with
rtnl_lock() held - so we don't need to care about other concurrent opens.
Verified also in LDD3 and by direct checking.  Also verified that the network
layer (through a state machine) guarantees us that nobody will close the
interface while it's being used.  Please correct me if I'm wrong.

Also, we must check we don't sleep with irqs disabled!!!  But anyway, this is
not news - we already can't sleep while holding a spinlock.  Who says this is
guaranted really by the present code?
Signed-off-by: default avatarPaolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>
Cc: Jeff Dike <jdike@addtoit.com>
Cc: Jeff Garzik <jeff@garzik.org>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 06837504
...@@ -114,8 +114,6 @@ static int uml_net_open(struct net_device *dev) ...@@ -114,8 +114,6 @@ static int uml_net_open(struct net_device *dev)
struct uml_net_private *lp = dev->priv; struct uml_net_private *lp = dev->priv;
int err; int err;
spin_lock(&lp->lock);
if(lp->fd >= 0){ if(lp->fd >= 0){
err = -ENXIO; err = -ENXIO;
goto out; goto out;
...@@ -149,8 +147,6 @@ static int uml_net_open(struct net_device *dev) ...@@ -149,8 +147,6 @@ static int uml_net_open(struct net_device *dev)
*/ */
while((err = uml_net_rx(dev)) > 0) ; while((err = uml_net_rx(dev)) > 0) ;
spin_unlock(&lp->lock);
spin_lock(&opened_lock); spin_lock(&opened_lock);
list_add(&lp->list, &opened); list_add(&lp->list, &opened);
spin_unlock(&opened_lock); spin_unlock(&opened_lock);
...@@ -160,7 +156,6 @@ static int uml_net_open(struct net_device *dev) ...@@ -160,7 +156,6 @@ static int uml_net_open(struct net_device *dev)
if(lp->close != NULL) (*lp->close)(lp->fd, &lp->user); if(lp->close != NULL) (*lp->close)(lp->fd, &lp->user);
lp->fd = -1; lp->fd = -1;
out: out:
spin_unlock(&lp->lock);
return err; return err;
} }
...@@ -169,15 +164,12 @@ static int uml_net_close(struct net_device *dev) ...@@ -169,15 +164,12 @@ static int uml_net_close(struct net_device *dev)
struct uml_net_private *lp = dev->priv; struct uml_net_private *lp = dev->priv;
netif_stop_queue(dev); netif_stop_queue(dev);
spin_lock(&lp->lock);
free_irq(dev->irq, dev); free_irq(dev->irq, dev);
if(lp->close != NULL) if(lp->close != NULL)
(*lp->close)(lp->fd, &lp->user); (*lp->close)(lp->fd, &lp->user);
lp->fd = -1; lp->fd = -1;
spin_unlock(&lp->lock);
spin_lock(&opened_lock); spin_lock(&opened_lock);
list_del(&lp->list); list_del(&lp->list);
spin_unlock(&opened_lock); spin_unlock(&opened_lock);
...@@ -246,9 +238,9 @@ static int uml_net_set_mac(struct net_device *dev, void *addr) ...@@ -246,9 +238,9 @@ static int uml_net_set_mac(struct net_device *dev, void *addr)
struct uml_net_private *lp = dev->priv; struct uml_net_private *lp = dev->priv;
struct sockaddr *hwaddr = addr; struct sockaddr *hwaddr = addr;
spin_lock(&lp->lock); spin_lock_irq(&lp->lock);
set_ether_mac(dev, hwaddr->sa_data); set_ether_mac(dev, hwaddr->sa_data);
spin_unlock(&lp->lock); spin_unlock_irq(&lp->lock);
return(0); return(0);
} }
...@@ -258,7 +250,7 @@ static int uml_net_change_mtu(struct net_device *dev, int new_mtu) ...@@ -258,7 +250,7 @@ static int uml_net_change_mtu(struct net_device *dev, int new_mtu)
struct uml_net_private *lp = dev->priv; struct uml_net_private *lp = dev->priv;
int err = 0; int err = 0;
spin_lock(&lp->lock); spin_lock_irq(&lp->lock);
new_mtu = (*lp->set_mtu)(new_mtu, &lp->user); new_mtu = (*lp->set_mtu)(new_mtu, &lp->user);
if(new_mtu < 0){ if(new_mtu < 0){
...@@ -269,7 +261,7 @@ static int uml_net_change_mtu(struct net_device *dev, int new_mtu) ...@@ -269,7 +261,7 @@ static int uml_net_change_mtu(struct net_device *dev, int new_mtu)
dev->mtu = new_mtu; dev->mtu = new_mtu;
out: out:
spin_unlock(&lp->lock); spin_unlock_irq(&lp->lock);
return err; return err;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment