CVS: cvs.openbsd.org: src

2018-01-25 Thread Carlos Cardenas
CVSROOT:/cvs
Module name:src
Changes by: ccarde...@cvs.openbsd.org   2018/01/25 22:34:41

Modified files:
usr.sbin/vmd   : config.c 

Log message:
Fix Failure on Start

When attempting to start a vm from vm.conf that fails due to inadequate
resources and such, do not remote vm entry from vm list.

Reported by mpi@

ok mpi@



CVS: cvs.openbsd.org: src

2018-01-25 Thread Kenneth R Westerback
CVSROOT:/cvs
Module name:src
Changes by: k...@cvs.openbsd.org2018/01/25 08:43:52

Modified files:
sbin/dhclient  : dhclient.c dhcpd.h dispatch.c packet.c 

Log message:
Add DPRINTF() and use it to replace log_debug() statements inside
#ifdef DEBUG/#endif blocks.

Suggestions on correct idiom (C99 vs gcc) from millert@

ok tom@



CVS: cvs.openbsd.org: src

2018-01-25 Thread Martin Pieuchot
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2018/01/25 08:19:33

Removed files:
sys/arch/alpha/alpha: mutex.c 
sys/arch/amd64/amd64: mutex.c 
sys/arch/arm64/arm64: arm64_mutex.c 
sys/arch/i386/i386: mutex.c 
sys/arch/mips64/mips64: mutex.c 
sys/arch/powerpc/powerpc: mutex.c 

Log message:
Remove mutex implementations that now live in MI code.



CVS: cvs.openbsd.org: src

2018-01-25 Thread Martin Pieuchot
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2018/01/25 08:06:29

Modified files:
sys/kern   : kern_lock.c 
sys/arch/alpha/conf: files.alpha 
sys/arch/amd64/conf: files.amd64 
sys/arch/arm64/conf: files.arm64 
sys/arch/i386/conf: files.i386 
sys/arch/mips64/conf: files.mips64 
sys/arch/powerpc/conf: files.powerpc 
sys/arch/alpha/include: mutex.h 
sys/arch/amd64/include: mutex.h 
sys/arch/arm64/include: mutex.h 
sys/arch/i386/include: mutex.h 
sys/arch/mips64/include: mutex.h 
sys/arch/powerpc/include: mutex.h 
sys/sys: mutex.h 

Log message:
Move common mutex implementations to a MI place.

Archs not yet converted can to the jump by defining __USE_MI_MUTEX.

ok visa@



CVS: cvs.openbsd.org: src

2018-01-25 Thread Martin Pieuchot
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2018/01/25 07:47:35

Modified files:
sys/netinet: ip_carp.c 

Log message:
Use a workaround for detached parent in carp_proto_input_c().

A NULL dereference can happen since processing protocol layer is
deffered to a second task.  In other words the NET_LOCK() is released
then regrabbed between ip_input() and carp_proto_input().

The same workaround is already in use in carp_output() due to deffered
processing in case of IPsec.

The real fix is to make carp(4) MP-safe and use if_get(9) there, any
taker?

Found & fix tested by Hrvoje Popovski.



CVS: cvs.openbsd.org: src

2018-01-25 Thread Martin Pieuchot
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2018/01/25 07:09:52

Modified files:
sys/arch/i386/conf: Makefile.i386 
sys/conf   : makegap.sh 

Log message:
Use a new LDFLAGS variable to pass "-melf_i386_obsd" on amd64.

Makes the linker generate a correct i386 gap.o file.

ok kettenis@, deraadt@



CVS: cvs.openbsd.org: src

2018-01-25 Thread Martin Pieuchot
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2018/01/25 07:04:36

Modified files:
sys/net: ifq.c 

Log message:
Assert that ifiq_destroy() is not called with the NET_LOCK() held.

Calling taskq_barrier() on a softnet thread while holding the lock
is clearly a deadlock.

ok visa@, dlg@, bluhm@