usb: xhci: remove the use of xhci->addr_dev
authorLu Baolu <baolu.lu@linux.intel.com>
Fri, 11 Nov 2016 13:13:30 +0000 (15:13 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 14 Nov 2016 09:18:23 +0000 (10:18 +0100)
xhci->addr_dev is used for the completion of both address device
and enable slot commands. It's shared by enumerations of all USB
devices connected to an xhci host. Hence, it's just a source for
possible races. Since we've introduced command structure and the
command queue to xhci driver.  It is time to get rid of addr_dev
and use the completion in the command structure instead.

Signed-off-by: Lu Baolu <baolu.lu@linux.intel.com>
Signed-off-by: Mathias Nyman <mathias.nyman@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/host/xhci-mem.c
drivers/usb/host/xhci.c
drivers/usb/host/xhci.h

index 8102a86f249cd020ee1e7799287197e25bb76161..5da765a23fe18e68f50a452cb078909b4c2b2ebc 100644 (file)
@@ -2535,7 +2535,6 @@ int xhci_mem_init(struct xhci_hcd *xhci, gfp_t flags)
         * something other than the default (~1ms minimum between interrupts).
         * See section 5.5.1.2.
         */
-       init_completion(&xhci->addr_dev);
        for (i = 0; i < MAX_HC_SLOTS; ++i)
                xhci->devs[i] = NULL;
        for (i = 0; i < USB_MAXCHILDREN; ++i) {
index ea338e05ae27f54769fa5abb141f46f9e4f8e93c..63c902a1f02ab5eb481609ea872a5733c9c8f930 100644 (file)
@@ -3687,20 +3687,19 @@ int xhci_alloc_dev(struct usb_hcd *hcd, struct usb_device *udev)
        int ret, slot_id;
        struct xhci_command *command;
 
-       command = xhci_alloc_command(xhci, false, false, GFP_KERNEL);
+       command = xhci_alloc_command(xhci, false, true, GFP_KERNEL);
        if (!command)
                return 0;
 
        /* xhci->slot_id and xhci->addr_dev are not thread-safe */
        mutex_lock(&xhci->mutex);
        spin_lock_irqsave(&xhci->lock, flags);
-       command->completion = &xhci->addr_dev;
        ret = xhci_queue_slot_control(xhci, command, TRB_ENABLE_SLOT, 0);
        if (ret) {
                spin_unlock_irqrestore(&xhci->lock, flags);
                mutex_unlock(&xhci->mutex);
                xhci_dbg(xhci, "FIXME: allocate a command ring segment\n");
-               kfree(command);
+               xhci_free_command(xhci, command);
                return 0;
        }
        xhci_ring_cmd_db(xhci);
@@ -3715,7 +3714,7 @@ int xhci_alloc_dev(struct usb_hcd *hcd, struct usb_device *udev)
                xhci_err(xhci, "Max number of devices this xHCI host supports is %u.\n",
                                HCS_MAX_SLOTS(
                                        readl(&xhci->cap_regs->hcs_params1)));
-               kfree(command);
+               xhci_free_command(xhci, command);
                return 0;
        }
 
@@ -3751,7 +3750,7 @@ int xhci_alloc_dev(struct usb_hcd *hcd, struct usb_device *udev)
 #endif
 
 
-       kfree(command);
+       xhci_free_command(xhci, command);
        /* Is this a LS or FS device under a HS hub? */
        /* Hub or peripherial? */
        return 1;
@@ -3759,6 +3758,7 @@ int xhci_alloc_dev(struct usb_hcd *hcd, struct usb_device *udev)
 disable_slot:
        /* Disable slot, if we can do it without mem alloc */
        spin_lock_irqsave(&xhci->lock, flags);
+       kfree(command->completion);
        command->completion = NULL;
        command->status = 0;
        if (!xhci_queue_slot_control(xhci, command, TRB_DISABLE_SLOT,
@@ -3820,14 +3820,13 @@ static int xhci_setup_device(struct usb_hcd *hcd, struct usb_device *udev,
                }
        }
 
-       command = xhci_alloc_command(xhci, false, false, GFP_KERNEL);
+       command = xhci_alloc_command(xhci, false, true, GFP_KERNEL);
        if (!command) {
                ret = -ENOMEM;
                goto out;
        }
 
        command->in_ctx = virt_dev->in_ctx;
-       command->completion = &xhci->addr_dev;
 
        slot_ctx = xhci_get_slot_ctx(xhci, virt_dev->in_ctx);
        ctrl_ctx = xhci_get_input_control_ctx(virt_dev->in_ctx);
@@ -3945,7 +3944,10 @@ static int xhci_setup_device(struct usb_hcd *hcd, struct usb_device *udev,
                       le32_to_cpu(slot_ctx->dev_state) & DEV_ADDR_MASK);
 out:
        mutex_unlock(&xhci->mutex);
-       kfree(command);
+       if (command) {
+               kfree(command->completion);
+               kfree(command);
+       }
        return ret;
 }
 
index 2561924c34f26e62f41b8c16c66f952b27831f4d..f24ae0ac12ae53cf7ab2b664fe6ebded2b8f7ddc 100644 (file)
@@ -1584,7 +1584,6 @@ struct xhci_hcd {
        /* slot enabling and address device helpers */
        /* these are not thread safe so use mutex */
        struct mutex mutex;
-       struct completion       addr_dev;
        int slot_id;
        /* For USB 3.0 LPM enable/disable. */
        struct xhci_command             *lpm_command;