usb: renesas_usbhs: add a workaround for a race condition of workqueue
authorYoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
Wed, 26 Jun 2019 13:06:33 +0000 (22:06 +0900)
committerFelipe Balbi <felipe.balbi@linux.intel.com>
Wed, 3 Jul 2019 08:00:50 +0000 (11:00 +0300)
commitb2357839c56ab7d06bcd4e866ebc2d0e2b7997f3
tree2c7d39d5ef4eda08f971cf77a7dab195c1f68b81
parent4aef7966060439bb4f9d440feacd5ce7697bce37
usb: renesas_usbhs: add a workaround for a race condition of workqueue

The old commit 6e4b74e4690d ("usb: renesas: fix scheduling in atomic
context bug") fixed an atomic issue by using workqueue for the shdmac
dmaengine driver. However, this has a potential race condition issue
between the work pending and usbhsg_ep_free_request() in gadget mode.
When usbhsg_ep_free_request() is called while pending the queue,
since the work_struct will be freed and then the work handler is
called, kernel panic happens on process_one_work().

To fix the issue, if we could call cancel_work_sync() at somewhere
before the free request, it could be easy. However,
the usbhsg_ep_free_request() is called on atomic (e.g. f_ncm driver
calls free request via gether_disconnect()).

For now, almost all users are having "USB-DMAC" and the DMAengine
driver can be used on atomic. So, this patch adds a workaround for
a race condition to call the DMAengine APIs without the workqueue.

This means we still have TODO on shdmac environment (SH7724), but
since it doesn't have SMP, the race condition might not happen.

Fixes: ab330cf3888d ("usb: renesas_usbhs: add support for USB-DMAC")
Cc: <stable@vger.kernel.org> # v4.1+
Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>
drivers/usb/renesas_usbhs/fifo.c