svn commit: r317316 - head/lib/libgeom

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sun Apr 23 03:36:03 2017 New Revision: 317316 URL: https://svnweb.freebsd.org/changeset/base/317316 Log: Minor style(9) fixups Delete trailing whitespace and sort headers. Leave libgeom.h's placement alone, per reasoning in r317289. MFC after:5 weeks Spons

svn commit: r317315 - head/lib/libc/gen

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sun Apr 23 03:17:32 2017 New Revision: 317315 URL: https://svnweb.freebsd.org/changeset/base/317315 Log: Note that getpagesize(3) can return -1 on failure MFC after:5 weeks Sponsored by: Dell EMC Isilon Modified: head/lib/libc/gen/getpagesize.3 Modified: head/li

svn commit: r317313 - head/tools/regression/geom_gpt

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sun Apr 23 02:30:06 2017 New Revision: 317313 URL: https://svnweb.freebsd.org/changeset/base/317313 Log: gctl_test.t: catch errors with the mdcfg directive While here, add a note about certain testcases relying on `count=1024` in the "create" portion. MFC after:

svn commit: r317312 - head/lib/libgeom

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sun Apr 23 01:50:47 2017 New Revision: 317312 URL: https://svnweb.freebsd.org/changeset/base/317312 Log: Fix type for `pagesize` to match the return type for getpagesize(3) to fix the build MFC after:5 weeks X-MFC with: r317311 Pointyhat to: ngie Sponsored b

svn commit: r317311 - head/lib/libgeom

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sun Apr 23 01:47:51 2017 New Revision: 317311 URL: https://svnweb.freebsd.org/changeset/base/317311 Log: Check for failures from getpagesize(3) Return errno on failure, similar to the open(2) call above it. MFC after:5 weeks Reported by: Coverity CID:

svn commit: r317310 - head/lib/libgeom

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sun Apr 23 01:17:59 2017 New Revision: 317310 URL: https://svnweb.freebsd.org/changeset/base/317310 Log: Minor style(9) fixups Delete trailing whitespace and sort headers. Leave libgeom.h's placement alone, per reasoning in r317289. MFC after:5 weeks Spons

Re: svn commit: r316978 - in head: contrib/zstd etc/mtree lib lib/libzstd share/mk usr.bin usr.bin/zstd

2017-04-22 Thread Ian Lepore
On Sun, 2017-04-23 at 01:27 +0200, Baptiste Daroussin wrote: > On Sat, Apr 22, 2017 at 04:05:28PM -0600, Ian Lepore wrote: > > > > On Sun, 2017-04-16 at 09:35 +0200, Baptiste Daroussin wrote: > > > > > > On Sat, Apr 15, 2017 at 04:37:02PM -0700, Rodney W. Grimes wrote: > > > > > > > > > > > > A

svn commit: r317309 - head/tools/regression/geom_gpt

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sat Apr 22 23:30:02 2017 New Revision: 317309 URL: https://svnweb.freebsd.org/changeset/base/317309 Log: gctl_test.t: improve error reporting with mdcfg and mount directives If the commands had failed previously, it would press on and result in a series of cascading fai

svn commit: r317308 - head/tools/regression/geom_gpt

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sat Apr 22 23:27:40 2017 New Revision: 317308 URL: https://svnweb.freebsd.org/changeset/base/317308 Log: gctl_test.t: more tweaks to try and update the code and get it functional (again?) - Make the logfile for $out be built off the basename for $cmd, instead of $cmd.

Re: svn commit: r316978 - in head: contrib/zstd etc/mtree lib lib/libzstd share/mk usr.bin usr.bin/zstd

2017-04-22 Thread Baptiste Daroussin
On Sat, Apr 22, 2017 at 04:05:28PM -0600, Ian Lepore wrote: > On Sun, 2017-04-16 at 09:35 +0200, Baptiste Daroussin wrote: > > On Sat, Apr 15, 2017 at 04:37:02PM -0700, Rodney W. Grimes wrote: > > > > > > And we need this in base for what great purpose??? > > > > > zstandard is the successor to l

svn commit: r317307 - head/tools/regression/geom_gpt

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sat Apr 22 23:06:53 2017 New Revision: 317307 URL: https://svnweb.freebsd.org/changeset/base/317307 Log: Use verb=delete not verb=remove The `remove` verb hasn't been present in geom_part*(4) for well over a decade, if ever. I couldn't find any references to it in ^/s

svn commit: r317306 - head/tools/regression/geom_gpt

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sat Apr 22 22:40:39 2017 New Revision: 317306 URL: https://svnweb.freebsd.org/changeset/base/317306 Log: gctl_test.t: minor tweaks - Declare $count with the `my` scope operator to permit `use strict`. - Add `use strict`. - Use `use warnings` instead of using `-w` in t

svn commit: r317305 - head/sys/fs/nfsclient

2017-04-22 Thread Rick Macklem
Author: rmacklem Date: Sat Apr 22 22:37:44 2017 New Revision: 317305 URL: https://svnweb.freebsd.org/changeset/base/317305 Log: Fix the NFSv4.1/pNFS client return layout on close. The "return layout on close" case in the pNFS client was badly broken. Fortunately, extant pNFS servers that

svn commit: r317304 - head/tools/regression/geom_gpt

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sat Apr 22 22:34:45 2017 New Revision: 317304 URL: https://svnweb.freebsd.org/changeset/base/317304 Log: gctl_test_helper: add diagnostic output for parse_retval(..) This will help end-users better diagnose issues with the function. MFC after:5 weeks Sponsored

Re: svn commit: r316978 - in head: contrib/zstd etc/mtree lib lib/libzstd share/mk usr.bin usr.bin/zstd

2017-04-22 Thread Ian Lepore
On Sun, 2017-04-16 at 09:35 +0200, Baptiste Daroussin wrote: > On Sat, Apr 15, 2017 at 04:37:02PM -0700, Rodney W. Grimes wrote: > > > > And we need this in base for what great purpose??? > > > zstandard is the successor to lz4, there are attempts to integrate > zstandard in > ZFS, Allan Jude is

svn commit: r317300 - head/usr.bin/grep/tests

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sat Apr 22 21:40:31 2017 New Revision: 317300 URL: https://svnweb.freebsd.org/changeset/base/317300 Log: Only expect :grep_r_implied to pass with bsdgrep(1) The test fails with gnu grep from base and ports. Sponsored by: Dell EMC Isilon Modified: head/usr.bin/grep

svn commit: r317299 - head/contrib/netbsd-tests/usr.bin/grep

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sat Apr 22 21:40:10 2017 New Revision: 317299 URL: https://svnweb.freebsd.org/changeset/base/317299 Log: Add more sanity tests for grep, egrep, and fgrep The test suite currently lacks basic sanity checks to ensure that egrep, fgrep, and grep are actually matching the r

svn commit: r317298 - head/bin/sh

2017-04-22 Thread Jilles Tjoelker
Author: jilles Date: Sat Apr 22 21:31:37 2017 New Revision: 317298 URL: https://svnweb.freebsd.org/changeset/base/317298 Log: sh: Simplify setinteractive(). setsignal() does nothing if the signal disposition is already set correctly. Modified: head/bin/sh/options.c head/bin/sh/trap.c

svn commit: r317297 - head/contrib/netbsd-tests/usr.bin/grep

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sat Apr 22 21:26:15 2017 New Revision: 317297 URL: https://svnweb.freebsd.org/changeset/base/317297 Log: Remove the expected failures for :context and :context2 with bsdgrep(1) They're no longer needed after recent fixes made to bsdgrep(1). Submitted by: Kyle Evans

Re: svn commit: r317290 - head/tools/regression/geom_gpt

2017-04-22 Thread Ngie Cooper (yaneurabeya)
> On Apr 22, 2017, at 14:09, Rodney W. Grimes > wrote: … > I had seen that already when I made my post, that docuement is not going > to stop someone from going "Oh, these are out of order I am going to sort > them since I am here" They well then probably have issues due to your next > statem

Re: svn commit: r317290 - head/tools/regression/geom_gpt

2017-04-22 Thread Rodney W. Grimes
-- Start of PGP signed section. [ Charset UTF-8 unsupported, converting... ] > > > On Apr 22, 2017, at 13:06, Rodney W. Grimes > > wrote: > > > >> Author: ngie > >> Date: Sat Apr 22 20:00:52 2017 > >> New Revision: 317290 > >> URL: https://svnweb.freebsd.org/changeset/base/317290 > >> > >> Log

Re: svn commit: r317290 - head/tools/regression/geom_gpt

2017-04-22 Thread Ngie Cooper (yaneurabeya)
> On Apr 22, 2017, at 13:06, Rodney W. Grimes > wrote: > >> Author: ngie >> Date: Sat Apr 22 20:00:52 2017 >> New Revision: 317290 >> URL: https://svnweb.freebsd.org/changeset/base/317290 >> >> Log: >> Fix -Wimplicit-function-declaration compilation warning by moving libgeom.h >> #include be

svn commit: r317296 - head/sys/fs/nfsclient

2017-04-22 Thread Rick Macklem
Author: rmacklem Date: Sat Apr 22 20:55:39 2017 New Revision: 317296 URL: https://svnweb.freebsd.org/changeset/base/317296 Log: Fix some krpc leaks for the NFSv4.1/pNFS client. The NFSv4.1/pNFS client wasn't doing a newnfs_disconnect() call for the connection to the Data Server (DS) under

svn commit: r317295 - head/tools/regression/geom_gpt

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sat Apr 22 20:51:54 2017 New Revision: 317295 URL: https://svnweb.freebsd.org/changeset/base/317295 Log: The GPT class no longer exists; use the PART class instead MFC after:5 weeks Sponsored by: Dell EMC Isilon Modified: head/tools/regression/geom_gpt/gctl_test_

svn commit: r317294 - head/tools/regression/geom_gpt

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sat Apr 22 20:29:56 2017 New Revision: 317294 URL: https://svnweb.freebsd.org/changeset/base/317294 Log: Bump WARNS to 6 per previous commits which fixed warnings MFC after:5 weeks Tested with: clang (4.0), gcc (4.2.1, 6.3.0) Sponsored by: Dell EMC Isilon Modifi

svn commit: r317293 - head/tools/regression/geom_gpt

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sat Apr 22 20:27:46 2017 New Revision: 317293 URL: https://svnweb.freebsd.org/changeset/base/317293 Log: gctl_test_helper: apply polish - Staticize variables to fix warnings. - Sprinkle asserts around for calls that can fail - Apply style(9) for main(..) definition.

svn commit: r317292 - head/tools/regression/geom_gpt

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sat Apr 22 20:15:47 2017 New Revision: 317292 URL: https://svnweb.freebsd.org/changeset/base/317292 Log: gctl_test.t: use make to compile gctl_test_helper instead of calling cc directly MFC after:5 weeks Sponsored by: Dell EMC Isilon Added: head/tools/regression

Re: svn commit: r317290 - head/tools/regression/geom_gpt

2017-04-22 Thread Rodney W. Grimes
> Author: ngie > Date: Sat Apr 22 20:00:52 2017 > New Revision: 317290 > URL: https://svnweb.freebsd.org/changeset/base/317290 > > Log: > Fix -Wimplicit-function-declaration compilation warning by moving libgeom.h > #include below the stdio.h #include. > > gctl_dump(3) needs stdio.h, per

svn commit: r317291 - head/tools/regression/geom_gpt

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sat Apr 22 20:06:11 2017 New Revision: 317291 URL: https://svnweb.freebsd.org/changeset/base/317291 Log: Rename gctl.t to gctl_test.t and test.c to gctl_test_helper.c This is being done to reduce ambiguity and to make the tests more portable in the future to other locat

svn commit: r317290 - head/tools/regression/geom_gpt

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sat Apr 22 20:00:52 2017 New Revision: 317290 URL: https://svnweb.freebsd.org/changeset/base/317290 Log: Fix -Wimplicit-function-declaration compilation warning by moving libgeom.h #include below the stdio.h #include. gctl_dump(3) needs stdio.h, per reasoning noted in r

svn commit: r317289 - head/lib/libgeom

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sat Apr 22 19:32:23 2017 New Revision: 317289 URL: https://svnweb.freebsd.org/changeset/base/317289 Log: libgeom(3): note that stdio.h is required when referencing gctl_dump(3) gctl_dump(3) is only exposed when stdio.h is #include'd first, per its addition in r112510. T

svn commit: r317288 - head/lib/libgeom

2017-04-22 Thread Ngie Cooper
Author: ngie Date: Sat Apr 22 19:11:37 2017 New Revision: 317288 URL: https://svnweb.freebsd.org/changeset/base/317288 Log: libgeom(3): apply minor polish - Use .Dv when mentioning NULL per mdoc(7). - Reword `g_device_path`, `g_open_by_ident`, and `g_providername`'s descriptions so t

svn commit: r317284 - head/usr.sbin/pmcstat

2017-04-22 Thread Pedro F. Giffuni
Author: pfg Date: Sat Apr 22 16:02:40 2017 New Revision: 317284 URL: https://svnweb.freebsd.org/changeset/base/317284 Log: pmcstat(8); unsign some allocation variables and use reallocarray(3). Use unsigned values in some internal variables that will be used during allocation. The variable

svn commit: r317283 - head/sbin/fsck_ffs

2017-04-22 Thread Pedro F. Giffuni
Author: pfg Date: Sat Apr 22 14:50:11 2017 New Revision: 317283 URL: https://svnweb.freebsd.org/changeset/base/317283 Log: fsck_ffs: Unsign some variables and make use of reallocarray(3). Instead of casting listmax and numdirs to unsigned values just define them as unsigned and avoid the

svn commit: r317282 - head/sys/netinet6

2017-04-22 Thread Kristof Provost
Author: kp Date: Sat Apr 22 13:04:36 2017 New Revision: 317282 URL: https://svnweb.freebsd.org/changeset/base/317282 Log: Rename variable for clarity Rename the mtu variable in ip6_fragment(), because mtu is misleading. The variable actually holds the fragment length. No functional chan

svn commit: r317280 - head/usr.bin/tip/tip

2017-04-22 Thread Edward Tomasz Napierala
Author: trasz Date: Sat Apr 22 09:38:43 2017 New Revision: 317280 URL: https://svnweb.freebsd.org/changeset/base/317280 Log: Add basic example to the cu(1) man page. Srsly, folks, you don't need to install minicom for this... MFC after:2 weeks Modified: head/usr.bin/tip/tip/cu.1

svn commit: r317279 - head/sys/net

2017-04-22 Thread Alexander Motin
Author: mav Date: Sat Apr 22 08:38:49 2017 New Revision: 317279 URL: https://svnweb.freebsd.org/changeset/base/317279 Log: Remove unneeded conditions. MFC after:2 weeks Modified: head/sys/net/if_lagg.c Modified: head/sys/net/if_lagg.c