This is one of those typical 'more is better' defaults that are just bad. Even on systems that are much better at parallelism than OpenBSD, people noticed that 'let me use as many threads as CPUs' isn't helping anything: https://github.com/llvm/llvm-project/commit/da68d2164efcc1f5e57f090e2ae2219056b120a0 This is the above LLVM commit (which was pointed out by mpi and others) with s/16/4/g. This reduces wall clock time and especially system time of builds significantly on machines with far more CPUs than our beloved OS can properly handle these days such as claudio's ampere box. On my 8 core bulk builder I don't see a significant change in wall clock time but a lot less spin during linking of monsters. Anything with more than 8 CPUs benefits from not calling sched_yield() until the cows come home. tested by claudio, landry and myself

Spotted by jsg@

TABLE_BITS is always currently defined as 4 - 8 is considered to be insecure due to timing leaks and 1 is considerably slower. Remove code that is not regularly tested, does not serve a lot of purpose and is making clean up harder than it needs to be. ok tb@

ok dlg@

ok dlg@

ok kn@ miod@ tedu@

Fixes compile errors with if_vio when other drivers are removed. From Crystal Kolipe. ok mvs@

Found by Tetsuya Isaki at NetBSD and nono project.

Found by Tetsuya Isaki at NetBSD and nono project.

This brings the wprintf family of functions back in sync with printf family of functions which have been already adjusted. While at it, also sync the documentation to explain possible errno values. OK deraadt@, millert@

OK kn@ mvs@, "Sure" mpi@

Rather than having defines for GCM_MUL/GHASH (along with the wonder that is GCM_FUNCREF_4BIT) then conditioning on their availability, provide and call gcm_mul()/gcm_ghash() unconditionally. This simplifies all of the call sites. ok tb@

Since struct gcm128_context is not exposed via a public header, there is no way CRYPTO_gcm128_init() can actually be used properly. Instead, use CRYPTO_gcm128_new() and CRYPTO_gcm128_free_bird()^WCRYPTO_gcm128_release() (naming consistency is apparently hard).

Remove `lockflags' argument from uvm_map_pageable() and add locking assertions in uvm_map_pageable_wire() and uvm_map_pageable(). ok dv@

o Mark functions that have no side effects with __pure. o settzname: Ensure that tzname values are set. OK tb@

o Make min_time and max_time constants. o Mark functions that have no side effects with __pure. o The zone offset at the end of version-2-format zone files is now allowed to be 24:00, as per POSIX.1-2008. o Fix zic bug that mishandled Egypt's 2010 changes. OK tb@

Make absolute_min_time and absolute_max_time constants. Mark __pure for functions that have no side effects. OK tb@

Reduce latency with a huge number of CPU and jobs. ok claudio@

ajacoutot@ says it looks fine from an rc.subr point of view

quoting is not strictly needed per se, i.e. those are equivalent: # tcpdump -i iwx0 port 22 and port 80 # tcpdump -i iwx0 'port 22 and port 80'