Re: svn commit: r324541 - in head: share/man/man9 sys/kern sys/sys

2017-10-13 Thread Matt Joras
On 10/13/2017 22:12, Ngie Cooper (yaneurabeya) wrote: >> Modified: head/sys/kern/subr_unit.c >> == >> --- head/sys/kern/subr_unit.cWed Oct 11 20:36:22 2017 >> (r324540) >> +++ head/sys/kern/subr_unit.c

Re: svn commit: r324541 - in head: share/man/man9 sys/kern sys/sys

2017-10-13 Thread Ngie Cooper (yaneurabeya)
> On Oct 11, 2017, at 14:53, Matt Joras wrote: > > Author: mjoras > Date: Wed Oct 11 21:53:50 2017 > New Revision: 324541 > URL: https://svnweb.freebsd.org/changeset/base/324541 > > Log: > Add clearing function for unr(9). > > Previously before you could call unrhdr_delete you needed to > i

svn commit: r324614 - head/sys/sys

2017-10-13 Thread Mateusz Guzik
Author: mjg Date: Sat Oct 14 04:27:58 2017 New Revision: 324614 URL: https://svnweb.freebsd.org/changeset/base/324614 Log: Fix wrong v_free_count annotation - (f) instead of (a) Reported by: alc Modified: head/sys/sys/vmmeter.h Modified: head/sys/sys/vmmeter.h =

Re: svn commit: r324609 - head/sys/sys

2017-10-13 Thread Mateusz Guzik
On Sat, Oct 14, 2017 at 2:30 AM, Ian Lepore wrote: > On Fri, 2017-10-13 at 23:38 +0200, Svatopluk Kraus wrote: > > MTX_UNOWNED is a flag. You did not change its value from 4 to 0, you > > removed it actually. I have very bad feeling about it. But maybe, it's > > really possible and in that case,

svn commit: r324613 - head/sys/kern

2017-10-13 Thread Mateusz Guzik
Author: mjg Date: Sat Oct 14 00:47:30 2017 New Revision: 324613 URL: https://svnweb.freebsd.org/changeset/base/324613 Log: mtx: fix up owner_mtx after r324609 Now that MTX_UNOWNED is 0 the test was alwayas false. Modified: head/sys/kern/kern_mutex.c Modified: head/sys/kern/kern_mutex.c

Re: svn commit: r324609 - head/sys/sys

2017-10-13 Thread Ian Lepore
On Fri, 2017-10-13 at 23:38 +0200, Svatopluk Kraus wrote: > MTX_UNOWNED is a flag. You did not change its value from 4 to 0, you > removed it actually. I have very bad feeling about it. But maybe, it's > really possible and in that case, a very good explanation should be > provided. > > Svata >

Re: svn commit: r324609 - head/sys/sys

2017-10-13 Thread Svatopluk Kraus
MTX_UNOWNED is a flag. You did not change its value from 4 to 0, you removed it actually. I have very bad feeling about it. But maybe, it's really possible and in that case, a very good explanation should be provided. Svata On Fri, Oct 13, 2017 at 10:31 PM, Mateusz Guzik wrote: > Author: mjg >

svn commit: r324610 - in head/sys: sys vm

2017-10-13 Thread Mateusz Guzik
Author: mjg Date: Fri Oct 13 21:54:34 2017 New Revision: 324610 URL: https://svnweb.freebsd.org/changeset/base/324610 Log: Reduce traffic on vm_cnt.v_free_count The variable is modified with the highly contended page free queue lock. It unnecessarily shares a cacheline with purely read-on

Re: svn commit: r324471 - head/sys/boot

2017-10-13 Thread Devin Teske
> On Oct 13, 2017, at 8:53 AM, Ngie Cooper wrote: > > >> On Oct 9, 2017, at 21:57, Ngie Cooper (yaneurabeya) >> wrote: >> >> >>> On Oct 9, 2017, at 21:56, Warner Losh wrote: >>> >>> DO NOT MAKE EDITS TO sys/boot. YOU ARE CREATING CONFLICTS FOR ME. >>> >>> DO NOT MAKE ANY COMMITS TO sys/b

Re: svn commit: r324609 - head/sys/sys

2017-10-13 Thread Mateusz Guzik
On Fri, Oct 13, 2017 at 10:54 PM, Ian Lepore wrote: > On Fri, 2017-10-13 at 20:31 +, Mateusz Guzik wrote: > > Author: mjg > > Date: Fri Oct 13 20:31:56 2017 > > New Revision: 324609 > > URL: https://svnweb.freebsd.org/changeset/base/324609 > > > > Log: > > mtx: change MTX_UNOWNED from 4 to

Re: svn commit: r324609 - head/sys/sys

2017-10-13 Thread Ian Lepore
On Fri, 2017-10-13 at 20:31 +, Mateusz Guzik wrote: > Author: mjg > Date: Fri Oct 13 20:31:56 2017 > New Revision: 324609 > URL: https://svnweb.freebsd.org/changeset/base/324609 > > Log: >   mtx: change MTX_UNOWNED from 4 to 0 >    >   The value is spread all over the kernel and zeroing a regi

svn commit: r324609 - head/sys/sys

2017-10-13 Thread Mateusz Guzik
Author: mjg Date: Fri Oct 13 20:31:56 2017 New Revision: 324609 URL: https://svnweb.freebsd.org/changeset/base/324609 Log: mtx: change MTX_UNOWNED from 4 to 0 The value is spread all over the kernel and zeroing a register is cheaper/shorter than setting it up to an arbitrary value. R

svn commit: r324608 - head/etc/devd

2017-10-13 Thread Kristof Provost
Author: kp Date: Fri Oct 13 20:29:35 2017 New Revision: 324608 URL: https://svnweb.freebsd.org/changeset/base/324608 Log: Regenerate usb.conf Modified: head/etc/devd/usb.conf Modified: head/etc/devd/usb.conf == --- h

svn commit: r324607 - in head/sys/dev/usb: . serial

2017-10-13 Thread Kristof Provost
Author: kp Date: Fri Oct 13 19:41:35 2017 New Revision: 324607 URL: https://svnweb.freebsd.org/changeset/base/324607 Log: Support the D-Link DWM-222 LTE Dongle Submitted by: Daniel Hänschke Modified: head/sys/dev/usb/serial/u3g.c head/sys/dev/usb/usbdevs Modified: head/sys/dev/usb/se

svn commit: r324606 - head/sys/compat/linuxkpi/common/src

2017-10-13 Thread Mark Johnston
Author: markj Date: Fri Oct 13 19:27:33 2017 New Revision: 324606 URL: https://svnweb.freebsd.org/changeset/base/324606 Log: Make the PHOLD in linux_wait_event_common() unconditional. After some in-progress work is committed, this would otherwise be the only instance of #if(n)def NO_SWAPP

svn commit: r324605 - head/targets/pseudo/userland/gnu

2017-10-13 Thread Ed Maste
Author: emaste Date: Fri Oct 13 18:30:32 2017 New Revision: 324605 URL: https://svnweb.freebsd.org/changeset/base/324605 Log: clean up remnants of removed GPL software send-pr r267486 texinfo r276551 ranlibr286332 sdiff r298823 rcs r307351 diff r317209 Modi

svn commit: r324604 - head/targets/pseudo/userland/gnu

2017-10-13 Thread Ed Maste
Author: emaste Date: Fri Oct 13 18:23:08 2017 New Revision: 324604 URL: https://svnweb.freebsd.org/changeset/base/324604 Log: Remove additional groff leftovers (r319664) Modified: head/targets/pseudo/userland/gnu/Makefile.depend Modified: head/targets/pseudo/userland/gnu/Makefile.depend

svn commit: r324603 - head

2017-10-13 Thread Ed Maste
Author: emaste Date: Fri Oct 13 17:15:17 2017 New Revision: 324603 URL: https://svnweb.freebsd.org/changeset/base/324603 Log: ObsoleteFiles.inc: correct year on libstand removal Modified: head/ObsoleteFiles.inc Modified: head/ObsoleteFiles.inc

svn commit: r324601 - head/sys/kern

2017-10-13 Thread Alan Cox
Author: alc Date: Fri Oct 13 16:31:50 2017 New Revision: 324601 URL: https://svnweb.freebsd.org/changeset/base/324601 Log: Address two problems with sendfile(..., SF_NOCACHE) and apply one "optimization". First, sendfile(..., SF_NOCACHE) frees pages without checking whether those pages are

svn commit: r324600 - head/sys/vm

2017-10-13 Thread Konstantin Belousov
Author: kib Date: Fri Oct 13 16:23:05 2017 New Revision: 324600 URL: https://svnweb.freebsd.org/changeset/base/324600 Log: Evaluate the real size of the sblk_zone. Submitted by: o...@j.email.ne.jp PR: 221356 Reviewed by: alc, markj MFC after:1 week Differential revision:

Re: svn commit: r324471 - head/sys/boot

2017-10-13 Thread Ngie Cooper
> On Oct 9, 2017, at 21:57, Ngie Cooper (yaneurabeya) > wrote: > > >> On Oct 9, 2017, at 21:56, Warner Losh wrote: >> >> DO NOT MAKE EDITS TO sys/boot. YOU ARE CREATING CONFLICTS FOR ME. >> >> DO NOT MAKE ANY COMMITS TO sys/boot. >> >> BACK OFF. >> >> Seriously, though, extra changes crea

svn commit: r324599 - head/release/arm

2017-10-13 Thread Glen Barber
Author: gjb Date: Fri Oct 13 15:16:57 2017 New Revision: 324599 URL: https://svnweb.freebsd.org/changeset/base/324599 Log: Increase the arm/armv6 and arm/armv7 images from 2.5GB to 3GB, since the RPI2 (at least) does not fit in 2.5GB. While here, add a missing BOARDNAME to RPI2.conf.

svn commit: r324598 - head/sys/dev/hwpmc

2017-10-13 Thread Ruslan Bukin
Author: br Date: Fri Oct 13 15:02:29 2017 New Revision: 324598 URL: https://svnweb.freebsd.org/changeset/base/324598 Log: o Support for Kabylake CPU PMCs (fall down to PMC_CPU_INTEL_SKYLAKE). o Fix bugs in events descriptions for Skylake, Skylake Xeon and Haswell. Reviewed by: kib Spon

Re: svn commit: r324580 - head/sys/net80211

2017-10-13 Thread Cy Schubert
In message <201710130649.v9d6n7ci036...@repo.freebsd.org>, Adrian Chadd writes: > Author: adrian > Date: Fri Oct 13 06:49:07 2017 > New Revision: 324580 > URL: https://svnweb.freebsd.org/changeset/base/324580 > > Log: > [net80211] don't try to follow a NULL rxs pointer down the sink. > > I

svn commit: r324597 - head/sys/compat/linuxkpi/common/src

2017-10-13 Thread Hans Petter Selasky
Author: hselasky Date: Fri Oct 13 14:14:46 2017 New Revision: 324597 URL: https://svnweb.freebsd.org/changeset/base/324597 Log: Don't call selrecord() outside the select system call in the LinuxKPI, because then td->td_sel is NULL and this will result in a segfault inside selrecord(). This h

svn commit: r324595 - head/sys/vm

2017-10-13 Thread Ed Maste
Author: emaste Date: Fri Oct 13 13:53:19 2017 New Revision: 324595 URL: https://svnweb.freebsd.org/changeset/base/324595 Log: ANSIfy vm_kern.c PR: 222673 Submitted by: o...@j.email.ne.jp MFC after:1 week Modified: head/sys/vm/vm_kern.c Modified: head/sys/vm/vm_kern.c =

svn commit: r324594 - head/usr.bin/truss

2017-10-13 Thread Ed Maste
Author: emaste Date: Fri Oct 13 13:47:55 2017 New Revision: 324594 URL: https://svnweb.freebsd.org/changeset/base/324594 Log: truss: mention 'H' in usage r295930 introduced the 'H' option to display thread IDs, but did not add the option to usage(). PR: 222837 Submitted b

svn commit: r324593 - head/sys/netpfil/ipfw

2017-10-13 Thread Andrey V. Elsukov
Author: ae Date: Fri Oct 13 11:11:53 2017 New Revision: 324593 URL: https://svnweb.freebsd.org/changeset/base/324593 Log: Fix regression in handling O_FORWARD_IP opcode after r279948. To properly handle 'fwd tablearg,port' opcode, copy sin_port value from sockaddr_in structure stored in t

svn commit: r324592 - head/sbin/ipfw

2017-10-13 Thread Andrey V. Elsukov
Author: ae Date: Fri Oct 13 11:01:33 2017 New Revision: 324592 URL: https://svnweb.freebsd.org/changeset/base/324592 Log: Return 'errno' value from the table_do_modify_record(), it is expected by table_modify_record(). This makes quiet operations with tables really quiet. PR:

svn commit: r324591 - head/sys/kern

2017-10-13 Thread Andriy Gapon
Author: avg Date: Fri Oct 13 09:42:05 2017 New Revision: 324591 URL: https://svnweb.freebsd.org/changeset/base/324591 Log: remove process and jail directory machinations from dounmount The manipulations done by mountcheckdirs() are not that useful during the unmount, they can bring about

svn commit: r324590 - head/usr.sbin/i2c

2017-10-13 Thread Andriy Gapon
Author: avg Date: Fri Oct 13 09:21:41 2017 New Revision: 324590 URL: https://svnweb.freebsd.org/changeset/base/324590 Log: i2c(8): clean up and clarify read operation The code went to a lot of trouble to issue either a start+stop condition or a repeated start condition only to follow it w

Re: svn commit: r324580 - head/sys/net80211

2017-10-13 Thread Hans Petter Selasky
On 10/13/17 09:45, Adrian Chadd wrote: no, it checks it. -a On 13 October 2017 at 00:11, Hans Petter Selasky wrote: On 10/13/17 08:49, Adrian Chadd wrote: if (ieee80211_check_rxseq_amsdu_more(rxs)) { ^^^ does this line also need a NULL check? ni->ni_rxseqs[t

Re: svn commit: r324580 - head/sys/net80211

2017-10-13 Thread Adrian Chadd
no, it checks it. -a On 13 October 2017 at 00:11, Hans Petter Selasky wrote: > On 10/13/17 08:49, Adrian Chadd wrote: >> >> if (ieee80211_check_rxseq_amsdu_more(rxs)) { > > ^^^ does this line also need a NULL check? > > >> ni->ni_rxseqs[tid] = rxseq; >> -

Re: svn commit: r324580 - head/sys/net80211

2017-10-13 Thread Hans Petter Selasky
On 10/13/17 08:49, Adrian Chadd wrote: if (ieee80211_check_rxseq_amsdu_more(rxs)) { ^^^ does this line also need a NULL check? ni->ni_rxseqs[tid] = rxseq; - if (ieee80211_check_rxseq_amsdu(rxs)) + if ((rxs != NULL) && ieee80211_check_rxseq_am