svn commit: r326656 - in head/sys: amd64/amd64 arm/arm arm64/arm64 i386/i386 kern mips/mips powerpc/powerpc riscv/riscv sparc64/sparc64

Author: bde Date: Thu Dec 7 07:55:38 2017 New Revision: 326656 URL: https://svnweb.freebsd.org/changeset/base/326656 Log: Move instantiation of msgbufp from 9 MD files to subr_prf.c. This variable should be pure MI except possibly for reading it in MD dump routines. Its initialization

svn commit: r326655 - head/sys/geom/raid

Author: eugen Date: Thu Dec 7 05:55:18 2017 New Revision: 326655 URL: https://svnweb.freebsd.org/changeset/base/326655 Log: Fix use-after-free that sometimes results in a garbage returned instead of right error code after requests to SINGLE/CONCAT volumes, f.e: # dd if=/dev/raid/r0

svn commit: r326654 - head/sbin/ifconfig

Author: bryanv Date: Thu Dec 7 04:55:31 2017 New Revision: 326654 URL: https://svnweb.freebsd.org/changeset/base/326654 Log: Use consistent name for the vxlan VNI parameter name and provide shorthand Submitted by: hrs MFC after:1 month Differential Revision:

Re: svn commit: r326626 - head/lib/libproc

On Wed, 6 Dec 2017, Mark Johnston wrote: Log: Use a global extern declaration to appease gcc. Why not do this to fix the bug which is reported by gcc as requested by using -Wnested-externs (given by WARNS?=6 in ../Makefile.inc). clang doesn't suuport this option except in a broken way by

Re: svn commit: r326651 - in head/bin: cat date kenv setfacl sh

[ Charset UTF-8 unsupported, converting... ] > Author: bjk (doc committer) > Date: Thu Dec 7 01:57:27 2017 > New Revision: 326651 > URL: https://svnweb.freebsd.org/changeset/base/326651 > > Log: > Fix mandoc -Tlint warnings in bin/ > > Many style-level issues are still reported. It helps

svn commit: r326652 - in head/usr.bin/dc: . tests

Author: asomers Date: Thu Dec 7 02:08:55 2017 New Revision: 326652 URL: https://svnweb.freebsd.org/changeset/base/326652 Log: dc(1): fix modulo operations with fractional inputs Our dc(1) has never correctly calculated remainders with fractional inputs. Both bmod and bdivmod seem to

svn commit: r326651 - in head/bin: cat date kenv setfacl sh

Author: bjk (doc committer) Date: Thu Dec 7 01:57:27 2017 New Revision: 326651 URL: https://svnweb.freebsd.org/changeset/base/326651 Log: Fix mandoc -Tlint warnings in bin/ Many style-level issues are still reported. Submitted by: Yuri Pankov Reviewed by: jilles

svn commit: r326650 - head/lib/libc/sys

Author: bjk (doc committer) Date: Thu Dec 7 01:50:17 2017 New Revision: 326650 URL: https://svnweb.freebsd.org/changeset/base/326650 Log: Note that old sys/event.h required manual sys/types.h inclusion ed fixed this in r313704 but older versions are still affected. Modified:

svn commit: r326649 - in head/targets/pseudo/userland: . lib

Author: bdrewery Date: Thu Dec 7 01:49:35 2017 New Revision: 326649 URL: https://svnweb.freebsd.org/changeset/base/326649 Log: Disconnect OFED after r326169 broke all DIRDEPS support for it. Modified: head/targets/pseudo/userland/Makefile.depend

Re: svn commit: r326169 - in head: . contrib/ofed lib/libc/locale share/mk sys/amd64/amd64 sys/amd64/conf sys/conf sys/contrib/rdma/krping sys/dev/cxgbe sys/dev/cxgbe/common sys/dev/cxgbe/iw_cxgbe sys

On 12/6/2017 4:53 PM, Hans Petter Selasky wrote: > On 12/07/17 01:38, Bryan Drewery wrote: >> 1. SUBDIR=../../somewhere else is inherently wrong. >> 2. Building a library from a directory named '2' is wrong. >> 3. Not just using proper SUBDIR_DEPEND and making all these subdirs is >> not needed

Re: svn commit: r326169 - in head: . contrib/ofed lib/libc/locale share/mk sys/amd64/amd64 sys/amd64/conf sys/conf sys/contrib/rdma/krping sys/dev/cxgbe sys/dev/cxgbe/common sys/dev/cxgbe/iw_cxgbe sys

On 12/6/2017 4:53 PM, Hans Petter Selasky wrote: > It would be nice if these utilities can be built standalone, outside the > buildworld target aswell. I've suggested adding an "all-install" variant > to make. Having to kick a buildworld to test-build contrib/ofed is quite > overkill, and that is

Re: svn commit: r326169 - in head: . contrib/ofed lib/libc/locale share/mk sys/amd64/amd64 sys/amd64/conf sys/conf sys/contrib/rdma/krping sys/dev/cxgbe sys/dev/cxgbe/common sys/dev/cxgbe/iw_cxgbe sys

On 12/6/2017 4:53 PM, Hans Petter Selasky wrote: > On 12/07/17 01:38, Bryan Drewery wrote: >> 1. SUBDIR=../../somewhere else is inherently wrong. >> 2. Building a library from a directory named '2' is wrong. >> 3. Not just using proper SUBDIR_DEPEND and making all these subdirs is >> not needed

Re: svn commit: r326169 - in head: . contrib/ofed lib/libc/locale share/mk sys/amd64/amd64 sys/amd64/conf sys/conf sys/contrib/rdma/krping sys/dev/cxgbe sys/dev/cxgbe/common sys/dev/cxgbe/iw_cxgbe sys

On 12/07/17 01:38, Bryan Drewery wrote: 1. SUBDIR=../../somewhere else is inherently wrong. 2. Building a library from a directory named '2' is wrong. 3. Not just using proper SUBDIR_DEPEND and making all these subdirs is not needed and complicates things MORE by splitting them all up. 4.

Re: svn commit: r326169 - in head: . contrib/ofed lib/libc/locale share/mk sys/amd64/amd64 sys/amd64/conf sys/conf sys/contrib/rdma/krping sys/dev/cxgbe sys/dev/cxgbe/common sys/dev/cxgbe/iw_cxgbe sys

On 12/6/2017 4:48 PM, Bryan Drewery wrote: > On 12/6/2017 4:38 PM, Bryan Drewery wrote: >> On 12/6/2017 4:28 PM, Hans Petter Selasky wrote: >>> On 12/07/17 01:10, Bryan Drewery wrote: I'm a bit speechless.  Why??  The old version was 90% fine and I sent you advice on how to make it work

Re: svn commit: r326169 - in head: . contrib/ofed lib/libc/locale share/mk sys/amd64/amd64 sys/amd64/conf sys/conf sys/contrib/rdma/krping sys/dev/cxgbe sys/dev/cxgbe/common sys/dev/cxgbe/iw_cxgbe sys

On 12/6/2017 4:38 PM, Bryan Drewery wrote: > On 12/6/2017 4:28 PM, Hans Petter Selasky wrote: >> On 12/07/17 01:10, Bryan Drewery wrote: >>> I'm a bit speechless.  Why??  The old version was 90% fine and I sent >>> you advice on how to make it work with the new layout. >> >> Hi, >> >> I tried to

Re: svn commit: r326169 - in head: . contrib/ofed lib/libc/locale share/mk sys/amd64/amd64 sys/amd64/conf sys/conf sys/contrib/rdma/krping sys/dev/cxgbe sys/dev/cxgbe/common sys/dev/cxgbe/iw_cxgbe sys

On 12/6/2017 4:28 PM, Hans Petter Selasky wrote: > On 12/07/17 01:10, Bryan Drewery wrote: >> I'm a bit speechless.  Why??  The old version was 90% fine and I sent >> you advice on how to make it work with the new layout. > > Hi, > > I tried to listen to your advice, splitting up the Makefiles,

svn commit: r326647 - head

Author: bdrewery Date: Thu Dec 7 00:31:10 2017 New Revision: 326647 URL: https://svnweb.freebsd.org/changeset/base/326647 Log: Revert r325529: No longer needed after r325699. Sponsored by: Dell EMC Modified: head/Makefile Modified: head/Makefile

Re: svn commit: r326169 - in head: . contrib/ofed lib/libc/locale share/mk sys/amd64/amd64 sys/amd64/conf sys/conf sys/contrib/rdma/krping sys/dev/cxgbe sys/dev/cxgbe/common sys/dev/cxgbe/iw_cxgbe sys

On 12/07/17 01:10, Bryan Drewery wrote: I'm a bit speechless. Why?? The old version was 90% fine and I sent you advice on how to make it work with the new layout. Hi, I tried to listen to your advice, splitting up the Makefiles, not listing every dependency. Further how the OFED libraries

svn commit: r326648 - head

Author: bdrewery Date: Thu Dec 7 00:31:28 2017 New Revision: 326648 URL: https://svnweb.freebsd.org/changeset/base/326648 Log: tinderbox/universe: Don't execute KERNCONFS lookup if not needed. Sponsored by: Dell EMC Modified: head/Makefile Modified: head/Makefile

Re: svn commit: r326169 - in head: . contrib/ofed lib/libc/locale share/mk sys/amd64/amd64 sys/amd64/conf sys/conf sys/contrib/rdma/krping sys/dev/cxgbe sys/dev/cxgbe/common sys/dev/cxgbe/iw_cxgbe sys

On 11/24/2017 6:50 AM, Hans Petter Selasky wrote: > Modified: head/Makefile.inc1 > == > --- head/Makefile.inc1Fri Nov 24 14:29:32 2017(r326168) > +++ head/Makefile.inc1Fri Nov 24 14:50:28 2017

svn commit: r326646 - head/lib/libcam

Author: asomers Date: Wed Dec 6 23:24:11 2017 New Revision: 326646 URL: https://svnweb.freebsd.org/changeset/base/326646 Log: Fix a null-pointer dereference and a tautological check in cam_get_device Reported by: Coverity CID: 1017964 MFC after:3 weeks Sponsored by:

svn commit: r326645 - in head/sys: cam cam/ata cam/mmc cam/nvme cam/scsi dev/hyperv/storvsc

Author: imp Date: Wed Dec 6 23:05:22 2017 New Revision: 326645 URL: https://svnweb.freebsd.org/changeset/base/326645 Log: Define xpt_path_inq. This provides a nice wrarpper around the XPT_PATH_INQ ccb creation and calling. Sponsored by: Netflix Differential Revision:

svn commit: r326644 - in head/sys/cam: ata nvme scsi

Author: imp Date: Wed Dec 6 23:05:15 2017 New Revision: 326644 URL: https://svnweb.freebsd.org/changeset/base/326644 Log: Now that cam_periph_runccb() can be called from situations where the kernel scheduler is stopped, replace the by hand calling of xpt_polled_action() with it.

svn commit: r326643 - head/sys/cam

Author: imp Date: Wed Dec 6 23:05:07 2017 New Revision: 326643 URL: https://svnweb.freebsd.org/changeset/base/326643 Log: Make cam_periph_runccb be safe to call when we can only do polling. Sponsored by: Netflix Differential Revision: https://reviews.freebsd.org/D13388 Modified:

svn commit: r326642 - in head: share/man/man9 sys/dev/firewire sys/dev/lmc sys/dev/smc sys/net

Author: glebius Date: Wed Dec 6 23:03:34 2017 New Revision: 326642 URL: https://svnweb.freebsd.org/changeset/base/326642 Log: Garbage collect IFCAP_POLLING_NOCOUNT. It wasn't used since very beginning of polling(4). The module always ignored return value from driver polling handler.

svn commit: r326641 - head/contrib/dma

Author: bapt Date: Wed Dec 6 22:08:35 2017 New Revision: 326641 URL: https://svnweb.freebsd.org/changeset/base/326641 Log: Split body of mails not respecting RFC2822 For mails which has a body not respecting RFC2822 (which often happen with crontabs) try to split by words finding the

svn commit: r326640 - in head/lib/libc: gen tests/gen

Author: asomers Date: Wed Dec 6 22:06:48 2017 New Revision: 326640 URL: https://svnweb.freebsd.org/changeset/base/326640 Log: Optimize telldir(3) Currently each call to telldir() requires a malloc and adds an entry to a linked list which must be traversed on future telldir(), seekdir(),

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

Oh good, I'll be able to sleep tonight after all. :-) -Ravi -Original Message- From: on behalf of Alan Somers Date: 2017-12-06, Wednesday at 10:15 To: Ravi Pokala Cc: "src-committ...@freebsd.org"

svn commit: r326636 - head/contrib/binutils/libiberty

Author: stevek Date: Wed Dec 6 21:18:45 2017 New Revision: 326636 URL: https://svnweb.freebsd.org/changeset/base/326636 Log: The function make_relative_prefix_1 does not properly free locally allocated memory when it returns early. Free the memory associated with the variables

svn commit: r326635 - head/lib/libc/stdio

Author: stevek Date: Wed Dec 6 21:12:24 2017 New Revision: 326635 URL: https://svnweb.freebsd.org/changeset/base/326635 Log: The function fwscanf() return value is wrong when encountering an early matching failure. According to the Open Group documentation for fwscanf: "Upon

svn commit: r326634 - head/share/mk

Author: bdrewery Date: Wed Dec 6 21:00:41 2017 New Revision: 326634 URL: https://svnweb.freebsd.org/changeset/base/326634 Log: AUTO_OBJ: Don't create nested OBJDIRS with print-dir or make -n. Sponsored by: Dell EMC Modified: head/share/mk/bsd.obj.mk Modified: head/share/mk/bsd.obj.mk

svn commit: r326633 - head/share/misc

Author: bapt Date: Wed Dec 6 20:23:38 2017 New Revision: 326633 URL: https://svnweb.freebsd.org/changeset/base/326633 Log: Update to 2017-12-06 MFC after:2 days Modified: head/share/misc/pci_vendors Modified: head/share/misc/pci_vendors

svn commit: r326632 - head/usr.bin/sdiff

Author: bapt Date: Wed Dec 6 20:10:05 2017 New Revision: 326632 URL: https://svnweb.freebsd.org/changeset/base/326632 Log: sdiff(1) can now be built defaults warning level Modified: head/usr.bin/sdiff/Makefile Modified: head/usr.bin/sdiff/Makefile

svn commit: r326631 - head/usr.bin/sdiff

Author: bapt Date: Wed Dec 6 20:09:30 2017 New Revision: 326631 URL: https://svnweb.freebsd.org/changeset/base/326631 Log: Rename variables confusing gcc 4.2.1 While there is no reason it shadows div(3) gcc 4.2.1 complains about it MFC after:1 week Modified:

svn commit: r326630 - head/usr.bin/sdiff

Author: bapt Date: Wed Dec 6 20:05:36 2017 New Revision: 326630 URL: https://svnweb.freebsd.org/changeset/base/326630 Log: Fix -Wincompatible-pointer-types-discards-qualifiers warnings MFC after:1 week Modified: head/usr.bin/sdiff/sdiff.c Modified: head/usr.bin/sdiff/sdiff.c

svn commit: r326629 - head/sys/vm

Author: markj Date: Wed Dec 6 18:36:54 2017 New Revision: 326629 URL: https://svnweb.freebsd.org/changeset/base/326629 Log: Use unique wait messages in the page daemon control loop. Discussed with: alc MFC after:1 week Modified: head/sys/vm/vm_pageout.c Modified:

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

Nope. The ATA spec prohibited them, and nobody ever manufactured any. But you might be able to fake one with CTL or QEMU or something similar. On Wed, Dec 6, 2017 at 10:11 AM, Ravi Pokala wrote: > > ... disks of >=1TiB that still use CHS addressing. > > Out of morbid

Re: svn commit: r326628 - head/contrib/tcpdump

On Wed, Dec 06, 2017 at 06:11:56PM +, Ed Maste wrote: E> Author: emaste E> Date: Wed Dec 6 18:11:56 2017 E> New Revision: 326628 E> URL: https://svnweb.freebsd.org/changeset/base/326628 E> E> Log: E> tcpdump: remove undesired svn:keywords property from contrib Thanks! -- Gleb Smirnoff

svn commit: r326628 - head/contrib/tcpdump

Author: emaste Date: Wed Dec 6 18:11:56 2017 New Revision: 326628 URL: https://svnweb.freebsd.org/changeset/base/326628 Log: tcpdump: remove undesired svn:keywords property from contrib Reported by: glebius MFC after:6 days MFC with: r326613 Modified: Directory Properties:

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

> ... disks of >=1TiB that still use CHS addressing. Out of morbid curiosity, are there any such monstrosities? -Ravi (rpokala@) -Original Message- From: on behalf of Alan Somers Date: 2017-12-06, Wednesday at 09:01 To:

svn commit: r326626 - head/lib/libproc

Author: markj Date: Wed Dec 6 17:52:01 2017 New Revision: 326626 URL: https://svnweb.freebsd.org/changeset/base/326626 Log: Use a global extern declaration to appease gcc. Reported by: gjb X-MFC with: r326498 Modified: head/lib/libproc/proc_create.c Modified:

svn commit: r326625 - head/usr.bin/logger

Author: glebius Date: Wed Dec 6 17:50:10 2017 New Revision: 326625 URL: https://svnweb.freebsd.org/changeset/base/326625 Log: Fix crash with a dotless hostname. Modified: head/usr.bin/logger/logger.c Modified: head/usr.bin/logger/logger.c

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

Author: asomers Date: Wed Dec 6 17:01:25 2017 New Revision: 326624 URL: https://svnweb.freebsd.org/changeset/base/326624 Log: cam: fix sign-extension error in adagetparams adagetparams contains a sign-extension error that will cause the sector count to be incorrectly calculated for ATA

svn commit: r326623 - head/sys/arm/allwinner

Author: kevans Date: Wed Dec 6 14:53:53 2017 New Revision: 326623 URL: https://svnweb.freebsd.org/changeset/base/326623 Log: aw_mp.c: use argument name in macros Rather than relying on 'cluster' existing in the context they're used in, use the argument name. Differential Revision:

svn commit: r326622 - head/usr.sbin/newsyslog

Author: bapt Date: Wed Dec 6 10:47:50 2017 New Revision: 326622 URL: https://svnweb.freebsd.org/changeset/base/326622 Log: Really fix typo and improve wording of the comment Modified: head/usr.sbin/newsyslog/newsyslog.c Modified: head/usr.sbin/newsyslog/newsyslog.c

Re: svn commit: r326621 - head/usr.sbin/newsyslog

On Wed, Dec 06, 2017 at 10:41:34AM +, Alexey Dokuchaev wrote: > On Wed, Dec 06, 2017 at 10:20:01AM +, Baptiste Daroussin wrote: > > New Revision: 326621 > > URL: https://svnweb.freebsd.org/changeset/base/326621 > > > > Log: > > Fix typo > > Still not fixed. > > > Reported by:

Re: svn commit: r326621 - head/usr.sbin/newsyslog

On Wed, Dec 06, 2017 at 10:20:01AM +, Baptiste Daroussin wrote: > New Revision: 326621 > URL: https://svnweb.freebsd.org/changeset/base/326621 > > Log: > Fix typo Still not fixed. > Reported by:danfe > > Modified: > head/usr.sbin/newsyslog/newsyslog.c > > Modified:

Re: svn commit: r326617 - head/usr.sbin/newsyslog

On Wed, Dec 06, 2017 at 10:15:35AM +, Alexey Dokuchaev wrote: > On Wed, Dec 06, 2017 at 09:44:35AM +, Baptiste Daroussin wrote: > > New Revision: 326617 > > URL: https://svnweb.freebsd.org/changeset/base/326617 > > > > Log: > > Allow newsyslog to execute compression commands which > >

svn commit: r326621 - head/usr.sbin/newsyslog

Author: bapt Date: Wed Dec 6 10:20:01 2017 New Revision: 326621 URL: https://svnweb.freebsd.org/changeset/base/326621 Log: Fix typo Reported by: danfe Modified: head/usr.sbin/newsyslog/newsyslog.c Modified: head/usr.sbin/newsyslog/newsyslog.c

Re: svn commit: r326617 - head/usr.sbin/newsyslog

On Wed, Dec 06, 2017 at 09:44:35AM +, Baptiste Daroussin wrote: > New Revision: 326617 > URL: https://svnweb.freebsd.org/changeset/base/326617 > > Log: > Allow newsyslog to execute compression commands which > have a semantic different than the traditional gzip(1) > > This is done to

svn commit: r326618 - head/sys/contrib/zstd/programs

Author: bapt Date: Wed Dec 6 09:53:10 2017 New Revision: 326618 URL: https://svnweb.freebsd.org/changeset/base/326618 Log: Revert local changes made to make zstd(1) frontend behave like gzip(1) and friends This change was made to allow zstd(1) to be a dropin replacement for gzip(1) and

svn commit: r326617 - head/usr.sbin/newsyslog

Author: bapt Date: Wed Dec 6 09:44:35 2017 New Revision: 326617 URL: https://svnweb.freebsd.org/changeset/base/326617 Log: Allow newsyslog to execute compression commands which have a semantic different than the traditional gzip(1) This is done to allow to use zstd(1) as a compression

Re: svn commit: r316980 - head/contrib/zstd/programs

On Tue, Dec 05, 2017 at 08:42:11PM -0500, Allan Jude wrote: > On 2017-12-05 20:17, Conrad Meyer wrote: > > Ping. Please revert this change. If you want zstd to behave this > > way, pursue it upstream first. > > > > Thanks, > > Conrad > > > > On Wed, Nov 15, 2017 at 7:38 PM, Conrad Meyer

Re: svn commit: r316980 - head/contrib/zstd/programs

On Tue, Dec 05, 2017 at 05:17:22PM -0800, Conrad Meyer wrote: > Ping. Please revert this change. If you want zstd to behave this > way, pursue it upstream first. This is planned, for now I first need to modify newsyslog to accept more arguments for the compression program, then I'll switch back