Use installboot -p in install.md for macppc, powerpc64

ok kn@

Remove pmap_collect() when a no-op, define __HAVE_PMAP_COLLECT otherwise.
Use that define to shunt uvm_swapout_threads(), which is a noop when
pmap_collect() does nothing.

ok mpi@

Increment the input and output position for EVP AES CFB1.

The length is decremented, however the input is repeatedly read from and
output written to the same position. Correct this by actually incrementing
the input and output pointers.

Found via OpenSSL 604e591ed7,

ok tb@

Use correct length for EVP CFB mode ciphers.

The BLOCK_CIPHER_* macros contained a bug where the total length is passed
to the underlying cipher implementation, rather than the length of the
current chunk. Correct this and use the chunk length instead.

Should address the remaining issues reported by Coverity.

ok tb@

Use correct option name.

top(1): remove last vestiges of "last pid" support

millert@ removed most of the "last pid" support from top(1) in 1997.
See, e.g. top/machine.c,v1.7:

http://cvsweb.openbsd.org/src/usr.bin/top/machine.c?rev=1.7&content-type=text/x-cvsweb-markup

Let's remove the rest of it:

- Eliminate system_info.last_pid.

- Remove mpid parameter and "last pid" printing code from i_loadave().

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

Get rid of the extra vnode reference known as UVM_VNODE_CANPERSIST.

Back in the 4.4BSD days the VM subystem had a OBJ_CANPERSIST flag to
enter objects in a global cached list.  Some of this logic seem to have
been copied to UVM but without the global list.

Unfortunately keeping UVM vnode objects alive after munmap(2)ing the
corresponding region without incrementing the reference count of the
related vnode led to many bugs when the vnode was recycled and/or when
it data where written back to disk (via the page daemon).

The problem is that VM pages might have a non-accounted reference to a
vnode via `pg->uobject'.

Fix "vref used where vget required" panic reported by bluhm@, gkoehler@
and Andrew Krasavinseen on bugs@.

Thanks a lot to semarie@ for co-debugging this issue!

Tested by bluhm@, tb@, miod@.

ok kettenis@, semarie@

Use CBS to parse TLS alerts in the legacy stack.

ok tb@

Provide a version of ssl_msg_callback() that takes a CBS.

Use this from the TLSv1.3 code.

ok tb@

Add another pin to the SC7180 map function.

Don't restrict GPIO events to pin numbers below 256.  With the _EVT()
method events corresponding to larger numbers are possible.

Fixes lid state detection on the x13s.

ok patrick@

Generalize the code such that it works on SC7180 SoCs as well.

ok mglocker@, patrick@

Add _HID for SC7180 SoCs.

ok mglocker@

carrier return character -> carriage return character
ok jmc@ miod@

fix repeated words
ok ok miod@ ack ack jmc@

Thinkpad -> ThinkPad

apm(1): Display unknown recharge time estimate instead of -1

OK miod@

Socs -> SoCs

dupex -> duplex

wireless LANS -> wireless LANs

fix repeated words
ok miod@ jmc@

remove apostrophe on possesive its
ok jmc@

Finally, only allocate the number of frames for isoc transfers which we
really need.  The static allocation workaround in the code
(DWC2_MAXISOCPACKETS) was required because the NetBSD allocx USB method
has a second argument to pass nframes already there, which we don't
have.

Regression tested with video play, audio play, audio record.

ok mpi@

Add EABI strong aliases for __fixunssfdi and __fixunsdfdi added recently on
armv7; ok mbuhl@

PACKAGES_REPOSITORY -> PACKAGE_REPOSITORY; from john verne

amd64, i386: lapic_initclocks: install i8254 before lapic_startclock()

In the future, the clock interrupt code will need a working
timecounter to do its job.  In lapic_initclocks(), call
i8254_inittimecounter_simple() before lapic_startclock().

The i8254 may be be a piece of junk, but one timecounter is better
than none.

init(8): fix signal handler boolean type and qualifier

The variable "clang" is modified from a signal handler.  Change it
from an 'int' to a 'sig_atomic_t' and mark it 'volatile', as we
recommend in signal(3).

ok millert@ kn@

Let pool_get(9) do the memory initialisation directly.

ok kettenis@

Recognize Qualcomm Krys 400 series cores.

ok jsg@

add recvmmsg to NAME;

Reflect script failure in exit code

installboot(8) runs newfs(8) and fsck(8) via system(3) but only checks
failures of the function itself, always returning zero no matter what the
programs/shell returned.

This is bad for regress tests relying on correct return codes.

create_filesystem() itself must not exit as write_filesystem() calls it and
cleans up temporary files upon failure.

Make it return -1 if the script returned non-zero so write_filesystem()
handles it as error, cleans up and makes installboot exit 1.

Stop ignoring create_filesystem()'s return code in md_prepareboot() and
exit the same way.

Here's the change in behaviour on arm64 (newfs fails because of the
vnd/disklabel race, see "Race in disk_attach_callback?" on tech@):

# installboot -vp vnd0 ; echo $?
newfsing 6694ae5b0d7596ed.i
newfs_msdos: /dev/r6694ae5b0d7596ed.i: No such file or directory
0
# ./obj/installboot -vp vnd0 ; echo $?
newfsing 6694ae5b0d7596ed.i
newfs_msdos: /dev/r6694ae5b0d7596ed.i: No such file or directory
1

Tested on amd64 arm64 macppc octeon powerpc64 sparc64
OK millert

sync

Add libc wrappers for the new sendmmsg and recvmmsg system calls.
Feedback tb@, miod@, jca@
OK jca@

Implement the F_SHORTER filter by doing explicit lookups for each possible
prefixlen. Even for IPv6 this is much faster than a full table walk.
OK tb@

Add mips64/octeon bits

Still fails due to the disklabel/vnd kernel race but running into failing
'installboot -p' helps exercising the fsck/newfs exit code fix.

REGRESS_EXIT_EARLY=1 runs past the prepare target without the diff but
properly stops with it as installboot now exits 1 if system(3) fails.

Clarify/typofix comments

OpenBSD is not FreeBSD and has no stf(4) interface.

No object change.

Add message-line option to control where message and prompt go, from
Varun Kumar E in GitHub issue 3324.

Bump version to 8.0

The DSDT on the snapdragon 7c based Acer Aspire 1 (A114-61) uses a bogus
compatible ID.  Add its hardware ID to the list to work around this.

ok patrick@, mlarkin@, mglocker@

sync

Adjust chunked encoding handling.

Add an extra state to distinguish in between chunks CRLF handling from
the last chunk which can optionally have some trailer fields included.
rpki-client ignores these trailer header fields (they are also not common
it seems).
Also remove the empty line handling in http_parse_chunked() for explicit
checks in http_read(). Because of the extra state the switch back to
non-chunked mode can now be delayed until the transfer is over.

OK tb@

drm/i915: Skip wm/ddb readout for disabled pipes

From Ville Syrjala
1787ec7dcc3761e040d0cd9896d3a40b4efac4cf in linux 5.15.y/5.15.66
0211c2a0ea600e25db3044daaeff4fe41c3ed6d9 in mainline linux

drm/i915/glk: ECS Liva Q2 needs GLK HDMI port timing quirk

From Diego Santa Cruz
f9a0f49be767a45aa1dc7d846a2745bb7849268a in linux 5.15.y/5.15.66
919bef7a106ade2bda73681bbc2f3678198f44fc in mainline linux

drm/i915/reg: Fix spelling mistake "Unsupport" -> "Unsupported"

From Colin Ian King
0522550aeed4ec899d16e237be0ae67f8aa1319b in linux 5.15.y/5.15.66
233f56745be446b289edac2ba8184c09365c005e in mainline linux

drm/i915/display: avoid warnings when registering dual panel backlight

From Arun R Murthy
7828b5d0080a4621973ab5c680c4df0bc6d96437 in linux 5.15.y/5.15.66
868e8e5156a1f8d92ca83fdbac6fd52798650792 in mainline linux

add backlight_device_get_by_name() used by 5.15.66 drm

drm/i915/backlight: extract backlight code to a separate file

From Jani Nikula
630ab29120b5dd1c543150da5e1aa304a5aece2e in linux 5.15.y/5.15.66
6cc42fbeb150ff33b17cbf108713ca4be23994d8 in mainline linux

notifier_complete(NULL, ...) is a noop, so no need to test
that ctx!=NULL; from Corinna Vinschen

There is no need to uncomment the clock gating exit/enter gadget functions,
since we define them void anyway.

The SC7180 (snapdragon 7c) needs the SMMU quirk as well.
It seems reasonable that SC8180X/XP needs the quirk too, so preemtively
add these to the list as well.

ok patrick@

There is no need to uncomment the call_gadget macro, since in our driver
it's getting defined void anyway.

Zap inactive gadget functions.

Move bitmap functions to a new home.

ok kettenis@

Remove old debug code knobs, wrong comments, and unused pv entry flags.
ok kettenis@

Support the sendmmsg and recvmmsg system calls.
Input guenther@
OK bluhm@

ssl_cipher_process_rulestr: return early if a cipher command is invalid
This is a safer fix for the bug where we might read outside rule_str
buffer and is how BoringSSL fixed it.  OK tb@

In http_get_line() additionally strip any trailing space or tab from lines.
In many places the HTTP allows for extra spaces which need to be ignored.
Similar the chunked encoding extensions are separated from the chunk size
by a ':' but the spec also allows for bad whitespaces in all shapes and
forms. Adjust the logic in http_parse_chunked() to stop when the first
space, tab or ':' is seen. There is no need to check for newlines since
those are stripped by http_get_line().
OK tb@

Remove useless -t

"Make if.c kvm free by fetching the interface stats via sysctl ..."
if.c r1.56 in 2008 neutered the -d and -t flags.

-d functionality was eventually restored with if.c r1.76 in 2015.

-t has been printing zeroes for over fourteen now; this cannot have been
useful to anyone and noone complained about a wrong watchdog timer values.

Remove -t entirely without printing a warning;  `netstat -t ...' now fails.

Feedback jsg
OK jsg bluhm

_rc_quirks is only called by _rc_parse_conf, so just merge the two.

ok robert@ sthen@ kn@

initialise inteldrm device params from module params

In inteldrm_attach() we need to call i915_params_copy().
Matching the changes made to i915_driver_create() in
linux 8a25c4be583d8a9e38fbadcc49d4de32b1aa8906 (5.10)
drm/i915/params: switch to device specific parameters

This makes DisplayPort MST work again as the value of
enable_dp_mst changes from false to true.

Set enable_guc as 0 as I suspect that needs more work.
Set request_timeout_ms to 0 as otherwise I see
"Fence expiration time out" running Xorg on Broadwell.

Problem reported and debugged by Paul Kelly.

Adjust HTTP header parsing to follow RFC more closely.

RFC9112 allows any amount of space/tabs between the ':' and the value.
Until now this code required exactly one space which works most of the
time but is not RFC compliant.
OK djm@

Rename global ifnet TAILQ

Naming the list like the struct itself makes for awful grepping.
Call the global variable "ifnetlist" from now on.

There used to be kvm(3) consumers in base picking up this symbol, but those
have long been converted to other interfaces.

A few potential ports users remain, same deal as sys/net/if_var.h r1.116
"Remove struct ifnet's unused if_switchport member":  they get bumped.

Previous users pointed out by deraadt
OK bluhm

Remove vm_page_md fields which were only needed for pre-v7 arm ports.

Adjust HTTP header parsing to follow RFC more closely.

RFC9112 allows any amount of space/tabs between the ':' and the value.
Until now this code required exactly one space which works most of the
time but is not RFC compliant.
Problem reported by Ties de Kock (tdekock (at) ripe.net)
OK tb@

Use generic PNP ID to match HID-over-I2C devices.

ok patrick@, drahn@

Add a few more pin mappings based on what is in some Linux device tree
updates that are floating around.  Remove some #ifdef'ed out debug code
that was left behind.

ok patrick@, drahn@

riscv64: cpu_initclocks: install tb_timecounter before cpu_startclock()

In the future, the clock interrupt code will need a real timecounter
to work correctly.  Nudge the tc_init(9) call for tb_timecounter up
before cpu_startclock().

macppc: cpu_initclocks: install tb_timecounter before cpu_startclock()

In the future, the clock interrupt code will need a real timecounter
to work correctly.  Nudge the tc_init(9) call for tb_timecounter up
before cpu_startclock().

recognise Realtek ALC245
from Neel Chauhan

match on Intel 600 Series LP HD Audio (0x51c8)

HP Spectre x360 16-f0013dx has subclass audio not hd audio
so azalia does not match

from Neel Chauhan

ssl_cipher_process_rulestr: don't read outside rule_str buffer
If rule_str ended in a "-", "l" was incremented one byte past the
end of the buffer.  This resulted in an out-of-bounds read when "l"
is dereferenced at the end of the loop.  OK tb@

Add EVP test coverage for RC2.

From Joshua Sing

Add output length validation for EVP

From Joshua Sing

Add a few more PNP IDs to handle variants of the Qualcomm dual role
controller based on AML dumps found online and some further detective
work looking at Windows drivers by jsg@.

ok jsg@, millert@

vmm(4): allow reading MSR_TSC on Intel hosts.

Add's MSR_TSC to the read bitmap for msr access. This was added to
AMD/SVM in Feb 2020, but never added to Intel VMX. Some guests use
rdmsr instead of rdtsc, so this prevents a #GP exception.

ok mlarkin@

On the Lenovo X13s the named components don't have the single mapping
flag set, which we so far had expected to be there.  This is allowed by
the IORT spec.  In that case simply make use of the first mapping if
there is a single one.

With feedback from and ok kettenis@

sync with arm64.html

Fix passing explicit stage files

Every platform ought to set `stages', `stage1' and optionally `stage2'
in md_init(), otherwise passing explicit files results won't work as
`stages' is zero-initialised and no default path is set:

# installboot -v sd0 /root/BOOTAA64.EFI
usage: installboot [-npv] [-r root] disk [stage1]

This is correct synopsis and ought to work, but efi_installboot.c has an
empty md_init().  Set stage bits to fix this:

# ./obj/installboot -nv sd0 /root/BOOTAA64.EFI
Using / as root
would install bootstrap on /dev/rsd0c
using first-stage /root/BOOTAA64.EFI
would copy /root/BOOTAA64.EFI to /tmp/installboot.2bGhLGT1eF/efi/boot/bootaa64.efi
would write /tmp/installboot.2bGhLGT1eF/efi/boot/startup.nsh

This makes regress/usr.sbin/installboot pass on armv7, arm64 and riscv64
(while being lucky or carrying miod's fix for the kernel disklabel race
manifesting on vnd).

Remove the .SILENT bits again after feedback from bluhm

Mention bsd.rd and install72.img

OK miod jsg

ansi

GBe -> GbE

remove history of when each du(1) flag was added
ok jmc@ schwarze@

StarFire -> StarFive

Document support for booting from RAID 1C on sparc64

The bootloader (ofwboot) got support some time ago, but installboot(8) had
to be fixed as well for multi-chunk volumes (1C requires at least two).

As of today, the latest sparc64 install72.img can perform a default install
onto a passphrase-protected 1C volume across four chunk disks, resulting in
a bootable system without requiring manual invervention, just like amd64.

Crank (sparse) disk size from 64M to 128M to make arm64 happy again

Add VERBOSE=No default, .IGNORE and .SILENT REGRESS_CLEANUP

Tiny bit less noise.  The many REGRESS_SETUP_ONCE targets could be merged
into a single one and then be added to .SILENT unless VERBOSE=yes to focus
on actual installboot output and ditch the dd/fdisk/disklabel noise by default.

keep noise down, discard usage text for expected failure cases

Use multi-chunk RAID 1C with keydisk on softraid enabled archs

Keydisks requires special treatment and should not be touched by installboot.
In real setups they are probably offline (temporary USB stick, only present
during boot to unlock), but when they are around (keydisk chunk on same disk
as crypto volume)...

installboot currently installs treats keydisk like any other chunk, which
must be fixed.

Thus change the tests into a should-work-but-currently-broken setup so an
installboot fix can be immediately tested against it.

i386/lapic.c: sync with amd64/lapic.c

In anticipation of merging the new clock interrupt code after release,
let's synchronize the lapic timer parts of i386's lapic.c with those
of amd64's lapic.c.  The two files will need similar changes to switch
to the new code, so the more alike they are the better.

Tested by mlarkin@ on an ESXi VM.  Tested by me on my amd64 laptop
running in x86 compatibility mode.

ok mlarkin@

Add a table-driven test for RC2 in ECB mode to replace the old one.

From Joshua Sing

Fix spacing/indent, no object change

Delete some unreferenced SMD, ST506 and archaic SCSI #define's
that use d_drivedata.

Since nothing else in the tree refers to d_drivedata other than
/etc/disktab parsing and no /etc/disktab entry utilizes the
:d[0-4]: attributes, stop disklabel(8) both printing "drivedata:
0" and paying attention to any "drivedata: ..." lines in ascii
labels being read.

ok jsg@ miod@

Attach aplmbox(4) early.

Properly free memory in filemode

OK tb@

Remove more mkerr.pl remnants, missed in previous

Noticed by jsg
Feedback OK jsg

Replace aes_{ccm,xts}_cleanup usage with NULL.

Only change in generated assembly is due to line numbers.

Stop casting a size_t to a long and then passing it as a size_t.

These cipher implementations take a size_t length argument, so stop
casting it to a long.

Found by Coverity.

ok tb@

sync

Fix copy-paste error