usb: renesas_usbhs: fix to avoid using a disabled ep in usbhsg_queue_done()
authorYoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
Mon, 4 Apr 2016 11:40:20 +0000 (20:40 +0900)
committerSasha Levin <sasha.levin@oracle.com>
Wed, 20 Apr 2016 05:03:44 +0000 (01:03 -0400)
commitbe851fafe2b011c725e107b8b70e6ec72bcf0d05
tree0b15e8c64c41c13abb44987775b3c1f8b459fcb2
parent413917195fe1c4f2e02e4a85515587f63cc65c8d
usb: renesas_usbhs: fix to avoid using a disabled ep in usbhsg_queue_done()

[ Upstream commit 4fccb0767fdbdb781a9c5b5c15ee7b219443c89d ]

This patch fixes an issue that usbhsg_queue_done() may cause kernel
panic when dma callback is running and usb_ep_disable() is called
by interrupt handler. (Especially, we can reproduce this issue using
g_audio with usb-dmac driver.)

For example of a flow:
 usbhsf_dma_complete (on tasklet)
  --> usbhsf_pkt_handler (on tasklet)
   --> usbhsg_queue_done (on tasklet)
    *** interrupt happened and usb_ep_disable() is called ***
    --> usbhsg_queue_pop (on tasklet)
     Then, oops happened.

Fixes: e73a989 ("usb: renesas_usbhs: add DMAEngine support")
Cc: <stable@vger.kernel.org> # v3.1+
Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>
Signed-off-by: Sasha Levin <sasha.levin@oracle.com>
drivers/usb/renesas_usbhs/mod_gadget.c