This commit was manufactured by cvs2svn to create tag
[linux-2.6.git] / drivers / usb / storage / usb.c
index bb044ff..1c49b7b 100644 (file)
@@ -50,7 +50,6 @@
 #include <linux/config.h>
 #include <linux/sched.h>
 #include <linux/errno.h>
-#include <linux/suspend.h>
 
 #include <scsi/scsi.h>
 #include <scsi/scsi_cmnd.h>
@@ -98,11 +97,6 @@ MODULE_AUTHOR("Matthew Dharm <mdharm-usb@one-eyed-alien.net>");
 MODULE_DESCRIPTION("USB Mass Storage driver for Linux");
 MODULE_LICENSE("GPL");
 
-static unsigned int delay_use = 5;
-module_param(delay_use, uint, S_IRUGO | S_IWUSR);
-MODULE_PARM_DESC(delay_use, "seconds to delay before using a new device");
-
-
 static int storage_probe(struct usb_interface *iface,
                         const struct usb_device_id *id);
 
@@ -318,8 +312,8 @@ static int usb_stor_control_thread(void * __us)
                /* lock access to the state */
                scsi_lock(host);
 
-               /* has the command timed out *already* ? */
-               if (test_bit(US_FLIDX_TIMED_OUT, &us->flags)) {
+               /* has the command been aborted *already* ? */
+               if (us->sm_state == US_STATE_ABORTING) {
                        us->srb->result = DID_ABORT << 16;
                        goto SkipForAbort;
                }
@@ -330,6 +324,8 @@ static int usb_stor_control_thread(void * __us)
                        goto SkipForDisconnect;
                }
 
+               /* set the state and release the lock */
+               us->sm_state = US_STATE_RUNNING;
                scsi_unlock(host);
 
                /* reject the command if the direction indicator 
@@ -390,15 +386,16 @@ SkipForAbort:
 
                /* If an abort request was received we need to signal that
                 * the abort has finished.  The proper test for this is
-                * the TIMED_OUT flag, not srb->result == DID_ABORT, because
-                * a timeout/abort request might be received after all the
+                * sm_state == US_STATE_ABORTING, not srb->result == DID_ABORT,
+                * because an abort request might be received after all the
                 * USB processing was complete. */
-               if (test_bit(US_FLIDX_TIMED_OUT, &us->flags))
+               if (us->sm_state == US_STATE_ABORTING)
                        complete(&(us->notify));
 
-               /* finished working on this command */
+               /* empty the queue, reset the state, and release the lock */
 SkipForDisconnect:
                us->srb = NULL;
+               us->sm_state = US_STATE_IDLE;
                scsi_unlock(host);
 
                /* unlock the device pointers */
@@ -798,6 +795,7 @@ static int usb_stor_acquire_resources(struct us_data *us)
        us->host->hostdata[0] = (unsigned long) us;
 
        /* Start up our control thread */
+       us->sm_state = US_STATE_IDLE;
        p = kernel_thread(usb_stor_control_thread, us, CLONE_VM);
        if (p < 0) {
                printk(KERN_WARNING USB_STORAGE 
@@ -825,6 +823,7 @@ void usb_stor_release_resources(struct us_data *us)
                /* Wait for the thread to be idle */
                down(&us->dev_semaphore);
                US_DEBUGP("-- sending exit command to thread\n");
+               BUG_ON(us->sm_state != US_STATE_IDLE);
 
                /* If the SCSI midlayer queued a final command just before
                 * scsi_remove_host() was called, us->srb might not be
@@ -883,46 +882,6 @@ static void dissociate_dev(struct us_data *us)
        kfree(us);
 }
 
-/* Thread to carry out delayed SCSI-device scanning */
-static int usb_stor_scan_thread(void * __us)
-{
-       struct us_data *us = (struct us_data *)__us;
-
-       /*
-        * This thread doesn't need any user-level access,
-        * so get rid of all our resources.
-        */
-       lock_kernel();
-       daemonize("usb-stor-scan");
-       unlock_kernel();
-
-       printk(KERN_DEBUG
-               "usb-storage: device found at %d\n", us->pusb_dev->devnum);
-
-       /* Wait for the timeout to expire or for a disconnect */
-       if (delay_use > 0) {
-               printk(KERN_DEBUG "usb-storage: waiting for device "
-                               "to settle before scanning\n");
-retry:
-               wait_event_interruptible_timeout(us->scsi_scan_wait,
-                               test_bit(US_FLIDX_DISCONNECTING, &us->flags),
-                               delay_use * HZ);
-               if (current->flags & PF_FREEZE) {
-                       refrigerator(PF_FREEZE);
-                       goto retry;
-               }
-       }
-
-       /* If the device is still connected, perform the scanning */
-       if (!test_bit(US_FLIDX_DISCONNECTING, &us->flags)) {
-               scsi_scan_host(us->host);
-               printk(KERN_DEBUG "usb-storage: device scan complete\n");
-       }
-
-       complete_and_exit(&us->scsi_scan_done, 0);
-}
-
-
 /* Probe to see if we can drive a newly-connected USB device */
 static int storage_probe(struct usb_interface *intf,
                         const struct usb_device_id *id)
@@ -944,8 +903,6 @@ static int storage_probe(struct usb_interface *intf,
        init_MUTEX_LOCKED(&(us->sema));
        init_completion(&(us->notify));
        init_waitqueue_head(&us->dev_reset_wait);
-       init_waitqueue_head(&us->scsi_scan_wait);
-       init_completion(&us->scsi_scan_done);
 
        /* Associate the us_data structure with the USB device */
        result = associate_dev(us, intf);
@@ -994,10 +951,12 @@ static int storage_probe(struct usb_interface *intf,
        if (result)
                goto BadDevice;
 
-       /* Acquire all the other resources and add the host */
+       /* Acquire all the other resources */
        result = usb_stor_acquire_resources(us);
        if (result)
                goto BadDevice;
+
+       /* Finally, add the host (this does SCSI device scanning) */
        result = scsi_add_host(us->host, &intf->dev);
        if (result) {
                printk(KERN_WARNING USB_STORAGE
@@ -1005,15 +964,10 @@ static int storage_probe(struct usb_interface *intf,
                goto BadDevice;
        }
 
-       /* Start up the thread for delayed SCSI-device scanning */
-       result = kernel_thread(usb_stor_scan_thread, us, CLONE_VM);
-       if (result < 0) {
-               printk(KERN_WARNING USB_STORAGE 
-                      "Unable to start the device-scanning thread\n");
-               scsi_remove_host(us->host);
-               goto BadDevice;
-       }
+       scsi_scan_host(us->host);
 
+       printk(KERN_DEBUG 
+              "USB Mass Storage device found at %d\n", us->pusb_dev->devnum);
        return 0;
 
        /* We come here if there are any problems */
@@ -1037,11 +991,6 @@ static void storage_disconnect(struct usb_interface *intf)
        usb_stor_stop_transport(us);
        wake_up(&us->dev_reset_wait);
 
-       /* Interrupt the SCSI-device-scanning thread's time delay, and
-        * wait for the thread to finish */
-       wake_up(&us->scsi_scan_wait);
-       wait_for_completion(&us->scsi_scan_done);
-
        /* Wait for the current command to finish, then remove the host */
        down(&us->dev_semaphore);
        up(&us->dev_semaphore);
@@ -1063,9 +1012,12 @@ static int __init usb_stor_init(void)
 
        /* register the driver, return usb_register return code if error */
        retval = usb_register(&usb_storage_driver);
-       if (retval == 0)
-               printk(KERN_INFO "USB Mass Storage support registered.\n");
+       if (retval)
+               goto out;
 
+       /* we're all set */
+       printk(KERN_INFO "USB Mass Storage support registered.\n");
+out:
        return retval;
 }