split the code in figure_out_kept:
we need a somewhat complete updateset so we can proceed with handling
dependencies.

The case where the signature is equal is not an issue, but the case
where we need to keep the package because it still contains oldlibs is!

So we do that AFTER dependencies so that dependencies have been updated
and we possibly depend on old libs (which may happen even if the signature
didn't change)

add a test case for the Imath update, along with notes about what's
exactly going on, if one day we trust wantlib enough to be accurate
(hello libsets)

"pretty print" verbose messages: more context, and indent them to make
clearer those message are grouped together

give a bit more context in verbose message

vgone() is vgonel() with curproc as 2nd argument. Use vgonel() like the
other call in vop_generic_revoke().
OK semarie@

vmm(4): dt(4) tracepoints for vm exit reporting previous exit.

On Intel/VMX, the tracepoint for vm exits was before extracting the
vm exit reason meaning we either reported stack garbage (on first
exit) or the prior exit reason. Move the tracepoint to after
extraction and refactor the logic to collect all exit info processing
in one place. (We were extracting the guest RFLAGS register state
before checking the exit info extraction success.)

On AMD/SVM, the tracepoint was always triggered even on unsuccessful
vm entry leading to garbage data. Tuck the tracepoint into the If
block and merge them.

ok mlarkin@

note to myself for 7.2

Fix three bugs regarding the interaction of \z and \h:

1. The combination \z\h is a no-op whatever the argument may be.
In the past, the \z only affected the first space character generated
by the \h, which was wrong.

2. For the conbination \zX\h with a positive argument, the first
space resulting from the \h is not printed but consumed by the \z.

3. For the combination \zX\h with a negative argument, application
of the \z needs to be completed before the \h can be started.
In the past, if this combination occurred at the beginning of an
output line, the \h backed up to the beginning of the line and
after that, the \z attempted to back up even further, triggering
an assertion.

Bugs found during an audit of assignments to termp->col that i
started after the bugfix tbl_term.c rev. 1.65.  The assertion
triggered by bug 3 was *not* yet found by afl(1).

typo: exceded -> exceed

Add authfd path to debug output.  ok markus@

ASN1_{,const_}check_infinite_end(3) were removed in the last major bump,
so there's no longer a need to document that they are undocumented.

Decode via c2i_ASN1_BIT_STRING_cbs() from asn1_ex_c2i().

ok inoguchi@ tb@

Bump __mp_lock_spinout to INT_MAX.

The previous value set years ago was causing amd64 kernels to spin
out when run with MP_LOCKDEBUG during boot.

ok kettenis@

typo in example text: unsused -> unused; noticed by tb@

Do not send a hostname option when we don't have a hostname.

The minimum length of a hostname is 1 according to RFC 2132.
Found the hard way by Ibrahim Khalifa (ibo AT trukonsult.se) with DHCP
Relay on a Cisco ASA in the installer where we don't have a hostname
yet.
Ibrahim also provided a diff which I simplified a bit.
OK tb

At the end of every tbl(7) cell, clear the \z state.
This is needed because the TERMP_MULTICOL mode is designed such
that term_tbl() buffers all the cells of the table row before the
normal reset logic near the end of term_flushln() can be reached.

This fixes an assertion failure triggered by \z near the end
of a table cell, found by tb@ using afl(1).

If a node is tagged explicitly, skip implicit tagging for that node.
Apart from making sense in the first place, this fixes an assertion
failure that happened when the calculated implicit tag did not match
the string value of the first child of the node,

Bug found by tb@ using afl(1).

No need for line wrap here.

On CPUs that have MPERF/APERF support use that information to install a
cpu frequency sensor for each core. This works on many "modern" Intel and
AMD cpus (probably anything that has some kind of turbo mode).
OK kettenis@

Check sshauthopt_new() for NULL.  bz#3425, from tessgauthier at
microsoft.com.  ok djm@

Reimplement the page allocation code using bus_dma(9) APIs to make sure
that on architectures with an IOMMU, DMA addresses are translated
properly.

Fixes radeondrm(4) and amdgpu(4) on powerpc64, sparc64 and arm64 machines
with smmu(4).

ok jsg@

Push DISK_[read|write]sectors() down to disk.c local functions
and expose new DISK_[read|write]bytes() functions.

Eliminates many bytes <-> sector i/o dances and makes the code
much easier to understand.

Be a bit more careful to consistently write only bytes that
checksums are calculated over.

No intentional functional change.

Extend ksmn(4) driver to also show CCD tempratures if available.
Based information from the linux k10temp driver.
OK brynet@

gh_csum an gh_part_csum are not used between initializing
the GPT and calculating them when writing the GPT to disk.

No need to update them in add_partition().

Pointed out by jmatthew@

Fix spelling error in warnx()

Thanks Marco D'Itri

zap extra blank line

Wrap two overlong lines

trailing whitespace

When we open a new .while loop, let's not attempt to close out
another enclosing .while loop at the same time.
Instead, postpone the closing until the next iteration of ROFF_RERUN.

This prevents one-line constructions like ".while 0 .while 0 something"
and ".while rx .while rx .rr x" (which admittedly aren't particularly
useful) from dying of abort(3), which was a bug tb@ found with afl(1).

If a .shift request has a negative argument, do not use a negative array
index but use 0 instead of the argument, just like groff.
Warn about the invalid argument.
While here, fix the column number in another warning message.

Segfault reported by tb@, found with afl(1).

In filemode emit the hash which uniquely identifies a given Signed Object

RPKI Signed Objects are not malleable; this means the SHA256 digest
of an input file containing a signed object is a very stable identifier
to associate with the decoded (validated) output shown in filemode.

The SHA256 in hash_id is base64 encoded (just like hashes on manifest listings).

OK tb@

Use warnx() if errno is not relevant. close() after errno of
interest is checked, not before.

Less confusing error messages.

Increase rx buffer size to 62kB (same as linux).  It turns out the device
doesn't fit packets into the host's rx buffer until it's full, it keeps
going as long as it has space in its own receive buffer, and then
truncates to fit the usb transfer.  The packet descriptors are at the end,
so this means losing all the packets in the buffer.  At 32kB, it's fairly
easy to overrun, but 62kB seems fine.

tested by and ok mlarkin@

Rewrite c2i_ASN1_BIT_STRING() using CBS.

Also switch to freeing and allocating, rather than attempting to recycle.
While here, factor out the flags ASN1_STRING_FLAG_BITS_LEFT bit bashing
and use the name "unused bits" rather than "bits left", to be more inline
with X.690 wording.

ok inoguchi@ tb@

Convert asn1_ex_c2i() to CBS.

This allows us to make direct use of c2i_ASN1_OBJECT_cbs().

ok inoguchi@ tb@

tweak previous: update .Bl -column line

Add ASN1_INTEGER test coverage.

Show which bwfm(4) chipsets are 11n and which are 11ac. ok miod@

Fix compilation on octeon, arm64, and riscv64.

Add missing uuid_dec_le() to init_gp() so -A works on
big-endian architectures too.

If the last data row of a tbl(7) contains nothing but a horizontal line,
do not skip closing the table and cleaning up memory at the end of the
table in the HTML output module.

This bug resulted in skipping the tblcalc() function and reusing
the existing roffcol array for the next tbl(7) processed.  If the
next table had more columns than the one ending with a horizontal
line in the last data row, uninitialized memory was read, potentially
resulting in near-infinite output.

The bug was introduced in rev. 1.24 (2018/11/26) but only fully exposed
by rev. 1.33 (2021/09/09).  Until rev. 1.32, it could only cause
misformatting and invalid HTML output syntax but not huge output
because up to that point, the function did not use the roffcol array.

Nasty bug found the hard way by Michael Stapelberg on the production
server manpages.debian.org.  Michael also supplied example files
and excellent instructions how to reproduce the bug, which was very
difficult because no real-world manual page is known that triggers
the bug by itself, so to reproduce the bug, mandoc(1) had to be
invoked with at least two file name arguments.

Verify sizes before arithmetic operations

Unsigned overflows are not a bug in C but we have to make sure that
requested buffer sizes will be actually available.

If not, set errno to ERANGE and return an error value.

ok deraadt, millert

escape the word "An" in an Rs/Re block, since it can be
interpreted as a macro;

Remove some of the workarounds for the tests by inputing the right numbers
(without impliciit conversion from int to long double).

The previous commit message talked about reading numbers, but the issue
(loss of precision due to too little digits) actually occurred when printing
the input to be read by bc.

arm64 has real IEEE quads, it needs more digits when reading the
bc results otherwise truncation and loss of precision occurs, making the
test fail.

Test failure on arm64 noted by anton@ and bluhmn@

checking for /mnt and /root in is_rootdisk() is excessively zealous,
the remaining directory list is good enough
ok sthen

Check tkp->output != NULL before taking strlen for both command mappings
and input mappings. This adds a missing check for command mappings and
simplifies the input mappings.

ok millert

vi: apply expandtab to the output of a ! command
This is consistent with vim's expandtab behavior.
From nvi2 (Craig Leres).  OK tb@

ARP, ND6 and PPPoE are not MP safe.  Protect them with kernel lock
for now.  Then IP forward can run in parallel and this can be fixed
later.
OK sashan@

Only run agent-ptrace.sh if gdb is available as all architectures do not
ship with gdb.

handle 16 bit kstat_kv types

add 16 bit types to kstat_kv structs.

of course it is a realtek card that needs this because they provide
some 16 bit counters.

Use memset() to initialize struct ieee80211_rxinfo properly.

Sven Wolf noticed that scans on ral(4) are buggy ever since I added a new
field to this struct. Turns out a lot of drivers were initializing fields
one-by-one, leaving any newly added fields uninitialized by default.

Affected drivers may report wrong channel numbers for received beacons.
The net80211 stack will discard such beacons, assuming they were received
on the wrong channel due to signal leakage. Scanning is broken as result.

ok miod@

Remove _switchd user/group/alias

ok millert deraadt

switchd(8) was removed back in November. Commit message was:

Retire switchd and switchctl. While interesting they never managed to
really get into a usable state. The OpenFlow API is mostly superseeded
by P4 and so this is a bit of a dead end.

Our malloc(3) guarantees 16-byte alignment.

ok sthen@, tb@

In vmm regress skip all corner cases that do not support vmm.  Link
dump_tables and vmm to the build.  Adopt all makefiles to skip tests
on non amd64 machines.
OK dv@

Add missing self. From anton

v_event_get: check qp->output for NULL before passing to e_memcmp().
Other users of qp->output already include a NULL check.
Avoids a crash when cursor key support is disabled in cl/cl_term.c.
From Jeremy Mates.  OK tb@

Introduce a dedicated link entries for snapshots in pfsync(4). The purpose
of snapshots is to allow pfsync(4) to move items from global lists
to local lists (a.k.a. snapshots) under a mutex protection. Snapshots
are then processed without holding any mutexes. Such idea does not fly
well if link entry is currently used for global lists as well as snapshots.
Feedback by bluhm@ Credits also goes to hrvoje@ for extensive testing.

OK bluhm@

-current recognizes that a symlink was deleted and just keep going with
a warning and no fuss.

Further refactor and cleanup filemode.c mainly remove the copies of
proc_parser_cert_validate() and proc_parser_root_cert() adjust
parse_load_certchain() and parse_load_ta() respectivly.
Also cleanup the functions in parser.c and make it possible to call
ta_parse and cert_parse with a NULL cert.
OK tb@

drm/i915: Sunset igpu legacy mmap support based on GRAPHICS_VER_FULL

From Matt Roper
14785927a1d46817b1648806ed5124be049bea75 in linux 5.15.y/5.15.35
1acb34e7dd7720a1fff00cbd4d000ec3219dc9d6 in mainline linux

drm/amdgpu: Enable gfxoff quirk on MacBook Pro

From Tomasz Mon
2dd7d2eddf8e1ff3556e333979af99e4c167a7b5 in linux 5.15.y/5.15.35
4593c1b6d159f1e5c35c07a7f125e79e5a864302 in mainline linux

drm/amd/display: don't ignore alpha property on pre-multiplied mode

From Melissa Wen
804c096d640ece718d6505c5252eb66544bbc2c9 in linux 5.15.y/5.15.35
e4f1541caf60fcbe5a59e9d25805c0b5865e546a in mainline linux

drm/amd/display: Fix allocate_mst_payload assert on resume

From Roman Li
6a03581ccffa571bfa1a9f3a097e1a4d7164fd2d in linux 5.15.y/5.15.35
f4346fb3edf7720db3f7f5e1cab1f667cd024280 in mainline linux

The filemode code is enough different from the regular parser code that it
makes sense to totally split it out. Duplicate proc_parser_cert_validate()
and proc_parser_root_cert() for now.

The valid_x509() plus the required static functions are moved to validate.c.
The crl_tree code moved into crl.c similar to the auth_tree handling in
cert.c. All the proc functions are now tagged with __attribute(noreturn)
which allows to remove the errx() after them.

OK tb@

drm/amd/display: Revert FEC check in validation

From Martin Leung
ac2eb310af0582428400186173d070320a8bfd6f in linux 5.15.y/5.15.35
b2075fce104b88b789c15ef1ed2b91dc94198e26 in mainline linux

drm/amd/display: Enable power gating before init_pipes

From Roman Li
eab8e585840f84c6a352eaab70e5495eda7ebb6f in linux 5.15.y/5.15.35
58e16c752e9540b28a873c44c3bee83e022007c1 in mainline linux

drm/amdkfd: Check for potential null return of kmalloc_array()

From QintaoShen
f2658d5966bcee8c3eb487875f459756d4f7cdfc in linux 5.15.y/5.15.35
ebbb7bb9e80305820dc2328a371c1b35679f2667 in mainline linux

drm/amdgpu/vcn: improve vcn dpg stop procedure

From Tianci Yin
9f0fabf30b486e3d5c9b578073cea369ffc05c43 in linux 5.15.y/5.15.35
6ea239adc2a712eb318f04f5c29b018ba65ea38a in mainline linux

drm/amdkfd: Fix Incorrect VMIDs passed to HWS

From Tushar Patel
25efb191d86b108f100f82f414229f8269d00b28 in linux 5.15.y/5.15.35
b7dfbd2e601f3fee545bc158feceba4f340fe7cf in mainline linux

drm/amd/display: Update VTEM Infopacket definition

From Leo (Hanghong) Ma
7a3bc11a40346987649018ecd82caa744117c347 in linux 5.15.y/5.15.35
c9fbf6435162ed5fb7201d1d4adf6585c6a8c327 in mainline linux

drm/amd/display: FEC check in timing validation

From Chiawen Huang
92951699a5f11043b9d1402ea21787f420b36094 in linux 5.15.y/5.15.35
7d56a154e22ffb3613fdebf83ec34d5225a22993 in mainline linux

drm/amd/display: fix audio format not updated after edid updated

From Charlene Liu
b054e8183fbdec15955311956df8a9ce47e0ab6d in linux 5.15.y/5.15.35
5e8a71cf13bc9184fee915b2220be71b4c6cac74 in mainline linux

drm/amdgpu/gmc: use PCI BARs for APUs in passthrough

From Alex Deucher
37bc29a445384f4bdfc734dfe7f763ede604acfe in linux 5.15.y/5.15.35
b818a5d374542ccec73dcfe578a081574029820e in mainline linux

drm/amdgpu: conduct a proper cleanup of PDB bo

From Guchun Chen
5a3b56a4a1c79111bc4aaa9caba5eb5e9cfaeaeb in linux 5.15.y/5.15.35
2d505453f38e18d42ba7d5428aaa17aaa7752c65 in mainline linux

drm/amd: Add USBC connector ID

From Aurabindo Pillai
6dded62e5aa6e29f18a1079002109cb6068e9bf6 in linux 5.15.y/5.15.35
c5c948aa894a831f96fccd025e47186b1ee41615 in mainline linux

Fix previous rc.d/spamlogd commit, the check in rc_pre is not needed
because rc.subr has special-casing for spamd/spamlogd.

Teach the script to detect the pflog interface from spamlogd flags and
create the correct interface if needed.

ok jturner@

drm/amd/display: Fix p-state allow debug index on dcn31

From Nicholas Kazlauskas
8e663865f5284124196bc04e010e7344d8a278f3 in linux 5.15.y/5.15.35
3107e1a7ae088ee94323fe9ab05dbefd65b3077f in mainline linux

drm/amd/display: Add pstate verification and recovery for DCN31

From Nicholas Kazlauskas
685a19fa6ae99dc7a18a9af2a7626162e1db7aff in linux 5.15.y/5.15.35
e7031d8258f1b4d6d50e5e5b5d92ba16f66eb8b4 in mainline linux

fix SEE ALSO;

Repair hw vlan tagging.  Previously swapped byte RGE_TDEXTSTS_VTAG, this
caused no tag was sent on the outgoing packet.

Bug reported and the fix tested by Andrew Laramore.

Clarify comments at the start of {asid,addr}_validate_path_internal()

Requested by jsing

Avoid expensive RFC 3779 checks during cert verification

X509v3_{addr,asid}_is_canonical() check that the ipAddrBlocks and
autonomousSysIds extension conform to RFC 3779. These checks are not
cheap. Certs containing non-conformant extensions should not be
considered valid, so mark them with EXFLAG_INVALID while caching the
extension information in x509v3_cache_extensions(). This way the
expensive check while walking the chains during X509_verify_cert() is
replaced with a cheap check of the extension flags. This avoids a lot
of superfluous work when validating numerous certs with similar chains
against the same roots as is done in rpki-client.

Issue noticed and fix suggested by claudio
ok claudio inoguchi jsing

Fix X509_get_extension_flags()

Ensure that EXFLAG_INVALID is set on X509_get_purpose() failure.

ok inoguchi jsing

It looks like we can't completely avoid waiting for processes to
exit so retrieve the pid via controlmaster and use that.

Refine the GPT partition entry table validity check to ensure
that the partition entry table associated with the primary GPT
header at sector 1 doesn't overwrite the header or intrude into
the sectors available for partitions.

Similarly ensure that the partition entry table associated with
the alternate header does not overwrite that header or intrude
into the sectors available for partitions.

Use glob to expand wildcards in "other device" paths rather than a
custom implementation that only allowed matching all files in a
directory.

ok millert

In pfsync there were some KASSERT hidden behind #ifdef PFSYNC_DEBUG.
Assertions should be active and rely on #ifdef DIAGNOSTIC.  Retire
PFSYNC_DEBUG.
OK sashan@ dlg@

Wrap a long line.

Fix mandoc -Tlint warning about trailing whitespace

Fix white space and wrap long lines.

While I'm here, prepare regress for claudio's upcoming diff

Fix rpki-client regress after -j -f change

Sigh. Another forgotten header file.

Remove unnecessary includes: openssl/hmac.h and openssl/evp.h.
From Martin Vahlensieck.

Add missing includes of stdlib.h and stdint.h.
We need stdlib.h for malloc(3) and stdint.h for SIZE_MAX.
Unlike the other xmss files, ssh-xmss.c does not include xmss_commons.h
so ssh-xmss.c must include those headers itself.
From Martin Vahlensieck

Avoid an unnecessary xstrdup in rm_env() when matching patterns.
Since match_pattern() doesn't modify its arguments (they are const),
there is no need to make an extra copy of the strings in options->send_env.
From Martin Vahlensieck

Simpify code manipulating GPT partition names by coalescing
logic into GPT_get_name(), string_to_name() and name_to_string()
functions. Remove unnecessarily abstract functions ask_string(),
utf16le_to_string() and string_to_utf16le().

No intentional functional change.

more whitespace cleanups