On Tue, 2008-01-22 at 20:20 +0000, Hugh Dickins wrote: > On Tue, 22 Jan 2008, James Bottomley wrote: > > > > Actually, I don't think it's a smaller I/O issue. The SMART protocol > > specifically mandates that the transfers for SMART READ DATA and SMART > > READ LOG shall be 512 bytes). However, the pio transfer routine does > > seem to be assuming sector alignment as well, which will be where your > > problems are coming from. I think we need to specify sector minimum > > alignment for ata (but not atapi, which has its own non sector size pio > > routine). How about the attached? > > > > We have to do this for all ATA devices, because they'll likely all > > support SMART, and SMART is defined to be a PIO command. > > Thanks, you've answered several of my uncertainties (why the PIO, > why the sector size). > > I've just tried it, and can confirm that your replacement patch > below fixes the issue for me in practice. Thanks! > What I can't say, you and Jeff and others will judge, is whether that's > actually the right placement for the blk_queue_update_dma_alignment call > (as an outsider, I'm not convinced that the SMART requirement should be > imposing this limitation on the rest). It's certainly the correct placement. The slight problem is that the actual alignment checking is only really done for commands coming down from the user, not for commands incoming from the kernel. That leaves us a potential nasty in IDENTIFY_DEVICE; that's also a PIO only 512 byte transfer command. libsas looks to be OK because it specifically kmallocs a 512 byte buffer which should (for off slab data) be 512 byte aligned. libata actually has an issue because the usual location for IDENTIFY_DEVICE data is inside a struct ata_device, which is highly unlikely to be correctly aligned. Fortunately, I think we can only get the bug if we actually cross a page boundary for non contiguous pages in the identify data, which a kernel allocation will never do, so libata should be safe as well. James - 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
