RE: [PATCH]: Flexible timeout infrastructure

James Bottomley
2004-06-16T17:38:34Z

On Wed, 2004-06-16 at 12:33, Smart, James wrote:
> I'll try to restate. The patch proposed to allow only 1 restart of the timer
> for a command. My comment was - why not 2, or 3? I would think - the number
> of restarts needed is a function of how long the timeout value is vs when
> the LLDD event occurs and what the duration of the LLDD event is. I can see
> capping the number of timer restarts, but am not sure 1 is the best choice.

It does. But the number of allowed restarts is linked to the command
retries (actually allowed retries + 1).

James