d0103273 mav Feb. 28, 2021, 2:08 p.m.
Too long name looks bad in `vmstat -m`.

MFC after:	1 week
cgit
99954552 freqlabs Feb. 28, 2021, 2:20 a.m.
There is already a globally defined name variable.

MFC after:	1 week
cgit
a0ebb915 freqlabs Feb. 28, 2021, 2:20 a.m.
Also trimmed an unused block of code that never prints out LAGG_PROTOS.
Reviewed by:	kp (earlier version)
MFC after:	2 weeks
Differential Revision:	https://reviews.freebsd.org/D28961
cgit
da393346 freqlabs Feb. 28, 2021, 2:20 a.m.
A trivial change now that ifconfig is already using libifconfig.
Reviewed by:	kp (earlier version)
MFC after:	2 weeks
Differential Revision:	https://reviews.freebsd.org/D28955
cgit
3fe2c68b rmacklem Feb. 28, 2021, 1:54 a.m.
Juraj Lutter (otis@) reported a panic "dvp != vp not true" in
cache_enter_time() called from the NFS client's nfsrpc_readdirplus()
function.
This is specific to an NFSv3 mount with the "rdirplus" mount
option. Unlike NFSv4, NFSv3 replies to ReaddirPlus
includes entries for the current directory.

This trivial patch avoids doing a cache_enter_time()
call for the current directory to avoid the panic.

Reported by:	otis
Tested by:	otis
Reviewed by:	mjg
MFC after:	3 days
Differential Revision:	https://reviews.freebsd.org/D28969
cgit
b5449c92 kib Feb. 27, 2021, 11:27 p.m.
for the which which definitely use membar to sync with interrupt handlers.

libc and rtld uses of __compiler_membar() seems to want compiler barriers
proper.

The barrier in sched_unpin_lite() after td_pinned decrement seems to be not
needed and removed, instead of convertion.

Reviewed by:	markj
MFC after:	1 week
Sponsored by:	The FreeBSD Foundation
Differential Revision:	https://reviews.freebsd.org/D28956
cgit
1d9ba697 freqlabs Feb. 27, 2021, 11:12 p.m.
This should return -1 with OTHER/ENOMEM set in the handle when malloc
fails, like everywhere else in libifconfig.

Reviewed by:	kp
MFC after:	1 week
Differential Revision:	https://reviews.freebsd.org/D28964
cgit
1d8510c1 mjg Feb. 27, 2021, 10:29 p.m.
It happens to trip over an assert but does not matter for correctness at
this time. However, do it for future proofing.

Reported by:	avg
cgit
1239a722 mjg Feb. 27, 2021, 10:29 p.m.
Historically it was allowed for any names, but arguably should never be
even attempted. Allow it again since there is a release pending and
allowing it is bug-compatible with previous behavior.

Reported by:	otis
cgit
70e95f0b tuexen Feb. 27, 2021, 10:27 p.m.
d656ce19 rew Feb. 27, 2021, 9:07 p.m.
MAX_SNAPSHOT_VMNAME is a macro used to set the size of a character
buffer that stores a filename or the path to a file - this file is used
by the save/restore feature.

Since the file doesn't have anything to do with a vm name, rename
MAX_SNAPSHOT_VMNAME to MAX_SNAPSHOT_FILENAME. Bump the size to PATH_MAX
while here.

Reviewed by:	jhb
Differential Revision:	https://reviews.freebsd.org/D28879
cgit
da971391 rew Feb. 27, 2021, 9:05 p.m.
Combine send_start_checkpoint() and send_start_suspend() into a
single function named snapshot_request().

snapshot_request() is equivalent to send_start_checkpoint() and
send_start_suspend() except that it takes an additional argument. The
additional argument, enum ipc_opcode, is used to determine the type of
snapshot request being performed. Also, switch to using strlcpy instead
of strncpy.

Reviewed by:	jhb
Differential Revision:	https://reviews.freebsd.org/D28878
cgit
b7fd9c4e rew Feb. 27, 2021, 9:03 p.m.
Generalize the naming here since the domain socket that uses these codes
might be used for purposes other than the save/restore feature.

- rename checkpoint_opcodes to ipc_opcode

Reviewed by:	jhb
Differential Revision:	https://reviews.freebsd.org/D28877
cgit
a9f11355 rwatson Feb. 27, 2021, 4:53 p.m.
MFC after:	3 days
cgit
7bfd8444 rwatson Feb. 27, 2021, 4:51 p.m.
MFC after:	3 days
cgit