> Hmm... I'm hearing different stories here. Some say 7sec is good enough > and the supporting argument that the other os is using 7 sec timeout is > pretty convincing. Firstly what is your goal ? > > After command timeout, EH will kick in, reset and revalidate then return > the control to SCSI HLD, sd in this case which will retry the command > several times. Would the reset make the drive to exit recovery mode > such that it has to start all over again when the command is retried? > Or does the drive stop responding to resets while in recovery mode? The > latter would be fine. libata gives and will continue to give more than > 30 secs for drive to respond to reset. > > Ideas? Some thinking based on discussion so far: Give it 7 seconds for a command to complete. At that point you need to have a chat with the drive since its probably gone AWOL. If you get an error within 7 seconds then its different. For a reported error within 7 seconds (CRC excepted) do fast error cleanup (not a reset) when we can if need be reset On the first media error/timeout , retry the command but with the segment list inverted On the second reset retry each block individually User specified time (allowing for recovery/queueing time) later, if we haven't got the drive back running then give up on those blocks. The reason I suggest issuing the segment list inverted is that its statistically likely we only hit one bad area. It's also likey that bad area is adjacent blocks (it might not be for some failures I grant). So if we retry from the other end of the list we have a good chance that a typical list of blocks (say 1-4, 300-305, 380-383, 440-456, 511, 535) that blows up in the middle somewhere is going to run through the other good segments and get them on disk or off disk and stuff up and going again before stalling at the other end of the faulty area. Alan - 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
