Jose Alberto Reguero wrote: > I found that the driver work well for Marvel 6121 without the flag > ATA_FLAG_SKIP_D2H_BSY. > What is the meaning of this flag? This is usually meaningful with hotplug. The problem is that, on some SATA controllers including ahci and sata_sil24, the wait for the first D2H Reg FIS indicating device readiness isn't reliable. It's supposed to work like the following (sata_sil behaves pretty good on this regard). 1. User hotplug device to port and PHY event interrupt raised, the controller should set BSY status bit. 2. libata EH kicks in and waits for BSY to clear (current timeout 30s). Done by the first D2H Reg FIS. Drives usually send this after spinup is complete. 3. After BSY is cleared, the device is reset. The problem with ahci and sata_sil24 is that there really is no TF status register is non-existent or emulated in some way and they doesn't let us reliably wait for !BSY after hotplug event. This often makes the wait in #2 timeout and that's 30 seconds. :-( So, the flag makes libata EH skip #2. Post-reset waiting is reliable, so there is no timeout. But this causes problems on some devices (the Quantum GoVault thing) which takes a looooong time to get ready and don't respond correctly to phy reset while it's busy initializing. The flag is there primarily for ICH ahci's. Marvell might be different. Care to remove SKIP_D2H_BSY and test a LOT of hotplugging with several different devices? :-) -- tejun - 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
