Hello, Jeff Garzik wrote: > On Tue, Jun 05, 2007 at 11:31:46PM +0200, Mikael Pettersson wrote: >> I can easily reproduce the problem in 2.6.22-rc4. There are no >> sata_promise changes between rc3 and rc4, but Tejun's libata >> polling SETXFER change was included in rc4. Reverting it makes >> sata_promise work again for me. > > Ugh.
Ugh... >> I suspect that sata_promise.c:pdc_interrupt() should detect >> a qc w/ ATA_TFLAG_POLLING, treat the interrupt as spurious, >> and just call ata_chk_status(qc), similar to how sata_inic162x.c, >> sata_nv.c, sata_sil.c, and sata_vsc.c do things. > > Yes, highly likely. I'm not sure whether that will work. I'll give a shot at it here. > SFF-like controllers (and in this case, Promise is included in that > list) with their own interrupt handlers need their own polling handling > code. One thing I don't understand is why IDENTIFY works. IDENTIFY uses polling too. Hmm... -- 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/
