From b957208807c3deeb0fbbdc80c5589863e9db06bb Mon Sep 17 00:00:00 2001 From: Felix Fietkau Date: Thu, 15 Mar 2012 19:32:31 +0000 Subject: [PATCH] swconfig: use a mutex instead of a spinlock, many swconfig calls sleep SVN-Revision: 30952 --- target/linux/generic/files/drivers/net/phy/swconfig.c | 10 +++++----- target/linux/generic/files/include/linux/switch.h | 2 +- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/target/linux/generic/files/drivers/net/phy/swconfig.c b/target/linux/generic/files/drivers/net/phy/swconfig.c index 1baccb1ba3..e772c94481 100644 --- a/target/linux/generic/files/drivers/net/phy/swconfig.c +++ b/target/linux/generic/files/drivers/net/phy/swconfig.c @@ -353,7 +353,7 @@ swconfig_get_dev(struct genl_info *info) break; } if (dev) - spin_lock(&dev->lock); + mutex_lock(&dev->sw_mutex); else DPRINTF("device %d not found\n", id); swconfig_unlock(); @@ -364,7 +364,7 @@ done: static inline void swconfig_put_dev(struct switch_dev *dev) { - spin_unlock(&dev->lock); + mutex_unlock(&dev->sw_mutex); } static int @@ -962,7 +962,7 @@ register_switch(struct switch_dev *dev, struct net_device *netdev) return -ENOMEM; } swconfig_defaults_init(dev); - spin_lock_init(&dev->lock); + mutex_init(&dev->sw_mutex); swconfig_lock(); dev->id = ++swdev_id; @@ -1000,11 +1000,11 @@ unregister_switch(struct switch_dev *dev) { swconfig_destroy_led_trigger(dev); kfree(dev->portbuf); - spin_lock(&dev->lock); + mutex_lock(&dev->sw_mutex); swconfig_lock(); list_del(&dev->dev_list); swconfig_unlock(); - spin_unlock(&dev->lock); + mutex_unlock(&dev->sw_mutex); } EXPORT_SYMBOL_GPL(unregister_switch); diff --git a/target/linux/generic/files/include/linux/switch.h b/target/linux/generic/files/include/linux/switch.h index 2c88bb3276..4f4085ef89 100644 --- a/target/linux/generic/files/include/linux/switch.h +++ b/target/linux/generic/files/include/linux/switch.h @@ -191,7 +191,7 @@ struct switch_dev { struct list_head dev_list; unsigned long def_global, def_port, def_vlan; - spinlock_t lock; + struct mutex sw_mutex; struct switch_port *portbuf; char buf[128]; -- 2.30.2