On Wed, Oct 19, 2011 at 5:07 PM, Tejun Heo <htejun@xxxxxxxxx> wrote: > Hello, > > On Wed, Oct 19, 2011 at 05:02:42PM -0700, Gwendal Grignou wrote: >> Reenable sending SRST to devices connected behind a Sil3726 PMP. >> This allow staggered spinups and handles drives that spins up slowly. >> >> While the drives spin up, the PMP will not accept SRST. >> Most controller reissues the reset until the drive is ready, while >> some [Sil3124] returns an error. >> In ata_eh_error, wait 10s before reset the ATA port and try again. > > While I agree with the change, the description doesn't seem too > accurate. > > * The behavior change applies to all PMPs. I just re-enable SRST for Sil3726, not the other PMPs. For the other ones where ATA_LFLAG_NO_SRST is still set, we will not issue softreset and therefore will not try to access the spining disk until we send the first identify. For the other PMP where SRST was allowed, I guess the Sil3132 controller would have offline the disks or downgrade the speed unnecessarily. > > * I hope 3726 change is in a separate patch. > > * It waits for reset deadline which currently happens to be 10s. That's correct. I update the description. Gwendal. > > Thanks. > > -- > 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