On Sun, May 26, 2013 at 08:20:24AM +0900, Tejun Heo wrote: > Hey, > > On Sun, May 26, 2013 at 03:16:53AM +0400, Sergei Shtylyov wrote: > > > This value should have been put into the 'dma_boundary' field of the > > >'struct scsi_host_template', IIUC. Tejun, if I do it, do I need > > >this check at all? > > > > You haven't replied, so I went and searched an answer myself. Indeed, > > it's enough to specify the right 'dma_boundary' and stop bothering about it. > > Expect two patches to the driver. I think the correct branch for > > them will be > > 'for-3.11'. > > I was just about to reply. Yes, it should be enough. Both block > layer and the dma machniery honors the DMA boundary and segment size, > which are configured from __scsi_alloc_queue(). But I'm a bit confused about ata_bmdma_fill_sg(). ATA_BMDMA_SHT sets the DMA boundary to 0xffff, so I have no idea why it's explicitly checking the DMA boundary in the function again. What am I missing? -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
