Hello, Alan. Alan Cox wrote: > On Iau, 2006-05-11 at 22:21 +0900, Tejun Heo wrote: >> Convert sata_sil to new EH. As these controllers have hardware >> interrupt mask and are known to have screaming interrupts issues, use >> hardware IRQ masking for freezing. sil_freeze() masks interrupts for >> the port and sil_thaw() unmasks them. As ports are automatically >> frozen before probing reset, there is no need to initialize interrupt >> masks sil_init_onde(). Remove related code. > > Word of caution. When you mask an interrupt on chip like this remember > that an IRQ may still be pending so the freeze is *not* safe for > synchronization, merely for shutting the chip up. Otherwise this patch > set looks pretty nice. I'm not sure what you mean by synchronization. But, freezing is indeed for shutting the chip up. If the port is frozen, EH doesn't touch active qcs nor tries to issue new command. The port is thawed/used only after the port & device are put into known good state via successful reset. Is this what you're talking about? > Patchsets all look good with my PATA hat on. The dev->ap stuff tidies up > the ->data_xfer methods I need to add and the other stuff fits nicely. Great. I haven't removed @ap from methods in ata_port_operations() yet because the change affects all LLDDs and later ata_link addition would require another sweep all over again. I think the change can wait until EH stuff settles down. Thanks for reviewing this. -- tejun - : 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
