Re: svn commit: r298036 - head/sys/cam

2016-04-15 Thread Howard Su
On Fri, Apr 15, 2016 at 9:26 PM Warner Losh wrote: > On Fri, Apr 15, 2016 at 12:05 AM, Howard Su wrote: > >> >> On Fri, Apr 15, 2016 at 1:10 PM Warner Losh wrote: >> >>> Author: imp >>> Date: Fri Apr 15 05:10:32 2016 >>> New Revision:

Re: svn commit: r298036 - head/sys/cam

2016-04-15 Thread Benjamin Kaduk
On Fri, Apr 15, 2016 at 8:26 AM, Warner Losh wrote: > On Fri, Apr 15, 2016 at 12:05 AM, Howard Su wrote: > > >> I suggest to remove NETFLIX from the name. > > > > It's the Netflix I/O scheduler as opposed to the default one. So the > name is appropriate.

Re: svn commit: r298036 - head/sys/cam

2016-04-15 Thread Howard Su
On Fri, Apr 15, 2016 at 1:10 PM Warner Losh wrote: > Author: imp > Date: Fri Apr 15 05:10:32 2016 > New Revision: 298036 > URL: https://svnweb.freebsd.org/changeset/base/298036 > > Log: > Put function only used by CAM_NETFLIX_IOSCHED under that ifdef. > > I suggest to remove

svn commit: r298036 - head/sys/cam

2016-04-14 Thread Warner Losh
Author: imp Date: Fri Apr 15 05:10:32 2016 New Revision: 298036 URL: https://svnweb.freebsd.org/changeset/base/298036 Log: Put function only used by CAM_NETFLIX_IOSCHED under that ifdef. Modified: head/sys/cam/cam_iosched.c Modified: head/sys/cam/cam_iosched.c