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

Remove unnecessary local initializers

Change "[ ! -z ... ]" => "[ ... ]" and "[ -z ... ]" => "[ ! ... ]"

Enable ACPI on arm64. It's not yet functional, but it will help keeping the
code building until it is ready.

Obtained from: ABT Systems Ltd
Sponsored by: The FreeBSD Foundation

Use $( ... ) instead of `...`

Comments

Eliminate every mention of PG_CACHED pages from the comments in the machine-
independent layer of the virtual memory system.  Update some of the nearby
comments to eliminate redundancy and improve clarity.

In vm/vm_reserv.c, do not use hyphens after adverbs ending in -ly per
The Chicago Manual of Style.

Update the comment in vm/vm_page.h defining the four types of page queues to
reflect the elimination of PG_CACHED pages and the introduction of the
laundry queue.

Reviewed by: kib, markj
Sponsored by: Dell EMC Isilon
Differential Revision: https://reviews.freebsd.org/D8752

Add basic ATF tests for Capability mode .. lookups

A follow-up to r309887.

Several tests copied verbatim from
https://github.com/emaste/snippets/blob/master/test_openat.c .

Reviewed by: kib@, ngie@ (earlier version)
X-MFC-With: r309887
Sponsored by: Dell EMC Isilon
Differential Revision: https://reviews.freebsd.org/D8748

Apply clear_mask correctly.

Reviewed by: jmcneill, kan
Sponsored by: DARPA, AFRL

On non-Intel platforms don't ignore the PCI root bridge mapping in
acpi_set_resource, the mappings are needed on arm64.

Obtained from: ABT Systenms Ltd
Sponsored by: The FreeBSD Foundation

clk_div: Add a div lookup table

Some clocks on SoC have a diff between the value written in the register
and the real divider.
Add a table that where we can lookup the real value of the divider.

Reviewed by: mmel (earlier revision)
Differential Revision: https://reviews.freebsd.org/D8728

vfs: add vrefact, to be used when the vnode has to be already active

This allows blind increment of relevant counters which under contention
is cheaper than inc-not-zero loops at least on amd64.

Use it in some of the places which are guaranteed to see already active
vnodes.

Reviewed by: kib (previous version)

Fix gic_cpu_mask() calculation in ARM GIC

r309616 changed the definition of GICD_ITARGETSR(n) to take the irq
id as argument, but the usage of the macro in gic_cpu_mask() was not
updated to reflect this. This causes the cpu mask to be computed
incorrectly.

Update the GICD_ITARGETSR() call to fix this, this fixes a hang seen
while booting freebsd on qemu-system-aarch64 with SMP enabled.

Avoid dereferencing NULL pointers in devtoname().  I've seen it panic,
called from ufs_print() in DDB.

MFC after: 1 month

Increase interrupt cells in generic_pcie_fdt_route_interrupt

ARM GIC specification in device trees use 3 cells, so the current
limit of 2 causes the last cell to be dropped. This in turn can
cause the interrupt polarity and trigger settings to be incorrect.

Increase the limit to 4 which should handle all reasonable cases.
This fixes issues seen in QEMU when registering PCI interrupts.

Modify IPv6 statistic accounting in ip6_input().

Add rcvif local variable to keep inbound interface pointer. Count
ifs6_in_discard errors in all "goto bad" cases. Now it will count
errors even if mbuf was freed. Modify all places where m->m_pkthdr.rcvif
is used to use local rcvif variable.

Obtained from: Yandex LLC
MFC after: 1 month

Enable lookup_cap_dotdot and lookup_cap_dotdot_nonlocal.

Requested and reviewed by: cem
Sponsored by: The FreeBSD Foundation
MFC after: 2 weeks
Differential revision: https://reviews.freebsd.org/D8746

When a zombie gets reparented due to the parent exit, send SIGCHLD to
the reaper.

The traditional reaper init(8) is aware of zombies silently reparented
to it after the parents exit, it loops around waitpid(2) to collect
them.  For other reapers, the silent reparenting is surprising and
collecting zombies requires a thread blocking in waitpid(2) just for
that purpose.  It seems that sending second SIGCHLD is a better
workaround than forcing all reapers to obey the setup.

Reported by:  Michael Zuo <muh.muhten@gmail.com>, jilles
PR: 213928
Reviewed by: jilles (previous version)
Sponsored by: The FreeBSD Foundation
MFC after: 2 weeks

Add ip6_tryforward() - a run to completion forwarding implementation
for IPv6.

It gets performance benefits from reduced number of checks. It doesn't
copy mbuf to be able send ICMPv6 error message, because it keeps mbuf
unchanged until the moment, when the route decision has been made.
It doesn't do IPsec checks, and when some IPsec security policies present,
ip6_input() uses normal slow path.

Reviewed by: bz, gnn
Obtained from: Yandex LLC
MFC after: 1 month
Sponsored by: Yandex LLC
Differential Revision: https://reviews.freebsd.org/D8527

hyperv/hn: Add polling support

MFC after: 1 week
Sponsored by: Microsoft
Differential Revision: https://reviews.freebsd.org/D8739

hyperv/vmbus: Add channel polling support.

MFC after: 1 week
Sponsored by: Microsoft
Differential Revision: https://reviews.freebsd.org/D8738

ed(1): Prevent possible overflows during allocation.

Make sure the parameters used for malloc(3) can hold size_t sizes.
This should help ed(1) handle bigger data in the future.

MFC after: 2 weeks

Merge changes from vendor to address several Coverity issues with
contrib/libarchive's tests

MFC after: 2 weeks
Obtained from: libarchive (ebe29c, fd0ea2, f9e3de)
Reported by: Coverity

We currently don't do TSO if ip options are present. In case of IPv6, we look at
in6p_options to check that. That is incorrect as we carry ip options in
in6p_outputopts. Also, just checking for in6p_outputopts being NULL won't
suffice as we combine ip options and ip header fields both in that one field.
The commit fixes this by using ip6_optlen() which correctly calculates length
of only ip options for IPv6.

Reviewed by:     ae, bz
MFC after:     3 weeks
Sponsored by:     Limelight Networks

Postpone ZVOL media/block size caching till first open.

At least on FreeBSD there are no legal way to access media or get its
size without opening device/provider first.  Postponing this caching
allows to skip several disk seeks per ZVOL/snapshot during import.

For HDD pool with 1 ZVOL in dev mode with 1000 snapshots this reduces
pool import time from 40 to 10 seconds.

MFC after: 2 weeks
Sponsored by: iXsystems, Inc.

When tmpfs and POSIX shm pagein a page for the sole purpose of performing
truncation, immediately queue the page for asynchronous laundering rather
than making the page pass through inactive queue first.

Reviewed by: kib, markj