r351175 mjg Aug. 17, 2019, 6:19 p.m.
- move allproc lock into the func, it is of no use prior to it
- the code would lock p1 and p2 while holding allproc to partially
construct it after it gets added to the list. instead we can do the
work prior to adding anything.
- protect lastpid with procid_lock

As a side effect we do less work with allproc held.

Sponsored by:	The FreeBSD Foundation
ViewVC
r351174 mjg Aug. 17, 2019, 5:56 p.m.
The limit is almost never reached. Do the check only on failure to see if
we can override it.

No change in user-visible behavior.

Sponsored by:	The FreeBSD Foundation
ViewVC
r351173 mjg Aug. 17, 2019, 5:42 p.m.
Code doing this is commented with a claim that these IDs are occupied by
daemons, but that's demonstrably false. To an extent the range is used by init
and kernel processes (and on sufficiently big machines it indeed is fully
populated).

On a sample box 40-way box the highest id in the range is 63. On a different one
it is 23. Just use the range.

Sponsored by:	The FreeBSD Foundation
ViewVC
r351172 asomers Aug. 17, 2019, 5:22 p.m.
Submitted by:	Ján Sučan <sucanjan@gmail.com>
MFC after:	2 weeks
Sponsored by:	Google, inc. (Google Summer of Code 2019)
Differential Revision:	https://reviews.freebsd.org/D21260
ViewVC
r351171 asomers Aug. 17, 2019, 3:25 p.m.
This is a preparation step for adding ATF tests of in_cksum(), which has been
modified to operate on unaligned data. ping.o cannot be linked to the test
executable because both of them contain 'main' symbol.

Submitted by:	Ján Sučan <sucanjan@gmail.com>
MFC after:	2 weeks
Sponsored by:	Google, inc. (Google Summer of Code 2019)
Differential Revision:	https://reviews.freebsd.org/D21288
ViewVC
r351170 asomers Aug. 17, 2019, 2:28 p.m.
Revision 350859 removed level of indirection that was needed for setting the
caller's `cp' pointer. dnsdecode() uses return value to indicate error or
success. It returns pointer to a buffer holding a decompressed DNS name or
NULL. The caller uses that value only to find out the result, not for accessing
the buffer.

We use the return value to propagate the new value of `cp' pointer to
the caller instead of using an output argument.

Submitted by:	Ján Sučan <sucanjan@gmail.com>
MFC after:	2 weeks
MFC-With:	350859
Sponsored by:	Google, Inc (Google Summer of Code 2019)
Differential Revision:	https://reviews.freebsd.org/D21266
ViewVC
r351169 cem Aug. 17, 2019, 10:11 a.m.
Lost in translation between different SDP stacks.

Reported by:	hselasky
ViewVC
r351168 avg Aug. 17, 2019, 9:23 a.m.
If vn_lock() failed, then the function returned the error but the vnode
obtained via zfs_zget() was never released.

MFC after:	10 days
Sponsored by:	Panzura
ViewVC
r351167 hselasky Aug. 17, 2019, 8:29 a.m.
Submitted by:	Kevin Zheng <kevinz5000@gmail.com>
PR:		239918
MFC after:	1 week
ViewVC
r351166 thj Aug. 17, 2019, 6:47 a.m.
Reduce the size of the EFI_STAGING area we allocate on arm to 32. On arm SBC
such as the NanoPi-NEOLTS the staging area allocation will fail on the 256MB
model with a staging size of 64.

Reviewed by:    bcran, manu
Approved by:    bz (mentor)
MFC after:      2 weeks
Differential Revision: https://reviews.freebsd.org/D21016
ViewVC
r351165 thj Aug. 17, 2019, 6:44 a.m.
Add common firewall NAT tests for pf, ipf and ipfw (using both in-kernel and
userspace NAT).

Submitted by:   Ahsan Barkati
Sponsored by:   Google, Inc. (GSoC 2019)
Reviewed by:    kp
Approved by:    bz (mentor)
MFC after:      2 weeks
Differential Revision: https://reviews.freebsd.org/D21199
ViewVC
r351163 cem Aug. 17, 2019, 3:09 a.m.
Sponsored by:	Dell EMC Isilon
ViewVC
r351162 cem Aug. 17, 2019, 3:07 a.m.
Sponsored by:	Dell EMC Isilon
ViewVC
r351161 cem Aug. 17, 2019, 3:06 a.m.
Sponsored by:	Dell EMC Isilon
ViewVC
r351160 cem Aug. 17, 2019, 3:05 a.m.
Sponsored by:	Dell EMC Isilon
ViewVC