Clean up memory handling in spawn_pager(), free(3)ing everything
that is malloc(3)ed.  In addition to being less confusing, the new
code is also shorter by two lines.

sync

sycn

In man(1) mode, properly clean up the resn[] result array
after processing each name given on the command line.

Failure to do so resulted in a memory leak of about 50 kilobytes
per name given on the command line.  Since man(1) uses a few
Megabytes of memory anyway and people rarely give hundreds of names
on the command line, this leak did not cause practical problems,
but cleaning up properly is better in any case.

Allegedly a "Marvell Armada 3700 Functional Errata, Guidelines, and
Restrictions" document exists that discusses an errata #251 in section
"3.12 PCIe Completion Timeout" and suggests that setting the DIS_ORD_CHK
flag in the Debug Mux Control register is necessary as a workaround:

https://lore.kernel.org/linux-pci/20210624222621.4776-6-pali@kernel.org

This workaround is still being discussed by the Linux developers, but it
does fix an issue I am seeing with athn(4), where an external abort
happens under load.  So apply this workaround since its potential side
effects seem to be significantly less severe than provoking an
external abort that hangs the machine.

ok patrick@

Provide a cleanup function for the term_tab module, freeing memory
and resetting the internal state to the initial state.
Call this function from the proper place in term_free().

With the way the module is currently used, this does not imply any
functional change, but doing proper cleanup is more robust, makes
it easier during code review to understand what is going on, and
makes it explicit that there is no memory leak.

3 groups of realtek firmware are now in base (with new filenames).
fw_update does not need to install the 3 realtek firmwares anymore.
We must keep them around during the 7.0 cycle, but 7.1 onwards will
not require the files.
discussed with sthen and kevlo

grow i386 media for new realtek firmwares

sync

store the operating system name obtained from uname(3) in the adequate
struct together with similar state date rather than in a function-scope
static variable, such that it can be free(3)d in roff_man_free();
no functional change

synch doc to tool

Do not leak 64 bytes of heap memory every time a manual page calls
a user-defined macro.  Calls of standard mdoc(7) and man(7) macros
were unaffected, so the effect on OpenBSD manual pages was small,
about 80 Kilobytes grand total for a full run of "makewhatis
/usr/share/man".

Argument expansion contexts for user-defined macros are stored on
a stack that grows as needed if calls of user-defined macros are
nested or recursive.  Individual stack entries contain dynamically
allocated arrays of pointers to arguments; these argument arrays
also grow as needed if user-defined macros take more than eight
arguments.  The mistake was that argument arrays of already
initialized expansion contexts were leaked rather than reused on
subsequent macro calls.

I found this issue in a systematic hunt for memory leaks after
Michael <Stapelberg at Debian> reported memory exhaustion problems
on the production server manpages.debian.org.  This sub-Megabyte
leak is not the cause of Michael's trouble, though, where Gigabytes
of memory are being wasted.  We are still investigating whether the
original problem may be related to his supervisor process, which is
written in Go, rather than to mandoc.

Simplify sys___thrsigdivert a bit. cursig() always moves the pending signal
to p_siglist and so there is no need to check ps_siglist for the signal.
OK mpi@

Use the fact the vnodes are locked when operations are inflight.
Remove the v_inflight member and alter the ffs and ext2fs sync code to
track inflight by checking if the node is locked or not (which it already
did before but for a different reason).
OK mpi@

Add firmwares for rsu(4), rtwn(4), and urtwn(4) drivers.
ok deraadt@

Build firmware for rsu(4), rtwn(4), and urtwn(4).
We have to install the files with new filenames (removal of "fw" from each of
the filenames) and change the driver to use the new filenames,
such that the package becomes irrelevant.

Discussed with deraadt@ and sthen@

ok deraadt@

Permission from Realtek to include wireless firmwares

After deraadt@ explained why the Realtek firmware cannot be put into our tree,
I reached out to Realtek to explain the situation.

According to the LICENCE.rtlwifi_firmware.txt [1], the rules permitted adding
their non-open-source firmware into open source operating systems but such
an addition (ironically) makes that open source operating system not entirely
open source.  Realtek understood the irony, and change the license.

Thanks to Realtek for this change which lets us put the firmware .h file
into our tree, this means Realtek wireless will work without requiring
a firmware download (which is difficult over a non-working Realtek network :)

[1] https://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git/tree/LICENCE.rtlwifi_firmware.txt?id=0f863ff1b388ad5b0f7d25decdbb642#n22

ok deraadt@

Unbreak btrace(8) regress, adding missing files.

bt(5)/btrace(8): add support for str()

Implement initial support for the str() function, which is used
primarily to truncate or NUL-terminate strings from either cli args
or args to tracepoints and syscalls.

Current implementation only supports cli args and is primarily for
compatability with bpftrace. Future work is needed once dt(4)
supports builtin args other than long values.

Adds a regress test and wires in argument-based tests again.

ok mpi@

Apparently some athn(4) variants are buggy and may hand us corrupt frames
that are marked "ok".  Linux has some workarounds for this and checks whether
the status word has error bits set in it regardless of the bit that marks
the frame as "ok".  Adapt this workaround to our driver and drop the frame
after setting input errors.  This doesn't filter out all corrupted frames,
but it does keep things down to a level where it doesn't fill up the node
cache anymore when athn(4) is used in hostap mode.

Seen with:

athn0 at pci1 dev 0 function 0 "Atheros AR9281" rev 0x01: intx
athn0: AR9280 rev 2 (2T2R), ROM rev 16, address xx:xx:xx:xx:xx:xx

ok stsp@

Extend filesystem tests to test mfs and tmpfs. Hook up mfs but leave tmpfs
out for now since it is not enabled by default.
mfs reports the same errors as ffs (no real surprise), tmpfs has a few
different errors.
OK bluhm@

sync

un.ifdef USB and PCI, since all our architectures (minus one) have models
which can have these busses, might as well make these distributable firmwares
available in case the drivers find devices.
ok kettenis

fuse: avoid namei_pool leaks in several functions

when calling namei(), cn_pnbuf is kept allocated when fs
implementation is setting SAVENAME flag. In such cases, it is expected
the fs implementation to also release memory when work is done.

fuse(4) didn't put back the memory to the pool. correct it.

ok mpi@

Extend workaround for reset on context closure from gen 7-8 to gen 4-8
as asavvycomputist@disroot.org reported this occurs on gm45 (gen 4).

remove dead variable from sys___realpath()

it is a leftover from LOCKPARENT removal in NDINIT() (in rev 1.337)

ok mpi@

Use SSL_CTX_get0_param() rather than reaching into the SSL_CTX.

vfs: merge *_badop to vop_generic_badop

It replaces spec_badop, fifo_badop, dead_badop and mfs_badop, which
are only calls to panic(9), to one unique function vop_generic_badop().

No intented behaviour changes (outside the panic message which isn't
the same).

ok mpi@

sync

Remove iwx_assoc() and iwx_disassoc(). Not needed because they duplicate
work that is already handled by state transitions involving AUTH or RUN.

Remove iwm_assoc() and iwm_disassoc(). Not needed because they duplicate
work that is already handled by state transitions involving AUTH or RUN.

Fix panic when iwx(4) firmware is not present at boot time.

Uncomfortable bug found the hard way by deraadt@

quote sudo

add missing sudo

hook up btrace

Dynamically allocate encoded HashKnownHosts and free as appropriate.
Saves 1k of static storage and prevents snprintf "possible truncation"
warnings from newer compilers (although in this case it's false positive
since the actual sizes are limited by the output size of the SHA1).
ok djm@

Add test for ssh hashed known_hosts handling.

unbreak FIDO sk-ed25519 key enrollment for OPENSSL=no builds;
ok dtucker@

amd/display: downgrade validation failure log level

From Simon Ser
526261c1b706fec0ea80ce9f14c8fe8468bee34d in linux 5.10.y/5.10.70
7bbee36d71502ab9a341505da89a017c7ae2e6b2 in mainline linux

treewide: Change list_sort to use const pointers

From Sami Tolvanen
55e6f8b3c0f5cc600df12ddd0371d2703b910fd7 in linux 5.10.y/5.10.70
4f0f586bf0c898233d8f316f471a21db2abd522d in mainline linux

drm/amd/pm: Update intermediate power state for SI

From Lijo Lazar
68d4fbe6220cd1f3d07cab0a4901e62f8c12cc68 in linux 5.10.y/5.10.70
ab39d3cef526ba09c4c6923b4cd7e6ec1c5d4faa in mainline linux

Mark another test as failing with the legacy verifier.

This test now fails with the legacy verifier, due to
X509_V_FLAG_TRUSTED_FIRST being enabled by default.

Enable X509_V_FLAG_TRUSTED_FIRST by default in the legacy verifier.

In order to work around the expired DST Root CA X3 certficiate, enable
X509_V_FLAG_TRUSTED_FIRST in the legacy verifier. This means that the
default chain provided by Let's Encrypt will stop at the ISRG Root X1
intermediate, rather than following the DST Root CA X3 intermediate.

Note that the new verifier does not suffer from this issue, so only a
small number of things will hit this code path.

ok millert@ robert@ tb@

delete expired DST Root CA X3 to work around bugs various libraries
ok sthen, beck, jsing, tb, etc etc

Quote SUDO value during propagation, allowing `doas -n' to be correctly
honored which I use while running regress. The same principle is already
applied to the PYTHON variable here.

In iwm(4) and iwx(4), prevent attemps to transition towards the same state
in cases where this would result in a redundant or illegal state transition.

jmc@ observed ASSOC -> ASSOC transitions which would result in a hang.
Such transitions are invalid and never intentionally triggered by net80211.
They imply a race between the Rx interrupt handler and the newstate task.

Tested by jmc@ on AX200 for a week and several known issues seem to be fixed.

Fix up whitespace left by previous change removing privsep.  No other
changes.

Remove (almost all) references to privsep.  This removes several do..while
loops but does not change the indentation of the now-shallower loops,
which will be done in a separate whitespace-only commit to keep changes
of style and substance separate.

Use "skip" instead of "fatal" if SUDO isn't set for the *-command tests.
This means running "make tests" without SUDO set will perform all of the
tests that it can instead of failing on the ones it cannot run.

Don't constrain -b specified block count or block size to be greater than 63.
Allow any value from 1 to UINT32_MAX.

MBR boot partition sizes/offsets are completely machine dependent.

Pointed out by loongson.

Global variables to track initialisation behave poorly with MP.
Move the tdb pool init into an init function.
OK mvs@

Add support for Cannon Lake H and Tiger Lake H platforms.

ok jcs@

sync

drm/amdgpu: add some additional RDNA2 PCI IDs

From Alex Deucher
8f0c93f454bd7ab04eaec1d3c436c4c7c2378f07 in mainline linux

regen

intel 0x5915 is a kaby lake graphics id not imaging unit
fix strings in some other graphics devices as well

regen

add amdgpu and inteldrm devices matched by 5.15 drm without force probe
includes amd codenames for codenames so subject to change

add some debug output showing how many key file/command lines
were processed. Useful to see whether a file or command actually has
keys present

Test certificate hostkeys held in ssh-agent too. Would have caught
regression fixed in sshd r1.575

ok markus@

Revert part of the previous diff to fix a regression (another endless loop)
reported by Michael <Stapelberg at Debian> in the Linux md(4) manual.

The reason the colwidth[] array is needed is not that it stores widths
different from those in tbl->cols[].width, but that only part of the
columns participate in the comparisons, i.e. only those intersecting
at least one span the still requires width distribution.

testcase for the avahi/avahi-lib/avahi-glib/cups update

Make prototype for rijndaelEncrypt match function including the bounds.
Fixes error in portable where GCC>=11 takes notice of the bounds.
ok deraadt@

Import regenerated moduli.

Fix timeout behaviour bug introduced in 1.241.
If the timespec is zero-valued sys___thrsigdivert() should just do the
check for pending signals and return immediatly.
OK kettenis@

Turns out that older SPARC CPUs (like the UltraSPARC II) do report an exact
fault address, while others (like the UltraSPARC T2) report an inexact
address.  Deal with this by using the EXPADDR_MASK on both the expected and
the reported fault address.

ok bluhm@

Use -Wl,-z,wxneeded instead of -z wxneeded such that this builds with base
gcc.

ok patrick@, millert@, jca@

Link libclang_rt.profile to the build.

"go for it" kettenis@

add missing sudo

Remove recent changes used to unblock the signal undergoing testing, I solved it
by changing my regress environment instead. This reduces the delta to the NetBSD
upstream.

Mark "failures" volatile to avoid a problem with sigsetjmp/siglongjmp.
This makes the test pass on sparc64 where the compiler may otherwise
store the variable in the strlcpy/strlcat function's delay slot.
OK kettenis@

Do not call recalculate_sizes while clearing a client session because it
needs to loop over the clients, instead do it after all clients are
cleared. Fixes a crash reported by martijn@ when a session with multiple
clients attached is destroyed, but there are other sessions so tmux does
not entirely exit. ok deraadt

These tests pass in a few seconds.  Remove REGRESS_SLOW_TARGETS.

Bring this regress into nicer shape.  Add all recent architectures.
For me it is still unclear what should actually be tested.  It runs
MAKEDEV, stores the result into an outfile and checks nothing.

Return 0 from main() otherwise the exit code is garbage on sparc64.
Collect status of the child process to detect test failures.
OK tb@

Make t_gettimeofday pass on sparc64.
OK bluhm@

drm/amdgpu: Disable PCIE_DPM on Intel RKL Platform

From Koba Ko
45bd9dd1bee8aedc4cbd409b1ba7f9b4f941eea6 in linux 5.10.y/5.10.69
b3dc549986eb7b38eba4a144e979dc93f386751f in mainline linux

drm/amd/amdgpu: Increase HWIP_MAX_INSTANCE to 10

From Ernst Sjoestrand
8f95553f0016c3994d9c022b5af4a1a433d6714e in linux 5.10.y/5.10.68
67a44e659888569a133a8f858c8230e9d7aad1d5 in mainline linux

drm/amd/display: Update bounding box states (v2)

From Jerry (Fangzhi) Zuo
b80a99e048275d566d63f2463a2f640065ccbf75 in linux 5.10.y/5.10.67
a7a9d11e12fcc32160d55e8612e72e5ab51b15dc in mainline linux

drm/amd/display: Update number of DCN3 clock states

From Aurabindo Pillai
583c4f3d09c3e980a683b59febbb0c775bdff1db in linux 5.10.y/5.10.67
0bbf06d888734041e813b916d7821acd4f72005a in mainline linux

drm/amdgpu: Fix BUG_ON assert

From Andrey Grodzovsky
7b1abace16a9dff6804d4eb94750beb60d9502b4 in linux 5.10.y/5.10.67
ea7acd7c5967542353430947f3faf699e70602e5 in mainline linux

drm/dp_mst: Fix return code on sideband message failure

From Rajkumar Subbiah
bb693c114e8b53e3e0b8228be218d907d35959a5 in linux 5.10.y/5.10.67
92bd92c44d0d9be5dcbcda315b4be4b909ed9740 in mainline linux

drm/amdkfd: Account for SH/SE count when setting up cu masks.

From Sean Keely
0e9f4492219f8f991163691aad43897da8478c4e in linux 5.10.y/5.10.67
1ec06c2dee679e9f089e78ed20cb74ee90155f61 in mainline linux

drm/display: fix possible null-pointer dereference in dcn10_set_clock()

From Tuo Li
83449db3aac0895147eac723bf23d0739720b968 in linux 5.10.y/5.10.67
554594567b1fa3da74f88ec7b2dc83d000c58e98 in mainline linux

gpu: drm: amd: amdgpu: amdgpu_i2c: fix possible uninitialized-variable access in amdgpu_i2c_router_select_ddc_port()

From Tuo Li
2254383788ff93a423e20068333b9f8376d56cb4 in linux 5.10.y/5.10.67
a211260c34cfadc6068fece8c9e99e0fe1e2a2b6 in mainline linux

drm/amd/display: fix incorrect CM/TF programming sequence in dwb

From Roy Chan
63ebc1f1df813ebb40d19449c356480555008166 in linux 5.10.y/5.10.67
781e1e23131cce56fb557e6ec2260480a6bd08cc in mainline linux

drm/amd/display: fix missing writeback disablement if plane is removed

From Roy Chan
d763afc4ea2b251217ec87cf4c1e006c9f0aef99 in linux 5.10.y/5.10.67
82367e7f22d085092728f45fd5fbb15e3fb997c0 in mainline linux

drm/amd/amdgpu: Update debugfs link_settings output link_rate field in hex

From Anson Jacob
6f51f4241253974a6a147daecd5c20beb7450330 in linux 5.10.y/5.10.67
1a394b3c3de2577f200cb623c52a5c2b82805cec in mainline linux

drm/amdgpu: Fix a printing message

From Oak Zeng
a5999d18a8d8c4c767c60d67fe6a6fe51b9a203d in linux 5.10.y/5.10.67
95f71f12aa45d65b7f2ccab95569795edffd379a in mainline linux

drm/amd/display: Fix timer_per_pixel unit error

From Oliver Logush
f462a39eb8334b52e332cc0cbffb705660b7d87b in linux 5.10.y/5.10.67
23e55639b87fb16a9f0f66032ecb57060df6c46c in mainline linux

drm: protect drm_master pointers in drm_lease.c

From Desmond Cheong Zhi Xi
34609faad0c9f9f08d4b59d25c94b78bf5710d93 in linux 5.10.y/5.10.67
56f0729a510f92151682ff6c89f69724d5595d6e in mainline linux

drm: serialize drm_file.master with a new spinlock

From Desmond Cheong Zhi Xi
06a553a99bacb00d3bc25f79e75c8e0fbf7a5025 in linux 5.10.y/5.10.67
0b0860a3cf5eccf183760b1177a1dcdb821b0b66 in mainline linux

drm: avoid blocking in drm_clients_info's rcu section

From Desmond Cheong Zhi Xi
54e51d288b38377e8cd645a83e1ad08cc9d20ccc in linux 5.10.y/5.10.67
5eff9585de220cdd131237f5665db5e6c6bdf590 in mainline linux

drm/amdgpu: Fix amdgpu_ras_eeprom_init()

From Luben Tuikov
10a135969fd7419695c003ddb67ef8a7820a808b in linux 5.10.y/5.10.67
dce4400e6516d18313d23de45b5be8a18980b00e in mainline linux

openssh-8.8

need initgroups() before setresgid(); reported by anton@, ok deraadt@

Revert '-b' change. Forgot required bsd.rd tweak.

'-g' no longer needs '-i'.

No functional change.

MBR partitions created with '-b' should be marked DOSACTIVE.
That's what 'bootable' means in the MBR world. GPT
partitions created by '-b' are unaffected.

Pointed out by kettenis@, obviating need for new option.

Fix a possible race condition in spc_msgin().

This comes from NetBSD:sys/dev/ic/mb89352.c fix by tsutsui:
http://cvsweb.netbsd.org/bsdweb.cgi/src/sys/dev/ic/mb89352.c.diff?r1=1.57&r2=1.58&f=h

Tested on LUNA-88K2 by me.