diff options
author | Hannes Reinecke <hare@suse.de> | 2014-11-24 09:37:26 -0500 |
---|---|---|
committer | Christoph Hellwig <hch@lst.de> | 2014-11-24 10:13:15 -0500 |
commit | 6df388f2d549d3a2a7ad58b632d2ecd25fc0ff3f (patch) | |
tree | 7b663c4ebd1da38079723da2989e7f44388a52f2 /drivers/scsi/am53c974.c | |
parent | 3a7e7be2a9a2105742c3c88d466ea2158a03a837 (diff) |
am53c974: BLAST residual handling
The am53c974 has an design issue where a single byte might be
left in the SCSI FIFO after a DMA transfer.
As the handling code is currently untested add a WARN_ON()
statement here.
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Acked-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Hannes Reinecke <hare@suse.de>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'drivers/scsi/am53c974.c')
-rw-r--r-- | drivers/scsi/am53c974.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/drivers/scsi/am53c974.c b/drivers/scsi/am53c974.c index a16c9cf0f0a2..a42e8db057a4 100644 --- a/drivers/scsi/am53c974.c +++ b/drivers/scsi/am53c974.c | |||
@@ -200,6 +200,12 @@ static void pci_esp_dma_drain(struct esp *esp) | |||
200 | } | 200 | } |
201 | pci_esp_write8(esp, ESP_DMA_CMD_DIR | ESP_DMA_CMD_IDLE, ESP_DMA_CMD); | 201 | pci_esp_write8(esp, ESP_DMA_CMD_DIR | ESP_DMA_CMD_IDLE, ESP_DMA_CMD); |
202 | esp_dma_log("DMA blast done (%d tries, %d bytes left)\n", lim, resid); | 202 | esp_dma_log("DMA blast done (%d tries, %d bytes left)\n", lim, resid); |
203 | /* BLAST residual handling is currently untested */ | ||
204 | if (WARN_ON_ONCE(resid == 1)) { | ||
205 | struct esp_cmd_entry *ent = esp->active_cmd; | ||
206 | |||
207 | ent->flags |= ESP_CMD_FLAG_RESIDUAL; | ||
208 | } | ||
203 | } | 209 | } |
204 | 210 | ||
205 | static void pci_esp_dma_invalidate(struct esp *esp) | 211 | static void pci_esp_dma_invalidate(struct esp *esp) |