CVS: cvs.openbsd.org: src

2017-12-13 Thread Jason McIntyre
CVSROOT:/cvs
Module name:src
Changes by: j...@cvs.openbsd.org2017/12/13 23:43:44

Modified files:
share/man/man9 : cond_init.9 

Log message:
zap trailing whitespace;



CVS: cvs.openbsd.org: src

2017-12-13 Thread Theo de Raadt
CVSROOT:/cvs
Module name:src
Changes by: dera...@cvs.openbsd.org 2017/12/13 23:21:04

Modified files:
sys/dev/ata: wd.c 

Log message:
Repair flushcache/standby operations to not convert AT_WAIT operations
to AT_POLL, which some chip driver underneath cannot handle.
tested by bluhm also



CVS: cvs.openbsd.org: src

2017-12-13 Thread Philip Guenther
CVSROOT:/cvs
Module name:src
Changes by: guent...@cvs.openbsd.org2017/12/13 20:30:43

Modified files:
sys/arch/alpha/alpha: mem.c 
sys/arch/amd64/amd64: mem.c 
sys/arch/arm/arm: mem.c 
sys/arch/arm64/arm64: mem.c 
sys/arch/hppa/hppa: mem.c 
sys/arch/i386/i386: mem.c 
sys/arch/m88k/m88k: mem.c 
sys/arch/macppc/macppc: mem.c 
sys/arch/mips64/mips64: mem.c 
sys/arch/sh/sh : mem.c 
sys/arch/socppc/socppc: mem.c 
sys/arch/sparc64/sparc64: mem.c 

Log message:
fcntl(F_SETFL) invokes the FIONBIO and FIOASYNC ioctls internally, so
the memory devices (/dev/null, /dev/zero, etc) need to permit them.

problem noted, tweak, and testing by jeremy@
ok deraadt@



CVS: cvs.openbsd.org: src

2017-12-13 Thread David Gwynne
CVSROOT:/cvs
Module name:src
Changes by: d...@cvs.openbsd.org2017/12/13 19:42:18

Modified files:
sys/kern   : kern_timeout.c 

Log message:
i forgot to convert timeout_proc_barrier to cond_signal



CVS: cvs.openbsd.org: src

2017-12-13 Thread David Gwynne
CVSROOT:/cvs
Module name:src
Changes by: d...@cvs.openbsd.org2017/12/13 19:40:51

Modified files:
sys/net: ifq.c 

Log message:
i forgot to convert ifq_barrier_task to cond_signal.



CVS: cvs.openbsd.org: src

2017-12-13 Thread David Gwynne
CVSROOT:/cvs
Module name:src
Changes by: d...@cvs.openbsd.org2017/12/13 17:45:16

Modified files:
sys/kern   : kern_task.c kern_timeout.c 
sys/net: ifq.c 

Log message:
replace the bare sleep state handling in barriers with wait cond code



CVS: cvs.openbsd.org: src

2017-12-13 Thread David Gwynne
CVSROOT:/cvs
Module name:src
Changes by: d...@cvs.openbsd.org2017/12/13 17:41:58

Modified files:
sys/sys: proc.h systm.h 
sys/kern   : kern_synch.c 
share/man/man9 : Makefile 
Added files:
share/man/man9 : cond_init.9 

Log message:
add code to provide simple wait condition handling.

this will be used to replace the bare sleep_state handling in a
bunch of places, starting with the barriers.



CVS: cvs.openbsd.org: src

2017-12-13 Thread Kenneth R Westerback
CVSROOT:/cvs
Module name:src
Changes by: k...@cvs.openbsd.org2017/12/13 11:53:04

Modified files:
sbin/dhclient  : clparse.c 

Log message:
Fix whitespace botch that snuck in.



CVS: cvs.openbsd.org: src

2017-12-13 Thread Kenneth R Westerback
CVSROOT:/cvs
Module name:src
Changes by: k...@cvs.openbsd.org2017/12/13 11:45:08

Modified files:
sbin/dhclient  : dhclient.conf.5 clparse.c 

Log message:
Simplifiy parser by not worrying about nesting 'interface  {}'
statements. They are silly but harmless and work fine.



CVS: cvs.openbsd.org: xenocara

2017-12-13 Thread Ingo Schwarze
CVSROOT:/cvs
Module name:xenocara
Changes by: schwa...@cvs.openbsd.org2017/12/13 10:45:49

Modified files:
app/xterm  : XTerm.ad 

Log message:
Disable Unicode canonical composition by default because that's
less surprising and makes copy and paste work better, even though
some none-default fonts currently do not display combining accents.

Problem reported by Philippe Meunier .
Root cause found by bentley@.
OK bentley@, and the general direction was also supported by stsp@.



CVS: cvs.openbsd.org: src

2017-12-13 Thread Bob Beck
CVSROOT:/cvs
Module name:src
Changes by: b...@cvs.openbsd.org2017/12/13 09:38:34

Modified files:
sys/ufs/ffs: ffs_softdep.c 

Log message:
Fix a softdep bug exposed by our recent changes to make reboot drop to read-only

The deadlock happens when softdep gets the same buffer in the BMSAFEMAP case
that it already called getdirtybuf() on and made busy at the top of the loop.
when this is the case, skip the BMSAFEMAP case and simply write the buffer
out at the bottom of the loop as always. This avoids calling getdirtybuf()
a second time on the same buffer we already took for exclusive use ourself
and have not yet written out.

While I'm in here add a KASSERT for the similar case above, which I don't
think can happen but we would deadlock in the same way if it does.

testing by and ok bluhm@



CVS: cvs.openbsd.org: src

2017-12-13 Thread Todd C . Miller
CVSROOT:/cvs
Module name:src
Changes by: mill...@cvs.openbsd.org 2017/12/13 09:07:54

Modified files:
usr.bin/sed: compile.c 

Log message:
Fix array index by signed char; from martijn@



CVS: cvs.openbsd.org: src

2017-12-13 Thread Todd C . Miller
CVSROOT:/cvs
Module name:src
Changes by: mill...@cvs.openbsd.org 2017/12/13 09:06:34

Modified files:
usr.bin/sed: compile.c defs.h main.c process.c 

Log message:
Fix sign compare warnings; OK martijn@



CVS: cvs.openbsd.org: src

2017-12-13 Thread Todd C . Miller
CVSROOT:/cvs
Module name:src
Changes by: mill...@cvs.openbsd.org 2017/12/13 09:06:05

Modified files:
usr.bin/sed: Makefile 

Log message:
Enable compiler warnings; OK martijn@



CVS: cvs.openbsd.org: xenocara

2017-12-13 Thread Okan Demirmen
CVSROOT:/cvs
Module name:xenocara
Changes by: o...@cvs.openbsd.org2017/12/13 08:10:17

Modified files:
app/cwm: search.c 

Log message:
move variable declarations up, to match everything else



CVS: cvs.openbsd.org: src

2017-12-13 Thread Helg Bredow
CVSROOT:/cvs
Module name:src
Changes by: h...@cvs.openbsd.org2017/12/13 05:30:18

Modified files:
lib/libfuse: Makefile debug.c debug.h fuse.c 

Log message:
Add support for the -d (-o debug) option to libfuse. Currently works for FUSE
file systems that call fuse_parse_cmdline(3), fuse_main(3) or
fuse_setup(3).

ok mpi@, jca@



CVS: cvs.openbsd.org: src

2017-12-13 Thread Helg Bredow
CVSROOT:/cvs
Module name:src
Changes by: h...@cvs.openbsd.org2017/12/13 05:23:48

Modified files:
lib/libfuse: fuse_opt.c 
regress/lib/libfuse: fuse-opt-parse.c 

Log message:
If the list of templates 'o' passed to fuse_opt_parse(3) is NULL then
the processing function should be called in the same way as if no match
was found.

ok mpi@



CVS: cvs.openbsd.org: src

2017-12-13 Thread Mark Kettenis
CVSROOT:/cvs
Module name:src
Changes by: kette...@cvs.openbsd.org2017/12/13 03:38:06

Modified files:
sys/dev/fdt: if_dwxe.c 

Log message:
Fix stray return; hopefully this makes the internal PHY actually work now.



CVS: cvs.openbsd.org: src

2017-12-13 Thread Martin Pieuchot
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2017/12/13 01:59:02

Modified files:
sys/net: rtsock.c 

Log message:
Spacing from ksh...@zoho.eu.



CVS: cvs.openbsd.org: src

2017-12-13 Thread Martin Pieuchot
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2017/12/13 01:34:04

Modified files:
sys/ddb: db_command.c 

Log message:
Add 'bt' an alias for 'trace'.

ok pirofti@



CVS: cvs.openbsd.org: src

2017-12-13 Thread Patrick Wildt
CVSROOT:/cvs
Module name:src
Changes by: patr...@cvs.openbsd.org 2017/12/13 01:27:06

Modified files:
sbin/iked  : util.c 

Log message:
getsockname(2) needs to be passed the length of the input struct.

ok jca@