Hello, Stuart_Hayes@xxxxxxxx wrote: > Tejun Heo wrote: >> Hello, >> >> akpm@xxxxxxxxxxxxxxxxxxxx wrote: >>> From: Stuart Hayes <stuart_hayes@xxxxxxxx> >>> >>> This fixes problems during resume with drives that take longer than >>> 1s to be ready. The ATA-6 spec appears to allow 5 seconds for a >>> drive to be ready. >>> >>> On one affected system, this patch changes "PM: resume devices >>> took..." message from 17 seconds to 4 seconds, and gets rid of a lot >>> of ugly timeout/error messages. >> Can you please attach log for this? Which controller was it? >> > > Here's the relevant bits of the log... I don't have a copy of a whole > log, but I should be able to get one if there's something missing that > you need from this. I had a more detailed description of what was > happening in a question I posted to linux-scsi back in January > (http://kerneltrap.org/index.php?q=mailarchive/linux-scsi/2009/1/20/4777 > 124). After looking at the code, I think your patch is doing the right thing. I was worried about controllers which don't have any mechanism to pass in the first D2H Reg FIS to the driver (sil24, for example) but in those cases the wait logic there isn't used at all. The only problem would be with some port multipliers which don't send first D2H Reg FIS after hardreset on controllers which can act on them (ahci). This will incur 5sec detection delay on those devices but well I think we can trade them off for more common cases. late-but-Acked-by: Tejun Heo <tj@xxxxxxxxxx> -- 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
