fix some poop that snuck into the last commit.

noticed by bijanebrahimi at riseup dot net and confirmed by anton@

Extract compile_mode command status with WEXITSTATUS.

Also tell the user if the parent shell spawned by popen(3)
was killed by a signal.

Discussed with jca@.

ok millert@ jca@ deraadt@

Stop grabing the KERNEL_LOCK() for running protocol input routines.

The NET_LOCK() is already held in this thread and is now enough.

People interested in ARP/bridge(4)/switch(4)/pipex(4)/pppoe(4)
performances can now push the KERNEL_LOCK() without depending on
other subsystems/drivers.

Tested by Hrvoje Popovski.

ok bluhm@, visa@

Creating a cloned interface could return ENOMEM due to temporary
memory shortage.  As it is invoked from a system call, it should
not fail and wait instead.
OK visa@ mpi@

Change `so_state' and `so_error' to unsigned int such that they can
be atomically read from any context.

ok bluhm@, visa@

lock->ctx != NULL => lock->ctx
requested by kettenis@

Work around a problem with ww_mutexes in the drm modeset lock.
Screen blanks often result in a NULL dereference in __ww_mutex_lock with
lock->acquired being non zero and lock->ctx NULL.  mpi@ also reported
it occuring when switching from Xorg to a virtual terminal.

ok mpi@

Do not truncate 64bit integers when pretty-printing types.

Use ip{,6}_send() instead of ip{,6}_output() to prevent a recursion.

if_start() is a boundary between the network stack and drivers.  The
states it modifies must be protected by the driver, so we should not
require the NET_LOCK() there.

ok bluhm@, visa@

Stop printing <not displayed> for wireless keys we know the kernel no
longer export them to userland.

ok stsp@, deraadt@, jca@

Do not silently truncate the firmware channel number.

Fix build with gcc, ok stsp@

make mpls_input take a struct ifnet *ifp argument.

this makes it like all our other protocol family input functions.

mpls_input always looks up the interface the mbuf was received on,
but it's always called by code that already has a reference to that
interface anyway. the result of this is a few less if_get/if_put
calls.

ok mpi@ bluhm@ visa@ claudio@

sync

Rename test file =.err to eq.err for simpler make file scripts.

Fix indentation.

Initial support for HOSTAP mode.  With this bwfm(4) can spawn an access
point including WPA2 support.  We now have a different event mask per
mode, so that events that are only useful for STA mode don't interfere
with HOSTAP mode.  Power savings is disabled when we act as AP. The
connection events generate 802.11 frames for handling auth/assoc and
deauth/deassoc so that our stack takes note of the connecting nodes.

Convert IF_CLONE_INITIALIZER() into C99 initializer.
OK mpi@

Do not pass a userland pointer to ktrabstimespec().

Prevents an infinite pagefault/pmap_enter() loop when ktracing apps
doing a lot of futex(2) calls like firefox & chrome.

update comments that were left behind in the recent unifdef JOB commit
from Klemens Nanni, thanks.

Fix typo in comment; ok mpi@

In AP mode multicast packets share the flowrings with broadcast
packets.

The bwfm(4) TX ring expects the ethernet header as part of the TX info
struct.  The data length is the length of the frame without the header.
In the previous version m_adj(9) is used, but since that was changed we
need to decrease the length ourselves.

Guard the debug printf function behind BWFM_DEBUG as well.  Also only
print the firmware's dmesg(8) if we're running with a higher debug
mode.

Prompted by Michael W. Bombardieri

Move SC_* defines into <machine/setjmp.h>, and rename them,
so that <machine/signal.h> is not needed in setjmp.S.

Suggested by kettenis@ long ago, OK deraadt@

Pass correct size to uvm_vnp_setsize() for large files.

Add kqueue support for ext2fs based on ffs.  OK deraadt@

group shared source files (e.g. SRCS_KEX) and allow compilation w/o OPENSSL
ok djm@

move subprocess() so scp/sftp do not need uidswap.o; ok djm@

switch ssh-pkcs11-helper to new API; ok djm@

only ssh-keygen needs uuencode.o; only scp/sftp use progressmeter.o

uuencode.h is not used

split client/server kex; only ssh-keygen needs uuencode.o;
only scp/sftp use progressmeter.o; ok djm@

Override membar_producer() for octeon.

OK dlg@

Drop two outdated entries

Namely POSIX character classes and tilde expansion within parameter
substitution.  While here, kill a useless line and fix a typo.
From Klemens Nanni

Remove unused "args" param from kb_add(), it is always NULL.
OK anton@ jca@

optimization and some cleanup; mostly from kshe (except the unmap() part)

Kill entries that are obviously outdated/out of scope

Pointed out by Klemens Nanni who had ~ the same diff

TIOCUCNTL is now allowed by promise "tty".

Reminded by jmc@

Enable TIOCUCNTL to be able to set ns8250's break detected condition.

It is now possible to send BREAK commands to vmd(8) independently of
the serial terminal emulator.

Happy virtual ddb(4) hacking!

No objection from mlarkin@, ok nicm@, ccardenas@, deraadt@

Allow TIOCUCNTL issued on a pty(4) master in promise "tty".

This will be soon be used to emulate BREAK commands in vmd(8).

ok nicm@, ccardenas@, deraadt@

Translate the TIOCSBRK & TIOCCBRK ioctl(2)s issued on a pty(4) slave to
corresponding user mode ioctls.

If the master part of the pseudo terminal previously enabled TIOCUCNTL,
it will now receive the TIOCUCNTL_{S,C}BRK commands.  This allows to
send BREAKS commands over a pty(4) independently of the serial terminal
emulator used.

Guidance and ok nicm@, ok ccardenas@, looks ok to deraadt@

Attach dwiic_pci on Dell Precision 7520. Also restrict ihidev reports
to those of 3 bytes or more (as per the hid-over-i2c spec).

discussed with jcs, kettenis

Mark up command strings with Cm, and variables with Va.

ok jmc@

Delete flowrings when we take the interface down or change its
settings.

Add regression tests for cdrom keyword

ok mlarkin@

Create multiple transmit flowrings in station mode, four in total, based
on TOS values.  In AP mode create multiple flowrings per connected node.

delete a wrong comment

remove all PG_G global page mappings from the kernel when running on
Intel CPUs. Part of an ongoing set of commits to mitigate the Intel
"meltdown" CVE. This diff does not confer any immunity to that
vulnerability - subsequent commits are still needed and are being
worked on presently.

ok guenther, deraadt

Add proper bounds checking to kb_add().  OK anton@

amd64_{get,set}_fsbase() are superfluous and unused; delete them.
Delete unused/never-implemented sysarch defines and structs while here.

ports check and ok naddy@
ok deraadt@ mlarkin@

Use __inline instead of inline as this header file might be included in a
"strict ansi" environment (even though it shouldn't be).  Found out the hard
way by phessler@ with a port that uses gcc -ansi.

ok millert@, phessler@

add arch to Dt;

On OpenBSD/armv7 we deliberately trap unaligned access.  Unfortunately
the assembly code in libcrypto assumes unaligned access is allowed for
ARMv7.  Make these paths conditional on __STRICT_ALIGNMENT not being
defined and define __STRICT_ALIGNMENT in arm_arch.h for OpenBSD.

ok tom@

Remove unused extern variable in openssl(1) s_time

This extern variable appears not to be used.
And it is overridden by local variable in doConnection().
This causes MSVC warning C4459 "declaration of 'verify_error'
hides global declaration".

OK millert@

Delete obsolete/untrue comments

remove a ridiculous comment and move another comment to the line it was
really intended to describe. no functional change

Assume that direct-mapped addresses come from the xkphys segment.

Add references to errata.

Add identcpu.c and specialreg.h definitions for the new Intel/AMD MSRs
that should help mitigate spectre. This is just the detection piece, these
features are not yet used.

Part of a larger ongoing effort to mitigate meltdown/spectre. i386 will
come later; it needs some machdep.c cleanup first.

ok kettenis@

Add fs to the application/octet-stream extensions, mostly for installXX.fs,
to help with a problem noticed by tj@ and diagnosed by job@, ok phessler

For OpenBSD distfile mirrors running httpd who want to do this before
getting the updated mime.types, you can set:

.  types {
.          application/octet-stream    fs
.  }

Handle %gs like %[def]s and reset set it in cpu_switchto() instead of on
every return to userspace.

ok kettenis@ mlarkin@

Fix an interrupt storm condition in vldcp(8) (sparc64).

If the Rx channel goes down with pending data on the ring, toss the pending
data to ensure that the firmware's rx ring head index matches the rx ring
tail index. Otherwise we trigger an interrupt storm since vldcpread() never
consumes pending data while the channel is down, and the firmware keeps
triggering the interrupt until all data on the ring is consumed.

Interrupt storm observed and fix tested on a T5220 (which can now run
ldom guests; without this fix starting ldomd would trigger the problem).

ok kettenis@

Don't advertise L0s if the "aspm-no-l0s" property is present.  Base available
bus numbers on the "bus-range" property.  Create outbound translations based
on the "ranges" property.

regen

Add Silicon Motion SM2260 NVMe.

sync

pckbd: don't change translation mode if controller is in table 2

This was changed a decade ago to forcibly try table 3 first in order
to make some now-long-gone hardware work.

Newer Lenovo machines seem to have trouble being asked to change
modes which manifests as a long boot delay as it waits for each
request to timeout, or by causing the keyboard to generate junk when
typing.

Assume table 2 by default and just leave it alone if it's already
there.  This is how Linux has operated for quite a while and seems
to help on these Lenovo machines.

Tested by a few with these machines and has been in snaps for a bit.

Bring back the sign compare changes, this time with a fix from otto@
that fixes the issues seen on hppa.  OK deraadt@ otto@

unifdef LOGIN_CAP, we always want to use login.conf
OK jca@, no objection deraadt@

Remove dead gettimeofday calls and timevals.

ok jca@

sxits(4)

Enable sxits(4).

Add sxits(4), a driver for the temperature sensor on the Allwinner A10/A20
touchpad controller.

Enable sxitemp(4).

Set the proper bus clock rate based on the "clock-frequency" property and
the rate of the parent clock.  If the "clock-frequency" property isn't
present, use the default standard mode clock of 100kHz.

tweak previous;

Import gilles@'s standalone spfwalk utility into smtpctl(8) as
'spf walk' command.

Ok gilles@

Move unpack functions into a seperate file.

Required for upcoming 'smtpctl spf walk'.
Ok eric@ gilles@ millert@

Remove redundant zeroing and dead code.

rework the transmit and receive paths to address reliability issues.

phessler@ has been having trouble with msk on overdrive 1000s. some
of the issues relate to the driver not coping with exhaustion of
mbufs for the rx ring, the other issues are corruption of the mcl9k
pool that msk uses.

this diff adds a timeout that the rx refill code uses when the rx
ring is empty and cannot be filled. it'll periodically retry the
ring refill until it can get some mbufs in the air again.

the current code made hunting for the mcl9k issue too hard, so this
rewrites it to be simpler and more like other drivers. there's now
just arrays of mbuf pointers and dmamaps to shadow the hardware
ring entries, and producer and consumer indexes. what was there
before had linkes lists of something to hold mbuf pointers and
dmamaps, and some way to go from the ring to go back to that. i
think, it was hard to tell what was happening.

this also copies the ADDR64 handling on the tx ring to the rx ring.
this potentially makes more rx descriptors available, but that can
happen later.

in hindsight the mcl9k problem could have been from letting if_rxr
allocate the entier ring. if every descriptor was filled, the chip
may have run around the ring when it shouldnt have. giving rxr one
less descriptor than there is on the ring may have fixed the problem
too.

this work also makes it easier to make msk mpsafe.

tested by an ok phessler@
ok kettenis@ deraadt@

To send out packets we need to create a flowring.  Acting as station,
we typically have about four flowrings per priority.  As access point
we apparently need one, or four considering the priorities, flowrings
per client.  For now let's start with a single TX flowring.  To setup
a flowring we need to send a create request and can only start sending
packets as soon as we are told that the ring is created.  With this we
can now do actual network traffic.

Drop incoming network packets as long as we are not in RUN state.  This
happens when we successfully associate and the AP tries to initiate the
WPA2 handshake but we haven't received the asynchronous ASSOC event yet.
Dropping the packet will make the AP retry, and at that point we should
have successfully associated.  While there, don't feed the event packets
to our network stack.  It's been helpful for debugging but now it's time
to let go.

some minor fixes;

Handle resets; needed on Allwinner R40/V40.

Implement R40/V40 SATA clock.

Add support for AXP221/223.

When we receive an AUTH or ASSOC event even though we have already
reached the RUN state, this probably means that we have roamed to
a different AP.  In that case throw us back into SCAN mode and let
the stack look for a new AP to connect to.  In the future it might
be worthwhile to use the ROAM event information to read the new AP
information to adjust our stack, but that is further down the road.

Sync with the code in libc

OK millert; original commit message by tedu@:

memcpy from the right place. at this point, the used variable is not
relevant. from Mark Karpilovskij.
ok millert

Make arm64 use the MI mplock implementation.  Avoid <sys/mplock.h> being
pulled in for assembly files by bringing <machine/cpu.h> and
<machine/param.h> in line with other architectures.  Some minor cleanup of
those files is included.

ok patrick@

unifdef JOBS support

Prompted by a mail from Klemens Nanni, who also had the same diff.
ok deraadt@ millert@

Complete and fix grammar in comments and bring grammar terms,
function names and function code into sync.

some 11n bits; help/ok stsp

@libset is going to be part of the dependency information.

Use log_warnx() in places where errno is irrelevant.

ok mlarkin@ ccardenas@

Typos in comments: 'it' -> 'if' and 'noityf' -> 'notify'.

from Klemens Nanni

Remove unnecessary delays.  There is no reason to wait after each and every
read or write to aregister.  There is also no reason to wait after
transmitting a STOP since the controller will wait until the bus is free
when transmitting the next START.  Based on a diff by Stephen Graf.

Also remove the interrupt code; it doesn't work on the newer variants of
the device.  The functionality will be put back in a future commit.

ok patrick@

Show uvm_fault and trace when typing show panic on a page fault'd kernel

Currently there is only support for amd64, if this change settles
I will add support for the rest of the architectures.

OK kettenis@.

No need for <sys/socketvar.h>

Fix DRM_DEBUG builds.

OK jsg@.

the value of bflag is a blockno, so use the proper type, avoids
an overflow seen with scanning for alternate superblocks; ok deraadt@