replace manual poll/ppoll timeout math with ptimeout API

feedback markus / ok markus dtucker

add ptimeout API for keeping track of poll/ppoll timeouts;
ok dtucker markus

more consistently name pf_state * variables "st".

pf_state ** are generally called "stp" now too.

discussed with and ok sashan@

do not blow up if NOMAN is set

after a few trap.c were fixed to fault with the right access, the
signal trampoline can now be PROT_EXEC (without PROT_READ) everywhere
ok kettenis

The access type for a prefetch abort should not include PROT_READ,
otherwise faults on executable pages mapped only as PROT_EXEC will
not work.

ok deraadt@

Add a dummy --no-execute-only option for compatibility with ld.lld.
This will allow ports people to sprinkle -Wl,--no-execute-only across the
ports tree when necessary without having to distinguish between ld.bfd and
ld.lld arcitectures.

ok deraadt@, sthen@

pfctl fails to add addresses to undefined/inactive table

pfr_add_tables() function must set PFR_TFLAG_ACTIVE flag
to table which is attached to rule. This will then allow
pfr_add_addrs() to populate the table with addresses.

without this pair of pfctl(8) commands fails as follows:
    # echo 'pass from <foo> to any' |pfctl -f -
    # pfctl -t foo -T add 192.168.1.0/24
    pfctl: Table does not exist

OK mbuhl@

suppress "Connection closed" message when in quiet mode

Rewrite BN_rshift()

This improves readability and eliminates special handling for various
cases, making the code cleaner and closer to constant time.

Basic benchmarking shows a performance gain on modern 64 bit architectures,
while there is a decrease on legacy 32 bit architectures (i386),
particularly for the zero bit shift case (which is now handled in the
same code path).

ok tb@

Add additional shift benchmarks that are useful on BN_BITS2 == 32 platforms

Make free_chunks() return 'const struct chunk *' since we don't
modify chunks outside of free_chunks(). Adjust local variables taking
the return value as needed.

No intentional functional change.

Delete remnants of removed "diff -l" option.
From Nathan Houghton

Copy bytes from the_banana[] rather than banana()

Fixes test failure due to segfault seen on arm64 with xonly snap.

ok djm

The access type for an instruction storage interrupt should not
include PROT_READ, otherwise faults on executable pages mapped only as
PORT_EXEC will not work.

ok deraadt@

Use 'const struct disklabel *' parameters where the disklabel is
not altered. And 'const struct partition *' for pointers into
newly const disklabels.

No intentional functional change.

minor text shuffle, for clarity; ok millert

Eliminate two unnecessary 'i' loop variables by incrementing
the pointer instead.

No intentional functional change.

Print RRDP serial + ID if nothing changed

OK tb@

Implement strscpy() and implement strscpy_pad() on top of that.  Fixes the
return value in case of truncation.

ok jsg@

Replace p_fstype to/from FS_UNUSED dances around invocations of
free_chunks() with a parameter that names the partition to treat
as FS_UNUSED when building the list of free space chunks. With -1
indicating that no partition should be ignored.

Simplifies code and enables further simplifications and a raft
of const'ifications.

No intentional functional change. Regress still happy.
SUN_CYLCHECK, SUN_AAT0 and D_VENDOR still happy.

Eduardo Horvath agreed to rescind clause 3 and 4 in
NetBSD fgen.1 rev 1.9 cpu_in_cksum.S rev 1.2

https://mail-index.netbsd.org/source-changes/2009/10/19/msg002130.html

Christos Zoulas removed his name from advertising clauses and copyrights
of some files in NetBSD.  Make the same changes here.

https://mail-index.netbsd.org/source-changes/2017/06/03/msg084953.html
https://mail-index.netbsd.org/source-changes/2017/06/03/msg084955.html
https://mail-index.netbsd.org/source-changes/2019/12/02/msg111431.html
https://mail-index.netbsd.org/source-changes/2019/12/02/msg111432.html

Adjust open policy integration test to the change of config in bgpd.

Add a per eBGP session role to the config.

This somewhat replaces the RFC 9234 open policy role. This is done because
ASPA requires the same role to be present to properly validate paths.
For iBGP sessions the role is forced to ROLE_NONE. If no role is set on
an ebgp session then 'announce policy' is forced to 'no'.
Also make sure the the role capability is only added if the role is set.
OK tb@

The redirectchain test actually works again. The test servr fixed the
redirect location to be https.

When building this regress test I copied a bad UUID from some other
RRDP file and so a few files have bad unique ids, fix them up to be valid.

Validate the session_id to be a real UUID.

RFC 8182 requires the session_id to be a version 4 random UUID (using
variant 1). Now checking the version and variant is currently disabled
because there is at least one CA with a session_id that is all random
and therefor the version check triggers there.
Joint work with job@. OK job@, tb@

remove unused version of ncr53cxxx.c
ok deraadt@ miod@

experiments for --execute-only on arm64 and riscv64 are going well,
which will prompt making that the default.  That means efiboot, which
build in a different way, must disable the link option.
ok kettenis

Christos Zoulas agreed to rescind clause 3 and 4 in
NetBSD fsck.8 rev 1.35 fsutil.h rev 1.14 pathnames.h rev 1.2
netgroup_mkdb.8 rev 1.9 netgroup_mkdb.c rev 1.18 str.c rev 1.7
str.h rev 1.4 rdate.8 rev 1.11 rdate.c rev 1.19 extern.h rev 1.14
getnetgrent.c rev 1.41 netgroup.h rev 1.10
fparseln.3 rev 1.4 fparseln.c rev 1.10

our stringlist.c/stringlist.h are derived from getnetgrent.c
rfc868time.c from rdate.c
newfs/pathnames.h from fsck/pathnames.h

https://mail-index.netbsd.org/source-changes/2009/10/21/msg002182.html
Not all files are covered as some had copyright assigned to TNF in 1998.

ugly white space

Leo Weppelman agreed to rescind clause 3 and 4 in
NetBSD kvm_dump.3 rev 1.15 kcore.h rev 1.3

https://mail-index.netbsd.org/source-changes/2009/10/20/msg002169.html

move the pf_state_tree_id type from pfvar.h to pfvar_priv.h.

the pf_state_tree_id type is private to the kernel.

while here, move it from being an RB tree to an RBT tree. this saves
about 12k in pf.o on amd64.

ok sashan@

Michael L. Hitch agreed to rescind clause 3 and 4 in
NetBSD ncr53cxxx.c rev 1.16 osiop.ss rev 1.2

https://mail-index.netbsd.org/source-changes/2009/10/21/msg002215.html

Gordon W. Ross and Bill Studenmund agreed to rescind clause 3 and 4 in
NetBSD zs.c rev 1.49

https://mail-index.netbsd.org/source-changes/2009/10/27/msg002419.html

Gordon W. Ross agreed to rescind clause 3 and 4 in
NetBSD dev_net.c rev 1.15 bootparam.c rev 1.19

https://mail-index.netbsd.org/source-changes/2009/10/21/msg002212.html

- use Fl instead of Ar for print0; from josiah frentsos
- while here, describe it as a primary rather than an option

Ichiro FUKUHARA agreed to rescind clause 3 and 4 in
NetBSD if_ath_cardbus.c rev 1.36

https://mail-index.netbsd.org/source-changes/2009/10/21/msg002189.html

Chuck Cranor rescinded the advertising clause of uvm_mmap.c in
NetBSD rev 1.134 and confirmed with Mike Hibler that the University of
Utah would do the same.

https://mail-index.netbsd.org/source-changes/2011/02/02/msg018021.html

ok deraadt@

fix a comment, no functional change.

Typos in vmd error message. No functional change.

move the pf_state_tree rb tree type from pfvar.h to pfvar_priv.h

the pf_state_tree types are kernel private, and are not used by
userland. make build agrees with me.

while here, move the pf_state_tree from the RB macros to the RBT
functions. this shaves about 13k off pf.o on amd64.

ok sashan@

Shuffle code a bit to eliminate need for inner loop
variable 'j' in editor_allocspace().

No intentional functional change. Regress still happy.

on sh/landisk, ELF binaries are currently unpadded between text and rodata.
Comment says because no nx support.  Well, we'll soon be able to something
like nx, and need the rodata to not be mingled into the same pages as
executable, so enforce padding.
with kettenis, ok miod

No need to call editor_countfree() when displaying all the free
chunks in the 'r' editor command. Just add up chunks as they are
displayed. Eliminates pointless second invocation of
free_chunks().

Increment the chunk pointer rather than using iteration
variable + indexing.

No intentional functional change.

do not mention dhclient as a dhcp route message sender

Print the RRDP Session ID and Serial in verbose mode

OK tb@ claudio@

Poll battery sensors less frequently.

The software id in the request must be copied as is to the response by the
hardware. Leverage the fact that the same id can be anything within [1, 15] by
using a different id per request in a round robin fashion. Makes it easier to
correlate requests and responses while making sense of the debug output.

Group more request and response constants.

Query the client terminal for foreground and background colours and if
OSC 10 or 11 is received but no colour has been set inside tmux, return
the colour from the first attached client (probably most people will
have all light or or all dark terminals).

Disable display backlight on Apple Silicon laptops when suspending.

ok kettenis@ patrick@

Add tfind_user(), for getting a proc* given a user-space TID and
the process* that it should be part of.  Use that in clock_get{time,res}(),
thrkill(), and ptrace().

ok jca@ miod@ mpi@ mvs@

The access type for an instruction storage/segment interrupt should not
include PROT_READ, otherwise faults on executable pages mapped only as
PORT_EXEC will not work.

"obviously correct" deraadt@

Let the EFI bootloader make a copy of the EFI System Resource Table (ESRT)
and pass it to the kernel.

ok jca@, patrick@

Fix the check for the BWX extension introduced in 1.92.
This repairs operation on 21164 processors (not 21164A!) which lack BWX.

Reported by Mark Butt on alpha@

regression test for PermitRemoteOpen

fix bug in PermitRemoteOpen which caused it to ignore its first
argument unless it was one of the special keywords "any" or "none".

Reported by Georges Chaudy in bz3515; ok dtucker@

use the pf generated toeplitz hash when setting the mbuf flow id.

before this it would use the pf state id, which is just an increasing
number. the toeplitz hash is generated/used by the rest of the
stack, so this encourages consistent flow of traffic through the
system.

Clear status bits when we receive a host system error.  Otherwise the
interrupt keeps firing even if the hos controller has been declared dead.

ok mpi@

With the introduction of the PMAP_PREFER_{ALIGN,OFFSET} macros a long time ago,
there are actually no more uses of the PMAP_PREFER() macro left in the kernel.
Remove that macro but keep PMAP_PREFER as a simple #define for it to let uvm
knows the PMAP_PREFER_{ALIGN,OFFSET} macros are available.

ok mpi@

Add explicit LL suffixes to large constants to appease some compilers on
32-bit systems.

Add explicit LL suffixes to large constants to appease some compilers on
32-bit systems.

Add explicit LL suffixes to large constants to appease some compilers on
32-bit platforms; NFCI

ok tb@

Round up fractional percentages, as per POSIX.
From nabijaczleweli, OK deraadt@

Enable power management for PCI devices.

ok mlarkin@, deraadt@

copyright++;

update drm to linux 6.1.2

new hardware support includes

AMD
Raphael, Ryzen 7000 desktop, gfx1036/GC 10.3.6
Mendocino, Ryzen & Athlon 7020 Series mobile APU, gfx1037/GC 10.3.7
Navi 31, gfx1100 dGPU, GC 11.0.0, Radeon RX 7900 XT/XTX
gfx1101 dGPU
gfx1102 dGPU
gfx1103 APU

Thanks to the OpenBSD Foundation for sponsoring this work.

timeout.9: document new interfaces, miscellaneous rewrites and cleanup

- Document timeout_abs_ts(9).
- Add the kclock arguments to timeout_set_flags(9) and
  TIMEOUT_INITIALIZER_FLAGS(9).
- Document KCLOCK_NONE and KCLOCK_UPTIME.
- Mention the static initialization macros alongside timeout_set(9) etc.;
  keep relevant information adjacent.
- Mention timeout_add_sec(9) etc. alongside timeout_add(9); keep
  relevant information adjacent.

... plus many other cleanups, rewrites, and rearrangements.

Prompted by mvs@ and many others.  With input from jmc@, mvs@, kn@,
schwarze@, and probably a few others I have forgotten.

v1: https://marc.info/?l=openbsd-tech&m=162449274513068&w=2
v2: https://marc.info/?l=openbsd-tech&m=165851505627764&w=2
v3: https://marc.info/?l=openbsd-tech&m=167250339811308&w=2

ok jmc@ mvs@ schwarze@

Document that -P disables BLOCKSIZE support.
With input from and OK jmc@

timeout: rename "timeout_at_ts" to "timeout_abs_ts"

I think "abs" ("absolute timeout") is a better mnemonic than
"at" ("at the given time").

The interface is undocumented and there are only two callers, so
renaming it is not a big deal.

probably ok kn@

sync

Look for the first 2 mem regions, ignore the rest

Some macppc nvidia graphics cards have a 3rd mem region, but nv(4)
wants to mmap the 1st and 2nd regions.

ok miod@

crank libfido2 major version, it depends on libcbor and it just
cranked. ok tb@

update to upstream libcbor v.0.10.0 and crank major. Also includes
e308674c5d to fix PR259. This release includes a number of memory
leak fixes.

Disable the upsteam custom allocators feature.

Feedback/ok tb@

Thanks also to Pedro Martelletto for pointing out the new release as
well as PR259.

Let luna88k's bootloader pass RB_GOODRANDOM to the kernel.

Current bootloader can pass boothowto information to the kernel.  It
also has the capability to load random seed data from /etc/random.seed
already.  So set RB_GOODRANDOM at the bootloader when loadrandom() has
been finished successfully.

Now the kernel says "random: good seed from bootblocks".

Tested by LUNA-88K2 and nono emulator, "Absolutely!" ok miod@

sysctl_clockintr: clear "sum" with memset before copyout(9)

Add machdep.lidaction to machdep names list.

ok mpi@

Add TABDLY, TAB0, TAB3 for better source compatibility.
These are XSI extensions but some code seems to expect them.
We don't currently implement TAB1 or TAB2.  Idea from FreeBSD.
OK guenther@ gnezdo@

add history for getpid(2) and getppid(2)

getpid() appeared (undocumented) in v5.  Between v6 and v7 there is an
extra return value for the parent process ID.

getppid() did not appear in v7, it appeared in 32v libc.  But getppid()
predates 32v.  It seems to have been in earlier USG releases such as
Generic 3 (PG-1C300 Issue 3) as the MERT Release 0 manual references it.

getppid() didn't become a system call until 4.3BSD-Reno

omit most of these details and just mention v5 and 32v

with and ok schwarze@

pause.3: miscellaneous rewrites, cleanup

Eliminate some redundant or extraneous pieces from the pause.3 page.
Say the "thread" "blocks", don't say the "process" "pauses".  No need
to enumerate the ways a signal can be delivered.  Add a few relevant
cross-references.

With input from millert@ and schwarze@.

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

accton.c: add missing $OpenBSD$ tag

Do not send (normal) packets before we reach the run state.  Logic copied
from iwm(4), which also looks at the TX_MGMT_ONLY flag.  We don't expect
that flag to be ever set for bwfm(4), but it shouldn't hurt and it keeps
things consistent across drivers.

This fixes issues with suspend/resume (including firmware crashes seen on
the M2 Macbook Air).

ok patrick@, stsp@

Add chip name for new revision of the BCM4378.

ok patrick@

Actually hide the clang-15 workaround behind the COMPILER_VERSION check

COMPILER_VERSION initially missed.  I'm not sure why we still have those
COMPILER_VERSION checks in sys/arch/i386 and sys/arch/amd64, when the
base system doesn't ship gcc any more, but let's stay consistent.

Neuter zlib fatal warnings when building kernels and bootloaders with clang 15

Disable -Wdeprecated-non-prototype instead of patching zlib.  Upstream
plans to drop the pre-ANSI syntax soon.  ok tb@ millert@

Support FTDI FT232R.  The upper 2 bits encode the fractional component of the
FT232R is either 0 or 0.125.

ok dlg@

sparc64: pull retry logic out of tickcmpr_set(), sys_tickcmpr_set()

Pull the retry logic out of tickcmpr_set() and sys_tickcmpr_set() into
C functions tick_rearm() and sys_tick_rearm(), respectively.  There is
nothing wrong with the retry logic in these assembly functions, but
it's better to keep equivalent code similar and this change realigns
the %TICK and %SYS_TICK rearm code with that of the Hummingbird
%STICK.

Tested by miod@ on UltraSPARC I, UltraSPARC II, and UltraSPARC IIe.

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

HISTORY: clarify that unmount(2) used to be called umount(2) from v1 to Tahoe;
joint work with and OK jsg@

Avoid doing cache flush/invalidate operations for DMA memory allocated with
the BUS_DMA_COHERENT flag.

ok miod@

Use an iorw fence since we're syncing cpu access and device access to
memory.

ok miod@

capital letter at sentence start;

FORK_SIGHAND and the handling of func==NULL disappeared in 2017 in
sys/kern/kern_fork.c rev 1.195.

profil(2) first appeared in fifth edition

tuhs/Distributions/Research/Dennis_v4/v4man.tar.gz has
manx/profil.2 but no man2/profil.2

the nsys sysent.c has '&nullsys, /* 44 = prof */'
https://minnie.tuhs.org/cgi-bin/utree.pl?file=V4/nsys/ken/sysent.c

profil(2) is first listed as v5 in the Combined Tables of Contents from
McIlroy's A Research UNIX Reader.
https://marc.info/?l=tuhs&m=158099986005023&w=2
and is present in
tuhs/Distributions/Research/Dennis_v5/v5man.pdf

ok schwarze@

fcntl(2) first appeared in System III
ok schwarze@

Add ktrace struct tracepoints for siginfo_t to the kernel side of
waitid(2) and __thrsigdivert(2) and teach kdump(1) to handle them.
Also report more from the siginfo_t inside PSIG tracepoints.

ok mpi@

{en,de}queing -> {en,de}queuing; from paul tagliamonte

spelling fixes; from paul tagliamonte
any parts of his diff not taken are noted on tech

iostat(8): implement periodic display with setitimer(2)

Prefer setitimer(2)+sigsuspend(2) to nanosleep(2) when performing
periodic work.  The latter drifts.

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