Re: svn commit: r326733 - head/usr.sbin/acpi/acpiconf

2017-12-09 Thread Bruce Evans
On Sat, 9 Dec 2017, Niclas Zeising wrote: Log: Improve options and error handling. Improve options handling and error out if multiple mutually exclusive options are passed to acpiconf. Switch from using atoi() to strtol() for argument parsing, and add error checking and handling, instead o

svn commit: r326739 - head/sys/powerpc/booke

2017-12-09 Thread Justin Hibbits
Author: jhibbits Date: Sun Dec 10 04:43:27 2017 New Revision: 326739 URL: https://svnweb.freebsd.org/changeset/base/326739 Log: Retrieve the page outside of holding locks pmap_track_page() only works with physical memory pages, which have a constant vm_page_t address. Microoptimize pmap_

Re: svn commit: r326731 - head/sys/ufs/ffs

2017-12-09 Thread Mark Johnston
On Sat, Dec 09, 2017 at 07:36:59PM -0700, Warner Losh wrote: > On Sat, Dec 9, 2017 at 11:03 AM, Andriy Gapon wrote: > > > On 09/12/2017 17:44, Mark Johnston wrote: > > > Some GEOMs do not appear to handle BIO_ORDERED correctly, meaning that > > the > > > barrier write may not work as intended. >

Re: svn commit: r326731 - head/sys/ufs/ffs

2017-12-09 Thread Warner Losh
On Sat, Dec 9, 2017 at 11:03 AM, Andriy Gapon wrote: > On 09/12/2017 17:44, Mark Johnston wrote: > > Some GEOMs do not appear to handle BIO_ORDERED correctly, meaning that > the > > barrier write may not work as intended. There's a few places we send down a BIO_ORDERED BIO_FLUSH command (see so

Re: svn commit: r326731 - head/sys/ufs/ffs

2017-12-09 Thread Warner Losh
On Sat, Dec 9, 2017 at 5:17 PM, Poul-Henning Kamp wrote: > > In message p...@mail.gmail.com>, Warner Losh writes: > > >That would be strange given that BIO_ORDERED is @gibbs baby ? > > > >Nah... I wrote the iosched code... and I find the concept somewhat flawed > >since it is at the dis

Re: svn commit: r326731 - head/sys/ufs/ffs

2017-12-09 Thread Poul-Henning Kamp
In message , Warner Losh writes: >That would be strange given that BIO_ORDERED is @gibbs baby ? > >Nah... I wrote the iosched code... and I find the concept somewhat flawed >since it is at the disk level, not the partition level, so it winds up >interfering with mixed traffic. And it re

Re: svn commit: r326738 - head/usr.sbin/pw

2017-12-09 Thread Eugene Grosbein
10.12.2017 6:34, Eugene Grosbein пишет: > Author: eugen > Date: Sat Dec 9 23:34:00 2017 > New Revision: 326738 > URL: https://svnweb.freebsd.org/changeset/base/326738 > > Log: > pw(8): correct expiration period handling and command line overrides > to preconfigured values for -e, -p and -w fl

Re: svn commit: r326731 - head/sys/ufs/ffs

2017-12-09 Thread Warner Losh
On Dec 9, 2017 4:27 PM, "Poul-Henning Kamp" wrote: In message , Warner Losh writes: >I also noticed that gsched doesn't take BIO_ORDERED into account when >sorting requests. Isilon has an I/O scheduler which has this problem >too > >I think the cam iosched ignores it too. That would be

svn commit: r326738 - head/usr.sbin/pw

2017-12-09 Thread Eugene Grosbein
Author: eugen Date: Sat Dec 9 23:34:00 2017 New Revision: 326738 URL: https://svnweb.freebsd.org/changeset/base/326738 Log: pw(8): correct expiration period handling and command line overrides to preconfigured values for -e, -p and -w flags. Use non-negative symbols instead of magic valu

Re: svn commit: r326731 - head/sys/ufs/ffs

2017-12-09 Thread Poul-Henning Kamp
In message , Warner Losh writes: >I also noticed that gsched doesn't take BIO_ORDERED into account when >sorting requests. Isilon has an I/O scheduler which has this problem >too > >I think the cam iosched ignores it too. That would be strange given that BIO_ORDERED is @gibbs baby ? -

Re: svn commit: r326731 - head/sys/ufs/ffs

2017-12-09 Thread Warner Losh
On Dec 9, 2017 3:37 PM, "Mark Johnston" wrote: On Sat, Dec 09, 2017 at 08:03:37PM +0200, Andriy Gapon wrote: > On 09/12/2017 17:44, Mark Johnston wrote: > > Some GEOMs do not appear to handle BIO_ORDERED correctly, meaning that the > > Nitpick: this should be "geoms" or, even better, "GEOM classe

svn commit: r326737 - head/sys/net80211

2017-12-09 Thread Adrian Chadd
Author: adrian Date: Sat Dec 9 23:16:02 2017 New Revision: 326737 URL: https://svnweb.freebsd.org/changeset/base/326737 Log: [net80211] add a method for checking if a VAP WME AC has a NOACK policy or not. A subsequent set of commits will introduce this instead of a whole lot of gymnasti

Re: svn commit: r326731 - head/sys/ufs/ffs

2017-12-09 Thread Mark Johnston
On Sat, Dec 09, 2017 at 08:03:37PM +0200, Andriy Gapon wrote: > On 09/12/2017 17:44, Mark Johnston wrote: > > Some GEOMs do not appear to handle BIO_ORDERED correctly, meaning that the > > Nitpick: this should be "geoms" or, even better, "GEOM classes" :-) Ok. :) > > barrier write may not work a

svn commit: r326736 - head/usr.bin/wc

2017-12-09 Thread Conrad Meyer
Author: cem Date: Sat Dec 9 21:55:19 2017 New Revision: 326736 URL: https://svnweb.freebsd.org/changeset/base/326736 Log: wc(1): Extend non-controversial optimizations to '-c' mode wc(1)'s slow path for counting words or multibyte characters requires conversion of the 8-bit input stream

svn commit: r326735 - head/sys/fs/nfs

2017-12-09 Thread Rick Macklem
Author: rmacklem Date: Sat Dec 9 21:04:56 2017 New Revision: 326735 URL: https://svnweb.freebsd.org/changeset/base/326735 Log: Define macros used by the pNFS server code. This commit defines some macros used by the pNFS server code. They will not be used until the main pNFS server code m

Re: svn commit: r326731 - head/sys/ufs/ffs

2017-12-09 Thread Andriy Gapon
On 09/12/2017 17:44, Mark Johnston wrote: > Some GEOMs do not appear to handle BIO_ORDERED correctly, meaning that the Nitpick: this should be "geoms" or, even better, "GEOM classes" :-) > barrier write may not work as intended. Could the loss of BIO_ORDERED in g_duplicate_bio() contribute to the

svn commit: r326734 - head/share/man/man5

2017-12-09 Thread Mike Karels
Author: karels Date: Sat Dec 9 17:27:36 2017 New Revision: 326734 URL: https://svnweb.freebsd.org/changeset/base/326734 Log: gifconfig_gif0 no longer works, document replacement rc.conf(5) documents the gifconfig_ keyword, which is no longer implemented. Document the replacement, which w

Re: svn commit: r326614 - in head: etc/mtree usr.bin usr.bin/sponge usr.bin/tee

2017-12-09 Thread Conrad Meyer
That isn't what was asked for. Conrad On Sat, Dec 9, 2017 at 12:22 AM, Eitan Adler wrote: > On 8 December 2017 at 20:23, Conrad Meyer wrote: >> On Fri, Dec 8, 2017 at 6:07 PM, Eitan Adler wrote: >> Great. Please re-apply the change with something like that in the >> commit message, when you g

svn commit: r326733 - head/usr.sbin/acpi/acpiconf

2017-12-09 Thread Niclas Zeising
Author: zeising (doc,ports committer) Date: Sat Dec 9 15:59:10 2017 New Revision: 326733 URL: https://svnweb.freebsd.org/changeset/base/326733 Log: Improve options and error handling. Improve options handling and error out if multiple mutually exclusive options are passed to acpiconf. S

svn commit: r326732 - head/sys/vm

2017-12-09 Thread Mark Johnston
Author: markj Date: Sat Dec 9 15:47:26 2017 New Revision: 326732 URL: https://svnweb.freebsd.org/changeset/base/326732 Log: Fix the act_scan_laundry_weight mechanism. r292392 modified the active queue scan to weigh clean pages differently from dirty pages when attempting to meet the inac

svn commit: r326731 - head/sys/ufs/ffs

2017-12-09 Thread Mark Johnston
Author: markj Date: Sat Dec 9 15:44:30 2017 New Revision: 326731 URL: https://svnweb.freebsd.org/changeset/base/326731 Log: Provide a sysctl to force synchronous initialization of inode blocks. FFS performs asynchronous inode initialization, using a barrier write to ensure that the inode

svn commit: r326730 - head/sys/dev/iscsi

2017-12-09 Thread Edward Tomasz Napierala
Author: trasz Date: Sat Dec 9 15:34:40 2017 New Revision: 326730 URL: https://svnweb.freebsd.org/changeset/base/326730 Log: Move the DIAGNOSTIC check for lost iSCSI PDUs from icl_conn_close() to icl_conn_free(). It's perfectly valid for the counter to be non-zero in the former. MFC af

Re: svn commit: r326729 - head

2017-12-09 Thread Rodney W. Grimes
[ Charset UTF-8 unsupported, converting... ] > Author: eadler > Date: Sat Dec 9 08:21:29 2017 > New Revision: 326729 > URL: https://svnweb.freebsd.org/changeset/base/326729 > > Log: > arc lint: Explain linting explain > > shell scripts in scripts don't need > to be chmod +x to work. In f

Re: svn commit: r326614 - in head: etc/mtree usr.bin usr.bin/sponge usr.bin/tee

2017-12-09 Thread Eitan Adler
On 8 December 2017 at 20:23, Conrad Meyer wrote: > On Fri, Dec 8, 2017 at 6:07 PM, Eitan Adler wrote: > Great. Please re-apply the change with something like that in the > commit message, when you get a chance. It shouldn't take more than a > minute or two. r326729 Once again, sorry about the

svn commit: r326729 - head

2017-12-09 Thread Eitan Adler
Author: eadler Date: Sat Dec 9 08:21:29 2017 New Revision: 326729 URL: https://svnweb.freebsd.org/changeset/base/326729 Log: arc lint: Explain linting explain shell scripts in scripts don't need to be chmod +x to work. In fact most are not. Of the tests I found from a simple search: