scsi: core: Delete scsi_{get,free}_host_dev()
authorJohn Garry <john.garry@huawei.com>
Mon, 13 Sep 2021 10:14:07 +0000 (18:14 +0800)
committerMartin K. Petersen <martin.petersen@oracle.com>
Wed, 29 Sep 2021 03:42:00 +0000 (23:42 -0400)
Since commit 0653c358d2dc ("scsi: Drop gdth driver"), functions
scsi_{get,free}_host_dev() no longer have any in-tree users, so delete
them.

Link: https://lore.kernel.org/r/1631528047-30150-1-git-send-email-john.garry@huawei.com
Reviewed-by: Bart Van Assche <bvanassche@acm.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: John Garry <john.garry@huawei.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Nacked-by: Hannes Reinecke <hare@suse.de>
drivers/scsi/scsi_scan.c
include/scsi/scsi_host.h

index fe22191..0d0381d 100644 (file)
@@ -1902,60 +1902,3 @@ void scsi_forget_host(struct Scsi_Host *shost)
        spin_unlock_irqrestore(shost->host_lock, flags);
 }
 
-/**
- * scsi_get_host_dev - Create a scsi_device that points to the host adapter itself
- * @shost: Host that needs a scsi_device
- *
- * Lock status: None assumed.
- *
- * Returns:     The scsi_device or NULL
- *
- * Notes:
- *     Attach a single scsi_device to the Scsi_Host - this should
- *     be made to look like a "pseudo-device" that points to the
- *     HA itself.
- *
- *     Note - this device is not accessible from any high-level
- *     drivers (including generics), which is probably not
- *     optimal.  We can add hooks later to attach.
- */
-struct scsi_device *scsi_get_host_dev(struct Scsi_Host *shost)
-{
-       struct scsi_device *sdev = NULL;
-       struct scsi_target *starget;
-
-       mutex_lock(&shost->scan_mutex);
-       if (!scsi_host_scan_allowed(shost))
-               goto out;
-       starget = scsi_alloc_target(&shost->shost_gendev, 0, shost->this_id);
-       if (!starget)
-               goto out;
-
-       sdev = scsi_alloc_sdev(starget, 0, NULL);
-       if (sdev)
-               sdev->borken = 0;
-       else
-               scsi_target_reap(starget);
-       put_device(&starget->dev);
- out:
-       mutex_unlock(&shost->scan_mutex);
-       return sdev;
-}
-EXPORT_SYMBOL(scsi_get_host_dev);
-
-/**
- * scsi_free_host_dev - Free a scsi_device that points to the host adapter itself
- * @sdev: Host device to be freed
- *
- * Lock status: None assumed.
- *
- * Returns:     Nothing
- */
-void scsi_free_host_dev(struct scsi_device *sdev)
-{
-       BUG_ON(sdev->id != sdev->host->this_id);
-
-       __scsi_remove_device(sdev);
-}
-EXPORT_SYMBOL(scsi_free_host_dev);
-
index 7536370..bc9c45c 100644 (file)
@@ -798,16 +798,6 @@ void scsi_host_busy_iter(struct Scsi_Host *,
 struct class_container;
 
 /*
- * These two functions are used to allocate and free a pseudo device
- * which will connect to the host adapter itself rather than any
- * physical device.  You must deallocate when you are done with the
- * thing.  This physical pseudo-device isn't real and won't be available
- * from any high-level drivers.
- */
-extern void scsi_free_host_dev(struct scsi_device *);
-extern struct scsi_device *scsi_get_host_dev(struct Scsi_Host *);
-
-/*
  * DIF defines the exchange of protection information between
  * initiator and SBC block device.
  *