tsc: configure LFENCE to serialize dispatch before testing TSC sync

On AMD CPUs, LFENCE does not serialize instruction dispatch until MSR
C001_1029[1] is properly configured.  We do this in identifycpu(); see
amd64/identcpu.c,v 1.103.

The upshot is that the first TSC synchronization test is currently
invalid on most AMD CPUs because the LFENCE in the test loop does not
ensure that the AP loads the BP's latest TSC value before executing
RDTSC.  So the synchronization test is yielding false positives on AMD
CPUs where the TSCs are actually synchronized.

The simplest fix is to wait until after the secondary CPU runs
identifycpu() in cpu_hatch() to test TSC synchronization.

Moving the TSC sync test after CPU identification means that we can
remove the CPUID() calls from tsc.c: the CPU feature flags are set in
identifycpu() so we no longer need to test for IA32_TSC_ADJUST support
by hand.

While we are at it, we should also pass the correct cpu_info pointer
to tsc_test_sync_bp().  It was unused before, so the bug was harmless,
but we definitely need the BP's cpu_info pointer, not the AP's pointer.

Unfortunately, this change does not fix the TSC sync problems we've
been seeing on e.g. dv@'s and jmc@'s Ryzen 5 machines.  Hopefully the
problem on those machines is buggy firmware and not another
architectural misunderstanding on my part.

Prompted by robert@.  Problem diagnosed by brynet@.  With input from
robert@, brynet@, and deraadt@.  Tested by robert@, brynet@, dv@,
phessler@, and jmc@.

ok robert@ brynet@ sthen@

Enable the keyboard on the Samsung Galaxy Book Go.

Help from kettenis@, "Nice!" deraadt@

Short names make for shorter and prettier lines.

Add support for Apple fn key combinations. Based on Apple fn key handling
in ukbd(4).

ok miod@

Use non-blocking connect() with ppoll() and timeout instead of alarm().
For hosts with multiple IP addrs this makes it possible to fall
over from an unresponsive IP to another.  This also replaces the
other connect(2) + connect_wait() calls with timed_connect() so the
-w option now works for more that just http.  OK sthen@ deraadt@

Add GPTPARTATTR_MS_* defines for Microsoft basic data attributes
and make 'fdisk -v' display their names (NoAutoMount, Hidden,
Shadow, ReadOnly).

Shift 1ULL instead of 1 to make it clear these are uint64_t
flags. Makes clang happier.

Remove unneeded interim DPRINTF() verbiage. Make DEBUG compile
again.

Add OID for RPKI signedTAL objects

IANA made a permanent registration in the SMI Security for S/MIME CMS
Content Type registry at
https://www.iana.org/assignments/smi-numbers/smi-numbers.xhtml#security-smime-1
for signed objects conforming to draft-ietf-sidrops-signed-tal.

OK tb@

Make kroute_matchgw() also work with connected routes.

Connected routes have no gateway set but only have ifindex set.
When an interface is deconfigured this makes sure the right route is
removed.
OK tb@

Ignore error when we try to delete an address that's already gone.

This will happen when an address expires because the vltime drops to
zero. The kernel then deletes the address and slaacd tries to do so,
too. The correct fix is to track in slaacd that the kernel already
deleted the address for us, but that's too much work shortly before a
release so just hide the ugly warning for now, it's harmless.
Problem reported by semarie some time ago.
OK deraadt, benno

Use LONG_MAX as the limit for ciphers with long based APIs.

These ciphers have long based APIs, while EVP has a size_t based API. The
intent of these loops is to handle sizes that are bigger than LONG_MAX.
Rather than using the rather crazy EVP_MAXCHUNK construct, use LONG_MAX
rounded down to a large block size, ensuring that it is a block size
multiple. Revert the recently added overflow checks now that this is
handled more appropriately.

ok tb@

regen

Add IDs for the JHL6240 Thunderbolt 3 controller found in my Thinkpad T490

ok jsg

recognise Neoverse V2 (Demeter)

AF_UNIX bind() must use UNVEIL_CREATE for namei() because it is creating
a file in the filesystem.  Spotted by martijn.  A review of AF_UNIX
binding programs has been done by benno, and we think it is worth commiting
this semantic change now and watching for fallout.

Backout "Reflect script failure in exit code"

amd64 install using (G)PT seems busted as reported by tb

remove an extraneous empty line

closer to potential release date

Merge common FORMAT_FDISK and USE_SOFTRAID default, simpler cleanup

Compare 'srcnat' when comparing policies.  Fixes a bug where policy lookup could
not differentiate between similar policies that only differ in srcnat.  Also
include srcnat when logging flows or policies.

ok markus@

Fold root disk setup targets into one

Fold vnd disk setup targets into a single loop

Make NDISKS an integer, simplify CLEANFILES with globbing

Testing with three softraid chunks now means NDISKS=3 as one would expect
and not NDISKS='1 2 3'.

This uses the powerful jot(1) -w and rs(1) -T commands and allows for more
simplifications in the Makefile.

Format softraid keydisk to make regress pass without installboot fix

Treat keydisks like real chunks until installboot properly skips it and
does not touch/install to them anymore.

clarify behaviour when the second address in a range is smaller than
or equal to the first;

diff from luka krmpotic
ok kn

sync

sk_enroll: never drop SSH_SK_USER_VERIFICATION_REQD flag from response

Now that all FIDO signing calls attempt first without PIN and then
fall back to trying PIN only if that attempt fails, we can remove the
hack^wtrick that removed the UV flag from the keys returned during
enroll.

By Corinna Vinschen

a little extra debugging

ssh-agent: attempt FIDO key signing without PIN and use the error
to determine whether a PIN is required and prompt only if necessary.
from Corinna Vinschen

add some initial docs for MODPY_PYBUILD, prompted by espie

== in [[ does pattern matching as well

OK millert

Split out the code that collects data from acpiac(4), acpibat(4) and
acpisbs(4) for apm(4) and hook it up to the arm64 version of apm(4) on
systems with ACPI.

ok kn@

Add (partial) support for agentx in vmd.

Metrics can be found under mib-2.236 and VM-MIB (RFC7666).

Stress tested by and happy noises from Mischa Peters
OK dv@

varbind was designed to allow both a ber NULL and a NULL pointer for
value. The ber NULL case is there for when it was received via a PDU.
The NULL pointer case can happen if application.c runs into a timeout
or when a backend runs into problems.

The NULL pointer case however was overlooked in appl_varbind_valid and
results in an "missing value" error, (needlessly) terminating the
connection to the backend.

Found the hard way by Mischa Peters while stress testing agentx support
for vmd.

OK tb@, sthen@

When a connection is reset while we still have an outstanding request,
the connection from the request to the rest of the structure is removed,
so we don't send any old data over the new connection.

However, the old code dereferences axc at a couple of places before
we check it for NULL.

Found the hard way by Mischa Peters while stress testing agentx support
for vmd.

OK tb@, sthen@

support more than one input file in llvm-ranlib by backporting
commit aa173573198e024b065c5f6523ce26bb865781b7 from upstream

ok kettenis@

Change pru_rcvd() return type to the type of void. We have no interest
on pru_rcvd() return value.

Drop "pru_rcvd != NULL" check within pru_rcvd() wrapper. We only call it
if the socket's protocol have PR_WANTRCVD flag set. Such sockets are
route domain, tcp(4) and unix(4) sockets.

ok guenther@ bluhm@

Do soreceive() with shared netlock for raw sockets.

ok bluhm@

document "configtest" in SYNOPSIS; from andrei
while here, sort SYNOPSIS at the behest of ajacoutot;

ok ajacoutot

fix Xr;

add missing quote;

add arch to Dt;

Stop pretending that EVP_CIPHER cleanup can fail.

Now that EVP_CIPHER is opaque, stop pretending that EVP_CIPHER cleanup can
fail.

ok tb@

SIOCDIFPARENT removes configuration not SIOCGIFPARENT
spotted by kn@

Enable acpiac(4) and acpibat(4).

ok deraadt@

Store mod/ref flags using md pg_flags values rather than a specific field in
vm_page_md, which allows this struct to shrink a bit.

Store mod/ref flags using md pg_flags values rather than a specific field in
vm_page_md, which allows this struct to shrink a bit.

Drop orphaned pv_flags values.

Add support for level-triggered GPIO events.

ok mlarkin@

sxirintc(4)

qcgpio(4) and qciic(4)

qcgpio(4) and qciic(4)

zap extra .Pp

Stop documenting i2c_ASN1_INTEGER.

This is no longer public API. Also remove some comments about i2c and c2i
functions being intentionally undocumented since they are no longer public.

disable Panel Self Refresh (PSR) by default in inteldrm

After i915_drv.c 1.144 PSR changed to being on by default.
On a TUXEDO InfinityBook Pro 14 Gen6 (Tiger Lake) this introduced screen
flicker.  Reported and tested by Matthias Schmidt.

Should also avoid flicker problem on Dell XPS 13 7390 (Comet Lake)
reported by James Cook.

Add CBC, CFB64 and OFB64 test coverage for RC2

From Joshua Sing

Hook up installboot(8) tests on all covered archs

Those that still fail (softraid+keydisk or explicit-stage-files) have fixes on tech@.

whitespace nits

Bump version for upcoming -portable release

Don't use options from pane if pane is NULL.

acpihpet(4): acpihpet_delay: only use lower 32 bits of counter

We can't use acpihpet_r() to implement acpihpet_delay().  Even if we
made acpihpet_r() atomic on amd64, i386 would still be incapable of
doing atomic 8-byte reads.  As-is, the code does a split read on all
platforms, which may or may not already be causing problems with TSC
calibration:

https://marc.info/?l=openbsd-tech&m=166220561709496&w=2

Switch from acpihpet_r() to bus_space_read_4() and only use the lower
32 bits of the counter.  This makes acpihpet_delay() slightly larger,
but unless we want two acpihpet_delay() implementations we have no
choice.

Link: https://marc.info/?l=openbsd-tech&m=166165347220077&w=2
ok jsg@

spelling

Introduce tree walkers that only walk a subtree of the RIB.

In some cases only a "small" part of the RIB needs to be looked at. Like
bgpctl show rib 10/8 or-longer that only needs to travers nodes under
10/8 all other RIB entries do not matter. By setting the start node to
the RB_NFIND(10/8) the all nodes below this point can be skipped.
Using prefix_compare() while walking the tree with RB_NEXT() the walker
know when it steps outside of the 10/8 subtree and stops.
With this the or-longer commands become a lot faster.

Looks good to tb@

SIOCGIFPARENT uses struct if_parent not ireq
ok jmc@

SIOCGVNETID uses struct ifreq not if_parent
ok jmc@

Move division by two out of sizeof()

Error checks for EVP_*

CID 356777

Move division by two out of sizeof()

CID 356778

Add regression tests for the sendmmsg and recvmmsg system calls.

Remove the DKF_LABELVALID flag from struct disk. Instead, trust disk drivers
to always be able to provide a duid, and keep ignoring whole zero duids.

This fixes a race in vnd setup where the disk_attach callback could run
before any I/O occurs on the vnd, thus not having a label available yet.

noticed by otto@ and kn@; ok kn@

Add the new inout vmm(4) tracepoint to dt(4).

Forgot to put it in the list of static tracepoints when I committed
the tracepoint at g2k22. Woops.

Enforce the minimum TLS version requirement for QUIC.

ok tb@

Adjust for opaque structs in ts.h

ok jsing

Adjust for opaque structs in pkcs12.h

ok jsing

Register the I2C controller with ACPI.  Skip this on the SC8280XP SoC for
now as the AML on the Lenovo X13S tries to do I2C transactions to a device
that doesn't respond leading to the ACPI thread spinning until the
transaction times out.

ok mlarkin@, deraadt@

sync

bump major after libcrypto and libssl major bump

Crank major after symbol addition and libcrypto major bump

Update Symbols.list

ok jsing

Expose SSL_get_share_{group,curve}() and related #defines

ok jsing

Expose some error codes needed for QUIC support

ok jsing

Define LIBRESSL_HAS_QUIC

ok jsing

Bump major after symbol addition and removal and struct visibility changes

Update Symbols.list

ok jsing

Make structs in ts.h opaque

ok jsing

Make structs in pkcs12.h opaque

ok jsing

Expose EVP_chacha20_poly1305()

ok jsing

Expose various EVP AEAD constants for EVP ChaCha and QUIC

ok jsing

Expose OPENSL_cleanup()

ok jsing

Make BIO_info_cb() identical to bio_info_cb()

Various projects use bio_info_cb and BIO_info_cb interchangeably, for
example mupdf and freerdp. This is because this was changed in OpenSSL
commit fce78bd4 (2017), triggered by new warnings in gcc 8.

https://github.com/openssl/openssl/pull/4493

This results in some scary compiler warnings and useless patches in ports.
Nobody seems to be using the old bio_info_cb() version.

ok jsing

Remove c2i_* and i2c_* from public visibility

This removes c2i_ASN1_OBJECT(), {c2i,i2c}_ASN1_BIT_STRING() and
{c2i,i2c}_ASN1_INTEGER(). These are not part of the OpenSSL 1.1
API and should never have been exposed in the first place.

ok jsing

link asn1object test statically in preparation for upcoming bump

Replace "echo 'w\ny\nq\n' | disklabel -E" with equivalent
'disklabel -dw'.

Tested & ok visa@

Be stricter with middlebox compatibility mode in the TLSv1.3 server.

Only allow a TLSv1.3 client to request middlebox compatibility mode if
this is permitted. Ensure that the legacy session identifier is either
zero length or 32 bytes in length. Additionally, only allow CCS messages
on the server side if the client actually requested middlebox compatibility
mode.

ok tb@

Only permit CCS messages if requesting middlebox compatibility mode.

Currently the TLSv1.3 client always permits the server to send CCS
messages. Be more strict and only permit this if the client is actually
requesitng middlebox compatibility mode.

ok tb@

drop the -beta

Use CBS when procesing a CCS message in the legacy stack.

ok tb@

Ensure there is no trailing data for a CCS received by the TLSv1.3 stack.

ok tb@

.Li in previous didn;t make sense;

Add #define's for GPT partition attribute bits REQUIRED, IGNORE
and BOOTABLE, set BOOTABLE attribute bit instead of using the
incorrect GPTDOSACTIVE value, have 'fdisk -v' print out GPT
partition attributes if any of the 64 bits are set, don't spoof
any partition with REQUIRED bit set.

Prompted by kettenis@ stumbling across a machine with 40+ (!!)
REQUIRED GPT partitions.

Tested & ok kettenis@