diff options
author | James Bottomley <James.Bottomley@suse.de> | 2009-11-05 14:33:12 -0500 |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2009-12-04 13:01:30 -0500 |
commit | d139b9bd0e52dda14fd13412e7096e68b56d0076 (patch) | |
tree | 518daa6384ff606ec77dd8fe4714ea2f177d92c4 /drivers/scsi/hosts.c | |
parent | 5917290ce9b376866b165d02a5ed88d5ecdb32d0 (diff) |
[SCSI] scsi_lib_dma: fix bug with dma maps on nested scsi objects
Some of our virtual SCSI hosts don't have a proper bus parent at the
top, which can be a problem for doing DMA on them
This patch makes the host device cache a pointer to the physical bus
device and provides an extra API for setting it (the normal API picks
it up from the parent). This patch also modifies the qla2xxx and lpfc
vport logic to use the new DMA host setting API.
Acked-By: James Smart <james.smart@emulex.com>
Cc: Stable Tree <stable@kernel.org>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers/scsi/hosts.c')
-rw-r--r-- | drivers/scsi/hosts.c | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/drivers/scsi/hosts.c b/drivers/scsi/hosts.c index 5fd2da494d08..28a753d796f3 100644 --- a/drivers/scsi/hosts.c +++ b/drivers/scsi/hosts.c | |||
@@ -180,14 +180,20 @@ void scsi_remove_host(struct Scsi_Host *shost) | |||
180 | EXPORT_SYMBOL(scsi_remove_host); | 180 | EXPORT_SYMBOL(scsi_remove_host); |
181 | 181 | ||
182 | /** | 182 | /** |
183 | * scsi_add_host - add a scsi host | 183 | * scsi_add_host_with_dma - add a scsi host with dma device |
184 | * @shost: scsi host pointer to add | 184 | * @shost: scsi host pointer to add |
185 | * @dev: a struct device of type scsi class | 185 | * @dev: a struct device of type scsi class |
186 | * @dma_dev: dma device for the host | ||
187 | * | ||
188 | * Note: You rarely need to worry about this unless you're in a | ||
189 | * virtualised host environments, so use the simpler scsi_add_host() | ||
190 | * function instead. | ||
186 | * | 191 | * |
187 | * Return value: | 192 | * Return value: |
188 | * 0 on success / != 0 for error | 193 | * 0 on success / != 0 for error |
189 | **/ | 194 | **/ |
190 | int scsi_add_host(struct Scsi_Host *shost, struct device *dev) | 195 | int scsi_add_host_with_dma(struct Scsi_Host *shost, struct device *dev, |
196 | struct device *dma_dev) | ||
191 | { | 197 | { |
192 | struct scsi_host_template *sht = shost->hostt; | 198 | struct scsi_host_template *sht = shost->hostt; |
193 | int error = -EINVAL; | 199 | int error = -EINVAL; |
@@ -207,6 +213,7 @@ int scsi_add_host(struct Scsi_Host *shost, struct device *dev) | |||
207 | 213 | ||
208 | if (!shost->shost_gendev.parent) | 214 | if (!shost->shost_gendev.parent) |
209 | shost->shost_gendev.parent = dev ? dev : &platform_bus; | 215 | shost->shost_gendev.parent = dev ? dev : &platform_bus; |
216 | shost->dma_dev = dma_dev; | ||
210 | 217 | ||
211 | error = device_add(&shost->shost_gendev); | 218 | error = device_add(&shost->shost_gendev); |
212 | if (error) | 219 | if (error) |
@@ -262,7 +269,7 @@ int scsi_add_host(struct Scsi_Host *shost, struct device *dev) | |||
262 | fail: | 269 | fail: |
263 | return error; | 270 | return error; |
264 | } | 271 | } |
265 | EXPORT_SYMBOL(scsi_add_host); | 272 | EXPORT_SYMBOL(scsi_add_host_with_dma); |
266 | 273 | ||
267 | static void scsi_host_dev_release(struct device *dev) | 274 | static void scsi_host_dev_release(struct device *dev) |
268 | { | 275 | { |