Commit ab3c81ff authored by Arjan van de Ven's avatar Arjan van de Ven Committed by Greg Kroah-Hartman

[PATCH] USBATM: semaphore to mutex conversion

This is the usbatm part of the Arjan, Jes and Ingo
mass semaphore to mutex conversion, reworked to apply on top
of the patches I just sent to you.  This time, with correct
attribution and signed-off lines.
Signed-off-by: default avatarArjan van de Ven <arjan@infradead.org>
Signed-off-by: default avatarIngo Molnar <mingo@elte.hu>
Signed-off-by: default avatarDuncan Sands <baldrick@free.fr>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent a3673d3c
...@@ -36,6 +36,7 @@ ...@@ -36,6 +36,7 @@
#include <linux/init.h> #include <linux/init.h>
#include <linux/device.h> /* FIXME: linux/firmware.h should include it itself */ #include <linux/device.h> /* FIXME: linux/firmware.h should include it itself */
#include <linux/firmware.h> #include <linux/firmware.h>
#include <linux/mutex.h>
#include "usbatm.h" #include "usbatm.h"
...@@ -160,7 +161,7 @@ struct cxacru_data { ...@@ -160,7 +161,7 @@ struct cxacru_data {
struct work_struct poll_work; struct work_struct poll_work;
/* contol handles */ /* contol handles */
struct semaphore cm_serialize; struct mutex cm_serialize;
u8 *rcv_buf; u8 *rcv_buf;
u8 *snd_buf; u8 *snd_buf;
struct urb *rcv_urb; struct urb *rcv_urb;
...@@ -219,7 +220,7 @@ static int cxacru_cm(struct cxacru_data *instance, enum cxacru_cm_request cm, ...@@ -219,7 +220,7 @@ static int cxacru_cm(struct cxacru_data *instance, enum cxacru_cm_request cm,
goto fail; goto fail;
} }
down(&instance->cm_serialize); mutex_lock(&instance->cm_serialize);
/* submit reading urb before the writing one */ /* submit reading urb before the writing one */
init_completion(&instance->rcv_done); init_completion(&instance->rcv_done);
...@@ -288,7 +289,7 @@ static int cxacru_cm(struct cxacru_data *instance, enum cxacru_cm_request cm, ...@@ -288,7 +289,7 @@ static int cxacru_cm(struct cxacru_data *instance, enum cxacru_cm_request cm,
ret = offd; ret = offd;
dbg("cm %#x", cm); dbg("cm %#x", cm);
fail: fail:
up(&instance->cm_serialize); mutex_unlock(&instance->cm_serialize);
return ret; return ret;
} }
...@@ -717,7 +718,7 @@ static int cxacru_bind(struct usbatm_data *usbatm_instance, ...@@ -717,7 +718,7 @@ static int cxacru_bind(struct usbatm_data *usbatm_instance,
instance->snd_buf, PAGE_SIZE, instance->snd_buf, PAGE_SIZE,
cxacru_blocking_completion, &instance->snd_done, 4); cxacru_blocking_completion, &instance->snd_done, 4);
init_MUTEX(&instance->cm_serialize); mutex_init(&instance->cm_serialize);
INIT_WORK(&instance->poll_work, (void *)cxacru_poll_status, instance); INIT_WORK(&instance->poll_work, (void *)cxacru_poll_status, instance);
......
...@@ -63,6 +63,7 @@ ...@@ -63,6 +63,7 @@
#include <linux/ctype.h> #include <linux/ctype.h>
#include <linux/kthread.h> #include <linux/kthread.h>
#include <linux/version.h> #include <linux/version.h>
#include <linux/mutex.h>
#include <asm/unaligned.h> #include <asm/unaligned.h>
#include "usbatm.h" #include "usbatm.h"
...@@ -358,7 +359,7 @@ struct intr_pkt { ...@@ -358,7 +359,7 @@ struct intr_pkt {
#define INTR_PKT_SIZE 28 #define INTR_PKT_SIZE 28
static struct usb_driver uea_driver; static struct usb_driver uea_driver;
static DECLARE_MUTEX(uea_semaphore); static DEFINE_MUTEX(uea_mutex);
static const char *chip_name[] = {"ADI930", "Eagle I", "Eagle II", "Eagle III"}; static const char *chip_name[] = {"ADI930", "Eagle I", "Eagle II", "Eagle III"};
static int modem_index; static int modem_index;
...@@ -1418,13 +1419,13 @@ static ssize_t read_status(struct device *dev, struct device_attribute *attr, ...@@ -1418,13 +1419,13 @@ static ssize_t read_status(struct device *dev, struct device_attribute *attr,
int ret = -ENODEV; int ret = -ENODEV;
struct uea_softc *sc; struct uea_softc *sc;
down(&uea_semaphore); mutex_lock(&uea_mutex);
sc = dev_to_uea(dev); sc = dev_to_uea(dev);
if (!sc) if (!sc)
goto out; goto out;
ret = snprintf(buf, 10, "%08x\n", sc->stats.phy.state); ret = snprintf(buf, 10, "%08x\n", sc->stats.phy.state);
out: out:
up(&uea_semaphore); mutex_unlock(&uea_mutex);
return ret; return ret;
} }
...@@ -1434,14 +1435,14 @@ static ssize_t reboot(struct device *dev, struct device_attribute *attr, ...@@ -1434,14 +1435,14 @@ static ssize_t reboot(struct device *dev, struct device_attribute *attr,
int ret = -ENODEV; int ret = -ENODEV;
struct uea_softc *sc; struct uea_softc *sc;
down(&uea_semaphore); mutex_lock(&uea_mutex);
sc = dev_to_uea(dev); sc = dev_to_uea(dev);
if (!sc) if (!sc)
goto out; goto out;
sc->reset = 1; sc->reset = 1;
ret = count; ret = count;
out: out:
up(&uea_semaphore); mutex_unlock(&uea_mutex);
return ret; return ret;
} }
...@@ -1453,7 +1454,7 @@ static ssize_t read_human_status(struct device *dev, struct device_attribute *at ...@@ -1453,7 +1454,7 @@ static ssize_t read_human_status(struct device *dev, struct device_attribute *at
int ret = -ENODEV; int ret = -ENODEV;
struct uea_softc *sc; struct uea_softc *sc;
down(&uea_semaphore); mutex_lock(&uea_mutex);
sc = dev_to_uea(dev); sc = dev_to_uea(dev);
if (!sc) if (!sc)
goto out; goto out;
...@@ -1473,7 +1474,7 @@ static ssize_t read_human_status(struct device *dev, struct device_attribute *at ...@@ -1473,7 +1474,7 @@ static ssize_t read_human_status(struct device *dev, struct device_attribute *at
break; break;
} }
out: out:
up(&uea_semaphore); mutex_unlock(&uea_mutex);
return ret; return ret;
} }
...@@ -1485,7 +1486,7 @@ static ssize_t read_delin(struct device *dev, struct device_attribute *attr, ...@@ -1485,7 +1486,7 @@ static ssize_t read_delin(struct device *dev, struct device_attribute *attr,
int ret = -ENODEV; int ret = -ENODEV;
struct uea_softc *sc; struct uea_softc *sc;
down(&uea_semaphore); mutex_lock(&uea_mutex);
sc = dev_to_uea(dev); sc = dev_to_uea(dev);
if (!sc) if (!sc)
goto out; goto out;
...@@ -1497,7 +1498,7 @@ static ssize_t read_delin(struct device *dev, struct device_attribute *attr, ...@@ -1497,7 +1498,7 @@ static ssize_t read_delin(struct device *dev, struct device_attribute *attr,
else else
ret = sprintf(buf, "GOOD\n"); ret = sprintf(buf, "GOOD\n");
out: out:
up(&uea_semaphore); mutex_unlock(&uea_mutex);
return ret; return ret;
} }
...@@ -1511,7 +1512,7 @@ static ssize_t read_##name(struct device *dev, \ ...@@ -1511,7 +1512,7 @@ static ssize_t read_##name(struct device *dev, \
int ret = -ENODEV; \ int ret = -ENODEV; \
struct uea_softc *sc; \ struct uea_softc *sc; \
\ \
down(&uea_semaphore); \ mutex_lock(&uea_mutex); \
sc = dev_to_uea(dev); \ sc = dev_to_uea(dev); \
if (!sc) \ if (!sc) \
goto out; \ goto out; \
...@@ -1519,7 +1520,7 @@ static ssize_t read_##name(struct device *dev, \ ...@@ -1519,7 +1520,7 @@ static ssize_t read_##name(struct device *dev, \
if (reset) \ if (reset) \
sc->stats.phy.name = 0; \ sc->stats.phy.name = 0; \
out: \ out: \
up(&uea_semaphore); \ mutex_unlock(&uea_mutex); \
return ret; \ return ret; \
} \ } \
\ \
...@@ -1737,9 +1738,9 @@ static void uea_disconnect(struct usb_interface *intf) ...@@ -1737,9 +1738,9 @@ static void uea_disconnect(struct usb_interface *intf)
* Pre-firmware device has one interface * Pre-firmware device has one interface
*/ */
if (usb->config->desc.bNumInterfaces != 1 && ifnum == 0) { if (usb->config->desc.bNumInterfaces != 1 && ifnum == 0) {
down(&uea_semaphore); mutex_lock(&uea_mutex);
usbatm_usb_disconnect(intf); usbatm_usb_disconnect(intf);
up(&uea_semaphore); mutex_unlock(&uea_mutex);
uea_info(usb, "ADSL device removed\n"); uea_info(usb, "ADSL device removed\n");
} }
......
...@@ -823,7 +823,7 @@ static int usbatm_atm_open(struct atm_vcc *vcc) ...@@ -823,7 +823,7 @@ static int usbatm_atm_open(struct atm_vcc *vcc)
return -EINVAL; return -EINVAL;
} }
down(&instance->serialize); /* vs self, usbatm_atm_close, usbatm_usb_disconnect */ mutex_lock(&instance->serialize); /* vs self, usbatm_atm_close, usbatm_usb_disconnect */
if (instance->disconnected) { if (instance->disconnected) {
atm_dbg(instance, "%s: disconnected!\n", __func__); atm_dbg(instance, "%s: disconnected!\n", __func__);
...@@ -867,7 +867,7 @@ static int usbatm_atm_open(struct atm_vcc *vcc) ...@@ -867,7 +867,7 @@ static int usbatm_atm_open(struct atm_vcc *vcc)
set_bit(ATM_VF_PARTIAL, &vcc->flags); set_bit(ATM_VF_PARTIAL, &vcc->flags);
set_bit(ATM_VF_READY, &vcc->flags); set_bit(ATM_VF_READY, &vcc->flags);
up(&instance->serialize); mutex_unlock(&instance->serialize);
atm_dbg(instance, "%s: allocated vcc data 0x%p\n", __func__, new); atm_dbg(instance, "%s: allocated vcc data 0x%p\n", __func__, new);
...@@ -875,7 +875,7 @@ static int usbatm_atm_open(struct atm_vcc *vcc) ...@@ -875,7 +875,7 @@ static int usbatm_atm_open(struct atm_vcc *vcc)
fail: fail:
kfree(new); kfree(new);
up(&instance->serialize); mutex_unlock(&instance->serialize);
return ret; return ret;
} }
...@@ -896,7 +896,7 @@ static void usbatm_atm_close(struct atm_vcc *vcc) ...@@ -896,7 +896,7 @@ static void usbatm_atm_close(struct atm_vcc *vcc)
usbatm_cancel_send(instance, vcc); usbatm_cancel_send(instance, vcc);
down(&instance->serialize); /* vs self, usbatm_atm_open, usbatm_usb_disconnect */ mutex_lock(&instance->serialize); /* vs self, usbatm_atm_open, usbatm_usb_disconnect */
tasklet_disable(&instance->rx_channel.tasklet); tasklet_disable(&instance->rx_channel.tasklet);
if (instance->cached_vcc == vcc_data) { if (instance->cached_vcc == vcc_data) {
...@@ -919,7 +919,7 @@ static void usbatm_atm_close(struct atm_vcc *vcc) ...@@ -919,7 +919,7 @@ static void usbatm_atm_close(struct atm_vcc *vcc)
clear_bit(ATM_VF_PARTIAL, &vcc->flags); clear_bit(ATM_VF_PARTIAL, &vcc->flags);
clear_bit(ATM_VF_ADDR, &vcc->flags); clear_bit(ATM_VF_ADDR, &vcc->flags);
up(&instance->serialize); mutex_unlock(&instance->serialize);
atm_dbg(instance, "%s successful\n", __func__); atm_dbg(instance, "%s successful\n", __func__);
} }
...@@ -1009,9 +1009,9 @@ static int usbatm_do_heavy_init(void *arg) ...@@ -1009,9 +1009,9 @@ static int usbatm_do_heavy_init(void *arg)
if (!ret) if (!ret)
ret = usbatm_atm_init(instance); ret = usbatm_atm_init(instance);
down(&instance->serialize); mutex_lock(&instance->serialize);
instance->thread_pid = -1; instance->thread_pid = -1;
up(&instance->serialize); mutex_unlock(&instance->serialize);
complete_and_exit(&instance->thread_exited, ret); complete_and_exit(&instance->thread_exited, ret);
} }
...@@ -1025,9 +1025,9 @@ static int usbatm_heavy_init(struct usbatm_data *instance) ...@@ -1025,9 +1025,9 @@ static int usbatm_heavy_init(struct usbatm_data *instance)
return ret; return ret;
} }
down(&instance->serialize); mutex_lock(&instance->serialize);
instance->thread_pid = ret; instance->thread_pid = ret;
up(&instance->serialize); mutex_unlock(&instance->serialize);
wait_for_completion(&instance->thread_started); wait_for_completion(&instance->thread_started);
...@@ -1110,7 +1110,7 @@ int usbatm_usb_probe(struct usb_interface *intf, const struct usb_device_id *id, ...@@ -1110,7 +1110,7 @@ int usbatm_usb_probe(struct usb_interface *intf, const struct usb_device_id *id,
/* private fields */ /* private fields */
kref_init(&instance->refcount); /* dropped in usbatm_usb_disconnect */ kref_init(&instance->refcount); /* dropped in usbatm_usb_disconnect */
init_MUTEX(&instance->serialize); mutex_init(&instance->serialize);
instance->thread_pid = -1; instance->thread_pid = -1;
init_completion(&instance->thread_started); init_completion(&instance->thread_started);
...@@ -1273,18 +1273,18 @@ void usbatm_usb_disconnect(struct usb_interface *intf) ...@@ -1273,18 +1273,18 @@ void usbatm_usb_disconnect(struct usb_interface *intf)
usb_set_intfdata(intf, NULL); usb_set_intfdata(intf, NULL);
down(&instance->serialize); mutex_lock(&instance->serialize);
instance->disconnected = 1; instance->disconnected = 1;
if (instance->thread_pid >= 0) if (instance->thread_pid >= 0)
kill_proc(instance->thread_pid, SIGTERM, 1); kill_proc(instance->thread_pid, SIGTERM, 1);
up(&instance->serialize); mutex_unlock(&instance->serialize);
wait_for_completion(&instance->thread_exited); wait_for_completion(&instance->thread_exited);
down(&instance->serialize); mutex_lock(&instance->serialize);
list_for_each_entry(vcc_data, &instance->vcc_list, list) list_for_each_entry(vcc_data, &instance->vcc_list, list)
vcc_release_async(vcc_data->vcc, -EPIPE); vcc_release_async(vcc_data->vcc, -EPIPE);
up(&instance->serialize); mutex_unlock(&instance->serialize);
tasklet_disable(&instance->rx_channel.tasklet); tasklet_disable(&instance->rx_channel.tasklet);
tasklet_disable(&instance->tx_channel.tasklet); tasklet_disable(&instance->tx_channel.tasklet);
......
...@@ -34,6 +34,7 @@ ...@@ -34,6 +34,7 @@
#include <linux/list.h> #include <linux/list.h>
#include <linux/stringify.h> #include <linux/stringify.h>
#include <linux/usb.h> #include <linux/usb.h>
#include <linux/mutex.h>
/* /*
#define VERBOSE_DEBUG #define VERBOSE_DEBUG
...@@ -171,7 +172,7 @@ struct usbatm_data { ...@@ -171,7 +172,7 @@ struct usbatm_data {
********************************/ ********************************/
struct kref refcount; struct kref refcount;
struct semaphore serialize; struct mutex serialize;
int disconnected; int disconnected;
/* heavy init */ /* heavy init */
......
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