OpenBSD src changes summary for 2017-07-09 to 2017-07-16 inclusive
==================================================================

bin/csh                                 bin/date
bin/expr                                bin/test
distrib/miniroot                        distrib/sets
distrib/syspatch                        etc/daily
etc/ksh.kshrc                           games/atc
games/backgammon                        games/fortune
games/grdc                              games/hack
gnu/lib/libiberty                       gnu/usr.bin/cc
gnu/usr.bin/clang                       gnu/usr.bin/gcc
lib/csu                                 lib/libc
lib/libcrypto                           lib/libcurses
lib/libedit                             lib/libevent
lib/libpcap                             lib/libssl
regress/lib                             regress/libexec
regress/sbin                            regress/sys
regress/usr.bin                         regress/usr.sbin
sbin/atactl                             sbin/bioctl
sbin/dhclient                           sbin/growfs
sbin/isakmpd                            sbin/mount
sbin/mount_vnd                          sbin/pfctl
sbin/route                              sbin/scan_ffs
sbin/slaacd                             sbin/wsconsctl
share/man                               share/mk
sys/arch/alpha/alpha                    sys/arch/alpha/conf
sys/arch/amd64/amd64                    sys/arch/amd64/conf
sys/arch/amd64/include                  sys/arch/arm/include
sys/arch/arm64/conf                     sys/arch/armv7/conf
sys/arch/hppa/conf                      sys/arch/hppa/hppa
sys/arch/hppa/include                   sys/arch/i386/conf
sys/arch/i386/i386                      sys/arch/i386/include
sys/arch/landisk/conf                   sys/arch/loongson/conf
sys/arch/loongson/loongson              sys/arch/macppc/conf
sys/arch/mips64/include                 sys/arch/octeon/conf
sys/arch/octeon/dev                     sys/arch/octeon/include
sys/arch/sgi/conf                       sys/arch/sparc64/conf
sys/conf                                sys/dev/fdt
sys/dev/ic                              sys/dev/pci
sys/dev/pv                              sys/dev/wscons
sys/kern                                sys/net
sys/netinet                             sys/netinet6
sys/sys                                 usr.bin/awk
usr.bin/cap_mkdb                        usr.bin/doas
usr.bin/encrypt                         usr.bin/fmt
usr.bin/gencat                          usr.bin/htpasswd
usr.bin/keynote                         usr.bin/less
usr.bin/m4                              usr.bin/make
usr.bin/mandoc                          usr.bin/mklocale
usr.bin/nc                              usr.bin/newsyslog
usr.bin/openssl                         usr.bin/pkill
usr.bin/rdist                           usr.bin/signify
usr.bin/ssh                             usr.bin/tcpbench
usr.bin/tee                             usr.bin/time
usr.bin/tmux                            usr.bin/true
usr.bin/tsort                           usr.bin/usbhidaction
usr.bin/xinstall                        usr.bin/ypwhich
usr.sbin/config                         usr.sbin/dhcpd
usr.sbin/ndp                            usr.sbin/npppd
usr.sbin/rebound                        usr.sbin/relayd
usr.sbin/rpc.bootparamd                 usr.sbin/rpc.lockd
usr.sbin/rpc.statd                      usr.sbin/rtadvd
usr.sbin/smtpd                          usr.sbin/vipw
usr.sbin/vmctl                          usr.sbin/vmd
usr.sbin/ypldap                         usr.sbin/ypserv

== bin =============================================================== 01/12 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/bin

csh

  ~ Makefile                              

  > remove misc. depend and yacc nits that no longer matter.
  > okay millert@ (espie@)

date

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

expr

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

test

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

== distrib =========================================================== 02/12 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/distrib

miniroot

  ~ install.sub                           

  > Rework vlan configuration in the installer, triggered by the vlan
  > releated changes in ifconfig. Put the vlan configuration into its
  > own function vlan_config() which is called from configure_ifs().
  > NOTE: Depending on how you use autoinstall(8) related to vlans
  > (espcially if you rely on the default answers) you should check
  > that everything still works.
  > Prompted by and OK dlg@
  > 'looks good' benno@
  > positive feedback tb@ deraadt@ (rpe@)

  ~ install.sub                           

  > - remove superfluous blanks from v[46]_info() output
  > - tweak comments
  > Based on a diff from and discussions with Klemens Nanni. (rpe@)

  ~ install.sub                           

  > add comment (rpe@)

sets

  ~ lists/man/mi                          

  > sync (visa@)

syspatch

  ~ bsd.syspatch.mk                       

  > Set XSRCDIR to /usr/xenocara if it's not defined. Matches what
  > /usr/X11R6/share/mk/bsd.xorg.mk does.
  > ok robert@ (ajacoutot@)

  ~ bsd.syspatch.mk                       

  > use the previous syspatch's fake root for getting the link order
  > instead of using / where the libraries are re-ordered on reboot
  > ok aja@ (robert@)

== etc =============================================================== 03/12 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/etc

daily

  ~ daily                                 

  > Test if an acct file exists before trying to rename it.  This silences
  > false warnings in the frist three daily mails after process accounting
  > has been turned on.
  > from Raf Czlonka (bluhm@)

ksh.kshrc

  ~ ksh.kshrc                             

  > Replace some `eval` with $(eval) for consistency and use
  > parameter expansion instead of basename(1). From Raf Czlonka
  > ok rpe (tb@)

== games ============================================================= 04/12 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/games

atc

  ~ lex.l                                 

  > have bsd.prog.mk/bsd.lib.mk generate .y -> .c/.h rules without intermediate
  > files. This fixes up parallel builds in the default case.
  > FreeBSD does something similar.
  > okay millert@ (espie@)

backgammon

  ~ backgammon/main.c                     

  > User count checks have been compiled out since 1998, get rid of them.
  > OK mlarkin@ (fcambus@)

fortune

  - tools/Do_spell                        - tools/Do_troff
  - tools/Troff.mac                       - tools/Troff.sed
  - tools/do_sort                         

  > Remove old junk. From Matthew Martin
  > ok bentley (tb@)

  ~ fortune/fortune.6                     ~ fortune/fortune.c

  > Inspect LC_CTYPE and if it isn't UTF-8, weed out bytes that are not
  > printable ASCII.  That makes using UTF-8 in fortune datfiles safe.
  > Potential usefulness of UTF-8 in fortune datfiles noticed by bentley@.
  > OK tedu@ millert@. (schwarze@)

  ~ Notes                                 ~ datfiles/fortunes
  ~ datfiles/fortunes-o.real              

  > Convert fortunes to UTF-8 from the old "backspace punctuation" method.
  > suggested by tedu, ok schwarze@ stsp@ tb@ (bentley@)

  ~ datfiles/fortunes                     ~ datfiles/fortunes2
  ~ datfiles/fortunes2-o                  ~ datfiles/limerick

  > Use an umlaut in Mobius and uppercase one instance of möbius visible in
  > bentley's previous commit. (tb@)

grdc

  ~ grdc.c                                

  > Convert fprintf(stderr, ...) to errx/warnx (with slight change of output).
  > From Scott Cheloha (tb@)

hack

  ~ Makefile                              

  > remove misc. depend and yacc nits that no longer matter.
  > okay millert@ (espie@)

== gnu =============================================================== 05/12 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/gnu

lib/libiberty

  ~ Makefile.bsd-wrapper                  

  > remove misc. depend and yacc nits that no longer matter.
  > okay millert@ (espie@)

usr.bin/cc

  ~ cc1plus/Makefile                      ~ cc_tools/Makefile
  ~ libobjc/Makefile                      

  > remove misc. depend and yacc nits that no longer matter.
  > okay millert@ (espie@)

usr.bin/clang

  ~ clang/Makefile                        ~ clang-tblgen/Makefile
  ~ include/clang/AST/Makefile            ~ include/clang/Basic/Makefile
  ~ include/clang/Driver/Makefile         ~ include/clang/Parse/Makefile
  ~ include/clang/Sema/Makefile           
  ~ include/clang/Serialization/Makefile
  ~ include/clang/intrin/Makefile         ~ include/llvm/AArch64/Makefile
  ~ include/llvm/ARM/Makefile             ~ include/llvm/Config/Makefile
  ~ include/llvm/IR/Makefile              ~ include/llvm/Mips/Makefile
  ~ include/llvm/PowerPC/Makefile         ~ include/llvm/Sparc/Makefile
  ~ include/llvm/X86/Makefile             ~ libLLVMAArch64AsmParser/Makefile
  ~ libLLVMAArch64AsmPrinter/Makefile     ~ libLLVMAArch64CodeGen/Makefile
  ~ libLLVMAArch64Desc/Makefile           ~ libLLVMAArch64Disassembler/Makefile
  ~ libLLVMAArch64Info/Makefile           ~ libLLVMAArch64Utils/Makefile
  ~ libLLVMARMAsmParser/Makefile          ~ libLLVMARMAsmPrinter/Makefile
  ~ libLLVMARMCodeGen/Makefile            ~ libLLVMARMDesc/Makefile
  ~ libLLVMARMDisassembler/Makefile       ~ libLLVMARMInfo/Makefile
  ~ libLLVMAnalysis/Makefile              ~ libLLVMAsmParser/Makefile
  ~ libLLVMAsmPrinter/Makefile            ~ libLLVMBitReader/Makefile
  ~ libLLVMBitWriter/Makefile             ~ libLLVMCodeGen/Makefile
  ~ libLLVMCore/Makefile                  ~ libLLVMCoroutines/Makefile
  ~ libLLVMCoverage/Makefile              ~ libLLVMDebugInfoCodeView/Makefile
  ~ libLLVMDebugInfoDWARF/Makefile        ~ libLLVMDebugInfoMSF/Makefile
  ~ libLLVMDemangle/Makefile              ~ libLLVMGlobalISel/Makefile
  ~ libLLVMIRReader/Makefile              ~ libLLVMInstCombine/Makefile
  ~ libLLVMInstrumentation/Makefile       ~ libLLVMLTO/Makefile
  ~ libLLVMLinker/Makefile                ~ libLLVMMC/Makefile
  ~ libLLVMMCDisassembler/Makefile        ~ libLLVMMCParser/Makefile
  ~ libLLVMMIRParser/Makefile             ~ libLLVMMipsAsmParser/Makefile
  ~ libLLVMMipsAsmPrinter/Makefile        ~ libLLVMMipsCodeGen/Makefile
  ~ libLLVMMipsDesc/Makefile              ~ libLLVMMipsDisassembler/Makefile
  ~ libLLVMMipsInfo/Makefile              ~ libLLVMObjCARCOpts/Makefile
  ~ libLLVMObject/Makefile                ~ libLLVMOption/Makefile
  ~ libLLVMPasses/Makefile                ~ libLLVMPowerPCAsmParser/Makefile
  ~ libLLVMPowerPCAsmPrinter/Makefile     ~ libLLVMPowerPCCodeGen/Makefile
  ~ libLLVMPowerPCDesc/Makefile           ~ libLLVMPowerPCDisassembler/Makefile
  ~ libLLVMPowerPCInfo/Makefile           ~ libLLVMProfileData/Makefile
  ~ libLLVMScalarOpts/Makefile            ~ libLLVMSelectionDAG/Makefile
  ~ libLLVMSparcAsmParser/Makefile        ~ libLLVMSparcAsmPrinter/Makefile
  ~ libLLVMSparcCodeGen/Makefile          ~ libLLVMSparcDesc/Makefile
  ~ libLLVMSparcDisassembler/Makefile     ~ libLLVMSparcInfo/Makefile
  ~ libLLVMSupport/Makefile               ~ libLLVMTableGen/Makefile
  ~ libLLVMTarget/Makefile                ~ libLLVMTransformUtils/Makefile
  ~ libLLVMVectorize/Makefile             ~ libLLVMX86AsmParser/Makefile
  ~ libLLVMX86AsmPrinter/Makefile         ~ libLLVMX86CodeGen/Makefile
  ~ libLLVMX86Desc/Makefile               ~ libLLVMX86Disassembler/Makefile
  ~ libLLVMX86Info/Makefile               ~ libLLVMX86Utils/Makefile
  ~ libLLVMipo/Makefile                   ~ libclangAST/Makefile
  ~ libclangAnalysis/Makefile             ~ libclangBasic/Makefile
  ~ libclangCodeGen/Makefile              ~ libclangDriver/Makefile
  ~ libclangEdit/Makefile                 ~ libclangFrontend/Makefile
  ~ libclangFrontendTool/Makefile         ~ libclangLex/Makefile
  ~ libclangParse/Makefile                ~ libclangRewrite/Makefile
  ~ libclangRewriteFrontend/Makefile      ~ libclangSema/Makefile
  ~ libclangSerialization/Makefile        ~ liblldConfig/Makefile
  ~ liblldCore/Makefile                   ~ liblldELF/Makefile
  ~ lld/Makefile                          ~ llvm-tblgen/Makefile

  > zap empty depends/silence empty installs.
  > okay millert@ (espie@)

usr.bin/gcc

  ~ Makefile.bsd-wrapper                  

  > protoize.1 isn't generated, so don't list it as a target, because it
  > fucks up make recognition of generated vs pre-existing file.
  > also zap .PATH.1   thingy that's no longer needed at all. (espie@)

== lib =============================================================== 06/12 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/lib

csu

  ~ Makefile                              

  > remove misc. depend and yacc nits that no longer matter.
  > okay millert@ (espie@)

libc

  ~ sys/kqueue.2                          

  > replace Fifos with FIFOs.
  > ok jmc@ visa@ (dlg@)

  ~ stdlib/malloc.c                       

  > one more instance of the previous commit; also initialize ->offset to a
  > definite value in the size == 0 case (otto@)

  ~ gen/sysctl.3                          

  > Get rid of ip6.maxifprefixes and ip6.maxifdefrouters, the kernel no
  > longer tracks prefixes or default routers from router advertisements.
  > Pointed out by jmc.
  > ports tree grepping sthen, who only found nsh
  > OK mpi, sthen (florian@)

libcrypto

  ~ Makefile                              

  > remove misc. depend and yacc nits that no longer matter.
  > okay millert@ (espie@)

libcurses

  ~ Makefile                              

  > remove misc. depend and yacc nits that no longer matter.
  > okay millert@ (espie@)

libedit

  ~ Makefile                              

  > remove misc. depend and yacc nits that no longer matter.
  > okay millert@ (espie@)

libevent

  ~ kqueue.c                              

  > if there are no changes for kevent, pass in NULL instead.
  > this has no effect except to make ktrace output prettier.
  > ok bluhm mpi (tedu@)

libpcap

  ~ Makefile                              ~ scanner.l

  > use working boilerplate for yacc/lex instead of homemade rules.
  > okay millert@
  > (forgot the obvious scanner.l tweak in my diff) (espie@)

libssl

  ~ s3_lib.c                              

  > Remove unused variable.
  > Reported by <dravion at ht-foss dot net> (jsing@)

  ~ Makefile                              ~ t1_lib.c
  + ssl_tlsext.c                          + ssl_tlsext.h

  > Start rewriting TLS extension handling.
  > Introduce a TLS extension handling framework that has per-extension type
  > functions to determine if an extension is needed, to build the extension
  > data and parse the extension data. This is somewhat analogous to BoringSSL,
  > however these build and parse functions are intentionally symetrical. The
  > framework is hooked into the existing TLS handling code in such a way that
  > we can gradual convert the extension handling code.
  > Convert the TLS Server Name Indication extension to the new framework,
  > while rewriting it to use CBB/CBS and be more strict in the process.
  > Discussed with beck@
  > ok inoguchi@ (jsing@)

== regress =========================================================== 07/12 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/regress

lib

  ~ libc/strtol/strtoltest.c              

  > Add a test that covers the recently fixed "0x" prefix handling issue.
  > (jsing@)

  ~ libssl/Makefile                       + libssl/tlsext/Makefile
  + libssl/tlsext/tlsexttest.c            

  > Provide a new regress test for TLS extension handlers, currently covering
  > the newly converted SNI code. (jsing@)

libexec

  ~ ld.so/initfirst/test1/prog1/Makefile

  > ld.so and nfs regress were using PROGS variable before it was added
  > to bsd.prog.mk.  Remove local additions and use global rules. (bluhm@)

sbin

  ~ pfctl/pfcmd1.opts                     

  > regress/sbin/pfctl - use -n for pfcmd group
  > Passing in the -n which just results in the rules being parsed.
  > The definition for the pfctlcmd testing group in the Makefile is:
  > # pfcmd: test pfctl command line parsing
  > This test passeed only because pfctl didn't exit with an error
  > when it was asked to clear tables from a non existing anchor.
  > with the -n flag we still test if the anchor command can be combined
  > with -Fa but don't actually attempt to run the clearing code.
  > OK mikeb@, sashan@ (awolk@)

sys

  ~ nfs/Makefile                          

  > ld.so and nfs regress were using PROGS variable before it was added
  > to bsd.prog.mk.  Remove local additions and use global rules. (bluhm@)

  ~ netinet/sendsrcaddr/Makefile          ~ netinet/sendsrcaddr/runtest.c

  > Rewrite regress sendsrcaddr so that is uses UDP sockets on localhost,
  > does not try to connect to google and avoids BPF programming. (bluhm@)

  ~ uvm/mmap_hint/mmap_hint.c             

  > We cannot mmap(2) memory at VM_MAXUSER_ADDRESS as the full allocated
  > page must be below that address.  Subtract PAGE_SIZE from the hint
  > address, then regress passes. (bluhm@)

  ~ kern/siginfo-fault/Makefile           ~ kern/siginfo-fault/siginfo-fault.c

  > Improve error messages of siginfo-fault regress to make clear which
  > tests fail.  First run all checks and report problmes, finally exit
  > with the total number of failures.  Unfortunately it still fails. (bluhm@)

  ~ uvm/mmap_hint/Makefile                ~ uvm/mmap_hint/mmap_hint.c

  > Round VM_MAXUSER_ADDRESS down to next page to make mmap_hint regress
  > pass on sparc64. (bluhm@)

usr.bin

  ~ pkg-config/Makefile                   

  > Make pkg-config regress tests pass:
  > - Allow to override PKG_CONFIG and check correct place in logfile.
  > - Replace "if ...; then false; fi" with a simple ! .
  > - Remove comments about failing test which do not fail.
  > - Adapt regress to changes in pkg-config rev 1.64 and 1.72 regarding
  > static and missing tests.
  > OK jasper@ (bluhm@)

  ~ libtool/Makefile                      

  > The libtool regress used PROGS as a regular variable.  Rename it
  > to avoid the new special meaning in bsd.prog.mk. (bluhm@)

  ~ mandoc/char/unicode/invalid.out_html
  ~ mandoc/char/unicode/latin1.out_html   
  ~ mandoc/char/unicode/latin1diff.out_html
  ~ mandoc/char/unicode/named.out_html    
  ~ mandoc/char/unicode/namediff.out_html
  ~ mandoc/char/unicode/nogroff.out_html
  ~ mandoc/eqn/fromto/basic.out_html      ~ mandoc/eqn/nullary/roman.out_html
  ~ mandoc/eqn/nullary/symbol.out_html    
  ~ mandoc/eqn/subsup/precedence.out_html
  ~ mandoc/eqn/unary/diacrit.out_html     

  > adapt to hex format of character entities,
  > committed by bentley@ in html.c rev. 1.86 (schwarze@)

  ~ mandoc/eqn/subsup/precedence.out_ascii
  ~ mandoc/eqn/subsup/precedence.out_html
  ~ mandoc/eqn/unary/diacrit.out_ascii    ~ mandoc/eqn/unary/diacrit.out_html

  > Catch up with eqn.c rev. 1.41 by bentley@:
  > Print the right character for mathematical overbar. (schwarze@)

  ~ mandoc/mdoc/Bl/Makefile               ~ mandoc/mdoc/Bl/diag.in
  ~ mandoc/mdoc/Bl/diag.out_ascii         ~ mandoc/mdoc/Bl/diag.out_markdown
  ~ mandoc/mdoc/Bl/inset.in               + mandoc/mdoc/Bl/diag.out_lint
  + mandoc/mdoc/Bl/inset.out_lint         

  > test -diag -width and -inset -width (schwarze@)

  ~ mandoc/mdoc/Bl/diag.out_ascii         ~ mandoc/mdoc/Bl/diag.out_markdown
  ~ mandoc/mdoc/Bl/inset.out_ascii        ~ mandoc/mdoc/Bl/inset.out_markdown

  > fix dates (schwarze@)

usr.sbin

  ~ pkg_add/Makefile                      

  > Disable failing pkg_add tests and document when they were added.
  > Targets are still executed, but are expected to fail. (bluhm@)

  + ifstated/Makefile                     + ifstated/ifstated
  + ifstated/statemachine                 

  > Add regression tests for ifstated.
  > Not hooked into regress/usr.sbin/Makefile yet.
  > From Rob Pierce <[email protected]> (benno@)

  ~ httpd/tests/LICENSE                   ~ httpd/tests/Makefile
  ~ httpd/tests/args-get-1048576.pl       ~ httpd/tests/args-get-1073741824.pl
  ~ httpd/tests/args-get-512.pl           ~ httpd/tests/args-get-range-512.pl
  ~ httpd/tests/args-get-range-multipart.pl
  ~ httpd/tests/args-get-slash.pl         
  ~ httpd/tests/args-tls-get-range-512.pl
  ~ httpd/tests/args-tls-get-range-multipart.pl
  ~ httpd/tests/funcs.pl                  

  > Do not mix Perl read() with sysread().  Data could get stuck in the
  > buffered IO and test run-regress-args-get-1073741824.pl would fail
  > on slow hardware.  Introduce a common function read_part() that
  > uses Perl read().  Limit debug output to one line per 1% of data.
  > Remove unused function http_server().  Fix whitespace.  Cleanup
  > Makefile. (bluhm@)

  ~ relayd/LICENSE                        
  ~ relayd/args-http-filter-null-host.pl
  ~ relayd/args-http-host3.pl             ~ relayd/args-http-slow-consumer.pl
  ~ relayd/funcs.pl                       

  > Do not mix Perl read() with sysread().  Data could get stuck in the
  > buffered IO and test run-regress-args-http-chunked.pl would fail
  > on slow hardware.  Limit debug output to one line per 1% of data.
  > Improve function write_syswrite().  Sync with httpd regress.  Fix
  > whitespace. (bluhm@)

== sbin ============================================================== 08/12 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sbin

atactl

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

bioctl

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

dhclient

  ~ dhclient.c                            ~ dhctoken.h
  ~ kroute.c                              

  > Whitespace tweaks to assuage auto-indent. (krw@)

  ~ clparse.c                             ~ dhcpd.h
  ~ parse.c                               

  > Some parsing code cleanup: add parse_boolean(); pass literal format
  > chars to parse_decimal() instead of less obvious *fmt; refactor
  > to eliminate need for the 'alloc:' and 'bad_flag:' labels and the
  > invidious backwards goto's to them. (krw@)

  ~ bpf.c                                 ~ conflex.c
  ~ dhclient.c                            ~ kroute.c
  ~ options.c                             ~ packet.c
  ~ parse.c                               

  > Be consistent. "return (e);" -> "return e;" (krw@)

  ~ clparse.c                             

  > Be consistent. "return (e);" -> "return e;" (krw@)

  ~ bpf.c                                 ~ clparse.c
  ~ conflex.c                             ~ dhclient.c
  ~ kroute.c                              ~ options.c
  ~ packet.c                              ~ parse.c

  > Use a modern spacious idiom on all function local variable
  > declarations. (krw@)

  ~ dhclient.c                            ~ kroute.c
  ~ privsep.c                             ~ privsep.h

  > Use ioctlfd/routefd sockets rather than rolling one's own
  > for resolving 'egress' and adding a route. Move creation
  > of main()'s ioctlfd up so it can be used for the
  > 'egress' resolution. (krw@)

  ~ bpf.c                                 ~ dhclient.c
  ~ dhcpd.h                               

  > Take some of the mystery out of who does what by
  > renaming "if_register_bpf()" to "get_bpf_sock()",
  > "if_register_send()" to "get_udp_sock()" and
  > "if_register_receive()" to "configure_bpf_sock()".
  > Pull back interface_info knowledge and just pass
  > the interface name, rdomain and bpf socket fd as
  > required. (krw@)

  ~ dispatch.c                            

  > bpf(4) *may* deliver more than one captured packet at a time. So when
  > the bpf socket signals that it has data, call packethandler() in a
  > loop until no more bpf packets are available. (krw@)

  ~ dhclient.c                            

  > Pour the few relevant bits of the current lease and the offered
  > renewal lease into canonical forms and compare those rather than the
  > leases themselves. Eliminates spurious interface and route
  > manipulation when irrelevant aspects of the leases are changed by the
  > DHCP server.
  > Problem found by Paul de Weerd when his DHCP server returned
  > renewed leases differing only by a decremented dhcp-lease-time option.
  > ok mpi@ (krw@)

  ~ clparse.c                             ~ dhclient.c
  ~ kroute.c                              ~ options.c
  ~ parse.c                               

  > Replace remaining "!<pointer>" expressions with
  > "<pointer> == NULL". And of course "<pointer>"
  > expressions with "<pointer> != NULL". (krw@)

  ~ bpf.c                                 ~ clparse.c
  ~ conflex.c                             ~ dhclient.c
  ~ dispatch.c                            ~ kroute.c
  ~ options.c                             ~ packet.c
  ~ parse.c                               ~ privsep.c

  > Replace remaining "!var" expressions with
  > "<var> == 0", "!(<var> & FLAG)" with
  > "(<var> & FLAG) == 0", "!<func()>"
  > with "<func()> == 0" and "!<define>" with
  > "<define> == 0". And the positive cases
  > as well.
  > A few stray == NULL and != NULL as well. (krw@)

  ~ bpf.c                                 

  > "ioctl() < 0" ==> "ioctl() == -1" as god and guenther@
  > intended. (krw@)

growfs

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

isakmpd

  ~ Makefile                              

  > remove misc. depend and yacc nits that no longer matter.
  > okay millert@ (espie@)

mount

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

mount_vnd

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

pfctl

  ~ pfctl.h                               ~ pfctl_table.c

  > sbin/pfctl: void functions and exit(3) on error
  > Changes:
  > voided:
  > - pfctl_clear_tables
  > - pfctl_show_tables
  > - pfctl_show_ifaces
  > Those functions now exit(3) in case of error instead
  > of passing it up to the callers (where it was ignored).
  > OK mikeb@, sashan@ (awolk@)

route

  ~ Makefile                              

  > zap depend ghost.
  > add a comment explaining why keywords.h is in the source tree even though
  > it's exactly what keywords.sh generates, so that people don't try adding
  > the generating rule again.
  > okay millert@ (espie@)

scan_ffs

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

slaacd

  ~ engine.c                              

  > If we lower the pltime for privacy addresses skew it by
  > ND6_PRIV_MAX_DESYNC_FACTOR like RFC 4941 tells us to.
  > Toss in a arc4random_uniform for good measure like the kernel always
  > did. (florian@)

  ~ engine.c                              

  > Ignore prefixes with a pltime < ND6_PRIV_MAX_DESYNC_FACTOR seconds
  > (512 in our case) for privacy addresses as per RFC 4941. (florian@)

  ~ engine.c                              ~ frontend.c
  ~ slaacd.c                              

  > run IMSG_UPDATE_IF through the main process, reduces upcomming diff
  > for RFC 7217 support (florian@)

wsconsctl

  ~ map_scan.l                            

  > have bsd.prog.mk/bsd.lib.mk generate .y -> .c/.h rules without intermediate
  > files. This fixes up parallel builds in the default case.
  > FreeBSD does something similar.
  > okay millert@ (espie@)

  ~ Makefile                              

  > remove misc. depend and yacc nits that no longer matter.
  > okay millert@ (espie@)

== share ============================================================= 09/12 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/share

man

  ~ man5/spamd.conf.5                     

  > Switch sample path for black/whitelist files from /var/mail (which is meant
  > to
  > be for mailboxes) to /etc/mail. Fritjof Bornebusch reported that
  > security(8)
  > complains about files in /var/mail and suggested a subdirectory, but /etc
  > seems a better idea all-round, and matches hier(7).  ok jmc@ (sthen@)

  ~ man4/man4.octeon/octmmc.4             

  > Update BUGS. (visa@)

  ~ man9/style.9                          

  > prefer getprogname to the magic global __progname.
  > from Matthew Martin (tedu@)

  ~ man7/ports.7                          

  > Fix non-portable .Lk usage that results in complete garbage with groff
  > and looks better, but still not good with mandoc.
  > Issue pointed out by Jan Stary <hans at stare dot cz>. (schwarze@)

  ~ man4/athn.4                           ~ man4/otus.4
  ~ man4/rsu.4                            ~ man4/rtwn.4
  ~ man4/uath.4                           ~ man4/urtwn.4

  > Mark up firmware pathnames with Pa.
  > ok jmc@ (bentley@)

  ~ man5/bsd.port.mk.5                    

  > consistently use "User settings" instead of a mixture of this and a
  > couple of "User-settings" in bsd.port.mk(5).  ok espie@ (sthen@)

  ~ man4/run.4                            

  > The 'Tenda W311M' USB wifi stick works with run(4). (stsp@)

  ~ man1/help.1                           

  > Missing trailing / in url
  > from Andras Farkas (jca@)

  ~ man4/man4.octeon/Makefile             + man4/man4.octeon/octcib.4

  > Add a manual page for octcib(4). (visa@)

  ~ man5/pf.conf.5                        

  > * Clarify that filter rules are evaluated once per packet and interface,
  > not only once per packet.
  > * Clarify that the syntax   anchor "name" { ... }   both loads and
  > evaluates the anchor, rather than merely loading it.
  > Triggered by questions from Benedikt Neuffer <bene at usta dot de>.
  > OK mikeb@ (schwarze@)

  ~ man5/changelist.5                     ~ man5/defaultdomain.5
  ~ man5/login.conf.5                     ~ man5/myname.5
  ~ man5/netgroup.5                       ~ man5/spamd.conf.5
  ~ man8/daily.8                          ~ man8/rc.8
  ~ man8/rc.shutdown.8                    ~ man8/security.8

  > man pages with pseudo synopses which list filenames end up creating
  > very ugly output in man -k; after some discussion with ingo, we feel
  > the simplest fix is to remove such SYNOPSIS sections: the info is hardly
  > helpful at page top, is contained already in FILES, and there are
  > sufficiently few that just zapping them is simple;
  > ok schwarze, who also helpfully ran things through a build to check
  > output; (jmc@)

mk

  ~ bsd.dep.mk                            ~ bsd.sys.mk

  > have bsd.prog.mk/bsd.lib.mk generate .y -> .c/.h rules without intermediate
  > files. This fixes up parallel builds in the default case.
  > FreeBSD does something similar.
  > okay millert@ (espie@)

  ~ bsd.prog.mk                           

  > add support to compile several progs at once us
  > by using alternate setup
  > PROGS = a b c along with SRCS_a = ... , SRCS_b = ... , SRCS_c = ...
  > okay millert@ (espie@)

  ~ bsd.README                            

  > document more reality.
  > proper english usage and okay jmc@ (espie@)

  ~ bsd.README                            

  > document PROGS, okay jmc@ (espie@)

  ~ bsd.README                            

  > Thinko. It's called BUILDAFTER
  > "noticed" by Ingo. (espie@)

  ~ bsd.dep.mk                            

  > re-add unintentionally removed cleandir part that rm tags from the .CURDIR
  > (espie@)

  ~ bsd.README                            

  > two minor tweaks and two minor fixes; OK espie@ (schwarze@)

  ~ bsd.lib.mk                            

  > use the previous syspatch's fake root for getting the link order
  > instead of using / where the libraries are re-ordered on reboot
  > ok aja@ (robert@)

  ~ bsd.README                            

  > Move the section about <bsd.sys.mk> up just below <sys.mk> which it is
  > closely related to.  Delete the example involving PARALLEL, which no
  > longer exists.  Instead, add a different example involving yacc(1).
  > OK espie@ (schwarze@)

  ~ bsd.README                            

  > explain the limitations of PROGS; joint work with espie@ (schwarze@)

  ~ bsd.README                            

  > add sys.mk to the table of contents, and make the description of
  > bsd.sys.mk more specific; wording with one tweak from espie@ (schwarze@)

== sys =============================================================== 10/12 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sys

arch/alpha/alpha

  ~ locore0.S                             

  > fix GET_CURPCB on !MULTIPROCESSOR builds.
  > this lets GENERIC work as well as GENERIC.MP. it should also fix
  > bsd.rd. tested by me on an es45.
  > from a grumpy alpha user (dlg@)

arch/alpha/conf

  ~ Makefile.alpha                        

  > Pass the linker from make environment to makegap.sh. This is (only)
  > a partial solution to cross compiling the kernel with KARL.
  > OK deraadt@ a while ago (visa@)

arch/amd64/amd64

  ~ codepatch.c                           

  > Replace codepatch nopsleds with trapsleds.
  > ok sf@ mlarkin@
  > suggested by deraadt@ (mortimer@)

  ~ machdep.c                             

  > remove CPU_LIDSUSPEND/machdep.lidsuspend
  > "fire away!" tedu (natano@)

  ~ trap.c                                

  > The regress test src/regress/sys/kern/siginfo-fault checks whether
  > the si_code is SEGV_ACCERR after memory access with wrong permissions
  > has triggert a SIGSEGV.  Adapt the behavior of i386 and amd64 kernel.
  > Remove the useless code that changed error from EACCES to EFAULT
  > in amd64.  Also convert variable name rv to errno in i386 to make
  > it look like amd64.
  > OK kettenis@ (bluhm@)

arch/amd64/conf

  ~ Makefile.amd64                        

  > Pass the linker from make environment to makegap.sh. This is (only)
  > a partial solution to cross compiling the kernel with KARL.
  > OK deraadt@ a while ago (visa@)

arch/amd64/include

  ~ vmmvar.h                              

  > Make max memory for VMs equal to MAXDSIZ to avoid failing later during
  > VM setup.
  > ok pd (mlarkin@)

  ~ cpu.h                                 

  > remove CPU_LIDSUSPEND/machdep.lidsuspend
  > "fire away!" tedu (natano@)

arch/arm/include

  ~ cpu.h                                 

  > remove CPU_LIDSUSPEND/machdep.lidsuspend
  > "fire away!" tedu (natano@)

arch/arm64/conf

  ~ Makefile.arm64                        

  > Pass the linker from make environment to makegap.sh. This is (only)
  > a partial solution to cross compiling the kernel with KARL.
  > OK deraadt@ a while ago (visa@)

arch/armv7/conf

  ~ Makefile.armv7                        

  > Pass the linker from make environment to makegap.sh. This is (only)
  > a partial solution to cross compiling the kernel with KARL.
  > OK deraadt@ a while ago (visa@)

arch/hppa/conf

  ~ Makefile.hppa                         

  > Pass the linker from make environment to makegap.sh. This is (only)
  > a partial solution to cross compiling the kernel with KARL.
  > OK deraadt@ a while ago (visa@)

arch/hppa/hppa

  ~ locore.S                              ~ vm_machdep.c

  > Clear the framepointer (r3) in switch_trampoline to avoid faults in stack
  > trace
  > Reduce size of switch stack frame to match normal 64byte alignment
  > Eliminate useless 'osp' variable when setting up initial frame in
  > cpu_fork()
  > ok visa@ kettenis@ (guenther@)

  ~ db_interface.c                        ~ lock_machdep.c

  > Add WITNESS support
  > ok visa@ kettenis@ (guenther@)

arch/hppa/include

  ~ mplock.h                              

  > Add WITNESS support
  > ok visa@ kettenis@ (guenther@)

arch/i386/conf

  ~ Makefile.i386                         

  > Pass the linker from make environment to makegap.sh. This is (only)
  > a partial solution to cross compiling the kernel with KARL.
  > OK deraadt@ a while ago (visa@)

arch/i386/i386

  ~ codepatch.c                           

  > Replace codepatch nopsleds with trapsleds.
  > ok sf@ mlarkin@
  > suggested by deraadt@ (mortimer@)

  ~ machdep.c                             

  > remove CPU_LIDSUSPEND/machdep.lidsuspend
  > "fire away!" tedu (natano@)

  ~ trap.c                                

  > The regress test src/regress/sys/kern/siginfo-fault checks whether
  > the si_code is SEGV_ACCERR after memory access with wrong permissions
  > has triggert a SIGSEGV.  Adapt the behavior of i386 and amd64 kernel.
  > Remove the useless code that changed error from EACCES to EFAULT
  > in amd64.  Also convert variable name rv to errno in i386 to make
  > it look like amd64.
  > OK kettenis@ (bluhm@)

  ~ bios.c                                

  > usingacpi might be referenced in apm only code, so adjust #if guard
  > from Nick Briggs (tedu@)

arch/i386/include

  ~ vmmvar.h                              

  > Make max memory for VMs equal to MAXDSIZ to avoid failing later during
  > VM setup.
  > ok pd (mlarkin@)

  ~ cpu.h                                 

  > remove CPU_LIDSUSPEND/machdep.lidsuspend
  > "fire away!" tedu (natano@)

arch/landisk/conf

  ~ Makefile.landisk                      

  > Pass the linker from make environment to makegap.sh. This is (only)
  > a partial solution to cross compiling the kernel with KARL.
  > OK deraadt@ a while ago (visa@)

arch/loongson/conf

  ~ Makefile.loongson                     

  > Pass the linker from make environment to makegap.sh. This is (only)
  > a partial solution to cross compiling the kernel with KARL.
  > OK deraadt@ a while ago (visa@)

arch/loongson/loongson

  ~ machdep.c                             

  > remove CPU_LIDSUSPEND/machdep.lidsuspend
  > "fire away!" tedu (natano@)

arch/macppc/conf

  ~ Makefile.macppc                       

  > Pass the linker from make environment to makegap.sh. This is (only)
  > a partial solution to cross compiling the kernel with KARL.
  > OK deraadt@ a while ago (visa@)

arch/mips64/include

  ~ cpu.h                                 

  > remove CPU_LIDSUSPEND/machdep.lidsuspend
  > "fire away!" tedu (natano@)

arch/octeon/conf

  ~ GENERIC                               ~ RAMDISK
  ~ files.octeon                          

  > Add a driver for the CIB interrupt controller. Certain device
  > controllers need it on CN70xx/CN71xx. (visa@)

  ~ Makefile.octeon                       

  > Pass the linker from make environment to makegap.sh. This is (only)
  > a partial solution to cross compiling the kernel with KARL.
  > OK deraadt@ a while ago (visa@)

arch/octeon/dev

  ~ octmmc.c                              

  > Keep refusing SDIO initialization. The driver cannot work properly
  > with SDIO because it does not handle card interrupts. (visa@)

  ~ octciu.c                              

  > Reorganize separate configuration fields into per-CPU structs
  > to compact the data indexing. This additionally improves the cache
  > locality of the data. (visa@)

  ~ octciu.c                              

  > Get cpuid from `ci' to avoid an extra function call. (visa@)

  ~ octciu.c                              

  > Add handling for the third interrupt summary vector,
  > needed by upcoming work. (visa@)

  + octcib.c                              

  > Add a driver for the CIB interrupt controller. Certain device
  > controllers need it on CN70xx/CN71xx. (visa@)

  ~ octeon_iobus.c                        

  > iobus_h is unused, remove it. (visa@)

arch/octeon/include

  ~ octeonreg.h                           

  > Add handling for the third interrupt summary vector,
  > needed by upcoming work. (visa@)

arch/sgi/conf

  ~ Makefile.sgi                          

  > Pass the linker from make environment to makegap.sh. This is (only)
  > a partial solution to cross compiling the kernel with KARL.
  > OK deraadt@ a while ago (visa@)

arch/sparc64/conf

  ~ Makefile.sparc64                      

  > Pass the linker from make environment to makegap.sh. This is (only)
  > a partial solution to cross compiling the kernel with KARL.
  > OK deraadt@ a while ago (visa@)

conf

  ~ makegap.sh                            

  > Pass the linker from make environment to makegap.sh. This is (only)
  > a partial solution to cross compiling the kernel with KARL.
  > OK deraadt@ a while ago (visa@)

dev/fdt

  ~ sxiccmu.c                             

  > Match /clocks node on Allwinner H5. (kettenis@)

dev/ic

  ~ wdc.c                                 ~ wdcvar.h

  > Introduce two quirks to prevent attaching ATA and ATAPI devices
  > Hyper-V and Xen have varying support for detaching emulated IDE
  > devices ranging from none on Hyper-V to only IDE disks but not
  > CDROM (Xen). The quirk mechanism provides a way of enforcing the
  > desired behavior.
  > With suggestions and OK kettenis, tedu, mlarkin (mikeb@)

dev/pci

  ~ pciide.c                              

  > Introduce two quirks to prevent attaching ATA and ATAPI devices
  > Hyper-V and Xen have varying support for detaching emulated IDE
  > devices ranging from none on Hyper-V to only IDE disks but not
  > CDROM (Xen). The quirk mechanism provides a way of enforcing the
  > desired behavior.
  > With suggestions and OK kettenis, tedu, mlarkin (mikeb@)

  ~ if_vic.c                              

  > Reshuffle vic_start and get rid of the dequeue begin/rollback/commit dance
  > Tested on ESXi 5.5.0, OK reyk (mikeb@)

  ~ if_txp.c                              ~ if_txpreg.h

  > Tweak txp to avoid ifq_deq_begin/commit/rollback
  > Setup the transmit ring the way FreeBSD driver does giving each
  > descriptor a 16 fragment chain and revamp the start routine to
  > use m_defrag for long chains and avoid the dequeue begin/commit/
  > rollback dance.
  > OK dlg (mikeb@)

  ~ drm/drm_crtc.c                        ~ drm/drm_crtc.h
  ~ drm/drm_linux.c                       ~ drm/drm_linux.h
  ~ drm/i915/i915_drv.c                   

  > Add a "Backlight" property to connectors with an associated backlight
  > controller for the inteldrm(4) driver.  If wscons(4) provides backlight
  > control, prefer ir over raw hardware control and attach it to LVDS, eDP
  > and DSI connectors which are the connector types typically connected to
  > laptop screens. (kettenis@)

  ~ if_tl.c                               

  > Fix multicast reception: use unsigned chars to avoid accidental sign
  > extension in hash calculation.  Reported & diagnosed by Nick Briggs.
  > ok mpi@ (naddy@)

  ~ if_wb.c                               

  > kill a prototype for a nonexisting function; ok mpi@ (naddy@)

  ~ drm/drm_dp_helper.c                   ~ drm/drm_dp_mst_topology.c
  ~ drm/drm_linux.h                       ~ drm/i915/intel_sdvo.c
  ~ drm/radeon/atombios_i2c.c             ~ drm/radeon/radeon_i2c.c

  > Add some more i2c glue to reduce the diffs to Linux. (kettenis@)

  ~ drm/i915/intel_display.c              

  > Disable the has_audio pipe config check.  We don't manage the HDMI/DP audio
  > capabilities yet so discrepancies are not entirely unexpected. (kettenis@)

  ~ if_iwm.c                              ~ if_iwmvar.h

  > Get rid of the device reset hack in iwm_newstate_task().
  > Instead, maintain firmware state in accordance to the current net80211
  > state by adding/updating/removing MAC config, PHY->MAC binding, and the
  > station in the firmare station table (we only add the AP to this table).
  > By eliminating the iwm_stop() call from iwm_newstate_task(), this change
  > prepares future work on fixing races between ioctls and tasks scheduled
  > by this driver.
  > There's a known new occasional fatal firmware error which results in
  > the message "iwm0: could not remove MAC context (error 35)"
  > I expect to fix this soon.
  > tested by tb@ on 7265, by jcs@ on 8265, and by myself on 7260/7265/8260
  > ok mpi@ tb@ (stsp@)

  ~ pcidevs                               

  > Update Skylake Graphics IDs. (kettenis@)

  ~ pcidevs.h                             ~ pcidevs_data.h

  > regen (kettenis@)

  ~ if_iwm.c                              

  > Some net80211 state changes are triggered by incoming frames, and it is
  > possible for such frames to move iwm(4) to the same state, e.g. AUTH->AUTH.
  > Do not bother the firmware with such no-op state changes because that runs
  > a high risk of hitting firmware errors.
  > Problem reported by Gregor Best. (stsp@)

  ~ if_iwm.c                              

  > Remove the AP from iwm's firmware station table when moving out of AUTH
  > state.
  > Fixes "could not remove MAC context" errors seen during AUTH->SCAN
  > transitions. (stsp@)

  ~ if_iwmvar.h                           

  > Add comments to the definitions of iwm(4) driver-private flags. (stsp@)

  ~ if_iwm.c                              

  > Fix missing splx() in iwm_newstate_task(). Bug introduced 2 hours ago in
  > r1.200. (stsp@)

dev/pv

  ~ hvs.c                                 

  > Prefer paravirtualized interface for IDE disks as well (mikeb@)

  ~ hvs.c                                 

  > Add a comment explaining why pciide(4) must skip attaching disks
  > Requested by kettenis@. (mikeb@)

  ~ if_xnf.c                              

  > Support out-of-order TX completion notifications
  > Apparently, just like the disk interface, the network backend may
  > post TX completions out-of-order, i.e. the ring position doesn't
  > determine which descriptor is being completed and the code must
  > look at the response id inside the completion message.  It might
  > seem obvious, but since networking hardware doesn't usually work
  > this way, it was something that has been overlooked.
  > Based on instability reports from Kirill Miazine <km at krot ! org>,
  > thanks! (mikeb@)

  ~ xen.c                                 ~ xenvar.h

  > Silence the interrupt source until the interrupt task has done its job
  > This small change significantly improves performance under load and halves
  > the number of received interrupts. (mikeb@)

  ~ xen.c                                 

  > Reduce the number of CAS loops from ludicrous to ridiculous
  > Now that the source of the delay with releasing grant table entries has
  > been identified and fixed the number of attempts to CAS entry flags can
  > be substantially reduced and while it's decreased by a factor of 100000,
  > it should go down at least a 100 more in the future. (mikeb@)

  ~ xen.c                                 

  > Spacing (mikeb@)

dev/wscons

  ~ wstpad.c                              

  > Minor changes in the setup.
  > 1. Add middle-button areas to the clickpad defaults.
  > 2. Handle the edge areas more uniformly.
  > 3. Don't disable sw buttons at the bottom edge. (bru@)

  ~ wsmouse.c                             

  > Explain the multi-touch tracking function. (bru@)

kern

  ~ kern_malloc.c                         ~ kern_malloc_debug.c

  > make malloc(9) mpsafe by using a mutex instead of splvm.
  > this is almost a straightforward change of spl ops with mutex ops,
  > except the accounting has been shuffled around. memory is counted
  > as used before an attempt to allocate it from uvm is made to prevent
  > overcommitting memory. this is modelled on how pools limit allocations.
  > the uvm bits have been eyeballed by kettenis@ who says they should be safe.
  > visa@ found some nits which have been fixed.
  > tested by chris@ and amit kulkarni
  > ok kettenis@ visa@ mpi@ (dlg@)

  ~ subr_pool.c                           

  > When there is no contention on a pool cache lock, lower the number
  > of items that a cache list is allowed to hold. This lets the cache
  > release resources back to the common pool after pressure on the cache
  > has decreased.
  > OK dlg@ (visa@)

  ~ subr_pool.c                           

  > Compute the level of contention only once.
  > Suggested by and OK dlg@ (visa@)

  ~ uipc_syscalls.c                       

  > Do not call fo_ioctl() in syscall that do, or will, take the socket
  > lock.
  > Prevents a future lock recursion since soo_ioctl() will need to grab
  > the lock.
  > ok bluhm@, visa@ (mpi@)

  ~ vfs_bio.c                             

  > Invalidate read-ahead buffers when read short
  > Buffercache performs read-ahead for cluster reads by extending
  > the length of an original read operation to the MAXPHYS (64k).
  > Upon I/O completion, the length is trimmed and the buffer is
  > returned to the filesystem and the remaining data is cached.
  > However, under certain circumstances, the underlying hardware
  > may fail to do a complete I/O operation and return with a non-
  > zero value of the residual length (i.e. data that wasn't read).
  > The residual length may exceed the size of an original request
  > and must be re-adjusted to uphold the contract with the caller,
  > e.g. the filesystem. At the same time, read-ahead buffers that
  > cover chunks of memory corresponding to the residual length
  > must be invalidated and not cached.
  > Discussed at length during d2k17, ok tedu (mikeb@)

  ~ uipc_socket.c                         

  > Do not unlock the netlock in the goto out error path before it has
  > been acquired in sosend().  Fixes a kernel lock assertion panic.
  > OK visa@ mpi@ (bluhm@)

net

  ~ if.c                                  

  > Purging is at last at hand. Day of Doom is here. All that is evil
  > shall all be cleansed.
  > Remove sending of router solicitations and processing of router
  > advertisements from the kernel. It's handled by slaacd(8) these days.
  > Input & OK bluhm@, mpi@ (florian@)

  ~ pf_lb.c                               

  > Use a 32 bit variable to detect integer overflow when searching for
  > an unused nat port.  Prevents a possible endless loop if high port
  > is 65535 or low port is 0.
  > report and analysis Jingmin Zhou; OK sashan@ visa@ (bluhm@)

  ~ pipex.c                               

  > Set the ipl of pipex pools to IPL_SOFTNET.
  > Fix floor splassert reported by mlarkin@ and Marko Cupać.
  > ok dlg@, visa@, yasuoka@ (mpi@)

  ~ pfkeyv2_parsemessage.c                

  > kernels don't build without MROUTING because ip_var.h only sometimes
  > introduces a forward decl for socket. turns out the affected file doesn't
  > need ip_var.h, so remove it. then move the decl to the bottom to prevent
  > the problem from recurring.
  > bug report by Nick Briggs
  > ok mpi (tedu@)

netinet

  ~ icmp6.h                               

  > Get rid of ICMPV6CTL_ND6_DRLIST and ICMPV6CTL_ND6_PRLIST sysctls
  > With this we can also get rid of in6_prefix and in6_defrouter. They
  > are meaningless, the kernel no longer tracks this information.
  > Pointed out by & OK mpi (florian@)

  ~ ip_var.h                              

  > kernels don't build without MROUTING because ip_var.h only sometimes
  > introduces a forward decl for socket. turns out the affected file doesn't
  > need ip_var.h, so remove it. then move the decl to the bottom to prevent
  > the problem from recurring.
  > bug report by Nick Briggs
  > ok mpi (tedu@)

netinet6

  ~ in6.c                                 ~ in6_ifattach.c
  ~ in6_var.h                             ~ nd6.c
  ~ nd6.h                                 ~ nd6_nbr.c
  ~ nd6_rtr.c                             

  > Purging is at last at hand. Day of Doom is here. All that is evil
  > shall all be cleansed.
  > Remove sending of router solicitations and processing of router
  > advertisements from the kernel. It's handled by slaacd(8) these days.
  > Input & OK bluhm@, mpi@ (florian@)

  ~ icmp6.c                               ~ nd6.c
  ~ nd6.h                                 

  > Get rid of ICMPV6CTL_ND6_DRLIST and ICMPV6CTL_ND6_PRLIST sysctls
  > With this we can also get rid of in6_prefix and in6_defrouter. They
  > are meaningless, the kernel no longer tracks this information.
  > Pointed out by & OK mpi (florian@)

  ~ in6.h                                 ~ in6_proto.c
  ~ ip6_var.h                             

  > Get rid of ip6.maxifprefixes and ip6.maxifdefrouters, the kernel no
  > longer tracks prefixes or default routers from router advertisements.
  > Pointed out by jmc.
  > ports tree grepping sthen, who only found nsh
  > OK mpi, sthen (florian@)

sys

  ~ socketvar.h                           

  > Kill 'rwlock' forward declaration, this should have been removed with
  > sbsleep(). (mpi@)

== usr.bin =========================================================== 11/12 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.bin

awk

  ~ Makefile                              

  > remove misc. depend and yacc nits that no longer matter.
  > okay millert@ (espie@)

cap_mkdb

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

doas

  ~ doas.conf.5                           

  > man pages with pseudo synopses which list filenames end up creating
  > very ugly output in man -k; after some discussion with ingo, we feel
  > the simplest fix is to remove such SYNOPSIS sections: the info is hardly
  > helpful at page top, is contained already in FILES, and there are
  > sufficiently few that just zapping them is simple;
  > ok schwarze, who also helpfully ran things through a build to check
  > output; (jmc@)

encrypt

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

fmt

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

gencat

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

htpasswd

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

keynote

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

less

  ~ lesskey/Makefile                      

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

m4

  ~ Makefile                              

  > have bsd.prog.mk/bsd.lib.mk generate .y -> .c/.h rules without intermediate
  > files. This fixes up parallel builds in the default case.
  > FreeBSD does something similar.
  > okay millert@ (espie@)

make

  ~ engine.c                              

  > there's no need to fork/exec to execute comments
  > okay millert@ (espie@)

  ~ varmodifiers.c                        

  > Also check for NUL termination at the first character of tstr in
  > the error path of VarModifiers_Apply().  This prevents a sporadic
  > crash in make's regress target t41.
  > OK espie@ (bluhm@)

mandoc

  ~ roff.c                                

  > eqn(7) .EQ has to break man(7) next-line scope, or tree corruption
  > and use after free many ensue; again found by jsg@ with afl(1) (schwarze@)

  ~ eqn_html.c                            

  > do not print <math class="eqn"><mrow></mrow></math> for empty .EQ;
  > issue reported by bentley@ (schwarze@)

  ~ html.c                                

  > Print HTML character references as 4+ digits hexadecimal, like Unicode.
  > tweaks + ok schwarze@ (bentley@)

  ~ html.c                                ~ mandoc.css
  ~ mdoc_html.c                           

  > Handle .Bl -compact via CSS rather than writing individual style
  > attributes into .It blocks; suggested by Steffen Nurpmeso <steffen
  > at sdaoden dot eu> on <groff at GNU dot org> in April 2017.
  > Delete margin-bottom and margin-top style names and the 'v' argument
  > letter from print_otag() because they are no longer used. (schwarze@)

  ~ html.c                                

  > Fix an assertion failure triggered by print_otag("sw+-l", NULL).
  > Even though we skip the style when the argument is NULL,
  > we must still consume the options.
  > Not found with afl(1), but during manual testing of the previous patch...
  > (schwarze@)

  ~ roff.c                                

  > Explicitly initialize a variable where the compiler is (understandably)
  > unable to figure out that it is never used uninitialized.
  > While here, tweak the content of the variable to make its usage
  > easier to understand.  No functional change. (schwarze@)

  ~ roff.c                                

  > The .Dd and .TH macros must interrupt .ce, too;
  > fixing tree corruption and assertion failure found by jsg@ with afl(1)
  > (schwarze@)

  ~ eqn.c                                 

  > always give commas their own <mo> element,
  > suggested by bentley@, improves e.g. glMultMatrix(3) (schwarze@)

  ~ mandocdb.c                            

  > Improve rev. 1.199 further:
  > If a database is empty, we already don't write a database file and
  > delete an existing one.  If none exists, that only means the database
  > was empty and remains so, which is not an error.  So don't nag about
  > it, in particular because that would be a weekly(8) annoyance.
  > Issue reported by jmc@. (schwarze@)

  ~ eqn.c                                 

  > Print the right character for mathematical overbar.
  > ok schwarze@ (bentley@)

  ~ mandoc.css                            

  > correct left margins for different classes of definition lists (schwarze@)

  ~ mdoc_validate.c                       

  > If -column, -diag, -inset, -item, or -ohang lists have a -width,
  > don't just talk about ignoring it, actually do ignore it.
  > No change for terminal output, improves HTML output. (schwarze@)

  ~ mandoc.css                            ~ mdoc_html.c

  > use the same width increase logic for .Bl -hang as for .Bl -tag (schwarze@)

  ~ html.c                                ~ mdoc_html.c

  > In .Bl -tag and -hang, do not print a margin-left style attribute
  > for each individual item if the -width argument matches the default
  > of 6n.  Suggested by Steffen Nurpmeso <steffen at sdaoden dot eu>
  > on <groff at GNU dot org> in April 2017. (schwarze@)

  ~ mandoc.css                            

  > The only elements using floats are dt.It-tag, so the only elements
  > allowed to the right of them are dd.It-tag.
  > Fixes overlapping rendering after long tag lists without bodies,
  > for example in the Xenocara apm(4) manual.  Issue reported by tj@.
  > (schwarze@)

mklocale

  ~ Makefile                              

  > remove misc. depend and yacc nits that no longer matter.
  > okay millert@ (espie@)

nc

  ~ nc.1                                  ~ netcat.c

  > Add a "-T tlscompat" option to nc(1), which enables the use of all TLS
  > protocols and "compat" ciphers. This allows for TLS connections to TLS
  > servers that are using less than ideal cipher suites, without having to
  > resort to "-T tlsall" which enables all known cipher suites.
  > Diff from Kyle J. McKay <mackyle at gmail dot com>
  > ok beck@ (jsing@)

  ~ nc.1                                  

  > grammar was ass backwards; (jmc@)

newsyslog

  ~ newsyslog.c                           

  > TAILQs instead of hand-rolled lists
  > ok millert@ (jca@)

  ~ newsyslog.c                           

  > Don't bother freeing memory just before exiting.
  > Suggested by & ok bluhm@ (jca@)

openssl

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

pkill

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

rdist

  ~ Makefile                              ~ client.c
  ~ docmd.c                               ~ rdist.c

  > have bsd.prog.mk/bsd.lib.mk generate .y -> .c/.h rules without intermediate
  > files. This fixes up parallel builds in the default case.
  > FreeBSD does something similar.
  > okay millert@ (espie@)

signify

  ~ signify.c                             

  > fix style compliance, in the keyname compliance function no less (tedu@)

  ~ zsig.c                                

  > adjust size to be one less than power of 2, so that doubling will never
  > roll to 0, but instead something very large, and thus fail in realloc
  > (tedu@)

ssh

  ~ scp/Makefile                          ~ sftp/Makefile
  ~ ssh-add/Makefile                      ~ ssh-agent/Makefile
  ~ ssh-keygen/Makefile                   ~ ssh-keyscan/Makefile

  > zap redundant Makefile variables.
  > okay djm@ (espie@)

  ~ ssh_config.5                          ~ sshd_config.5

  > man pages with pseudo synopses which list filenames end up creating
  > very ugly output in man -k; after some discussion with ingo, we feel
  > the simplest fix is to remove such SYNOPSIS sections: the info is hardly
  > helpful at page top, is contained already in FILES, and there are
  > sufficiently few that just zapping them is simple;
  > ok schwarze, who also helpfully ran things through a build to check
  > output; (jmc@)

  ~ clientloop.c                          

  > Make ""Killed by signal 1" LogLevel verbose so it's not shown at the
  > default
  > level.  Prevents it from appearing during ssh -J and equivalent
  > ProxyCommand
  > configs. bz#1906, bz#2744, feedback&ok markus@ (dtucker@)

tcpbench

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

tee

  ~ tee.c                                 

  > Replace manual list with SLIST.
  > from Klemens Nanni (bluhm@)

time

  ~ time.c                                

  > use monotonic clock for computing elapsed time
  > from Scott Cheloha. ok dlg. (tedu@)

tmux

  ~ cmd-kill-session.c                    ~ server-fn.c
  ~ server.c                              ~ session.c
  ~ tmux.h                                

  > Some extra logging to show why tmux might exit. (nicm@)

  ~ proc.c                                ~ server-client.c
  ~ server-fn.c                           ~ tmux.h

  > proc_send_s now seems unnecessary. (nicm@)

  ~ client.c                              ~ tmux.c
  ~ tmux.h                                

  > Make shell_command a global like other stuff rather than making it an
  > exception and using callback argument. (nicm@)

  - signal.c                              ~ Makefile
  ~ client.c                              ~ cmd-pipe-pane.c
  ~ job.c                                 ~ proc.c
  ~ server.c                              ~ tmux.h
  ~ window.c                              

  > Move signal code into proc.c. (nicm@)

  ~ cmd-pipe-pane.c                       ~ job.c
  ~ server.c                              ~ window.c

  > Block signals between forking and clearing signal handlers (or calling
  > event_reinit) - if the child gets a signal and fires the libevent signal
  > handler during this period it could write a signal into the parent's
  > signal pipe. GitHub issue 1001 from Aaron van Geffen. (nicm@)

  ~ proc.c                                

  > Do not need to set up USR2 twice. (nicm@)

  ~ mode-tree.c                           ~ window-tree.c

  > Line up keys in tree mode, and don't expand windows with one pane. From
  > Thomas Adam. (nicm@)

  ~ proc.c                                ~ server-client.c

  > Fix redraw defer code in the presence of multiple clients - the timer
  > may be needed for all of them, so don't delete it on the first; and
  > don't skip setting the redraw flag if the timer is already running.
  > Reported by Pol Van Aubel in GitHub issue 1003. (nicm@)

  ~ client.c                              ~ cmd-pipe-pane.c
  ~ job.c                                 ~ proc.c
  ~ server.c                              ~ tmux.h
  ~ window.c                              

  > Because ignore SIGCHLD early, letting signal_del restore it doesn't work
  > correctly, so set it explicitly back to default (and the others for good
  > measure). (nicm@)

true

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

tsort

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

usbhidaction

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

xinstall

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

ypwhich

  ~ Makefile                              

  > remove redundant variable declarations in Makefiles, since those are
  > the default.
  > okay millert@ (espie@)

== usr.sbin ========================================================== 12/12 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.sbin

config

  ~ scan.l                                

  > have bsd.prog.mk/bsd.lib.mk generate .y -> .c/.h rules without intermediate
  > files. This fixes up parallel builds in the default case.
  > FreeBSD does something similar.
  > okay millert@ (espie@)

dhcpd

  ~ dhcp.c                                

  > Handle DHCPINFORM from clients behind a DHCP relay.
  > The dhcpinform() function has assumed that ciaddr matches the packet's
  > IP source address and didn't consider a relay, such as dhcrelay(8) -
  > indicated by giaddr, has forwarded the request.
  > Tested by landry@
  > OK krw@ (reyk@)

ndp

  ~ ndp.8                                 ~ ndp.c

  > Remove defunct code, the kernel no longer provides
  > ICMPV6CTL_ND6_DRLIST and ICMPV6CTL_ND6_PRLIST sysctls since it no
  > longer tracks that information.
  > We can no longer flush or list the prefix list or the default router
  > list.
  > Pointed out by & OK mpi (florian@)

  ~ ndp.8                                 ~ ndp.c

  > Simplify usage and man page synopsis after removal of prefix list and
  > default router flags.
  > Pointed out and diff from jcm@, OK me
  > While reviewing the diff I stumbled upon the H flag, which is now
  > useless, too. jmc suggested that I commit his diff and zap H at the
  > same time. (florian@)

npppd

  ~ npppd/npppd-users.5                   

  > man pages with pseudo synopses which list filenames end up creating
  > very ugly output in man -k; after some discussion with ingo, we feel
  > the simplest fix is to remove such SYNOPSIS sections: the info is hardly
  > helpful at page top, is contained already in FILES, and there are
  > sufficiently few that just zapping them is simple;
  > ok schwarze, who also helpfully ran things through a build to check
  > output; (jmc@)

rebound

  ~ rebound.8                             ~ rebound.c

  > add an option to listen to an address other than localhost,
  > upgrading to a mini recursive resolver for small networks. (tedu@)

relayd

  ~ relayd.conf.5                         

  > The config option is called "no splice", the parser rejects "nosplice".
  > (bluhm@)

  ~ check_icmp.c                          

  > Correctly set ttl for IPv4 and IPv6.
  > Problem noted and fix from Kapetanakis Giannis, thanks!
  > Input & OK jca. (florian@)

  ~ check_icmp.c                          

  > Consistently use the variable "ttl" for get/setsockopt
  > ok benno@ on an earlier version, input from Kapetanakis Giannis (jca@)

rpc.bootparamd

  ~ bootparams.5                          

  > man pages with pseudo synopses which list filenames end up creating
  > very ugly output in man -k; after some discussion with ingo, we feel
  > the simplest fix is to remove such SYNOPSIS sections: the info is hardly
  > helpful at page top, is contained already in FILES, and there are
  > sufficiently few that just zapping them is simple;
  > ok schwarze, who also helpfully ran things through a build to check
  > output; (jmc@)

rpc.lockd

  ~ Makefile                              

  > remove misc. depend and yacc nits that no longer matter.
  > okay millert@ (espie@)

rpc.statd

  ~ Makefile                              

  > remove misc. depend and yacc nits that no longer matter.
  > okay millert@ (espie@)

rtadvd

  ~ if.c                                  

  > Don't clobber errno, we want to log_warn what went wrong.
  > OK benno, millert (florian@)

  ~ log.c                                 ~ log.h

  > Bring over fatal{,x} from bgpd so that we can have format strings.
  > OK benno (florian@)

  ~ config.c                              ~ if.c
  ~ rtadvd.c                              

  > replace
  > log_warnx("foo");
  > exit(1);
  > with
  > fatalx("foo);
  > OK benno (florian@)

  ~ if.c                                  

  > errno is not set, use log_warnx
  > OK benno (florian@)

smtpd

  ~ parse.y                               ~ smtpd.conf.5

  > Remove "listen secure" syntax from smtpd.conf. It's broken since a couple
  > of
  > months and noone complained.
  > Users should replace existing "listen secure" directives with two separate
  > "tls" and "smtps" listeners. i.e. a line like
  > listen on $iface tls pki $pki
  > has to be replaced with
  > listen on $iface tls pki $pki
  > listen on $iface smtps pki $pki
  > Relaying syntax is not affected by this change.
  > suggested by eric
  > ok gilles (natano@)

vipw

  ~ vipw.c                                

  > Useless include
  > from Matthew Martin (jca@)

vmctl

  ~ main.c                                ~ vmctl.c
  ~ vmctl.h                               

  > vmd/vmctl: Add ability to pause / unpause vms
  > With help from Ashwin Agrawal
  > ok reyk@ mlarkin@ (pd@)

  ~ Makefile                              ~ main.c
  ~ vmctl.8                               ~ vmctl.c
  ~ vmctl.h                               

  > Add vmctl send and vmctl receive
  > ok reyk@ and mlarkin@ (pd@)

  ~ vmctl.8                               

  > consistent case; (jmc@)

vmd

  ~ control.c                             ~ i8253.c
  ~ i8253.h                               ~ mc146818.c
  ~ mc146818.h                            ~ vm.c
  ~ vmd.c                                 ~ vmd.h
  ~ vmm.c                                 

  > vmd/vmctl: Add ability to pause / unpause vms
  > With help from Ashwin Agrawal
  > ok reyk@ mlarkin@ (pd@)

  ~ config.c                              ~ control.c
  ~ ns8250.c                              ~ vm.c
  ~ vmd.c                                 ~ vmd.h
  ~ vmm.c                                 

  > Add vmctl send and vmctl receive
  > ok reyk@ and mlarkin@ (pd@)

ypldap

  ~ entries.c                             

  > Log when we have finished pushing users and groups
  > Helps people testing new configurations.  From Kurt Mosiejczuk (jca@)

ypserv

  ~ ypserv/Makefile                       

  > remove misc. depend and yacc nits that no longer matter.
  > okay millert@ (espie@)

===============================================================================
_______________________________________________
owc mailing list
[email protected]
http://www.squish.net/mailman/listinfo/owc

Reply via email to