On Thu, 09 Apr 2009 01:25:40 -0400 Jeff Garzik <jeff@xxxxxxxxxx> wrote: > Alan, > > I keep meaning to mention this (and get it into the mail archive)... > The 32-bit IO patch was incomplete, because it rendered > ATA_IOC_[GS]ET_IO32 handling in ata_sas_scsi_ioctl() -- our base ioctl > handler -- incorrect. > > Previously, it simply ensured the ioctls reflected hardcoded 16-bit I/O. > Now, behavior no longer matches what the ioctls report. > > At a minimum, drivers probably need to set an "I'm using 32bit I/O" > flag, that the ioctl then reads, replacing ata_sas_scsi_ioctl()'s > assumption that val==0. Put onto my TODO list. I missed the fact you were even faking that rather bogus ioctl (what for example does a PIO over DMA device claim ??) -- 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
