mwifiex: fix SDIO interrupt lost issue
authorAmitkumar Karwar <[email protected]>
Fri, 27 Sep 2013 17:55:38 +0000 (10:55 -0700)
committerJohn W. Linville <[email protected]>
Mon, 30 Sep 2013 20:19:28 +0000 (16:19 -0400)
commit453b0c3f6910672f79da354077af728d92f95c5b
tree282902e2841cdb85d2cbcee388c7236f70f9fe7e
parentdfb6b7c109a7f98d324a759599d1b4616f02c79f
mwifiex: fix SDIO interrupt lost issue

601216e "mwifiex: process RX packets in SDIO IRQ thread directly"
introduced a command timeout issue which can be reproduced easily on
an AM33xx platform using a test application written by Daniel Mack:

https://gist.github.com/zonque/6579314

mwifiex_main_process() is called from both the SDIO handler and
the workqueue. In case an interrupt occurs right after the
int_status check, but before updating the mwifiex_processing flag,
this interrupt gets lost, resulting in a command timeout and
consequently a card reset.

Let main_proc_lock protect both int_status and mwifiex_processing
flag. This fixes the interrupt lost issue.

Cc: <[email protected]> # 3.7+
Reported-by: Sven Neumann <[email protected]>
Reported-by: Andreas Fenkart <[email protected]>
Tested-by: Daniel Mack <[email protected]>
Reviewed-by: Dylan Reid <[email protected]>
Signed-off-by: Amitkumar Karwar <[email protected]>
Signed-off-by: Bing Zhao <[email protected]>
Signed-off-by: Paul Stewart <[email protected]>
Signed-off-by: John W. Linville <[email protected]>
drivers/net/wireless/mwifiex/main.c