Use AF_UNSPEC not 0

Some spaces have been removed from pfctl print.  Adapt expected
output in regress.

retire the old cron socket path; ok jca millert

Detect vmm(4) in the bootloader and automatically switch to the serial
console at 115200 baud.

tested by phessler and myself, ok deraadt

Regress update to match changes for chrome

Fix comment about VLAN encapsulation and checksum offload

Document that some chips actually could do hardware checksum offload for
encapsulated packets, though that would need special handling in those
drivers.

discussions and ok naddy@

add option "network ... priority number" to announce prefixes from the
kernel routing table selected by priority.
For example to import all ospfd/ospf6d routes into bgp.
tested by remi@
ok remi@ henning@ and maybe a little claudio@

Detect when a router advertisement packet changes due to config
change and if it does send a new advertisement.

The way this is implemented gives us various things for free:
- periodic sending of router advertisements
- send initial advertisement for every interface on startup

no longer needed

Sync comment

Makes it a tad easier to read through and compare with BN_swap_ct().

OK tb

Add -w option to vmctl stop to wait for completion of VM termination.

Use it in /etc/rc.d/vmd accordingly.

OK sthen@

vmm(4): return proper cache topology for cpuid(0x4)

Make the cache neighbor fields match the number of VCPUs present
(currently 1)

ok reyk

Retire RTM_LOSING, it no longer makes sense and on busy servers the
route socket is flooded with those messages. Instead maek sure that the
removal of the dynamic route that can happen is actually also sent to
the routing socket.
OK mpi@ henning@

rtm_send() the cloned routes because of ICMP mtu changes. Until now
these changes to the routing table have not been visible whereas the
RTM_DELETE of those routes have been. Remove this inconsistency.
Input and OK mpi@
OK henning@

vmm(4): respect argument size when reading from undefined ports.

vmm(4): small cleanup in vm_rwregs.

Clarify error values and change a panic into a debug printf (which will
in turn just kill the VM).

Drop a const-bomb on regexec. It's probably not a good idea to remove a
const promise when processing it in the regex engine.

Minor tweak and OK schwarze@

When in incremental search handle ^M (a.k.a. <cr>) like ^[ (a.k.a.
<esc>). i.e. exit incremental search and set the mark. This is what
emacs does.

pointers, suggestions and ok florian@

Change the control socket to ospfd.sock.<rdomain>.

ok friehm@ jca@

After opening all the needed files, tighten the pledge(2)
from "stdio rpath" to just "stdio", before parsing any user data.
It may not matter that much just yet, but parsing will become
slightly more complicated soon when i shall add UTF-8 handling.
OK millert@

the STATE_LOOKUP macro made sense ages ago. It stopped making sense
when we moved most of the functionality into a function. g/c the macro
and just call the function. ok mpi jca

Repair the regression introduced by the recent refactoring
revision 1.11 date: 2004/07/03 21:00:37;
for -p/-P, the argument was no longer parsed, causing segfaults.
OK millert@

Rename function to vmd_check_vmh

remove wrong comment

ok jca@ tb@

allow default from state

allow state->new to deduce the command name from $0

Add -f option to vmctl stop to forcefully kill a VM.

This also fixes a bug in vmm_sighdlr where it might have missed
forwarding the TERMINATE_EVENT to the vmd parent after a VM child
died, leading to an abandoned VM in the vmd parent process.

OK ccardenas@ mlarkin@ benno@ kn@

in if_addgroup(), call the new pfi_group_addmember() instead of
pf_group_change() - the latter is called by _addmemeber now to update dynaddr.
before this, "set skip on lo", ifconfig lo1 create -> no skip on lo1 until
pf rueset got reloaded. Now lo1 gets the skip flag as intended. This has
caused much confusion with i. e. gif interfaces in the past.
ok benno, very excited ok phessler

Convert AH & IPcomp to ipsec_input_cb() and count drops on input.

ok markus@

provide pfi_group_addmember(), which makes the new member interface inherit
set flags from the group. ok phessler benno

there is nothing secret about rad.conf

Document behavior change of EC_POINTs_mul(3) from EC constant time changes.

ok beck on earlier version, markup help from Schwarze.

Don't hide errors when IPv6 forwarding is not enabled.
OK(failed) phessler
OK deraadt

Expand formats in load-buffer and save-buffer.

s/wuth/with/ in comment

sync

Do for most running out of memory err() what was done for most running
out of memory log_warn(). i.e. ("%s", __func__) instead of manual
function names and redundant verbiage about which wrapper detected the
out of memory condition.

ok henning@

Turn yesterday's optimistic ! in an XXX comment into a more cautious ?

Using resolved after realpath(3) has failed is dangerous. Don't do it!

Minor tweak and OK jca@
OK beck@ deraadt@

Make the output of the list command more sensible for the output device.
We now output $COLUMNS - 8 characters of the string and a newline.
This is similar to the behaviour in ed(1).

Discussed with and OK schwarze@

Helper function to shorten history.

implement media type detection and forcing of link speed, lightly tested
with a variety of 10g optics.

Rephrase the wording on the replacement string of the substitute command.
Cover more cases with less wording.

Joint work with schwarze@

OK millert@

Add function comments.

move declarations of hwrm message functions up to the top and make them
non-static.

don't bother checking the rx index matches what we expect, it works
properly.

Remove an old and false comment. REALLOC now free(3)s the code if realloc
fails.

OK millert@

zap trailing whitespace;

Update EC regression tests.

Part of https://github.com/libressl-portable/openbsd/pull/94
from Billy Brumley and his team.

ok jsing

remove newd control leftovers

remove unused variable; pointed out by llvm

add ractl, the rad(8) control program

Indent labels by a space so they don't obliterate function names in diffs.

ECC constant time scalar multiplication support. First step in overhauling
the EC module.

From Billy Brumley and his team, via
https://github.com/libressl-portable/openbsd/pull/94

With tweaks from jsing and me.

ok jsing

Provide BN_swap_ct(), a constant time function that conditionally swaps
two bignums. It's saner and substantially less ugly than the existing
public BN_constantime_swap() function and will be used in forthcoming work
on constant time ECC code.

From Billy Brumley and his team. Thanks!

ok jsing

Factor out a bit of ugly code that truncates the digest to the order_bits
leftmost bits of a longer digest, according to FIPS 183-6, 6.4. Eliminate
a microoptimization that only converts the relevant part of the digest to
a bignum.

ok beck, jsing

pledge(2)

Looks great! deraadt@
OK florian
OK remi@

style (single-line ifs don't need braces)

$OpenBSD$

Now that all *_free() functions are NULL safe, we can generate the
freenull test from Symbols.list.

Suggested by jsing, discussed with beck and bluhm.

Return the VM pid to the vmd parent.  This pid field already existed
in the result but wasn't filled in by the vmm process.  No functional
change.

Remove a debug message

When an interface doesn't have a layer 2 address in6_get_soii_ifid()
failes and then later on a in in6_get_ifid() a layer 2 address is
"borrowed" from from another interface.

Do the "borrowing" in in6_get_soii_ifid(), too so that semantically
opaque interface identifiers work for these kind of interfaces, too.

OK phessler, benno

When an interface doesn't have a layer 2 address in6_get_ifid()
tries to "borrow" one from another interface.
But then it checks if the U bit is set int the generated EUI64
address and rejects it.
On the other hand for interfaces that do have a layer 2 address this
check is skipped, so relax it for the "borrowing" case, too.

With this one gets stable link local addresses on e.g. gre(4)
interfaces on certain virtualisation environments depending which mac
addresses get picked for the vio(4) interfaces while previously we
would end up with a random IP on every reboot.

Reported by Aaron A. Glenn via phessler.

OK phessler, benno

vmd already had DEBUG/DPRINTF, there is no need for VMD_DEBUG/dprintf

Replace all occurences of dprintf with DPRINTF (defined in proc.h).

Remove raw_usrreq and raw_cb, nothing is using them anymore.
OK mpi@

Remove net/raw_cb.h from includes and replace the RAWSNDQ, RAWRCVQ with
protocol specific ones.
OK mpi@

machine/fpu.h is not needed on amd64 and does not exist on i386.
Remove the include.

we were refering to 10k states by default here as well, pt out by claudio

where we were showing "set limit states 10000" make that 100k as well,
and adjust adaptive.start/end as well (just like in the code)

Fix a few, but not all, clang warnings: Use "%s" to print modifiable
strings, add a couple of braces, ansify a few functions, add and remove
a few extra parens.

ok jcs

Make legacy interrupts work in acpipci(4).

ok patrick@

+addsub

Add simple regression tests for BN_{,u}{add,sub}(3). With input from jca

The year is 2018.
Mercury, Bowie, Cash, Motorola and DEC all left us.
Just pf still has a default state table limit of 10000.
Had! Now it's a tiny little bit more, 100k.
lead guitar: me
ok chorus: phessler theo claudio benno
background school girl laughing: bob

"%%s: s" -> "%s: %s" in log_warn()

Import rad(8).
It's a Router Advertisement Daemon written using the standard 3
process privsep pattern and a parse.y based config file.
Commit early to continue work in tree.
OK jca
"it's totally rad" phessler@
"usr.sbin never runs out of space" deraadt@

Tweak debug log messages

- Turn tracing messages into DPRINTF (only compiled with DEBUG).

- Pass __func__ to vm_stop and vm_remove: this way we can track who
called the function in the async context.  It replaces the manual
log_debug in front of each vm_stop/vm_remove.  This debug logging
trick can be removed in the future once we are more confident about
it.

OK ccardenas@ mlarkin@

check-problems -> pkg_check-problems;

In asm.h ensure NENTRY uses the old-school nop-sled align, but change standard
ENTRY is a trapsled.  Fix a few functions which fall-through into an ENTRY
macro. amd64 binaries now are free of double+-nop sequences (except for one
assember nit in aes-586.pl).  Previous changes by guenther got us here.
ok mortimer kettenis

rde_update_get_prefix() and friends should also verify the prefixlen.
This way the check can be removed from rde_update_dispatch() which is
just a duplicate of the general failure case of rde_update_get_prefix().
OK benno@ phessler@

Tweak comments and explain some not so obvious things.

Error out if -netmask/-prefixlen does not follow the destination parameter

Since the address string comes last, `-prefixlen 56 2001:db8::' silently
installs a route for /64 since that's the currently implied prefix length.

The manual page already states that these options must follow the
destination parameter in order to have any effect.

Discussed at length with many
OK benno sthen bluhm jca

sync

Always use PSCI to enable secondary CPUs if it is detected.

__func__ log statements and some whitespace knf
ok^H^HHmpf, whatever claudio@

Add a "machine acpi" command to force booting with ACPI enabled.

Initialize the rtable in the requests send to bgpd to the current
rtables process. With this you dont need to add "table N" to commands
when talking to a bgpd not in rdomain 0.
ok claudio@ phessler@

move the top level pledge, getrtable() works with pledge stdio.
ok claudio@ phessler@

in pf_set_protostate(), only decrement the half-open states counter when
the state was created on this host, i. e. not for those pfsync-imported.
whether pfsync-imported states should be accounted is a seperate discussion,
but as things are, we only increment the counter in pf_create_state(), and
imported states don't excercise that path.
probably fixes the half-open states accounting underflow-wraparounds that
some people have been seeing.
ok sashan

You can run multiple copies of bgpd in seperate rdomains.

However, the processes will see each others route messages. Some
structures are not initialized correctly for that, causing at least
useless log messages.

This is an attempt to use the default_tableid where its needed.

A few hardcoded uses of rtable 0 remain.

ok claudio@

the check "if (nc->rtableid)"

is used as an indicator that we are dealing with network statements from within
rdomain <n> { } blocks.

That needs to check against the rdomain we are running in instead.

ok claudio@

If intra area prefixes move from one router to another router, cloning routes
may become gateway routes and contrary. The kernel does not allow to change the
flags RTF_GATEWAY / RTF_CLONING in RTM_CHANGE messages, but ospf6d tried this
anyway. The result is a broken route.
Instead of modifying such routes remove the old route and insert a new one.

Thanks to Raimund Specht for reporting the problem and testing the fix.

OK claudio@

Since we use multiple cloning routes (mpath) if more than one ip address
exists in the same network, the routes are distinguished by their gateway
address, which is the associated interface address.
The ospfd has to track the gateway addresses so that kroute_matchgw() is able to
find the correct routes.

OK claudio@

Introduce new IPsec (per-CPU) statistics and refactor ESP input
callbacks to be able to count dropped packet.

Having more generic statistics will help troubleshooting problems
with specific tunnels.  Per-TDB counters are coming once all the
refactoring bits are in.

ok markus@

Remove DELAY(1000) from carp_send_arp() / carp_send_na() since it is not clear
why it was necessary.

OK bluhm@
'ok but watch for fallouts' mpi@

sync

Let kroute_matchgw() select the correct route before checking flags.

OK claudio@

unify option parsing a bit more

zap old comments that are unlikely to ever come back