scsi: lpfc: remove a bogus pci_dma_sync_single_for_device call
authorChristoph Hellwig <hch@lst.de>
Thu, 11 Oct 2018 07:16:13 +0000 (09:16 +0200)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 16 Oct 2018 02:41:00 +0000 (22:41 -0400)
dma_alloc_coherent allocates memory that can be used by the cpu and the
device at the same time, calls to pci_dma_sync_* are not required, and in
fact actively harmful on some architectures like arm.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/lpfc/lpfc_bsg.c

index d53a704..7bd7ae8 100644 (file)
@@ -2844,9 +2844,6 @@ diag_cmd_data_alloc(struct lpfc_hba *phba,
 
                if (nocopydata) {
                        bpl->tus.f.bdeFlags = 0;
-                       pci_dma_sync_single_for_device(phba->pcidev,
-                               dmp->dma.phys, LPFC_BPL_SIZE, PCI_DMA_TODEVICE);
-
                } else {
                        memset((uint8_t *)dmp->dma.virt, 0, cnt);
                        bpl->tus.f.bdeFlags = BUFF_TYPE_BDE_64I;