Commit a70fee3b authored by Julian Wiedmann's avatar Julian Wiedmann Committed by David S. Miller

s390/qeth: use DEFINE_MUTEX for qeth_mod_mutex

Consolidate declaration and initialization of a static variable.
While at it reduce its scope in qeth_core_load_discipline(), and simplify
the return logic accordingly.
Signed-off-by: default avatarJulian Wiedmann <jwi@linux.ibm.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 4fda3354
...@@ -62,7 +62,6 @@ static struct kmem_cache *qeth_qdio_outbuf_cache; ...@@ -62,7 +62,6 @@ static struct kmem_cache *qeth_qdio_outbuf_cache;
static struct device *qeth_core_root_dev; static struct device *qeth_core_root_dev;
static struct lock_class_key qdio_out_skb_queue_key; static struct lock_class_key qdio_out_skb_queue_key;
static struct mutex qeth_mod_mutex;
static void qeth_send_control_data_cb(struct qeth_channel *, static void qeth_send_control_data_cb(struct qeth_channel *,
struct qeth_cmd_buffer *); struct qeth_cmd_buffer *);
...@@ -5589,11 +5588,11 @@ static int qeth_register_dbf_views(void) ...@@ -5589,11 +5588,11 @@ static int qeth_register_dbf_views(void)
return 0; return 0;
} }
static DEFINE_MUTEX(qeth_mod_mutex); /* for synchronized module loading */
int qeth_core_load_discipline(struct qeth_card *card, int qeth_core_load_discipline(struct qeth_card *card,
enum qeth_discipline_id discipline) enum qeth_discipline_id discipline)
{ {
int rc = 0;
mutex_lock(&qeth_mod_mutex); mutex_lock(&qeth_mod_mutex);
switch (discipline) { switch (discipline) {
case QETH_DISCIPLINE_LAYER3: case QETH_DISCIPLINE_LAYER3:
...@@ -5607,14 +5606,15 @@ int qeth_core_load_discipline(struct qeth_card *card, ...@@ -5607,14 +5606,15 @@ int qeth_core_load_discipline(struct qeth_card *card,
default: default:
break; break;
} }
mutex_unlock(&qeth_mod_mutex);
if (!card->discipline) { if (!card->discipline) {
dev_err(&card->gdev->dev, "There is no kernel module to " dev_err(&card->gdev->dev, "There is no kernel module to "
"support discipline %d\n", discipline); "support discipline %d\n", discipline);
rc = -EINVAL; return -EINVAL;
} }
mutex_unlock(&qeth_mod_mutex);
return rc; return 0;
} }
void qeth_core_free_discipline(struct qeth_card *card) void qeth_core_free_discipline(struct qeth_card *card)
...@@ -6623,7 +6623,6 @@ static int __init qeth_core_init(void) ...@@ -6623,7 +6623,6 @@ static int __init qeth_core_init(void)
INIT_LIST_HEAD(&qeth_core_card_list.list); INIT_LIST_HEAD(&qeth_core_card_list.list);
INIT_LIST_HEAD(&qeth_dbf_list); INIT_LIST_HEAD(&qeth_dbf_list);
rwlock_init(&qeth_core_card_list.rwlock); rwlock_init(&qeth_core_card_list.rwlock);
mutex_init(&qeth_mod_mutex);
qeth_wq = create_singlethread_workqueue("qeth_wq"); qeth_wq = create_singlethread_workqueue("qeth_wq");
if (!qeth_wq) { if (!qeth_wq) {
......
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