summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSarah Sharp2009-11-04 20:22:19 +0100
committerGreg Kroah-Hartman2009-11-18 01:46:34 +0100
commit5294bea40666db5c5d6c336b8e4e55d69fa576ca (patch)
treeeb2bcd2cc90b9318d8f61e389918b0479945d822
parentUSB: xhci: Fix TRB physical to virtual address translation. (diff)
downloadkernel-qcow2-linux-5294bea40666db5c5d6c336b8e4e55d69fa576ca.tar.gz
kernel-qcow2-linux-5294bea40666db5c5d6c336b8e4e55d69fa576ca.tar.xz
kernel-qcow2-linux-5294bea40666db5c5d6c336b8e4e55d69fa576ca.zip
USB: xhci: Fix scratchpad deallocation.
The scratchpad_free() function uses xhci->page_size to free some memory with pci_free_consistent(). However, the page_size is set to zero before the call, causing kernel oopses on driver unload. Call scratchpad_free() before setting xhci->page_size to zero. Signed-off-by: Sarah Sharp <sarah.a.sharp@linux.intel.com> Acked-by: John Youn <John.Youn@synopsys.com> Cc: stable <stable@kernel.org> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-rw-r--r--drivers/usb/host/xhci-mem.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/usb/host/xhci-mem.c b/drivers/usb/host/xhci-mem.c
index 732580c6cbea..b8fd270a8b0d 100644
--- a/drivers/usb/host/xhci-mem.c
+++ b/drivers/usb/host/xhci-mem.c
@@ -843,9 +843,9 @@ void xhci_mem_cleanup(struct xhci_hcd *xhci)
xhci->dcbaa, xhci->dcbaa->dma);
xhci->dcbaa = NULL;
+ scratchpad_free(xhci);
xhci->page_size = 0;
xhci->page_shift = 0;
- scratchpad_free(xhci);
}
int xhci_mem_init(struct xhci_hcd *xhci, gfp_t flags)