sync

For KARL, include a linkkit into the base set, as usr/share/compile.tgz.
This contains the relevant pieces from all the GENERIC* compile directories
(*.o ld.script Makefile gap.S).  It also includes the machine/ subdir for
now, to support re-randomizing of gap.S, though other methods are being
investigated.  (Any binutils ld.script hackers out there?)
collaboration with rpe

newbsd: target which re-links a kernel.  We cannot use the bsd: target
since it has source dependencies, and relinking doesn't have any.  Also
the bsd: target is created by config(8)...
worked on with rpe.

The KARL compile.tgz linkkit adds some growth to the base set.
Prepare the install*.fs files for this growth.

If the layout is empty except for requesting a left vertical frame,
record that detail in struct tbl_opts, such that term_tbl() can do
correct column calculations and doesn't prematurely break lines.
Fixes the tbl/layout/empty regression test that got broken when
line breaking in text block cells was implemented.

Kill unused #define IFI_NOMEDIA.

Rename 'packet' field to 'recv_packet' and 'bootrequest_packet'
field to 'sent_packet'. Also 'bootrequest_packet_length' to
'sent_packet_length'.

Adopt consistent idiom of a local variable 'packet' to point at
packet being manipulated.

No intentional functional change.

needs a fake ld.script for KARL tooling

Delete the arbitrary range restriction for -Owidth.
We provide users with tools.  We don't attempt to prevent them from
using them in stupid ways: depending on the context, not every
stupid-looking use is necessarily actually stupid, and not every
stupidity can be automatically detected anyway, so don't even try.

fix the interaction of the allbox option with spanned cells in the layout

Do not invalidate remote TLB entries with new mappings because the
virtual addresses really should not have TLB hits. This reduces IPIs.

Explicitly ignore .br, .ce, and .sp inside tbl(7) text blocks.
With the current code structure, they would appear at the wrong
place in the syntax tree, so it is better to not insert them
into the tree at all and issue an UNSUPP message instead.

Tweak order of operands to check constant `cache_valias_mask' first.

Do not require that unknown EXT_INFO extension values not contain
\0 characters. This would cause fatal connection errors if an
implementation sent e.g. string-encoded sub-values inside a value.

Reported by Denis Bider; ok markus@

when enabling cpu caches, check the item size against the right thing

lists of free items on the per cpu caches are built out the pool items
as struct pool_cache_items, not struct pool_cache. make the KASSERT
in pool_cache_init check that properly.

missing prototype.

Increase WSFONT_NAME_SIZE size, this will allow storing longer font
names in wsdisplay_font structures.

OK deraadt@

Remove xterm flag from key before checking prefix, reported by Peter
Fern in GitHub issue 974.

use size_t for the size of things in memory, not int.

this tweaks the len argument to sysctl_rdstring, sysctl_struct, and
sysctl_rdstruct.

there's probably more to fix.

ok millert@

Initialize the length parameter for radius_get_vs_raw_attr() since
it's read/write.  diff from IIJ.

Split early startup code out of locore.S into locore0.S.  Adjust link
run so that this locore0.o is always at the start of the executable.
But randomize the link order of all other .o files in the kernel, so
that their exec/rodata/data/bss segments land all over the place.

As a result, the internal layout of every newly build bsd kernel is
different from past kernels.  Internal relative offsets are not known
to an outside attacker.  The only known offsets are in the startup code
(which will be gone when it is smashed).

The bootstrap code unmapping or smashing isn't done yet, and there is
some potential for more bits from locore.S to move to locore0.S

add a dummy ld.script.  macppc does not use one of these yet, but
infrastructure that is coming requires an architecture to carry such a
file.

Prepare install_extint() for the vector being relative-negative (ie
lower in the address space).  Previously it could only be higher
than... uhm locore.o
Discussed with drahn and tom

First step of randomized link for sparc64:  gap.S and randomize files.
locore isn't split up yet, nor is the startup code smashed.
ok dlg, pieces seen before by kettenis

pledge(2) bpf has been in use for some time now on tcpdump(8), this will enable
it also for pflogd(8)'s priv proc.

OK deraadt@

have the timeout call if qstart via the serialiser isntead of directly

hfsc schedules a timeout to keep traffic moving if somethings has been
delayed an no other tx activity has occurred. that timeout was calling
(*ifp->if_qstart)(ifq) rather than ifq_start. the latter prevents
concurrent calls to if_qstart.

without this change bjorn ketelaars on misc@ was experiencing weird
pauses in traffic and lockups because the tx ring was corrupted
because re_start was run concurrently, once from the stack and once
from hfsc.

thanks to bjorn ketelaars for debugging, and mikeb@ for most of the
legwork in diagnosing the problem.
ok mikeb@

test the new allbox and wrapping features

Two minor fixes for the "allbox" modifier:
1. It does not reduce explicit "||" in the layout to "|".
2. It does not cause three horizontal lines at the end of a table,
even if the table ends with an explicit "_" data line.

If a tbl(7) layout contains a 'w' (minimum width) modifier for a
given column, that column contains no literal or numeric cell of
larger width, and all text block cells in that column can be line
wrapped to fit into that minimum width, groff does not increase
that column width beyond the specified minimum: so do the same.

Pledge is fairly done, so the kernel printf's can be converted to log()
calls.  They'll be a little less visible, but still in the system logs.
ok bluhm

/usr/games/lib doesn't exist any more, use the new path.

OK tb@ deraadt@

implement the tbl(7) "allbox" option;
used for example by curs_getch(3) and GLwDrawingArea(3)

fix column width calculation for text block cells

Use unlinkat() when removing the contents of a directory so we don't
need to construct paths.  OK deraadt@

Implement automatic line breaking
inside individual table cells that contain text blocks.
This cures overlong lines in various Xenocara manuals.

redundant, gets BINDIR from ../Makefile.inc

destroy lint remnants.
okay millert@ deraadt@

Use openat() and unlinkat() instead of chdir()ing to the recovery dir.
Since we use flock() and not fcntl() locking we can open the recovery
file read-only.  OK martijn@

Add dprintf() and vdprintf() RETURN VALUES.  OK deraadt@

Backout last change to remove esym .data patching because it breaks
boot for e.g. the BBB and the C.H.I.P.; 'panic: initarm: out of memory'.
Better solution to be found as discussed with patrick@.

ok jsg@

Add some tests from kshe48@zoho.com. Currently failing, but will be fixed soon.

Limit the maximum size of softraid crypto volumes that can be created to
the size that is currently supported (a data area of 16TB, or
2^32 * 32 + 528 blocks including the softraid metadata/boot area), rather
than successfully creating the volume and then failing during resource
allocation (without properly reporting why it failed).

Found the hard way by sharon s. <mymlact at gmx dot com>

Upon failure, only return zero from the softraid ioctl handler when there
is at least one bio status message. It is a much nicer user experience when
we properly report failures, however in the case that we fail to do this,
getting an errno back from the ioctl is somewhat better than simply
printing "unknown error" from bioctl(8).

Fix another long standing softraid crypto bug where if all 32 keys are in
use, when freeing crypto sessions we run straight off the end of the array
and start blatting memory - clearly no one has a softraid crypto volume
that exceeds 15.5TB in size...

Factor out and reuse the code that frees the crypto sessions, instead of
duplicating it.

Fix a long standing off-by-one bug in the softraid crypto number of keys
calculation - we allow one key per 0.5TB, which should allow up to 16TB
disks, however the disk size was treated like a block offset and shifted.
This meant that the maximum size was actually 16TB minus one block.

While here also calculate the number of keys as an absolute value, rather
than as the upper inclusive bound - adjust the logic in the associated for
and if statements to match.

ok krw@ tb@

rejname[] is also -r option buffer, and should be PATH_MAX
ok millert

specify free() size from the old allocation, not new allocation.
fix from C Turt

device path should be PATH_MAX.  Any NAME_MAX without +1 is suspect
to begin with anyways.

ifq_start does OACTIVE and RUNNING checks for the driver

Reminded by dlg@.

Need to flush out the linefeed after wrapper. GitHub issue 970.

Add explicit keys for the bracketed paste sequences, both to avoid mix
ups with other keys and to make logs clearer.

typo; from bryan vyhmeister

From FreeBSD (r227593, r307982):
More and more RealTek controllers started to implement EEE feature.
Vendor driver seems to load a kind of firmware for EEE with
additional PHY fixups.  It is known that the EEE feature may need
ASPM support.  Unfortunately there is no documentation for EEE of
the controller so enabling ASPM may cause more problems.

The Realtek vendor drivers for FreeBSD and Linux also disable ASPM and
clock request.  While here, add a define for the ECPM (Enable Clock Power
Management) bit.

Tested by stsp@ and myself.
ok stsp@

make vers.o not depend on gap.o using a trick, because it is piece of
fiction participating only in the linkphase.
tricks from rpe and espie

Use umask 077 instead of cp -p when installing the kernel as root.
Otherwise owner of /bsd could be the build user with permissions
inherited from the compile directory.
OK deraadt@ tb@

ooops, fix a glitch in the previous commit...

Use a common 'goto bad' style and set mp to NULL after freeing it
in ipip_input_gif().  This prevents a use-after-free if there is a
bug in the IP input functions.
OK mpi@

churn caused by the new Mdocdate messages, no easy way to avoid this :(

Fix a test race, wait after kill in case writing the core dump takes
a while.  Adjust wrong comments.  Mention regress- in program name
to make clear where violations seen in process accounting happend.

Style message about legacy man(7) date format in mdoc(7) documents
and operating system dependent messages about missing or unexpected
Mdocdate; inspired by mdoclint(1).

request -> promises; from scott cheloha

usr/bin/ktrace: replace snprintf(3)/write(2) with dprintf(3)

Brought to attention by BlackFrog on #openbsd-daily

OK deraadt@

style message about missing .Fn markup; inspired by mdoclint

missing .Fn macros; found with mandoc -Tlint

missing .Fn markup for main(); found with mandoc -Tlint

Add missing .Fn markup to in() and out(); found with mandoc -Tlint.
Delete useless \*(Gt and \*(Lt while here.

Continue the flattening of the pledge logic started in r1.184 and place
a blank space somewhere else.

suggested by and ok jsing

Do not issue the message "no blank before trailing delimiter" for .No.
In practice, that message only matters inside .Bf, and even there, it
can occasionally be a false positive.  In all other cases, it usually
is a false positive, so it is better to drop it outright.
Suggested by jmc@.

make two macros more semantic

Simple style(9) fixes from Juuso Lapinlampi, mostly whitespace and
omitting parentheses in return statements. Binary change because of
return instead of exit(3) from main and because help() is now __dead.

ok awolk

Define register_splx_handler() in one place.

Fix TLB size computation on OCTEON II and III. The CPUs have utilized
the whole TLB space even before this. However, TLB initialization on
boot and TLB flush on ASID wraparound have been incomplete. These have
caused crashes of processes.

Add a memory barrier to ensure that stores become visible
in a proper order.

integer overflow for two range checks
fix from C Turt, ok miod

If -P and -c were given, a second pledge call tried to add "rpath" to the
first pledge promises, so nc exited with EPERM. To fix this, merge the
pledge of the Pflag && usetls case into the first pledge block. This
allows us to get rid of the second pledge block and thus to simplify the
logic a bit. While there, add a missing blank to an error string.

Joint effort by the #openbsd-daily code reading group, problem found and
initial patch by <rain1 openmailbox org>.

ok awolk

Nuke unused field bootp_policy and associated enum{}.

Nuke unused global warnings_occurred.

Reduce false positives for the "no blank before trailing delimiter" message.
This brings us down to one false positive for about every 18 pages.

minor markup simplifications

improve semantic markup for __attribute__

missing blank before full stop, found with mandoc -Tlint

add missing blanks before several delimiters; found with mandoc -Tlint

missing space before trailing punctuation; found with mandoc -Tlint

remove lots of bogus escaping, some of which even broke the output;
found with mandoc -Tlint

fix imprecise .Fa markup; found with mandoc -Tlint

Fix broken markup of function pointer invocations; found
with mandoc -Tlint.  While here, delete .Tn macros.

fix broken markup of callback arguments; found with mandoc -Tlint

Fix broken escaping: "\." is almost never what you want; found with
mandoc -Tlint.  While here, make macro usage more consistent.

fix a sentence that used unusual terminology, the wrong macro,
and broken delimiter syntax; found with mandoc -Tlint

repair broken markup of callback argument; found with mandoc -Tlint

Pass M_CANFAIL to malloc(9) calls which use M_WAITOK but are tested
for failure.

ok armani@

Use software interrupt to process TX/RX data between sio and tty(4)
layer.

This is a straightforwad port of current NetBSD/luna68k implementation
by Izumi Tsutsui.  Tested on both LUNA-88K and LUNA-88K2.

Report processes that were killed due to pledge or memory access
violations in the daily mail.
OK millert@ jmc@

perl(1)'s ...

use mandoc annotations for flags and such

Don't describe AppleTalk's output format and bugs.

OK claudio@
jmc@ doesn't object

print '?' instead of incorrect link count (that the protocol doesn't
provide) for remote listings. bz#2710 ok dtucker@

implement sorting for globbed ls; bz#2649 ok dtucker@

style message about missing blank before trailing delimiter;
inspired by mdoclint(1), and jmc@ considers it useful