Some Intel xhci(4) controllers don't fully power down unless they've seen
a "save state" command.  So use that command when we suspend (and don't
reset the controller at that point such that it doesn't forget about it).
Note that on resume we don't restore the state.  Instead we just reset
the controller and bring it up from scratch.  There isn't much state to
save anyway since we detach all USB devices when we suspend.

ok mlarkin@, deraadt@

Unlock sysctl net.inet.ip.directed-broadcast.

ip_directedbcast is read once in either ip_input() or pf_test()
during packet processing.  So writing the variable does not need
net lock.

OK mvs@

On the RK3588 eMMC controller it appears that after a failed command a
completion signal shows up some time after the error signal.  This can
lead to the case that when the next command is being executed, the old
completion signal is still recorded and the driver may think the command
already executed.  Resetting the status before executing a new command
makes sense, albeit it's technically still possible that the controller
issues another completion signal.  It shouldn't happen because a reset
was issued and we're also waiting for the command and data inhibit bits
to be cleared, but who knows.  This makes the eMMC come up reliably.

ok kettenis@

Unlock KERN_CLOCKRATE.

Read-only access to local `clkinfo' filled with immutable data.

ok bluhm

The devicetree spec says in section 2.4:

If an interrupt-generating device does not have an interrupt-parent
property, its interrupt parent is assumed to be its devicetree
parent.

Implement this aspect of the spec in the same way as Linux does by not
just looking at the immediate parent but keep on walking up the tree
until we hit a node with an "interrupt-controller" property or a node
that has an "interrupt-parent" property.

similar diff as committed for arm64 a couple of days ago
ok jca@

Stop using KERNEL_LOCK to protect the per process kqueue list

Instead of the KERNEL_LOCK use the ps_mtx for most operations.
If the ps_klist is modified an additional global rwlock (kqueue_ps_list_lock)
is required. This includes the knotes with NOTE_FORK and NOTE_EXIT since
in either cases a ps_klist is changed. In the NOTE_FORK | NOTE_TRACK case
the call to kqueue_register() can sleep this is why a global rwlock is used.

Adjust the reaper() to call knote_processexit() without KERNEL_LOCK.
Double lock idea from visa@
OK mvs@

Define _LOCORE before including machine/ctlreg.h in ASM code.
OK miod@ kettenis@

Replace sys/time.h with proper includes for using localtime

Sure deraadt@

restrict the maximum wait time you can set via BIOCSWTIMEOUT to 5 minutes.

this is avoids passing excessively large values to timeout_add_nsec.

Reported-by: syzbot+f650785d4f2b3fe28284@syzkaller.appspotmail.com

timeout_add_nsec takes a uint64_t now.

change the nsec argument to timeout_add_nsec from int to uint64_t

you can only fit a couple of nanonseconds into an int, which limited
the usefulness of the api. worse, if a large nsec value was passed
in it could be cast to a negative int value which tripped over a
KASSERT at the top of timeout_add that ends up being called. avoid
this footgun by working in the bigger type and doing the same range
checks/fixes for other timeout_add wrappers.

ok claudio@ mvs@

Unlock KERN_BOOTTIME.

microboottime() and following binboottime() are mp-safe and `mb' is
local data.

ok bluhm

Adjust for arm64_intr_get_parent() changes.

ok patrick@

Fix reporting if critical battery state.

ok jca@, mlarkin@

Add battery sensors.

ok phessler@

Fix bridging IPv6 fragments with pf reassembly.

Sending IPv6 fragments over a bridge with pf did not work.  During
input pf reassembles the packet, and at bridge output it should be
refragmented.  This is only done for PF_FWD direction, but bridge(4)
and veb(4) called pf_test() with PF_OUT argument.

OK sashan@

Unlock most of `kern_vars' variables.

Add corresponding cases to the kern_sysctl() switch and unlock read-only
variables from `kern_vars'. Unlock KERN_SOMAXCONN and KERN_SOMINCONN
which are atomically read-only accessed only from solisten().

ok kettenis

The devicetree spec says in section 2.4:

If an interrupt-generating device does not have an interrupt-parent
property, its interrupt parent is assumed to be its devicetree
parent.

Implement this aspect of the spec in the same way as Linux does by not
just looking at the immediate parent but keep on walking up the tree
until we hit a node with an "interrupt-controller" property or a node
that has an "interrupt-parent" property.

same diff as committed for arm64 yesterday
ok patrick@

Take `sysctl_lock' before kernel lock.

ok bluhm

sync with base

sync with upstream (noop for openbsd)

Remove bogus connect(s, NULL, 0) call

The only thing it does is error because of a check added in sockargs() in
uipc_syscalls r1.155. As guenther pointed out, this may have been added
because of a misreading of the last sentence of the first paragraph of the
connect(2) manual.

Instead of erroring, this will keep listening if -k is given and otherwise
it will close the socket and exit with success.

ok guenther jeremy

Allow our linker warnings regarding misuse of libc functions to be
elevated to errors using the -Wl,--fatal-warnings option.

ok robert@

drm/i915/dp: Don't switch the LTTPR mode on an active link

From Imre Deak
12880cc086deef91e62e2f010750087a3c23fae3 in linux-6.6.y/6.6.44
509580fad7323b6a5da27e8365cd488f3b57210e in mainline linux

drm/i915/dp: Reset intel_dp->link_trained before retraining the link

From Imre Deak
a90e900f71da23a354e9d17f1dc0c654d0ae9ceb in linux-6.6.y/6.6.44
d13e2a6e95e6b87f571c837c71a3d05691def9bb in mainline linux

drm/amd/amdgpu: Fix uninitialized variable warnings

From Ma Ke
e150f0171c0c0c45a373a658358c51c940ed4fd9 in linux-6.6.y/6.6.44
df65aabef3c0327c23b840ab5520150df4db6b5f in mainline linux

drm/amdgpu: reset vm state machine after gpu reset(vram lost)

From ZhenGuo Yin
323790535237927e1b6a7bc35ddf662c6e7c25cc in linux-6.6.y/6.6.44
5659b0c93a1ea02c662a030b322093203f299185 in mainline linux

drm/dp_mst: Fix all mstb marked as not probed after suspend/resume

From Wayne Lin
e290feb8b749a06b93ed01d4cf66567cbdf0e85d in linux-6.6.y/6.6.44
d63d81094d208abb20fc444514b2d9ec2f4b7c4e in mainline linux

drm/amdgpu/sdma5.2: Update wptr registers as well as doorbell

From Alex Deucher
9d74e50098492e89f319ac6922db3c2062f69340 in linux-6.6.y/6.6.44
a03ebf116303e5d13ba9a2b65726b106cb1e96f6 in mainline linux

drm/i915/gt: Do not consider preemption during execlists_dequeue for gen8

From Nitin Gote
05c0fb20ca3e000a7e966584845731ba4fc950da in linux-6.6.y/6.6.44
65564157ae64cec0f527583f96e32f484f730f92 in mainline linux

drm/amd/display: Check for NULL pointer

From Sung Joon Kim
4ccd37085976ea5d3c499b1e6d0b3f4deaf2cd5a in linux-6.6.y/6.6.44
4ab68e168ae1695f7c04fae98930740aaf7c50fa in mainline linux

drm/amdgpu: Remove GC HW IP 9.3.0 from noretry=1

From Tim Van Patten
6d72626808325c1986cbf90b0ce27a59b4291876 in linux-6.6.y/6.6.44
1446226d32a45bb7c4f63195a59be8c08defe658 in mainline linux

drm/amdgpu: Check if NBIO funcs are NULL in amdgpu_device_baco_exit

From Friedrich Vock
67b4592a7d74e57a5a0929eaf3ae30414ebd39ae in linux-6.6.y/6.6.44
0cdb3f9740844b9d95ca413e3fcff11f81223ecf in mainline linux

drm/amdgpu: Fix memory range calculation

From Lijo Lazar
ae1dd0a379e41db7d4773880e6918948668ef3ad in linux-6.6.y/6.6.44
ce798376ef6764de51d8f4684ae525b55df295fa in mainline linux

drm/amd/pm: Fix aldebaran pcie speed reporting

From Lijo Lazar
0af2e5cb74514877326448becf40365b073b44e2 in linux-6.6.y/6.6.44
b6420021e17e262c57bb289d0556ee181b014f9c in mainline linux

drm/amdkfd: Fix CU Masking for GFX 9.4.3

From Mukul Joshi
cd1885ae4cbf191332ae66f8a214e52fb226ba43 in linux-6.6.y/6.6.44
85cf43c554e438e2e12b0fe109688c9533e4d93f in mainline linux

ppoll() is in POSIX-2024.  Update #include visibility and manpages
and add restrict qualifiers.  While here, rename the BUGS section
to CAVEATS since they're really "these probably don't do what you
want" notes and not things that need fixing.

ok millert@

Fix typo.

ok kettenis@

bump datasize for armv7's pbuild user, some software has grown over the years

OK jca@

sync

add intelpmc(4) entry;

Add battery status support for the x1e80100.  This SoC needs a second
firmware and requires us to shut down an initial "light" firmware that
was presumably loaded bu the system firmware.  For now the required
firmwares are not packaged and need to be copied from the Windows
install.

ok patrick@

intelpmc(4)

The devicetree spec says in section 2.4:

  If an interrupt-generating device does not have an interrupt-parent
  property, its interrupt parent is assumed to be its devicetree
  parent.

Implement this aspect of the spec in the same way as Linux does by not
just looking at the immediate parent but keep on walking up the tree
until we hit a node with an "interrupt-controller" property or a node
that has an "interrupt-parent" property.

ok patrick@

Add intelpmc(4), a driver for the power management controller found on
various Intel SoCs.  The driver takes care of calling the AML methods
needed to enter low power idle states during suspend-to-idle (S0i).
The driver also implements some debug code that prints the residency of
various power states in dmesg.  Based on some earlier code by jcs@

ok jcs@

Make a little effort to treate CRLF as LF in config files. GitHub issue
3720.

The Linux console has some bugs with bright colours. It seems likely
that it is emulating them by setting a bright (or bold) flag; however,
when the colour is changed from a bright colour (say SGR 96) to a
non-bright (say SGR 36), the flag is not reset, so the new colour
remains as bright. SGR 39 (default colour) also does not reset, so you
end up with the bright default colour. Work around this by sending SGR 0
when switching away from a bright colour, and disable AX for TERM=linux.
Also make the check for AX simpler and do not check for the op
capability is not actually used. GitHub issue 3976.

-l should be before -r, pointed out by jmc a while ago.

Adjust the logic when deleting last buffer to better preserve the
selection: if selecting the element below the deleted one fails (because
as the last one), select the one above it instead. From Daniel Mueller,
GitHub issue 4043.

Use paste_is_empty to check if there are no buffers since paste_get_top
will return NULL if there are no automatic buffers.

x509_vfy.c: drop some unnecessary parentheses

- sort SEE ALSO
- radiusd_file(8) is section 8, not 5

Use "module" for the modules.  Add a link to radiusd_file(8) from
radiusd_eap2mschap(8).

A dozen interfaces in <endian.h> were standardized in POSIX-2024 as expected

Set system ID field in the PVD to OpenBSD

This matches what we use in mkybrid.  From Crystal Kolipe.

The maximum count is SKEY_MAX_SEQ + 1 not SKEY_MAX_SEQ - 1.

Sequence numbers are in the range [0, SKEY_MAX_SEQ] inclusive.
This means the maximum value for the -n option (count) should be
SKEY_MAX_SEQ + 1.  From Denis Bodor.

Update awk to the July 28, 2024 version.
 * Fixed readcsvrec resize segfault when reading csv records longer than 8k.
 * Rewrite if-else chain in quoted as a switch.

The improbable occurred: strlcpy(3) and strlcat(3) are in POSIX-2024.
memmem(3) was also added.  Update #include visibility and manpages
and add restrict qualifiers.

"never thought I'd see this day" millert@

mkostemp(3) and reallocarray(3) are in POSIX-2024:
adjust #include visibility and update the reallocarray(3) manpage

ok millert@

The EC_KEY method should use the EC_KEY index, not RSA

Unbreaks ssh's t-agent-pkcs11-cert regress reported by anton.

ok jsing

Bump CRYPTO_EX_DATA_MAX_INDEX to 32

rust-openssl tests do something weird and need lots of ex data (one index
for each registered callback, for example). This makes the regress pass
again.

noticed by anton
ok jsing

Use proper size for allocating indexes

It's a double pointer, so we should allocate a pointer size, not the entire
struct. This saves roughly 500B per class.

CID 507397

ok jsing

Document SSL_CTX_set_cert_store

From Kenjiro Nakayama

Prepare to provide SSL_CTX_set1_cert_store()

SSL_CTX_set_cert_store() should have been called SSL_CTX_set0_cert_store()
since it takes ownership of the store argument. Apparently a few people ran
into the issue of not bumping the refcount themselves, leading to use after
frees about 10 years ago. This is a quite rarely used API and there are no
misuses in the ports tree, but since someone did the work of writing a diff,
we can still add it.

Needless to say that SSL_CTX_get_cert_store() obviously has the exact same
issue and nobody seems to have thought of adding a get0 or get1 version to
match...

Fixes https://github.com/libressl/openbsd/issues/71
From Kenjiro Nakayama

Code patch out the retpoline in the signal trampoline just
like the others.

Nudged, test, and ok deraadt@

The {get,set}res[ug]id(2) family are in POSIX-2024's XSI option,
so adjust the #include visibility and update the manpage.

ok millert@

nc(1): remove useless function pointer dereferencing

ok tb@

Fix previous

Arguably the want_protocol entries in various of these tests are incorrect
but I'll leave that for another day.

Adjust tls regress for protocol parsing fixes

This mostly reverts what was done by beck in Tallinn and adjust tlstest
to add new test cases and now failing connection tests.

libtls: fix legacy protocol parsing

Redefining TLS_PROTOCOL_TLSv1_0 and TLS_PROTOCOL_TLSv1_1 to be the same
as TLS_PROTOCOL_TLSv1_2 had undesired side effects, as witnessed in the
accompanying regress tests. The protocol string all:tlsv1.0 would disable
TLSv1.2 (so only enable TLSv1.3) and tlsv1.2:!tlsv1.1 would disable all
protocols.

It makes more sense to ignore any setting of TLSv1.0 and TLSv1.1, so if
you request 'tlsv1.1' you get no protocol, but 'all:!tlsv1.1' will enable
the two supported protocols TLSv1.3 and TLSv1.2.

Restore the defines to their original values and adjust the parsing code
to set/unset them.

Issue reported by Kenjiro Nakayama
Fixes https://github.com/libressl/openbsd/issues/151

with/ok jsing

regen

Push kernel lock down to sysctl(2).

Unlock few obvious immutable or read-only variables from "kern.*" and
"hw.*" paths. Keep the rest variables locked as before, include pages
wiring. Use new sysctl_vs{,un}lock() functions introduced for thar
purpose.

In kern.* path:

 - KERN_OSTYPE, KERN_OSRELEASE, KERN_OSVERSION, KERN_VERSION -
   immutable;
 - KERN_NUMVNODES - read-only access to integer;
 - KERN_MBSTAT - read-only access to per-CPU counters;

In hw.* path:

 - HW_MACHINE, HW_MODEL, HW_NCPUONLINE, HW_PHYSMEM, HW_VENDOR,
   HW_PRODUCT, HW_VERSION, HW_SERIALNO, HW_UUID, HW_PHYSMEM64 -
   immutable;
 - HW_USERMEM and HW_USERMEM64 - `physmem' is immutable, uvmexp.wired
   is mutable but integer; read-only access to localy stored difference
   between `physmem' and uvmexp.wired;
 - `hw_vars' - read-only access to integers; some of them like
   HW_BYTEORDER and HW_PAGESIZE are immutable;

ok bluhm kettenis

free class->indexes in CRYPTO_cleanup_all_ex_data()

ok jsing

Backout previous.  It was incompleted and too early.

Rewrite CRYPTO_EX_DATA.

CRYPTO_EX_DATA exists as a way to allow an application to attach data to
various components in libcrypto and libssl. The general idea is that there
are various "classes" (e.g. RSA) and an application can get an "index"
(which can have new/dup/free functions provided). The application can then
use the index to store a pointer to some form of data within that class,
for later retrieval.

However, even by OpenSSL standards, this is an insane API. The current
implementation allows for data to be set without calling new, indexes
can be used without allocation, new can be called without actually getting
an index and dup can be called either after new or without new (see regress
and RSA_get_ex_new_index(3)/CRYPTO_set_ex_data(3) for more details). On
top of this, the previous "overhaul" of the code was written to be
infinitely extensible.

For now, the rewrite intends to maintain the existing behaviour - once we
bed this down we can attempt to ratchet the API requirements and require
some sort of sensible sequence. The only intentional change is that there
is now a hard limit on the number of indexes that can be allocated
(previously there was none, relying only on ENOMEM).

ok tb@

Switch off unreferenced power resources in DVACT_POWERDOWN instead of
DVACT_SUSPEND.  Fixes an issue with (un)hibernate where we don't attach
xhci(4) controllers, which would make is turn the xhci(4) off inadvertedly
because its power resource wasn't referenced.

ok mlarkin@

sync

bump minor after yesterday's symbol addition

requested by kettenis

Put exported symbols of libz under our own control

This adds a Symbols.list containing the exported symbols like in other
libraries in base. If upstream adds new API this won't necessarily need
a bump on our side.

help/ok kettenis

getentropy(2) is in POSIX-2024, so adjust the #include visibility,
change the "too much" error to EINVAL, add GETENTROPY_MAX to
<limits.h> (via sys/syslimits.h), and update the manpage.

ok deraadt@

Add qcgpio at fdt.

ok kettenis@

Run socket splice idle timeout without kernel lock.

OK mvs@

Bump libsndio pkg-config version to 1.10.0

ok tb@

bump posix spec from 2008 -> 2024;
no documented changes.

sndiod: Fix insufficent check of input from clients

Fixes possible sndiod(8) crashes caused by a global table overread
triggered by the client.

Found and analysed by Henry Ford <henryfordkjv at gmail.com>, thanks!

ok blumh, help from millert, mlarkin

bump posix spec 2008 -> 2024;

no change noted from 2008 spec, but i've added -h to the list
of extensions. that was an omission from our 2008 notes,
rather than a change in the spec.

bump posix spec 2008 -> 2024;

bump posix spec 2008 -> 2024;

Add device tree mapping for Samsung Galaxy Book4 Edge.

ok kettenis@

virtio: Fix dmamap_sync calls

Add some missing bus_dmamap_sync calls, noticed with SEV and based on
an earlier diff by hshoexer@.

Some of the required syncing is done in virtio_check_vq(). Make sure
to use that function instead of calling call the virtqueue done
function directly from device specific drivers.

For viogpu this means that we cannot poll with virtio_dequeue() but
must use virtio_check_vq() instead. To make this more clear, rename
viogpu_vq_wait() into viogpu_vq_done(). While there, set the DRIVER_OK
flag even earlier. It must be set before using any virtqueue.

ok kettenis@

Remove a comment that was committed by accident

sync with src

sync with upstream

This adds a new public API, deflateUsed(), so is technically a minor bump.
Nothing will be using this anytime soon, so no shared library bump.

discussed with deraadt during c2k24

Possible to read the core file of a process.

Use arc4random_uniform(n) instead of "% n".

Change the routing entry of the pool addresses from blackhole to
reject.  This is better because the routing cache can be purged by the
reject.

bump posix spec 2008 -> 2024;

add a random amount of time (up to 4 seconds) to the grace login time.

ok deraadt@ djm@

Add ufshci(4) at fdt support.

This lets me boot the Samsung Galaxy Book4 Edge in DT mode with ufshci(4),
based on a WIP DTB file.

ok kettenis@

quote the book title (%T) reference, to prevent formatters (in this case
groff) stumbling over literal words that match macro names;