mwifiex: Adjust calling place of mwifiex_terminate_workqueue
authorMarc Yang <yangyang@marvell.com>
Wed, 31 Dec 2014 10:36:39 +0000 (02:36 -0800)
committerKalle Valo <kvalo@codeaurora.org>
Tue, 6 Jan 2015 19:19:31 +0000 (21:19 +0200)
Workqueue needs to be flushed early when removing card, otherwise
soft lockup issue may happen because main_process is triggered by
interrupt while card is being removed.

Signed-off-by: Marc Yang <yangyang@marvell.com>
Signed-off-by: Cathy Luo <cluo@marvell.com>
Signed-off-by: Amitkumar Karwar <akarwar@marvell.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/mwifiex/main.c

index 2568395..2d4047f 100644 (file)
@@ -1166,6 +1166,8 @@ int mwifiex_remove_card(struct mwifiex_adapter *adapter, struct semaphore *sem)
 
        adapter->surprise_removed = true;
 
+       mwifiex_terminate_workqueue(adapter);
+
        /* Stop data */
        for (i = 0; i < adapter->priv_num; i++) {
                priv = adapter->priv[i];
@@ -1208,8 +1210,6 @@ int mwifiex_remove_card(struct mwifiex_adapter *adapter, struct semaphore *sem)
        wiphy_unregister(adapter->wiphy);
        wiphy_free(adapter->wiphy);
 
-       mwifiex_terminate_workqueue(adapter);
-
        /* Unregister device */
        dev_dbg(adapter->dev, "info: unregister device\n");
        if (adapter->if_ops.unregister_dev)