whitespace

first step towards keep-alive/persistent connections support

Now that the relayd timeouts have been fixed, make the http timeout
test more strict again.  Backout rev 1.2 of args-timeout-http.pl.

When a connection was spliced in one direction and in copy mode in
the other direction, the timeouts did not work.  They were longer
than specified.  Link the splicing and non-splicing timeouts.
Found by make run-regress-args-timeout-http.pl
OK reyk@

Improve RAND_write_file(), chmod crud, etc.
ok tedu

Update regress test to work with ressl API changes.

use mallocarray for multiplied value checking

fix regress tests after log changes. noticed by bluhm@.
everybody stand back. i know regular expressions.

Add configuration handling for certificate and key files.

KASSERTMSG(9): New kernel assertion with message

KASSERT() is annoying as it only prints the expression as a string.  If you
(developers) want to know a little more information, you have to do:

#ifdef DIAGNOSTIC
if (bad)
panic(...);
#endif

KASSERTMSG() replaces it into a single line:

KASSERTMSG(!bad, ...);

Taken from NetBSD.

(There is a concern that KASSERT() messages are too long; consume more memory,
and not friendly for small monitors.  This have to be considered & revisited
later.)

"Like" from henning@
Man page review & advices from jmc@ and schwarze@

Add stubs for the proposed server API.

add RDC R1012 support

Stop leaking internal library pointers in error messages.

Requested by miod@

pass correct sizes to free()

add missing whitespace between .Fa macro argument and trailing punctuation

use mallocarray()

pciide.c, pciide_rdc_reg.h(new): ported rdcide(4) from NetBSD.
it supports RDC's R1012 IDE controller.

tested on 86duino EduCake (DM&P Vortex86EX SoC)

ok by deraadt@

Tabs, not spaces.

use mallocarray

Rename the context allocation from ressl_new to ressl_client, which makes
it completely obvious what the context is for. Ensure client functions are
used on client contexts.

one .Fn argument per function argument

Some reallocarray() use; review Jean-Philippe Ouellet, patrick keshishian
ok tedu

Split the context allocation out from the configuration. This will allow
us to properly report errors that occur during configuration processing.

Discussed with tedu@

Do not fold multiple function arguments into the same .Fn argument:
That may cause indexing and formatting issues.
Buggy mdoc(7) code mentioned by uebayasi@ to jmc@.

boot(9): Cosmetic changes to improve diff'ability.

Move the client code into a separate file.

Rename various configuration handling functions.

Requested by and discussed with tedu@.

Comment out option GPT until the matching userland bits are in place.

Use a single ressl.h header file.

Discussed with beck@ and tedu@.

Cosmetic changes to reduce diffs.

Update procflags list, add PS_SYSTEM, PS_EMBRYO, PS_ZOMBIE and
PS_NOBROADCASTKILL. The resulting table is shifted so far right
that a few additional lines had to be wrapped. Not ideal but the
best we can do at the moment.

The correct place to call _bus_dmamap_sync() is after we copy data *to* the
bounce buffer and before we copy data *from* the bounce buffer.  Currently
_bus_dmamap_sync() is a no-op, but keeping it #ifdef'ed out in the wrong
place makes no sense.

ok deraadt@, miod@

sync

Use lerrx instead of errx since the logging subsystem is already initialized.
OK florian@

Fix sched_stop_secondary_cpus() to properly drain CPUs

TAILQ_FOREACH() isn't safe to use in sched_chooseproc() to iterate
over the run queues because within the loop body we remove the threads
from their run queues and reinsert them elsewhere.  As a result, we
end up only draining the first thread of each run queue rather than
all of them.

ok kettenis

yes indeed, it returns void *.  from Jean-Philippe Ouellet, i also had
this lurking in a tree

Explicitly initialise slen - this was not previously done due to a missing
M_ASN1_D2I_begin macro.

Convert error handling to SSLerr and ERR_asprintf_error_data.

Some functions need to dup() before sending a socket by imsg and don't
need to close() after sending socket since imsg_compose() closes the
passing socket.

Only set machdep.allowaperture if 'vga1: aperture needed' is found
in dmesg output. Use that information to decide whether or not to
ask the user if he intends to use X.

initial diff from and OK halex@
OK deraadt@

include stdint.h for standard ints. from Jean-Philippe Ouellet

fix even more trailing tabs

#$%#@$# CONSPEED

Display zero page hit and miss counters in vmstat -s.

ok deraadt@

more hanging tabs

ugly hanging tabs; ok jasper

Show an error if cmd_find_session can't find the current session, like
the other functions.

No libsa putchar wanted here actually

An EOF is a good reason to close a connection.

ok nicm@

obvious conversion to mallocarray()

tweak previous;

missing bracket in previous;

Still needs <sys/reboot.h>

If a client is killed while suspended with ^Z so has gone through the
MSG_EXITED dance, don't try to resume it since a) it's pointless and b)
the tty structures have been cleaned up and tmux will crash.

remove all crypt choices other than bcrypt. ok afresh1 deraadt

calling this "encryption" makes me cringe. "slightly obfuscated" is better.

compare snprintf return value with -1. this isn't really necessary because
int promotion rules guarantee the correct result when compared with sizeof,
but it is perhaps easier for some people to understand it this way.
from Doug Hogan.

sync

sync with kernel, root= -> rootdev=

consistency with upcoming bootloader device parsing of the uboot arguments,
root= becomes rootdev=

Convert d2i_SSL_SESSION to ASN1 primitives, instead of the horrific
asn1_mac.h macros. This still needs a lot of improvement, but immediately
becomes readable.

ok miod@ (sight unseen!)

Use errx() after getpwnam() failure since errno may not be set.
All other privsep / privdrop daemons do this the same way.
OK florian@ some time ago

Document KERN_PROC_NOBROADKILL

When reassembled IPv6 fragments are NATed or RDRed by pf, the
checksum has to be recalculated before the packet is fragmented
again.  Put a missing in6_proto_cksum_out() into pf_refragment6().
This makes run-regress-frag6 and run-regress-frag6-ext pass again.
From Matthias Pitzl; OK henning@

tweak previous;

do not need malloc.h

use mallocarray where arguments are multiplied; ok tedu

-N for SYNOPSIS and usage(), and some small changes to previous;

bye bye src url. Never amounted to much, we went another road a while ago

Keep track of port connections and port resets.

Also notify upstream when a port finished reseting and when the
connection status changes.

Gets things further along to the point where pipe device transfer
and control methods are required.

bundling base + packages was slow, as it needed to find lots of files.
different approach: mark the base system in one go, then hand pick the
files from the pkglocatedb.

quite a few unaccounted files yet, this will get better

Set KERN_PROC_NOBROADCASTKILL so that iscsid is not killed by init
when terminating multi-user so that we have a chance to sync the
file systems mounted via iscsi.

sync

kill fs2hXX/h2fsXX macros with letohXX/htoleXX

The reason being that ext2 structures are little-endian but JBD2 journal
is big-endian.  Don't confuse readers by talking about "file system endian".

Some KNF while there.

ok guenther

Put back a m_copyback() that got lost in rev 1.837 of pf.c.  This
fixes the rewrite of an IPv6 header of an ICMP6 packet in the payload
of an ICMP6 error packet.  Path MTU discovery with ping6 over pf
nat or rdr works again.
Found by src/regress/sys/net/pf_fragment make run-regress-fragping6
OK henning@

GPTSIGNATURE must be LL, for operation on non-punk machines
ok punk

Add iscsid_flags to rc.conf so we do not try to start iscsid all the time.
Noticed by naddy@

kill fs2hXX/h2fsXX macros with letohXX/htoleXX

The reason being that ext2 structures are little-endian but JBD2 journal
is big-endian.  Don't confuse readers by talking about "file system endian".

While here, nuke the copy of bswap.c and link it against the kernel's like
fsck_ext2fs has done, for better maintainability.

ok guenther

Remove license introduced with the PSK code, which has since been removed.

ok deraadt@

When the three possible return values are -1, 0, and 1, != 1 is the same
as <= 0. And the latter is the normal idiom so use that.

ok claudio@ henning@

Introduce PS_NOBROADCASTKILL a process flag that excludes processes from
receiving broadcast signals (kill -1). The flag can be set via a new
sysctl KERN_PROC_NOBROADCASTKILL. This will be used by iscsid to survive
the mass killing by init(8) when terminating multi-user operations.
With and OK guenther@

Another compression remnant.

Expand the tlsext_sigalg macros. The end result is about the same number
of lines and much more readable.

ok miod@

Rewrite i2d_SSL_SESSION to use the ASN1 primitives, rather than using the
horrific macros from asn1_mac.h.

This is a classic example of using macros to obfuscate code, in an attempt
to reduce the line count. The end result is so ridiculously convoluted that
it is completely unreadable and it takes hours to deconstruct the macros
and figure out what is actually going on behind the scenes.

ok miod@

kill fs2hXX/h2fsXX macros with letohXX/htoleXX

The reason being that ext2 structures are little-endian but JBD2 journal
is big-endian.  Don't confuse readers by talking about "file system endian".

ok guenther

Add a regress test for the ASN1 handling of SSL session tickets.

The bell tolls for BUF_strdup - Start the migration to using
intrinsics. This is the easy ones, a few left to check one at
a time.
ok miod@ deraadt@

tedu setlist support, superseded by the new locate dbs.

stuff no longer needed, now that base/x release install its own locate dbs

No need for <netinet/in_systm.h> here.

bzero -> memset. for the speeds.

trim some casts

Remove stale comment which used to explain why we had a special 31-bit freelist
for DMA, back when this was applicable.

Fix memory leak.

OK from beck@ and miod@

KERNEL_ASSERT_LOCKED(9): Assertion for kernel lock (Rev. 3)

This adds a new assertion macro, KERNEL_ASSERT_LOCKED(), to assert that
kernel_lock is held.  In the long process of removing kernel_lock, there will
be a lot (hundreds or thousands) of use of this; virtually almost all functions
in !MP-safe subsystems should have this assertion.  Thus this assertion should
have a short, good name.

Not only that "KERNEL_ASSERT_LOCKED" is consistent with other KERNEL_* and
SCHED_ASSERT_LOCKED() macros.

Input from dlg@ guenther@ kettenis@.

OK dlg@ guenther@

OPENSSL_{malloc,free} -> {malloc,free}

Warn about the use of BUF_strdup.

Remove a debug message

Since the event(s) passed to a callback can be a mask of all events
of interest and EV_WRITE is or'ed into the interesting events, it
is more correct to check both events each time. Pointed out by
Claudio.

ok henning@ claudio@

Compatibility hack for the old "manpath=OpenBSD<blank>" query parameter format;
unfortunate, more than 400 links needing this are scattered all around
the www.openbsd.org website, and CVSweb needs this as well.