OpenBSD cvs log

created 2019-05-04T02:17:02Z
begin 2017-10-12T00:00:00Z
end 2017-10-13T00:00:00Z
path src/sys
commits 7

date 2017-10-12T07:24:46Z
author anton
files src/sys/dev/acpi/dsdt.c log diff annotate
message Add missing size to free(); ok deraadt@ mpi@

date 2017-10-12T09:03:15Z
author mpi
files src/sys/sys/exec_elf.h log diff annotate
message Add Section Attirbute flag defines, some default values for n_type
and a couple of missing EM_* required to build devel/libdwarf and
devel/valgrind.

ok deraadt@, jasper@

date 2017-10-12T09:10:52Z
author mpi
files src/sys/net/if.c log diff annotate
message Fix a singed vs unsigned comparison resulting in an overflow of the
routing socket.

When clang became the default compiler, `if_flags' from `struct ifnet'
was changed from "short" to "unsigned short", to silence a warning.
Sadly the copy of these flags on the stack was still a "short" which
made the flags comparison always true, which in turn made ifioctl()
generates a RTM_INFO message for many ioctl(2).

Since my last commit, the flag comparison is done for every ioctl(2).
This made the kernel generate at least one routing message per ioctl,
resulting in a lot of RTM_DESYNC.

RTM_DESYNC problem reported by krw@ thanks to dhclient(8)'s noisiness!

ok krw@, patrick@

date 2017-10-12T09:14:16Z
author mpi
files src/sys/kern/uipc_mbuf.c log diff annotate
src/sys/net/if.c log diff annotate
src/sys/net/if_var.h log diff annotate
src/sys/sys/sysctl.h log diff annotate
message Move sysctl_mq() where it can safely mess with mbuf queue internals.

ok visa@, bluhm@, deraadt@

date 2017-10-12T09:19:45Z
author mpi
files src/sys/kern/kern_rwlock.c log diff annotate
message Use a temporary variable in rw_status() to dereference only once the
volatile member of the struct.

Not forcing a memory read on every access, 3 in this function, might
reduce cache traffic in some cases.

Micro-optimization and diff provided by Mateusz Guzik.

ok visa@

date 2017-10-12T11:54:37Z
author patrick
files src/sys/dev/sdmmc/sdmmc_io.c log diff annotate
message Apparently one of the main concepts in the SDMMC I/O subsystem is that
the driver attached to an SDIO card always holds the lock and only
releases it once it detaches. Now with that in mind, some time ago
sdmmc_io_function_disable() and sdmmc_io_function_ready() were changed
to only assert the write lock and not take it, but not all of the tree
was converted. Change sdmmc_io_function_enable() as well, and remove
the enter/exit dance in the interrupt code. Apparently there is no
SDIO driver yet/anymore which would trigger those issues.

ok kettenis@

date 2017-10-12T15:04:33Z
author bluhm
files src/sys/kern/kern_pledge.c log diff annotate
message Print the word pledge in the kernel log when there is a violation.
This should make it easier to figure out what is going on. Note
that the pledgecode it shows is only a guess which pledge(2) might
help.
OK deraadt@ semarie@