d5bf1cf1 behlendorf1 June 22, 2019, 11:30 p.m.
30af21b025 broke build on Fedora. gcc can detect potential overflow
on compile-time. Consider strlen of already copied string.

Also change strn to strl variants per suggestion from @behlendorf
and @ofaaland.

libzfs_input_check.c: In function 'test_redact':
libzfs_input_check.c:711:2: error: 'strncat' specified bound 288 equals
 destination size [-Werror=stringop-overflow=]
  strncat(bookmark, "#testbookmark", sizeof (bookmark));

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: Olaf Faaland <faaland1@llnl.gov>
Signed-off-by: Tomohiro Kusumi <kusumi.tomohiro@gmail.com>
Closes #8939
31d2b1cf sevan June 22, 2019, 10:43 p.m.
07fb2bcc sevan June 22, 2019, 10:34 p.m.
As linked to in bug 26137 as a source
"We already synchronize our code periodically with NetBSD for most of our user commands"
cgit ViewVC
9035b225 dougm June 22, 2019, 7:27 p.m.
Fix a style violation with regard to header file arrangement.

Improved by: alc
Approved by: markj, kib (mentor)
Differential Revision: https://reviews.freebsd.org/D20723
cgit ViewVC
6d4d6573 mav June 22, 2019, 7:09 p.m.
I don't want to be regularly notified that my enclosure violates standards
until there is some real problem I want to debug.

MFC after:	2 weeks
cgit ViewVC
05b62601 imp June 22, 2019, 4:54 p.m.
Adds a list of valid CPUTYPE flags for arm and arm64 architectures
List taken from share/mk/bsd.cpu.mk

Submitted by: Daniel Engberg
Reviewed By: imp
Differential Revision: https://reviews.freebsd.org/D20315
cgit ViewVC
36c5a4cb alc June 22, 2019, 4:26 p.m.
(1) pmap_remove(), where it eliminates redundant TLB invalidations by
pmap_remove() and pmap_remove_l3(), and (2) pmap_enter_l2(), where it may
optimize the TLB invalidations by batching them.

Reviewed by:	markj
MFC after:	1 week
Differential Revision:	https://reviews.freebsd.org/D12725
cgit ViewVC
6ab631e8 dteske June 22, 2019, 3:39 p.m.
Latter is undesired when including <sys/param.h> according to style(9)

Submitted by:	Faraz Vahedi
Reviewed by:	cem
Differential Revision:	https://reviews.freebsd.org/D20637
cgit ViewVC
a6d2a24c rlibby June 22, 2019, 5:35 a.m.
b8038d78 mav June 22, 2019, 3:50 a.m.
MFC after:	2 weeks
cgit ViewVC
a616b253 dougm June 22, 2019, 3:16 a.m.
'-E' appears on the swapon command line, or if "trimonce" appears as
an fstab option.

Discussed at: BSDCAN
Tested by: markj
Reviewed by: markj
Approved by: markj (mentor)
Differential Revision:https://reviews.freebsd.org/D20599
cgit ViewVC
df840654 vangyzen June 22, 2019, 1:20 a.m.
Until r349278, bhyve presented a seg_max to the guest that was too large.
Detect this case and clamp it to the virtqueue size.  Otherwise, we would
fail the "too many segments to enqueue" assertion in virtqueue_enqueue().

I hit this by running a guest with a MAXPHYS of 256 KB.

Reviewed by:	bryanv cem
MFC after:	1 week
Sponsored by:	Dell EMC Isilon
Differential Revision:	https://reviews.freebsd.org/D20703
cgit ViewVC
6805c9b7 mav June 22, 2019, 1:06 a.m.
SES specifications tell: "The Additional Element Status descriptors shall
be in the same order as the status elements in the Enclosure Status
diagnostic page".  It allows us to question ELEMENT INDEX that is lower
then values we already processed.  There are many SAS2 enclosures with
this kind of problem.

While there, add more specific error messages for cases when ELEMENT INDEX
is obviously wrong.  Also skip elements with INVALID bit set.

MFC after:	2 weeks
cgit ViewVC
0feb46b0 scottl June 21, 2019, 11:40 p.m.
visible functional changes, though code flow was modified a bit
internally to lessen the need for goto jumps and chained if
cgit ViewVC
44f654fd asomers June 21, 2019, 11:29 p.m.
Even if a short read is caused by EOF, it's still necessary to bzero the
remaining buffer, because that buffer could become valid as a result of a
future ftruncate or pwrite operation.

Reported by:	fsx
Sponsored by:	The FreeBSD Foundation
cgit ViewVC