Re: svn commit: r298011 - head/sys/cam/ata

2016-04-15 Thread Ravi Pokala
-Original Message- From: on behalf of Warner Losh Date: 2016-04-15, Friday at 16:15 To: Dmitry Morozovsky Cc: Ronald Klop , "svn-src-head@freebsd.org" , "svn-src-...@freebsd.org" , src-committers , Warner Losh Subject: Re: svn commit: r298011 - head/sys/c

Re: svn commit: r298011 - head/sys/cam/ata

2016-04-15 Thread Warner Losh
On Fri, Apr 15, 2016 at 3:45 PM, Dmitry Morozovsky wrote: > On Fri, 15 Apr 2016, Warner Losh wrote: > > > A shorter timeout dramatically cuts the length of time that the > > queue has to be frozen for error recovery. This allows one, > > in theory, to recover the system more quickly after a drive

Re: svn commit: r298011 - head/sys/cam/ata

2016-04-15 Thread Dmitry Morozovsky
On Fri, 15 Apr 2016, Warner Losh wrote: > A shorter timeout dramatically cuts the length of time that the > queue has to be frozen for error recovery. This allows one, > in theory, to recover the system more quickly after a drive > goes off into the weeds. May be make some more generic changes?

Re: svn commit: r298011 - head/sys/cam/ata

2016-04-15 Thread Warner Losh
A shorter timeout dramatically cuts the length of time that the queue has to be frozen for error recovery. This allows one, in theory, to recover the system more quickly after a drive goes off into the weeds. Warner > On Apr 15, 2016, at 9:23 AM, Ronald Klop wrote: > > Hello, > > Out of curio

Re: svn commit: r298011 - head/sys/cam/ata

2016-04-15 Thread Ronald Klop
Hello, Out of curiosity, what are these big problems? Broken device or other things (timeouts/triggers/full queues) going on in the kernel? Regards, Ronald. On Fri, 15 Apr 2016 00:13:46 +0200, Warner Losh wrote: Author: imp Date: Thu Apr 14 22:13:46 2016 New Revision: 298011 URL: https:/

svn commit: r298011 - head/sys/cam/ata

2016-04-14 Thread Warner Losh
Author: imp Date: Thu Apr 14 22:13:46 2016 New Revision: 298011 URL: https://svnweb.freebsd.org/changeset/base/298011 Log: Add a comment about why the timeout for flush was lowered to 5s. Modified: head/sys/cam/ata/ata_da.c Modified: head/sys/cam/ata/ata_da.c