r325790 jhibbits Nov. 14, 2017, 3:23 a.m.
ofw_parse_bootargs() was added in r306065 as an attempt to unify the
various copies of the same code.  This simply migrates BERI to use it.

Reviewed By: brooks
Differential Revision: https://reviews.freebsd.org/D12962
r325789 jhibbits Nov. 14, 2017, 3:21 a.m.
r325788 tuexen Nov. 14, 2017, 2:30 a.m.
MFC after:	1 week
r325787 kevlo Nov. 14, 2017, 1:57 a.m.
Submitted by:	Kris G <netsick@gmail.com>
r325786 brd Nov. 14, 2017, 1:48 a.m.
Approved by:	bdrewery
r325785 imp Nov. 14, 2017, 12:29 a.m.
kern.poweroff_on_panic which, when enabled, instructs a system to
power off on a panic instead of a reboot.

kern.powercyle_on_panic which, when enabled, instructs a system to
power cycle, if possible, on a panic instead of a reboot.

Sponsored by: Netflix
Differential Revision: https://reviews.freebsd.org/D13042
r325782 glebius Nov. 13, 2017, 10:16 p.m.
shorten variable name to avoid extra long lines.
No functional changes.
r325781 feld Nov. 13, 2017, 9:55 p.m.
MFC after:	1 week
r325780 imp Nov. 13, 2017, 9:25 p.m.
what was tested...
r325779 imp Nov. 13, 2017, 9:23 p.m.
loader.conf for uboot have this in the list, but the default one
didn't. Since there's no harm and it's a failsafe, add it to the list.

Sponsored by: Netflix
r325778 jhb Nov. 13, 2017, 9:22 p.m.
The PT_DETACH case above the sendsig: label already looped over all
threads clearing flags in td_dbgflags.  Reuse this loop to clear
TDB_SUSPEND and move the logic out of the sendsig: block.
r325777 jhb Nov. 13, 2017, 9:09 p.m.
Most of the conditionals in the 'sendsig:' block are now only different
for PT_ATTACH vs other continue requests.  Pull the PT_ATTACH-specific
logic up into the PT_ATTACH case and simplify the 'sendsig:' block.  This
also permits moving the unlock of proctree_lock above the sendsig: label
since PT_KILL doesn't hold the lock and and the other cases all fall
through to the label.

Reviewed by:	kib
Differential Revision:	https://reviews.freebsd.org/D13073
r325776 bdrewery Nov. 13, 2017, 8:49 p.m.
Reported by:	ngie
Sponsored by:	Dell EMC Isilon
r325775 imp Nov. 13, 2017, 8:39 p.m.
loader.conf is also different between machines. On arm it's a minimal
one that's not quite compatible with the default one. On arm it's
minimal for speed, which is good, but there's also extra things in it
relative to the default on which break loading FDT which is bad. This
doesn't address that issue, but instead ensures the minimal one for
arm is used.

A similar issue for mips exists, but since we can have a beri variant
of /boot/loader and a uboot variant, I'm leaving that mess alone for
the moment.

Sponsored by: Netflix
r325772 arichardson Nov. 13, 2017, 8:07 p.m.
Before this patch libmagic was always built in the build-tools phase
even if -DWITHOUT_FILE was specified.

Reviewed by:	emaste, jhb
Approved by:	jhb (mentor)
Differential Revision: https://reviews.freebsd.org/D12925