bba7ed20 kib Jan. 19, 2017, 2:27 p.m.
Edit comments which explain no longer relevant details, and add
locking annotations to the struct tmpfs_node members.

Tested by:	pho (as part of the larger patch)
Sponsored by:	The FreeBSD Foundation
MFC after:	1 week
cgit ViewVC
b328ce00 loos Jan. 19, 2017, 2:05 p.m.
Sponsored by:	Rubicon Communications, LLC (Netgate)
cgit ViewVC
9e3ff5c5 kib Jan. 19, 2017, 1:35 p.m.
Sponsored by:	The FreeBSD Foundation
MFC after:	1 week
cgit ViewVC
805ca177 delphij Jan. 19, 2017, 8:01 a.m.
result from & and the pattern instead of just assuming it's one bit
value).

Pointed out by Tianjie Mao <tjmao tjmao com>.

MFC after:	2 weeks
Differential Revision:	https://reviews.freebsd.org/D4827
cgit ViewVC
34a6b428 behlendorf1 Jan. 18, 2017, 11:10 p.m.
Two threads send_traverse_thread() and receive_writer_thread() should
end with thread_exit();

Mostly a cosmetic issue under IllumOS.

Authored by: Jorgen Lundman <lundman@lundman.net>
Reviewed by: Paul Dagnelie <pcd@delphix.com>
Reviewed by: Matt Ahrens <mahrens@delphix.com>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Ported-by: George Melikov <mail@gmelikov.ru>

OpenZFS-issue: https://www.illumos.org/issues/7659
OpenZFS-commit: https://github.com/openzfs/openzfs/commit/a569268
Closes #5603
cgit
9df99d6e asomers Jan. 18, 2017, 10:10 p.m.
It's pretty unlikely to actually hit this, but good to check it anyway

Reported by:	Coverity
CID:		1362018
MFC after:	4 weeks
Sponsored by:	Spectra Logic Corp
cgit ViewVC
daa0d9dd asomers Jan. 18, 2017, 8:24 p.m.
CID 1362055, 1362054: File descriptor leaks during shutdown
CID 1362013: Potential null-termination fail with long network device names
CID 1362097: Uncaught exception during memory pressure
CID 1362017, 1362016: Unchecked errors, possibly resulting in weird behavior
	if two devd instances start at the same time.
CID 1362015:	Unchecked error that will probably never fail

Reported by:	Coverity
CID:	1362055 1362054 1362013 1362097 1362017 1362016 1362015
MFC after:	4 weeks
Sponsored by:	Spectra Logic Corp
cgit ViewVC
c9bf8148 cem Jan. 18, 2017, 6:16 p.m.
UFS2 extended attribute names are not NUL-terminated.  Handle
appropriately.

Correct the EXTATTR_BASE_LENGTH() macro, which handled ea_namelength ==
one (mod eight) extended attributes incorrectly.

PR:		216127
Reported by:	dewayne at heuristicsystems.com.au
Reviewed by:	kib@
Sponsored by:	Dell EMC Isilon
Differential Revision:	https://reviews.freebsd.org/D9208
cgit ViewVC
4a42ab7d ngie Jan. 18, 2017, 6:14 p.m.
MFC after:	3 weeks
Sponsored by:	Dell EMC Isilon
cgit ViewVC
6e02fac9 cem Jan. 18, 2017, 5:55 p.m.
The ea_name string is not nul-terminated.  Correct the documentation.

Because the subsequent field is padded to 8 bytes, and the padding is
zeroed, the ea_name string will appear to be nul-terminated whenever the
length isn't exactly one (mod eight).

This was introduced in r167010 (2007).

Additionally, mark the length fields as unsigned.  This particularly
matters for the single byte ea_namelength field, which can represent
extended attribute names up to 255 bytes long.

No functional change.

PR:		216127
Reported by:	dewayne at heuristicsystems.com.au
Reviewed by:	kib@
Sponsored by:	Dell EMC Isilon
Differential Revision:	https://reviews.freebsd.org/D9206
cgit ViewVC
c5f61e6f mjg Jan. 18, 2017, 5:55 p.m.
Discussed with:	jhb
Tested by:	pho (previous version)
cgit ViewVC
3f0a0612 mjg Jan. 18, 2017, 5:53 p.m.
Discussed with:     jhb
Tested by:	pho (previous version)
cgit ViewVC
f3b3aa83 glebius Jan. 18, 2017, 5:21 p.m.
functional change.
cgit ViewVC
cf66bb8d glebius Jan. 18, 2017, 5:09 p.m.
lead to delayed send of data sent with sendto(MSG_NOSIGNAL).

Submitted by:	rrs
cgit ViewVC
1c6fccc1 br Jan. 18, 2017, 2:41 p.m.
Sponsored by:	DARPA, AFRL
cgit ViewVC