Reduce diff from Illumos by better variables mapping.

Use the platform_*_t typedefs to help check the platform function types are
correct.

Sponsored by: ABT Systems Ltd

Add the missing void to function signatures in much of the arm code.

Sponsored by: ABT Systems Ltd

Use platform_*_t to check the platform function signatures are correct in
the Rockchip platform code and correct the one place they differ.

Sponsored by: ABT Systems Ltd

Switch Rockchip RK3188 SoC to use the platform code.

Reviewed by: andrew, manu
Differential Revision: https://reviews.freebsd.org/D8769

[gpiospi] add clock delay to avoid smashing of bits

Submitted by: Hiroki Mori <yamori83@yahoo.co.jp>
Reviewed by: loos, ray, mizhka
MFC after: 2 weeks
Differential Revision: https://reviews.freebsd.org/D8749

[spi] reformat message and ar5315_spi minor fix

This commit corrects print of nomatch (newline was too early) and fix
unit number for new child in ar5315_spi (was 0, now is -1 to calculate it
according to actual system state)

Submitted by: Hiroki Mori <yamori813@yahoo.co.jp>
Reviewed by: ray, loos, mizhka
MFC after: 2 weeks
Differential Revision: https://reviews.freebsd.org/D8749

Remove the only user of sysctl_add_oid().

My plan is to change this function's prototype at some point in the
future to add a new label argument, which can be used to export all of
sysctl as metrics that can be scraped by Prometheus. Switch over this
caller to use the macro wrapper counterpart.

Check that blkfront devices have a non-zero number of sectors and a
non-zero sector size.  Such a device would be a virtual disk of zero
bytes; clearly not useful, and not something we should try to attach.

As a fortuitous side effect, checking that these values are non-zero
here results in them not *becoming* zero later on the function.  This
odd behaviour began with r309124 (clang 3.9.0) but is challenging to
debug; making any changes to this function whatsoever seems to affect
the llvm optimizer behaviour enough to make the unexpected zeroing of
the sector_size variable cease.

PR: 215209
Security: The potential for variables to unexpectedly become zero
has worrying consequences for security in general, but
not so much in this particular context.

[iMX6] Add compatibility string for GPT timer on i.MX6 Dual

Up until r295436 GPT timer in i.MX6 Dual dts used the same compatiblity
string as i.MX6 Quad. After the sync up with Linux in r295436, GPT timer
stopped getting attached on the i.MX6 Dual

MFC after: 3 days

Remove a too strict test and instead, just filter the passed flags with the
supported capabilities.

Spotted by: yamori813@yahoo.co.jp (Hiroki Mori)
MFC after: 2 weeks

[iMX6] Fix platform compatibility string for i.MX6 Dual

i.MX6 Dual boot was broken since r308533 because ofw_bus_node_is_compatible
is more strict than fdt_is_compatible and does not accept partial matches

It's completely pointless to replace newlines with space
(this is done automatically for you upon shell expansion)

The flags of a WLAN need to be quoted (they contain things like brackets)

Simplify single-line if statements

Simplify loop by moving predicate to clause

Wordsmithing

Why test $? when you can test the command

Restore previous comment

Both simplify bringup of interface after changes and catch errors in debug

Calculate proper size of menu list dialog

There's an API function for catching errors and displaying them or
logging them to debug output

There's an API function for displaying pauses

There's an API function for displaying yes/no dialogs

There's an API function for displaying errors

Comment

Whitespace alignment

Relying on dialog auto-sizing (width/height/rows = 0) is a mistake
Use the provided API for calculating the appropriate size of menus

Remove unnecessary quotes

Add missing quotes

In awk, if you're going to append a newline to your printf
AND you're going to print only the argument, just use print

This statement has too many backslashes

Neither printf (and as is commonly known) nor print need parens in awk

Whitespace and alignment

You don't need parentheses for awk's printf

Continued resolution of conveluted statement
We shouldn't be coding things like "x || (x && x) || x || x || x ..."

These two error messages have always been backwards since inception

Why use $? when you can use the command itself

If the first ping succeeded, why on Earth should we ping it again?

Start deconstructing a conveluted hunk of code

Remove completely unnecesary parentheses

Why repeat yourself when you can send stderr to the same place as stdout?

Properly quote variable

Use more generic f_yesno() from provided API

The output of dialog needs to be sanitized
for portability/compatibility requirements

Whitespace alignment

Sort the domains

The --no-items and --stdout options are non-standard and should be avoided

From the dialog(1) manual:
using [--stdout] in portable scripts is not recommended

More efficiently make use of the exit status

Stop repeating strings (centralize prompt string)
NB: Changes to strings now only affect a single line

Add missing backslash (no real effect; it's pedantic and correct for
the interpolation level)

Use the oft-neglected awk syntax "startcondition, stopcondition { ... }" to
process the range of country labels which appear as columnar list from the
"ifconfig DEV list countries" command. Not only improving maintainability,
but also properly encapsulating arguments in single-quotes instead of
trying to escape whitespace. It is also completely unnecessary to collapse
newlines into whitespace (shell will do this for you automatically upon
expansion of the contents where necessary).

NB: This also changes the sorting algorithm to sort on the country code,
not the country name. The type-ahead feature of dialog is destroyed if the
tags are not sorted properly.

Quote WLAN_IFACE (pedantic)

Add tests for reaper receiving SIGCHLD (r309886).

PR: 213928
MFC after: 2 weeks

In awk, casting a variable as a boolean condition is the same as testing if
the length of the variable contents is greater than zero

This allows us to also move the secondary condition into the action clause

Remove an unnecessary call to f_dialog_title_restore()

Whitespace

Utilize provided i18n strings

Remove incomplete and unnecessary creation of fd3

The provided API already provides a passthru descriptor and even
gives you a varaible for referring to it.

Remove unnecessary semi-colons

Use awk the following (more succinct) awk syntax:

condition1 { action1 }
condition2 { action2 }

instead of the following syntax:

{
if (condition1) { action1 }
else if (condition2) { action2 }
}

1 is the default descriptor for redirects without an fd prefix

Fix invalid parameter expansion (change $@ to "$@")

Without quotes, $@ loses its special meanining (see below)

% sh -c 'echo $@' /bin/sh "   1   " "   2   "
1 2
% sh -c 'echo "$@"' /bin/sh "   1   " "   2   "
   1       2

The quotes are required to get ARGV to be unperterped

Allow the script path to contain whitespace and special characters

Use provided API to centralize dialog title strings

Reorder dialog parameters based on commonality for readability

Fix incorrect use of provided API
The result of which was incorrectly sized menu dialogs

Use provided API (change "dialog" to "$DIALOG")

Whitespace (dialog options separated to minimize diffs)

ed(1): Simplify some checks.

The return type for both fread(3) and fwrite(3) cannot be negative, this
renders some checks invalid and variable 'ct' unnecessary.
Also bump 'len' to size_t to avoid signed/unsigned comparison warnings.

Use the spibus accessor when applicable.

MFC after: 3 days

Consolidate redirects into here documents, with proper code indentation

- Refactor listening socket list.  All of the listening sockets are
  now maintained in a single linked-list in a transport-independent manner.
- Use queue.h for linked-list structure.
- Use linked-list for AllowedPeers.
- Use getaddrinfo(8) even for Unix Domain sockets.
- Use macros to type-casting from/to struct sockaddr{,_in,_in6}.
- Define fu_* macro for union f_un to shorten the member names.
- Remove an extra #include <sys/type.h>.
- Add "static" to non-exported symbols.
- !INET support is still incomplete but will be fixed later.

There is no functional change except for some minor debug messages.

Remove an unnecessary "return $?" at end of function

Temporarily backout the previous commit because it was totally broken due to
unresolved merge conflicts.

Pointy hat to: hrs

Use ternary operator

vfs: use vrefact in getcwd and fchdir

If you're not going to make use of the products of a match() in awk
(e.g., RSTART and RLENGTH variables) then use ~ instead of match()

Undo r309891.  Konstantin is right in that this condition normally
cannot happen - the um_dev field is assigned at mount and never written
to afterwards.

Now that these variables do not contain the --default-item flag itself,
change the name of the variable from $def_item_... to $default_...

- Refactor listening socket list.  All of the listening sockets are
  now maintained in a single linked-list in a transport-independent manner.
- Use queue.h for linked-list structure.
- Use linked-list for AllowedPeers.
- Use getaddrinfo(8) even for Unix Domain sockets.
- Use macros to type-casting from/to struct sockaddr{,_in,_in6}.
- Define fu_* macro for union f_un to shorten the member names.
- Remove an extra #include <sys/type.h>.
- Add "static" to non-exported symbols.
- !INET support is still incomplete but will be fixed later.

There is no functional change except for some minor debug messages.

Always pass --default-item parameter to dialog

There is zero harm in always passing --default-item to dialog

Centralize backtitle string

Fix regression when stdin/out/err fds are are overridden by shell.

Submitted by: Pawel Biernacki <pawel.biernacki@gmail.com>
Reported by: ngie
Approved by: ngie
Sponsored by: Mysterious Code Ltd.
Differential Revision: https://reviews.freebsd.org/D8543

Use provided API instead of hard-coded status integers

Whitespace

Remove unnecessary `-n' parameter to head/tail

awk(1) match() takes a regex, use /.../ to remind ourselves of this
NB: The difference between "..." and /.../ for a regex in awk is that
quoted strings go through escape expansion first (e.g., \\ becomes \)

Be internally consistent (": > ..." is used elsewhere in this file)

"echo | sed | sed | awk" is silly (changed to "echo | awk")

Add missing `-e' parameter to sed invocations

Allow $BSDINSTALL_TMPETC to contain whitespace or special chars

CS ivar is uint32_t, not int.

MFC after: 3 days

Group fallbacks together

Remove unnecessary quotes around number in test

Change "[ ! ... ] && ..." to "[ ... ] || ..."

Collapse tiny if statements

Replace funny block with something easy to digest

Consolidate locals