Re: svn commit: r326036 - head/sys/cam/scsi

2017-11-22 Thread Justin T. Gibbs
I'm sure it was to improve the performance of some SSD we were testing at Spectra. -- Justin > On Nov 22, 2017, at 9:16 AM, Alan Somers wrote: > > I believe the original motivation was to reduce FLUSH CACHE EXT > activity with some old SSDs that handled that command very slowly. > gibbs might

Re: svn commit: r326036 - head/sys/cam/scsi

2017-11-22 Thread Alan Somers
I believe the original motivation was to reduce FLUSH CACHE EXT activity with some old SSDs that handled that command very slowly. gibbs might know more. On Tue, Nov 21, 2017 at 1:14 PM, Ronald Klop wrote: > Out of curiosity, What is the use case which this improves? Does happen in > daily usage

Re: svn commit: r326036 - head/sys/cam/scsi

2017-11-21 Thread Ronald Klop
Out of curiosity, What is the use case which this improves? Does happen in daily usage or mostly in testing? Regards, Ronald. On Mon, 20 Nov 2017 23:27:34 +0100, Alan Somers wrote: Author: asomers Date: Mon Nov 20 22:27:33 2017 New Revision: 326036 URL: https://svnweb.freebsd.org/change

svn commit: r326036 - head/sys/cam/scsi

2017-11-20 Thread Alan Somers
Author: asomers Date: Mon Nov 20 22:27:33 2017 New Revision: 326036 URL: https://svnweb.freebsd.org/changeset/base/326036 Log: da(4): Short-circuit unnecessary BIO_FLUSH commands sys/cam/scsi/scsi_da.c Complete BIO_FLUSH commands immediately if the da(4) device hasn't been w