Jeff Garzik wrote: > Tejun Heo wrote: >> [cc'ing Albert and linux-ide] >> >> Alan Cox wrote: >>> /from the media. */ >>>> > + if (qc->nbytes < 2048) >>>> > + return -EOPNOTSUPP; >>>> > + >>>> > /* No ATAPI DMA in smart mode */ >>>> > if (itdev->smart) >>>> > return -EOPNOTSUPP; >>>> > >>>> This looks like a gross hack. Aren't you supposed to inspect >>>> the command instead and whitelist the ones you know are OK, >>>> like pata_pdc2027x.c and sata_promise.c do? >>> It does seem to be about transfer size in the IT821x case not commands. >>> It may be to do with how we issue ATAPI command transfer sizes from high >>> up (patch went to Jeff) but for now this is definitely the right >>> approach >>> >>> Reviewed-by: Alan Cox <[EMAIL PROTECTED]> >> >> I wonder whether we should be using similar check in generic path too. >> We have quite a few cases where MWDMA ATAPI devices choking on commands >> with small transfer sizes. I don't think we'll experience significant >> performance regression with this applied and even if there is some, it's >> far better to have slightly slower working device. >> >> What do you guys think? > > Need to look at, or know, a standard profile of submitted commands. It's > quite possible some high performance commands want this, where possible.
Yeah, agreed. I'm quite confused what's going on with all these MWDMA failures and would really like to know what's broken. The reason can be one of the following two. 1. libata itself isn't doing anything wrong but using SCSI for mid and high level drivers are causing problems as more types of commands are used. 2. Mode programming for MWDMA is broken. This doesn't seem too likely as we're seeing MWDMA failures across different low level drivers but is still a possibility. However, it's now like drivers/ide drivers didn't have MWDMA problems. Some drives show similar behavior on IDE too. It's less noticeable because IDE is much quicker to downstep to PIO mode. libata EH logic definitely can use some improvements here. I think the correct thing to do regarding command DMA filtering is to follow what the other OS does. Apparently, that's all ATAPI device vendors test against. Thanks. -- tejun - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/