Shrink cluster size for FAT32 to fullfil the new minimum cluster
limit of newfs_msdos for our small 64 MB vnd disk image.
suggested by jsg@

LibreSSL error message has changed, adapt test.

Remove unneeded calls to tls_init(3)

As per the manual and lib/libtls/tls.c revision 1.79 from 2018
"Automatically handle library initialisation for libtls." initialisation
is handled automatically by other tls_*(3) functions.

Remove explicit tls_init() calls from base to not give the impression of
it being needed.

Feedback tb
OK Tests mestre

Restructure the getopt_long options. Sort them mostly by name with the
exception of no-xyz options that are grouped with the coresponding xyz
option. Fix --no-motd to use the internal flag setting of getopt_long.
Also use some defines instead of numbers for pure long options that
need special handling.
OK benno@

timeout_add() always sets a new expiry time unrelated to the previous
one and requeues as needed so no need to call timeout_del() before
timeout_add()

spotted by kettenis@

Move default value for TERM into tmux.h.

reorder SessionType; ok djm

When mod_timer() is called with a value less than or equal to the
current number of jiffies delete the timeout and reschedule for the next
tick.  Avoids timeout_add() failing the ticks >= 0 assertion when
mod_timer() is called from i915_utils.c set_timer_ms().

Reported and earlier version tested by Tom Murphy on Kaby Lake R.

Adjust expected certificate error message. Caused by recent work to support
partial chains in libcrypto.

ok bluhm@

fix "logically dead code" flagged by coverity

It looks like twe was refactored in 2011 and one error check was missed.
While the device may no longer be widely used, this helps reduce the
coverity alert count.

CID 1453371

ok krw@

add a SessionType directive to ssh_config, allowing the configuration
file to offer equivalent control to the -N (no session) and -s
(subsystem) command-line flags.

Part of GHPR#231 by Volker Diels-Grabsch with some minor tweaks;
feedback and ok dtucker@

No need to manually reinvent nitems() 5 times to scan part_types.

No functional change.

Give #() commands a one second grace period where the output is empty
before telling the user they aren't doing anything. GitHub issue 2774.

Ensure that we don't overrun the TX FIFO for longer transfers.  Check
the limit after every character, and wait for the FIFO to empty before
sending out more bytes.  With this I can now use ipmitool(1) to change
IPMI passwords on the Ampere eMAG.

ok kettenis@

Beauharnios -> Beauharnois

Add more checks for eContent 'version' fields.

Input from deraadt@, tb@, claudio@

OK deraadt@ claudio@ tb@

Fix TIMEOUT_INITIALIZER_{FLAGS,KCLOCK}() macro.

ok cheloha@

modernize province abbreviations

update canadian area codes according to cnac.ca

Disk sector addresses are normally stored/provided in uint64_t
variables/parameters, not off_t.

Adjust various parameters and variables accordingly. A few missed const's
and verbiage tweaks in passing.

No functional change.

sync

Replace a stray use of letoh64() on gh_part_num and
gh_part_size, which are uint32_t fields, with letoh32().

Only use client for sizing when not detached, GitHub issue 2772.

Order the MRT table dump versions by preference also the text refers
to the latter two (table-mp and table) to be only available for
compatibility. People should really only use table-v2 since it is
the only method which will properly dump everything.

Remove unused `PolicyHead' from 'sockaddr_encap' structure.

ok tobhe@

Add f_modify and f_process callbacks to FIFO filterops.

OK millert@ mpi@

Toss 'const' in anywhere the compiler doesn't complain about, thus identifying
places that *should* be const but currently aren't and preventing the rot from
spreading further while those issues are dealt with.

No functional change.

Add uaudio(4) and umidi(4). ok kettenis@, mlarkin@

Final batch of struct field name tweaks. 'cmd_' for cmd, ut_' for
unit_types, 'pt_' for part_type, 'pg_' for protected_guid.

No functional change.

new manual page X509_print_ex(3)

Use the x509_verify_cert_cache_extensions fuction instead of manually
calling the OpenSSL legacy cache extensions goo.

Requested by tb@
ok tb@

Change the error reporting pattern throughout the tree when unveil
fails to report the path that the failure occured on. Suggested by
deraadt@ after some tech discussion.

Work done and verified by Ashton Fagg <ashton@fagg.id.au>

ok deraadt@ semarie@ claudio@

document X509V3_extensions_print(3)

Add 'dk_' prefix to struct disk field names.

No functional change.

fix "evaluation order violation" flagged by coverity

Coverity flagged the self-assignment introduced in the previous commit.
This is CID 1505120.

ok kevlo@

document X509V3_EXT_print(3)

Remember to set CPUF_RUNNING on secondary CPUs.

Make hw_cpu_hatch() more similar on loongson and octeon.

Make limit for time_t test unconditional in the format_absolute_time
fix for bz#3329 that allows printing of timestamps past INT_MAX.
This was incorrectly included with the previous commit.   Based on
discussion with djm@.

Use existing format_absolute_time() function when printing cert validity
instead of doing it inline.  Part of bz#3329.

wrap a long line

fix some broken tests; clean up output

Add 'prt_' prefix to struct prt field names.

Use 'dp' instead of 'prt' for struct dos_partition *
paramters, and 'prt' instead of 'partn' for struct prt *
parameters.

No functional change.

Optimize gadget fixups for MOV instructions.

Instead of swapping registers around, we can just use the REV version of
the same instruction, which has the same effect but encodes differently and
does not result in return bytes in the binary. This reduces the number
of xchg instructions resulting from gadget fixing.

Prompted by ratchov@, with input from millert@ and sthen@.

ok sthen@

While the traditional OpenSSL return value and behaviour of BIO_dump(3)
is pure comedy gold, and now documented as such, sadly this bit of pure
Muppet genius can't really in good consience stay in the tree as is.

Change BIO_dump to always return the number of bytes printed on success
and to stop printing and return -1 on failure if a writing function
fails.

ok tb@, jsing@

Add 'mbr_' prefix to struct mbr field names.

Change 'offset' to mbr_lba_self and 'reloffset'
to mbr_lba_firstembr to make their use more evident.
Adjust a few parameter names to match.

Change 'part[]' to mbr_prt[] to reflect that it is an
array of struct prt.

No functional change.

new manual page ASN1_parse_dump(3)

Correct various min/max cluster numbers for FAT12/16/32.  From NetBSD.

ok krw@, jsg@

sync

document ASN1_get_object(3)

Move ask_cmd(), ask_num(), ask_pid(), ask_string(), parse_b()
and crc32().

No functional change.

Add tabs to make variable declarations easier
to read.

No functional change.

Add tabs to make struct definitions and function prototypes easier
to read.

No functional change.

'return' is not a function call.

No functional change.

convert db_addr_t to vaddr_t

correct comment
from Jonathan Kollasch in NetBSD

tweak indentation of conditional in db_validate_address().
this also matches arm64 now/again.

Fix a read buffer overrun in X509_CERT_AUX_print(3),
which by implication also affects X509_print(3).

The ASN1_STRING_get0_data(3) manual explitely cautions the reader
that the data is not necessarily NUL-terminated, and the function
X509_alias_set1(3) does not sanitize the data passed into it in
any way either, so we must assume the alias->data field is merely
a byte array and not necessarily a string in the sense of the C
language.

I found this bug while writing manual pages for these functions.

OK tb@

As an aside, note that the function still produces incomplete and
misleading results when the data contains a NUL byte in the middle
and that error handling is consistently absent throughout, even
though the function provides an "int" return value obviously intended
to be 1 for success and 0 for failure, and even though this function
is called by another function that also wants to return 1 for success
and 0 for failure and even does so in many of its code paths, though
not in others.  But let's stay focussed.  Many things would be nice
to have in the wide wild world, but a buffer overflow must not be
allowed to remain in our backyard.

Pass expected result first when diff(1)ing for result.

new manual page BIO_dump(3)

Add a bunch of workarond in the verifier to support partial chains and
the saving of the first error case so that the "autochain" craziness from
openssl will work with the new verifier. This should allow the new verification
code to work with a bunch of the autochain using cases in some software.
(and should allow us to stop using the legacy verifier with autochain)

ok tb@

Add test case for single static address in configuration payload in
addition to existing "config address" test that uses an address pool.

honor objdir

minor nits:
* compile the edit utility once
* emit a warning during timeout

Sync maxusers with other 64bit archs.

From miod@, ok deraadt@

sync

use vaddr_t as type for frames as is commonly used elsewhere too

ok kettenis@

fix scentence in db_printsym comment

Remove unused `cc_koperations' and `cc_queued' members from 'cryptocap'
struct.

ok bluhm@

Parse /soc/dma-ranges to populate the openbsd,dma-constraint property.
On the MNT Reform, which uses a i.MX8MQ with 4 GB of memory, this makes
sure that we don't allocate DMA buffers above the 32-bit boundary.  As
it turns out the i.MX8MQ's I/O devices are limited to 32-bit addresses.
The i.MX8MP seems to be better in that regard, though at least the USB
controller is still limited.  That's a bit harder to fix in a dynamic
fashion, but we'll take care of that as soon as someone shows up with
an i.MX8MP with that much memory.

ok kettenis@

Use SLIST macro for cryptosoft sessions instead of hand rolled list.
OK mpi@ tobhe@

Fix mixup between localKeyID and friendlyName.

"please commit" schwarze

KNF: remove whitespace between functions and parentheses

new manual page for X509_keyid_set1(3), X509_keyid_get0(3),
X509_alias_set1(3), X509_alias_get0(3)

Switch iwm(4) to new firmware images available in iwm-firmware-20210512.

This updates firmware for 3165, 3168, 7265, 8260, 8265, 9260, 9560 devices.
Other devices did not receive firmware updates from Intel.
New firmware images should contain revelant fixes for fragattacks:
https://www.intel.com/content/www/us/en/security-center/advisory/intel-sa-00473.html

Running fw_update(1) may be required before rebooting into a new kernel.
sysupgrade(8) will take care of this.

Blocking issues reported during the previous attempt at using these firmware
images have been addressed.

When reporting issues, please enable 'ifconfig iwm0 debug', reproduce the
error once more, and include the full dmesg in your bug report.

Print new iwm(4) firmware version numbers like Linux iwlwifi does it.

Starting with major version 35 the Linux driver prints the minor version
number in hexadecimal.

remove unused lines of code from iwm(4) which were specific to iwx(4) devices

Fix errors in the probe request template we provide to iwm(4) firmware.

The SSID should not be inserted into the template itself. The template
just needs an empty SSID information element. Firmware fills in the rest.

Fix the length specification of 2GHz band-specific data in the template.

Do not ask iwm(4) firmware to send probe requests on passive channels.

This seems to fix a problem where newer 9k firmware versions become
unresponsive after association and eventually raise a fatal error.

Make iwm(4) set various scan command flags just like how iwlwifi sets them.
Discrepancies found while investigating an issue related to active scanning.

Fix iwm(4) PHY context updates for newer firmware versions.

Firmware which advertises the BINDING_CDB_SUPPORT capability
needs a remove+add dance when the channel band has changed.

See Linux commit 730a18912bcbde0b94ae7f1b554a9908b3424a22
and Linux commit 91109f42d0ad0c0c282d1fa1257a1548977aa895

Make iwm(4) use only antenna B for single-antenna Tx on 9k devices.

This matches what Linux and FreeBSD do.
According to Linux commit logs using antenna A for single-antenna Tx
on 9k devices is "wrong" (with no further explanation given).
I suspect it only matters if Bluetooth is active, since antenna A has
obviously been working for us. But who knows...

Clear the "persistence bit" on iwm(4) 9k devices during hardware init.

According to iwlwifi commit messages this fixes an edge case where
9k family devices fail to resume after system suspend.
See Linux commit 8954e1eb2270fa2effffd031b4839253952c76f2

Ignore undocumented TLVs found in the iwm(4) 9000-43 firmware image.

Reset sc_ucode_api flags before iwm(4) loads firmware.
Fixes firmware re-loading after an initial attempt to load firmware failed.

Simplify code supporting the new iwm(4) time quota firmware command.
There was no real need to use a separate function for this.

silence redundant error message; reported by Fabian Stelzer

ifq_hdatalen can return 0 if ifq_empty is true, which avoids locks.

whitespace fixes, no code change.

fix the hibernate io path.

a botched merge meant i was posting the previously used slot to the
chip to process before posting the current slot.

ok deraadt@

Avoid a potential buffer overflow in backslash escaping.
https://github.com/onetrueawk/awk/issues/121

The xformsw array never changes.  Declare struct xformsw constant
and map data read only.
OK deraadt@ mvs@ mpi@

sys/lib/libz isn't configured for lazy optimizations to save space because
those options are incompatible with the kernel anymore.  Set DYNAMIC_CRC_TABLE
and BUILDFIXED for these bootblocks, to save space on the media
ok tb mlarkin

In recent libz update, tb@ overzealously attempted codesize reduction with
option BUILDFIXED -- which is incompatible with kernel hibernate unpack since
it places side-effects into BSS, rather than inside z_streamp or using
the private allocator.  While here DYNAMIC_CRC_TABLE could cause similar
problems, so disable this also.
Space savings for the media is best done with bootblock-specific libz
compile options, coming in the following commit.
ok tb mlarkin

typo: hibenate -> hibernate

sync

Support MCC update response used by newer iwm(4) firmware, and verify
the size of the response we receive for MCC_UPDATE commands (even
though we aren't doing anything with this response yet).

With the correct expected response length this time. The driver needs
to expect the maximum response size, not sizeof() the response struct.
The actual response size depends on the number of channels stored
in the "world" regulatory profile of the device.

ok sthen@

Initialize `ipsec_acquire_pool' pool (9) within pfkey_init() instead of
doing that in runtime within ipsp_acquire_sa().

ok bluhm@

Debug printfs in encdebug were inconsistent, some missing newlines
produced ugly output.  Move the function name and the newline into
the DPRINTF macro.  This simplifies the debug statements.
OK tobhe@

Remove the code to store intermediary vnodes in the unveil list.
These traversed vnodes are a leftover from early times where realpath(3)
was still all done in userland.
OK semarie@