Similarly, for sessions use a callback to free rather than checking
every loop.

Missing no-pie logic for bootblocks.

Change deref to the more sensible unref, and add a couple I missed before.

Instead of putting dead clients on a list and checking it every loop,
use event_once to queue a callback to deal with them. Also dead clients
with references would never actually be freed because the wrap-up
functions (the callback for stdin, or status_prompt_clear) would never
be called. So call them in server_client_lost.

Do not unconditionally clear pcb_onfault after a uvm_fault.  That will
permit the active copyout/copyin to continue work on subsequent faulting
pages and not misinterpret & fault them as kernel bcopy against userland
addresses.  Old bug -- fall of 1996.  This should fix getentropy issues
on MP systems which have become more apparent recently, probably due to
some combo of increased ASLR with unlocked getentropy happening very soon
after vfork/fork...
ok miod

these days, curproc is never NULL, so skip those checks.
Discussed with kettenis

trunc_page() generally.... succeeds at clearing the page offset
bits the first time it is called, so don't do it again.
ok miod

These days p->p_addr will never be NULL.

ok deraadt@

For "ssh -L 12345:/tmp/sock" don't fail with "No forward host name."
(we have a path, not a host name).  Based on a diff from Jared Yanovich.
OK djm@

Do not use the key variable uninitialized (in a debug log statement),
reported by jungleboogie0 at gmail dot com.

Fix coupling and decoupling operations.

With help and ok from mikeb@

Improve error handling and recovery during state insertion

Reshuffle the code around a bit and greatly improve error handling
fixing a few bugs along the way.

Problem reported by and fix was written with Alexandr Nedvedicky.
OK henning

Add bits missed in previous... I suck at cvs.

Put spaces and commas where they belong.

Finally protect VP lookups to guarantee that a pted won't be freed or
reused by a CPU while another CPU is manipulating it.

This races occurs because the virtual spill handlers are run without
taking the KERNEL_LOCK for obvious reasons.  So use a per-pmap mutex
that CPUs must hold when modifying a pted in order to guarantee the
atomicity of operations *and* the coherence between pmap VPs tree and
what's in the HASH.

Thanks to dlg@ for assisting me debugging this.  This change ends your
PowerPC pmap SMP show of the week.  GENERIC.MP on macppc should now be
stable enough to build ports without corrupting its own memory.

ok kettenis@, deraadt@, dlg@

Don't try to be clever when unrolling the loop in pmap_remove().

Needed for upcoming locking.

Replace the per-entry locks by a global HASH lock.

Since this lock is recursive we can now guarantee the atomicity of
pte_inser{32,64}() when a pted has to be removed first.  This fixes
one of the races.

Using a __mp_lock here also allowed dlg@ to provide me useful traces
to fix the next race.  Thanks for your help!

ok kettenis@, deraadt@, dlg@

Call pte_spill_v() from the real mode fault handler instead of rerolling
it.  This will reduce the number of places to audit for locking.

Note that for profiling purposes pte_spill_v() is now marked a __noprof
since per-CPU profiling buffers are not guaranteed to be 1:1 mapped and
cannot be accessed from the real mode fault handler.

ok kettenis@, deraadt@, dlg@

Rewrite PTE manipulation routines to better match the PEM.

Document every operation, make sure to call "sync" when appropriate so
that other CPUs see the bit changes and finally grab a lock where it was
missing to grantee atomicity.

ok kettenis@, deraadt@, dlg@

Split pteclrbits() into pmap_{test,clear}_attrs().

This should not introduce any behavior change but makes the code easier
to read and later easier to protect.  This also brings this pmap closer
to what others do.

Thanks to kettenis@ for spotting a bad typo!

ok kettenis@, deraadt@, dlg@

More usages of pmap_ptedinhash().

If you wonder why pte_insert{32,64}() is not using pmap_hash_remove() if
it finds a conflicting PTE in the HASH, it's because in the current state
trying to grab the same lock a second time would lead to a deadlock.

This is much easier to reproduce on G5 (or G4 with BAT disabled).

ok kettenis@, deraadt@, dlg@

Remove DEBUG stuff.

Make use of ptesr() instead of rerolling it.

Merge various copies of the same code into a new function to determine
if a PTE is present in the HASH.

Note that atomicity is currently not guaranteed between this check and
the following operations.

ok kettenis@, deraadt@, dlg@

Introduce pmap_pted_ro() a simple wrapper for the 32/64 bits versions
that does not call pmap_vp_lookup().

Carreful readers would have notice the removal of the bits on the virtual
address with a page mask, this change allows me to find the 13 years old
bug fixed in r1.145.

ok kettenis@, deraadt@, dlg@

Do only one VP lookup when removing a page.

This simplify pmap_remove() & friends by re-using an already fetched PTE
descriptor.

There's currently a race on MP system where one CPU can reuse a pted
while another one is still trying to insert it in the HASH.  This commit
starts reducing the number of pmap_vp_lookup() calls to help fix this
race.

ok kettenis@, deraadt@, dlg@

Remove the MANAGED flag when removing a PV entry.

Even if this change is not strickly needed, because the memory will be
returned to the pool it helped me track the use-after-free.

Fix a warning.

Remove unneeded splvm() calls and the pool_setipl(9) hack of r1.140.

By instructing spl(9) calls on MP machines I figured out that their high
cost was hiding a race condition involving PTE reuse in our pmap.  Thanks
to deraadt@ for finding a way to trigger such panic by adding a couple of
splvm().

This should make the races easier to trigger but will be addressed
shortly.

This commit starts your PowerPC pmap SMP show of the week.

ok kettenis@, deraadt@, dlg@

Make it so that if a window or session target is prefixed with an =,
only an exact name or index match is accepted, no special character,
prefix match, or fnmatch.

Use p15 for the coprocessor name not 15.  gas accepts both forms, clang
won't accept just a number which a comment in the gas code mentions is
for backward compatibility.

regen

Add another PCI ID for Intel Dual Band Wireless AC 3165. From brad.

typo: accidental repetition; bz#2386

sync

Move the nested check from client to server and compare the client tty
name to all the pane pty names instead of comparing socket paths. This
means that "new -d" will work without unsetting $TMUX.

Mention rtwn(4) in pci(4). Reminded by brad.

systrace(4) STRIOCATTACH can only fail for file(1) if it is already
systraced. If so, silently ignore the error and do not attempt to apply
our own systrace policy. From Patrick Keshishian.

man page for rtwn(4)

enable rtwn(4) on i386

Enable rtwn(4) on amd64.

attach rtwn at pci

New rtwn(4) driver for RTL8188CE wifi cards.

This is a PCI card from the same chip family as supported by urtwn(4) on USB.
Development started in 2013 using urtwn(4) as a starting point but was dormant
for much of the time since. I finally unslacked after uwe@ provided help with
lifting this driver on its feet. As usual we got helpful hints from Theo.

Requires firmware which will be available in ports soon.

There are rate adaptation issues that still need to be fixed, cause unknown.
In my testing the hardware rarely transmits more than 1Mbit/s.

Committing over MAC/BB RTL8188CE, RF 6052 1T1R.

tweak SYNOPSIS and usage();

force reseeding if pid has changed.

ok deraadt@

make sure to check for resolv.conf update the first time the resolver
is used after pid has changed.

ok deraadt@

sync

+ REALTEK RT8191SE

Add support for em(4) on Teak 3020, a Tolopai (EP80579)
based devices. This introduces Realtek PHY into em driver
code and is only a temporary solution to the problem.

OK deraadt@

The (no quite so) new kernel perfpolicy code calls cpu_setperf() from a
timeout.  Unfortunately the smu(4) CPU voltage slewing code sleeps, which
causes a kernel panic.  Prevent this by delegating the CPU frequency switching
and voltage slewing to a task.

ok mpi@

regen

Add more of Tolopai (EP80579) Gigabit controllers id.

Check if interface was stopped before calling rx/tx interrupt routines.

Report & tests by mxb@alumni.chalmers.se, thanks!
OK deraadt, chris

Make unsetting a global option restore it to the default. Diff lying
around for a while, I have forgotten who suggested it :-/.

spacing, makes example fit on display.

no objection from gilles@

Add support for a single "marked pane". There is one marked pane in the
server at a time; it may be toggled or cleared with select-pane -m and
-M (the border is highlighted). A new target '~' or '{marked}' specifies
the marked pane to commands and it is the default target for the
swap-pane and join-pane -s flag (this makes them much simpler to use -
mark the source pane and then change to the target pane to run swapp or
joinp).

Remove unused sa_dl.

teminal -> terminal, from Corey Farwell.

Switch intel_gmbus_is_port_valid and intel_gmbus_is_forced_bit from
extern inline back to static inline so the kernel will build on
compilers that default to c99 inline semantics.

use mallocarray instead of malloc in drm_calloc

remove the now unused alawtolin16 table
ok ratchov@

Remove unused defines and functions.
"Looks fine." deraadt@

Adjust yielding condition slightly in the if_input_process thread.

ok mpi, kettenis

Turn off -Werror in relayctl.

Remove -Werror from the Makefile: as it happened with asprintf, it
breaks the tree if any new compiler warnings get enabled, especially
with -Wall.  We do not want to use it in the tree, except for some
very special places.

As discussed with deraadt@

It is better to check the asprintf return value for -1 instead of the
input buffer for NULL.
Found by adding __attribute((__warn_unused_result__)) to asprintf.

OK deraadt@

Treat a missing mail spool the same as a zero-length mail spool
unless the -f option was specified.  OK deraadt@

snprintf(3) is available on all modern systems and asprintf(3) is
available on more systems these days.  OK deraadt@

Unify .Nd blocks, prodded by and ok jmc@

Fix pf_map_addr() not to cause dividing by 0.  This fixes problem when
using table or dynamic interface addresses for source-hash.  Also
avoid calling arc4random_uniform() with upper_bound == 0.

ok mikeb

Actually commit the bits that make the compiler emit visibility information
for (undefined) references with non-default visibility on mips64.

Support for U4, missed in previous.

ok kettenis@, miod@, dlg@

Check for the correct bit in the interrupts property of the device-tree
to determin if interrupts should be established as edge/level triggered.

Fix audio interrupts on U4 systems.  On such systems the "interrupts"
fields in the device-tree has more bits set and cannot be compared to
0.

Add hpb(4), reminded by Brad.

Explicitly request the sockaddr_dl when doing a RTM_GET rather than
assuming that it will be in the gateway sa.

Fixes a regression introduced with the support of multiple connected
routes, found the hardway by deraadt@.

ok claudio@

Move warning on open failure to open_mbox() where we have the file name.

Do not assume that asprintf() clears the pointer on failure, which
is non-portable.  Also add missing asprintf() return value checks.
OK deraadt@ guenther@ doug@

better description of internal copyn() function

missing __EOT spotted by rpe@

there's been a long standing issue in ppp on a tty/serial line where it allocates mbufs at IPL_SOFTTTY, which is above the IPL_NET the mbuf layer protects itself at.

recent improvements to diagnostics in pools and mbufs now panic
instead of letting these things silently corrupt.

this reworks the ppp handling in the tty layer so it has its own
private pool to allocate packet memory out of. these packets get
built and then queued for softnet to process. softnet dequeues the
packet and attaches it to mbufs as external storage before handing
it on to the rest of the stack.

this was reported on bugs@ and tested by both Walter Daugherity and
Martin van den Nieuwelaar
ok deraadt@ mpi@

Don't leak value returned by cgetstr().  OK deraadt@

Remove the $FSTABFLAG variable and use -F directly. The installer
uses disklabel UIDs unconditionally for a while already.

OK krw@
"looks good" deraadt@

Replace
  [[ -n $(foo | grep 'bar') ]]
with
  foo | grep -q 'bar'

OK halex@ krw@

sync

fix variable name typo

ok ajacoutot@

write Mobile IP{,v6} in a consistent way, as given in RFCs 5944 and 6275.

ok jmc@

Modernize from(1) a bit.  Use getline() to support longer lines and
document the environment variables used.  OK deraadt@

Mention hpb(4), prodded by jmc@

Add missing define for hpb(4).

Man page for hpb(4).

Add a new HT-PCI bridge driver and the necessary glue to openpic(4)
to get interrupts working on U4 machines.

With this OpenBSD can run on PowerMac11,2 (Quad G5).

With inputs from and ok kettenis@, miod@, dlg@

RIP ether_input_mbuf().

Use if_input() instead of ether_input_mbuf().

ok jasper@, bluhm@

Take carp(4) out of ether_input().

Tested by <mxb AT alumni DOT chalmers DOT se>, thanks!

ok bluhm@, dlg@

mention CheckHostIP adding addresses to known_hosts;
bz#1993; ok dtucker@

missing " spotted by Kyle Milz

Make sparc go PIE (not `static PIE' yet).

Override pie range constants (well, only VM_PIE_MAX_ADDR) with a variable
which value is decided at runtime, in order to only enable PIE on sun4m
which has a large enough address space.

with binutils 2.17 we can change some raw opcodes into instruction names
ok miod@ deraadt@

Rename the imx miniroot to nitrogen as it creates "6x_bootscript".
Add a miniroot for the CuBox-i which needs u-boot at a particular
offset in the sd image to boot.  Based on changes made by Patrick
Wildt in bitrig.