--- orig/drivers/scsi/scsi_syms.c 2003-01-29 17:13:18.000000000 -0600 +++ linux-2.4.20/drivers/scsi/scsi_syms.c 2003-01-29 17:13:18.000000000 -0600 @@ -103,3 +103,9 @@ extern int scsi_delete_timer(Scsi_Cmnd *); EXPORT_SYMBOL(scsi_add_timer); EXPORT_SYMBOL(scsi_delete_timer); + +/* Support for hot plugging and unplugging devices -- safe for + * ieee1394 or USB devices, but probably not for normal SCSI... */ +EXPORT_SYMBOL(scsi_add_single_device); +EXPORT_SYMBOL(scsi_remove_single_device); + --- orig/drivers/scsi/hosts.h 2003-01-29 17:13:18.000000000 -0600 +++ linux-2.4.20/drivers/scsi/hosts.h 2003-01-29 17:13:18.000000000 -0600 @@ -535,6 +535,13 @@ int scsi_register_device(struct Scsi_Device_Template * sdpnt); void scsi_deregister_device(struct Scsi_Device_Template * tpnt); +/* Support for hot plugging and unplugging devices -- safe for + * ieee1394 or USB devices, but probably not for normal SCSI... */ +extern int scsi_add_single_device(struct Scsi_Host *shpnt, + int channel, int id, int lun); +extern int scsi_remove_single_device(struct Scsi_Host *shpnt, + int channel, int id, int lun); + /* These are used by loadable modules */ extern int scsi_register_module(int, void *); extern int scsi_unregister_module(int, void *); --- orig/drivers/scsi/scsi.c 2003-01-29 17:13:18.000000000 -0600 +++ linux-2.4.20/drivers/scsi/scsi.c 2003-01-29 17:13:18.000000000 -0600 @@ -1553,6 +1553,156 @@ } } + +static DECLARE_MUTEX(scsi_host_internals_lock); +/* + * Function: scsi_add_single_device() + * + * Purpose: Support for hotplugging SCSI devices. This function + * implements the actual functionality for + * echo "scsi add-single-device 0 1 2 3" >/proc/scsi/scsi + * + * Arguments: shpnt - pointer to the SCSI host structure + * channel - channel of the device to add + * id - id of the device to add + * lun - lun of the device to add + * + * Returns: 0 on success or an error code + * + * Lock status: None needed. + * + * Notes: This feature is probably unsafe for standard SCSI devices, + * but is perfectly normal for things like ieee1394 or USB + * drives since these busses are designed for hotplugging. + * Use at your own risk.... + */ +int scsi_add_single_device(struct Scsi_Host *shpnt, int channel, + int id, int lun) +{ + Scsi_Device *scd; + + /* Do a bit of sanity checking */ + if (shpnt==NULL) { + return -ENXIO; + } + + /* We call functions that can sleep, so use a semaphore to + * avoid racing with scsi_remove_single_device(). We probably + * need to also apply this lock to scsi_register*(), + * scsi_unregister*(), sd_open(), sd_release() and anything + * else that might be messing with with the Scsi_Host or other + * fundamental data structures. */ + down(&scsi_host_internals_lock); + + /* Check if they asked us to add an already existing device. + * If so, ignore their misguided efforts. */ + for (scd = shpnt->host_queue; scd; scd = scd->next) { + if ((scd->channel == channel && scd->id == id && scd->lun == lun)) { + break; + } + } + if (scd) { + up(&scsi_host_internals_lock); + return -ENOSYS; + } + + scan_scsis(shpnt, 1, channel, id, lun); + up(&scsi_host_internals_lock); + return 0; +} + +/* + * Function: scsi_remove_single_device() + * + * Purpose: Support for hot-unplugging SCSI devices. This function + * implements the actual functionality for + * echo "scsi remove-single-device 0 1 2 3" >/proc/scsi/scsi + * + * Arguments: shpnt - pointer to the SCSI host structure + * channel - channel of the device to add + * id - id of the device to add + * lun - lun of the device to add + * + * Returns: 0 on success or an error code + * + * Lock status: None needed. + * + * Notes: This feature is probably unsafe for standard SCSI devices, + * but is perfectly normal for things like ieee1394 or USB + * drives since these busses are designed for hotplugging. + * Use at your own risk.... + */ +int scsi_remove_single_device(struct Scsi_Host *shpnt, int channel, + int id, int lun) +{ + Scsi_Device *scd; + struct Scsi_Device_Template *SDTpnt; + + /* Do a bit of sanity checking */ + if (shpnt==NULL) { + return -ENODEV; + } + + /* We call functions that can sleep, so use a semaphore to + * avoid racing with scsi_add_single_device(). We probably + * need to also apply this lock to scsi_register*(), + * scsi_unregister*(), sd_open(), sd_release() and anything + * else that might be messing with with the Scsi_Host or other + * fundamental data structures. */ + down(&scsi_host_internals_lock); + + /* Make sure the specified device is in fact present */ + for (scd = shpnt->host_queue; scd; scd = scd->next) { + if ((scd->channel == channel && scd->id == id && scd->lun == lun)) { + break; + } + } + if (scd==NULL) { + up(&scsi_host_internals_lock); + return -ENODEV; + } + + /* See if the specified device is busy. Doesn't this race with + * sd_open(), sd_release() and similar? Why don't they lock + * things when they increment/decrement the access_count? */ + if (scd->access_count) { + up(&scsi_host_internals_lock); + return -EBUSY; + } + + SDTpnt = scsi_devicelist; + while (SDTpnt != NULL) { + if (SDTpnt->detach) + (*SDTpnt->detach) (scd); + SDTpnt = SDTpnt->next; + } + + if (scd->attached == 0) { + /* Nobody is using this device, so we + * can now free all command structures. */ + if (shpnt->hostt->revoke) + shpnt->hostt->revoke(scd); + devfs_unregister (scd->de); + scsi_release_commandblocks(scd); + + /* Now we can remove the device structure */ + if (scd->next != NULL) + scd->next->prev = scd->prev; + + if (scd->prev != NULL) + scd->prev->next = scd->next; + + if (shpnt->host_queue == scd) { + shpnt->host_queue = scd->next; + } + blk_cleanup_queue(&scd->request_queue); + kfree((char *) scd); + } + + up(&scsi_host_internals_lock); + return 0; +} + #ifdef CONFIG_PROC_FS static int scsi_proc_info(char *buffer, char **start, off_t offset, int length) { @@ -1605,8 +1755,6 @@ static int proc_scsi_gen_write(struct file * file, const char * buf, unsigned long length, void *data) { - struct Scsi_Device_Template *SDTpnt; - Scsi_Device *scd; struct Scsi_Host *HBA_ptr; char *p; int host, channel, id, lun; @@ -1721,13 +1869,12 @@ /* * Usage: echo "scsi add-single-device 0 1 2 3" >/proc/scsi/scsi * with "0 1 2 3" replaced by your "Host Channel Id Lun". - * Consider this feature BETA. + * + * Consider this feature pre-BETA. + * * CAUTION: This is not for hotplugging your peripherals. As * SCSI was not designed for this you could damage your - * hardware ! - * However perhaps it is legal to switch on an - * already connected device. It is perhaps not - * guaranteed this device doesn't corrupt an ongoing data transfer. + * hardware and thoroughly confuse the SCSI subsystem. */ if (!strncmp("add-single-device", buffer + 5, 17)) { p = buffer + 23; @@ -1745,33 +1892,11 @@ break; } } - err = -ENXIO; - if (!HBA_ptr) - goto out; - - for (scd = HBA_ptr->host_queue; scd; scd = scd->next) { - if ((scd->channel == channel - && scd->id == id - && scd->lun == lun)) { - break; - } - } - - err = -ENOSYS; - if (scd) - goto out; /* We do not yet support unplugging */ - - scan_scsis(HBA_ptr, 1, channel, id, lun); - - /* FIXME (DB) This assumes that the queue_depth routines can be used - in this context as well, while they were all designed to be - called only once after the detect routine. (DB) */ - /* queue_depth routine moved to inside scan_scsis(,1,,,) so - it is called before build_commandblocks() */ - - err = length; + if ((err=scsi_add_single_device(HBA_ptr, channel, id, lun))==0) + err = length; goto out; } + /* * Usage: echo "scsi remove-single-device 0 1 2 3" >/proc/scsi/scsi * with "0 1 2 3" replaced by your "Host Channel Id Lun". @@ -1781,7 +1906,6 @@ * CAUTION: This is not for hotplugging your peripherals. As * SCSI was not designed for this you could damage your * hardware and thoroughly confuse the SCSI subsystem. - * */ else if (!strncmp("remove-single-device", buffer + 5, 20)) { p = buffer + 26; @@ -1797,58 +1921,8 @@ break; } } - err = -ENODEV; - if (!HBA_ptr) - goto out; - - for (scd = HBA_ptr->host_queue; scd; scd = scd->next) { - if ((scd->channel == channel - && scd->id == id - && scd->lun == lun)) { - break; - } - } - - if (scd == NULL) - goto out; /* there is no such device attached */ - - err = -EBUSY; - if (scd->access_count) - goto out; - - SDTpnt = scsi_devicelist; - while (SDTpnt != NULL) { - if (SDTpnt->detach) - (*SDTpnt->detach) (scd); - SDTpnt = SDTpnt->next; - } - - if (scd->attached == 0) { - /* - * Nobody is using this device any more. - * Free all of the command structures. - */ - if (HBA_ptr->hostt->revoke) - HBA_ptr->hostt->revoke(scd); - devfs_unregister (scd->de); - scsi_release_commandblocks(scd); - - /* Now we can remove the device structure */ - if (scd->next != NULL) - scd->next->prev = scd->prev; - - if (scd->prev != NULL) - scd->prev->next = scd->next; - - if (HBA_ptr->host_queue == scd) { - HBA_ptr->host_queue = scd->next; - } - blk_cleanup_queue(&scd->request_queue); - kfree((char *) scd); - } else { - goto out; - } - err = 0; + err=scsi_remove_single_device(HBA_ptr, channel, id, lun); + goto out; } out: