diff options
| author | Russell King <rmk+kernel@arm.linux.org.uk> | 2011-01-06 17:31:35 -0500 |
|---|---|---|
| committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2011-01-06 17:32:03 -0500 |
| commit | 4ec3eb13634529c0bc7466658d84d0bbe3244aea (patch) | |
| tree | b491daac2ccfc7b8ca88e171a43f66888463568a /drivers/ata/libata-scsi.c | |
| parent | 24056f525051a9e186af28904b396320e18bf9a0 (diff) | |
| parent | 15095bb0fe779c0403091bda7adce5fb3bb9ca35 (diff) | |
Merge branch 'smp' into misc
Conflicts:
arch/arm/kernel/entry-armv.S
arch/arm/mm/ioremap.c
Diffstat (limited to 'drivers/ata/libata-scsi.c')
| -rw-r--r-- | drivers/ata/libata-scsi.c | 24 |
1 files changed, 13 insertions, 11 deletions
diff --git a/drivers/ata/libata-scsi.c b/drivers/ata/libata-scsi.c index d050e073e570..66aa4bee80a6 100644 --- a/drivers/ata/libata-scsi.c +++ b/drivers/ata/libata-scsi.c | |||
| @@ -2552,8 +2552,11 @@ static void atapi_qc_complete(struct ata_queued_cmd *qc) | |||
| 2552 | * | 2552 | * |
| 2553 | * If door lock fails, always clear sdev->locked to | 2553 | * If door lock fails, always clear sdev->locked to |
| 2554 | * avoid this infinite loop. | 2554 | * avoid this infinite loop. |
| 2555 | * | ||
| 2556 | * This may happen before SCSI scan is complete. Make | ||
| 2557 | * sure qc->dev->sdev isn't NULL before dereferencing. | ||
| 2555 | */ | 2558 | */ |
| 2556 | if (qc->cdb[0] == ALLOW_MEDIUM_REMOVAL) | 2559 | if (qc->cdb[0] == ALLOW_MEDIUM_REMOVAL && qc->dev->sdev) |
| 2557 | qc->dev->sdev->locked = 0; | 2560 | qc->dev->sdev->locked = 0; |
| 2558 | 2561 | ||
| 2559 | qc->scsicmd->result = SAM_STAT_CHECK_CONDITION; | 2562 | qc->scsicmd->result = SAM_STAT_CHECK_CONDITION; |
| @@ -3163,8 +3166,8 @@ static inline int __ata_scsi_queuecmd(struct scsi_cmnd *scmd, | |||
| 3163 | 3166 | ||
| 3164 | /** | 3167 | /** |
| 3165 | * ata_scsi_queuecmd - Issue SCSI cdb to libata-managed device | 3168 | * ata_scsi_queuecmd - Issue SCSI cdb to libata-managed device |
| 3169 | * @shost: SCSI host of command to be sent | ||
| 3166 | * @cmd: SCSI command to be sent | 3170 | * @cmd: SCSI command to be sent |
| 3167 | * @done: Completion function, called when command is complete | ||
| 3168 | * | 3171 | * |
| 3169 | * In some cases, this function translates SCSI commands into | 3172 | * In some cases, this function translates SCSI commands into |
| 3170 | * ATA taskfiles, and queues the taskfiles to be sent to | 3173 | * ATA taskfiles, and queues the taskfiles to be sent to |
| @@ -3174,37 +3177,36 @@ static inline int __ata_scsi_queuecmd(struct scsi_cmnd *scmd, | |||
| 3174 | * ATA and ATAPI devices appearing as SCSI devices. | 3177 | * ATA and ATAPI devices appearing as SCSI devices. |
| 3175 | * | 3178 | * |
| 3176 | * LOCKING: | 3179 | * LOCKING: |
| 3177 | * Releases scsi-layer-held lock, and obtains host lock. | 3180 | * ATA host lock |
| 3178 | * | 3181 | * |
| 3179 | * RETURNS: | 3182 | * RETURNS: |
| 3180 | * Return value from __ata_scsi_queuecmd() if @cmd can be queued, | 3183 | * Return value from __ata_scsi_queuecmd() if @cmd can be queued, |
| 3181 | * 0 otherwise. | 3184 | * 0 otherwise. |
| 3182 | */ | 3185 | */ |
| 3183 | int ata_scsi_queuecmd(struct scsi_cmnd *cmd, void (*done)(struct scsi_cmnd *)) | 3186 | int ata_scsi_queuecmd(struct Scsi_Host *shost, struct scsi_cmnd *cmd) |
| 3184 | { | 3187 | { |
| 3185 | struct ata_port *ap; | 3188 | struct ata_port *ap; |
| 3186 | struct ata_device *dev; | 3189 | struct ata_device *dev; |
| 3187 | struct scsi_device *scsidev = cmd->device; | 3190 | struct scsi_device *scsidev = cmd->device; |
| 3188 | struct Scsi_Host *shost = scsidev->host; | ||
| 3189 | int rc = 0; | 3191 | int rc = 0; |
| 3192 | unsigned long irq_flags; | ||
| 3190 | 3193 | ||
| 3191 | ap = ata_shost_to_port(shost); | 3194 | ap = ata_shost_to_port(shost); |
| 3192 | 3195 | ||
| 3193 | spin_unlock(shost->host_lock); | 3196 | spin_lock_irqsave(ap->lock, irq_flags); |
| 3194 | spin_lock(ap->lock); | ||
| 3195 | 3197 | ||
| 3196 | ata_scsi_dump_cdb(ap, cmd); | 3198 | ata_scsi_dump_cdb(ap, cmd); |
| 3197 | 3199 | ||
| 3198 | dev = ata_scsi_find_dev(ap, scsidev); | 3200 | dev = ata_scsi_find_dev(ap, scsidev); |
| 3199 | if (likely(dev)) | 3201 | if (likely(dev)) |
| 3200 | rc = __ata_scsi_queuecmd(cmd, done, dev); | 3202 | rc = __ata_scsi_queuecmd(cmd, cmd->scsi_done, dev); |
| 3201 | else { | 3203 | else { |
| 3202 | cmd->result = (DID_BAD_TARGET << 16); | 3204 | cmd->result = (DID_BAD_TARGET << 16); |
| 3203 | done(cmd); | 3205 | cmd->scsi_done(cmd); |
| 3204 | } | 3206 | } |
| 3205 | 3207 | ||
| 3206 | spin_unlock(ap->lock); | 3208 | spin_unlock_irqrestore(ap->lock, irq_flags); |
| 3207 | spin_lock(shost->host_lock); | 3209 | |
| 3208 | return rc; | 3210 | return rc; |
| 3209 | } | 3211 | } |
| 3210 | 3212 | ||
