On Mon, 2014-03-17 at 12:52 -0400, Josh Boyer wrote: > Hi All, > > We've had a few reports[1][2] now on the vmw_pvscsi driver throwing > DMA-API mapping errors when the DMA_API_DEBUG option is enabled. I've > included one from a recent 3.14-rc6 kernel below. Looking at > vmw_pvscsi.c, I can see pvscsi_map_buffers is missing the calls to > pci_dma_mapping_error, which is what causes the warnings to be thrown. > > However, I'm not familiar with this driver and I can't see what the > proper error path should be in this case. pvscsi_map_buffers is a > void function and doesn't currently have the ability to return an > error to the caller. Even if it did, I'm not sure what the proper > response to an error should be. Thoughts? It's missing checking in two places: from pci_map_single() and on scsi_dma_map(). In the latter case, pvscsi_map_buffers needs to return an error code and then pvscsi_queue_ring can return -1. James -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
