OpenBSD src changes summary for 2015-05-10 to 2015-05-17 inclusive ==================================================================
distrib/miniroot distrib/notes distrib/sets gnu/lib/libstdc++-v3 gnu/lib/libsupc++-v3 gnu/usr.bin/binutils-2.17 lib/libc lib/libcrypto lib/libkvm lib/libpthread lib/librthread lib/libssl lib/libtls regress/sys regress/usr.bin regress/usr.sbin sbin/bioctl sbin/route share/man share/mk sys/arch/amd64/amd64 sys/arch/amd64/include sys/arch/arm/include sys/arch/arm/xscale sys/arch/armish/armish sys/arch/armish/include sys/arch/armv7/armv7 sys/arch/armv7/exynos sys/arch/armv7/imx sys/arch/armv7/include sys/arch/armv7/omap sys/arch/armv7/sunxi sys/arch/hppa/conf sys/arch/hppa/gsc sys/arch/luna88k/cbus sys/arch/luna88k/conf sys/arch/macppc/conf sys/arch/macppc/dev sys/arch/sgi/dev sys/arch/sgi/hpc sys/arch/sh/sh sys/arch/sparc/conf sys/arch/sparc/dev sys/arch/sparc64/conf sys/arch/sparc64/dev sys/arch/vax/conf sys/arch/vax/if sys/arch/vax/vax sys/arch/vax/vsa sys/arch/zaurus/conf sys/arch/zaurus/dev sys/arch/zaurus/include sys/arch/zaurus/zaurus sys/conf sys/dev sys/dev/ata sys/dev/hil sys/dev/ic sys/dev/isa sys/dev/pci sys/dev/pcmcia sys/dev/sbus sys/dev/tc sys/dev/usb sys/kern sys/miscfs/fifofs sys/net sys/netinet sys/netinet6 sys/sys usr.bin/aucat usr.bin/audioctl usr.bin/calendar usr.bin/kdump usr.bin/nm usr.bin/renice usr.bin/ssh usr.bin/tmux usr.sbin/npppd usr.sbin/ntpd usr.sbin/pkg_add usr.sbin/relayd usr.sbin/smtpd == distrib =========================================================== 01/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/distrib miniroot ~ install.sub > Extend autoinstall(8) to allow for <hostname>-<mode>.conf response files > and to put response files in a subdir of the webserver's document root. > Based on diffs from Nathanael Rensen, thanks! > While here fix a buglet introduced by the $_server -> $AI_SERVER change. > OK krw, halex (rpe@) ~ install.sh ~ install.sub > Remove the 'Use DUIDs rather than device names in fstab?' question > and use DUIDs unconditionally. DUIDs in the installed /etc/fstab > has been the default for quite some time now. > OK deraadt@, krw@, guenther@, beck@ (rpe@) notes ~ armv7/hardware ~ armv7/whatis > mention CuBox-i4Pro, Utilite and Novena (jsg@) sets ~ lists/comp/mi > sync (miod@) ~ lists/base/md.alpha ~ lists/base/md.amd64 ~ lists/base/md.armish ~ lists/base/md.armv7 ~ lists/base/md.aviion ~ lists/base/md.hppa ~ lists/base/md.hppa64 ~ lists/base/md.i386 ~ lists/base/md.landisk ~ lists/base/md.loongson ~ lists/base/md.luna88k ~ lists/base/md.macppc ~ lists/base/md.octeon ~ lists/base/md.sgi ~ lists/base/md.socppc ~ lists/base/md.sparc ~ lists/base/md.sparc64 ~ lists/base/md.vax ~ lists/base/md.zaurus ~ lists/comp/mi > sync (deraadt@) ~ lists/comp/md.i386 ~ lists/comp/md.sparc64 > sync (deraadt@) == gnu =============================================================== 02/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/gnu lib/libstdc++-v3 ~ Makefile ~ Makefile.common > Stop including CFLAGS in CXXFLAGS it creates problems when mixing > C and C++ when wanting to pass flags to only the C compiler. > Tested in a ports bulk build by naddy and the handful of problems > found were fixed by naddy, jca and pascal. (jsg@) lib/libsupc++-v3 ~ Makefile > Stop including CFLAGS in CXXFLAGS it creates problems when mixing > C and C++ when wanting to pass flags to only the C compiler. > Tested in a ports bulk build by naddy and the handful of problems > found were fixed by naddy, jca and pascal. (jsg@) usr.bin/binutils-2.17 ~ bfd/archive.c ~ bfd/archive64.c ~ bfd/libbfd-in.h ~ bfd/libbfd.h ~ bfd/som.c > Add _bfd_ar_spacepadll() for formatting long long values in archive > headers. > ok kettenis@ (guenther@) ~ bfd/elflink.c > Revert rev 1.3. While that change appears to make the behaviour similar to > binutils 2.15, it introduces bogus failures when inter-library dependencies > are present. > With the binutils 2.17 behaviour restored, --as-needed will cause linker > failures when necessary inter-library dependencies are not present. This > is > actually a good thing as binutils 2.15 will silently produce executables > that will fail at runtime in those cases. > ok miod@ (kettenis@) == lib =============================================================== 03/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/lib libc ~ db/db/db.c ~ gen/shm_open.c ~ stdlib/posix_pty.c > When checking flags that will be passed to open(), test the O_ACCMODE > portion > separately to avoid false negatives. > ok miod@ millert@ (guenther@) ~ db/db/db.c > Whoops, need to pass through O_ACCMODE flags to the underlying __*_open() > problem pointed out by Mark Patruck (mark (at) wrapped.cx) (guenther@) ~ sys/sigaction.2 ~ gen/signal.3 > chflagsat() is also async-signal-safe (guenther@) ~ arch/vax/sys/cerror.S > Put the right number of leading underscores in ___cerror and __cerror, for > the sake of libpthread. Probably missed during the a.out->ELF switch, I > can't believe this went unnoticed for so long. (miod@) ~ stdlib/atexit.3 > Add dlclose(3) to SEE ALSO > ok millert@ jmc@ schwarze@ (guenther@) ~ crypt/crypt.c > If crypt(3) is called with an unknown setting, return NULL instead > of some undefined value. > OK tedu@ (bluhm@) ~ net/inet_net_ntop.c > rev 1.3 introduced a check to an if statement without adding braces. > Claudio points out the size is checked by an earlier test so just > remove it to restore the original handling of the partial octet case. > Discussed with claudio and gilles. (jsg@) ~ arch/i386/string/strchr.S ~ arch/i386/string/strrchr.S > Use STRONG_ALIAS instead of ALTENTRY. > ok millert@ (kettenis@) - string/rindex.c - string/index.c ~ arch/alpha/string/Makefile.inc ~ arch/amd64/string/strchr.S ~ arch/amd64/string/strrchr.S ~ arch/arm/string/Makefile.inc ~ arch/hppa/string/Makefile.inc ~ arch/hppa64/string/Makefile.inc ~ arch/i386/string/strchr.S ~ arch/i386/string/strrchr.S ~ arch/m88k/string/Makefile.inc ~ arch/mips64/string/strchr.S ~ arch/mips64/string/strrchr.S ~ arch/powerpc/string/Makefile.inc ~ arch/sh/string/Makefile.inc ~ arch/sparc/string/Makefile.inc ~ arch/sparc64/string/Makefile.inc ~ arch/vax/string/Makefile.inc ~ arch/vax/string/strchr.S ~ string/strchr.c ~ string/strrchr.c > Make index/rindex weak aliases of strchr/strrchr since they are not > part of the ISO C standard and have also been dropped from POSIX. > OK guenther@ kettenis@ (millert@) ~ rpc/clnt_udp.c > Use fcntl() to set non-blocking-mode, rather ioctl(). This has a better > chance of working in systrace restricted environments. > ok guenther (deraadt@) ~ gen/isatty.c > isatty() is used by stdio to determine the buffering mode. Add a F_ISATTY > option to fcntl(), so that isatty() can use this rather than than the > bloated > ioctl() interface. Reducing uses of ioctl() by libc makes it easier to > constrain programs with various kinds of systrace sandboxes. > ok guenther, previously discussed as a concept with nicm (deraadt@) ~ gen/isatty.c > Of course, fcntl errno case returns -1, which must be converted to 0 > with guenther (deraadt@) libcrypto ~ crypto/shlib_version > (kettenis@) libkvm ~ kvm.c > When checking flags that will be passed to open(), test the O_ACCMODE > portion > separately to avoid false negatives. > ok miod@ millert@ (guenther@) libpthread ~ man/pthread_atfork.3 > Document pthread_atfork(3)'s interaction with dlclose(3) > Use Xr instead of Fn for functions documented on other manpages > ok millert@ jmc@ schwarze@ (guenther@) ~ man/pthread_atfork.3 > trailing whitespace; (jmc@) librthread ~ rthread.c ~ rthread_fork.c > In the child after fork, the dl lock has to be forced as its inner spinlock > may have been grabbed by another thread in the parent before the fork > problem report from dcoppa@, ok kettenis@ (guenther@) ~ arch/vax/cerror.S > Put the right number of leading underscores in ___cerror and __cerror, for > the sake of libpthread. Probably missed during the a.out->ELF switch, I > can't believe this went unnoticed for so long. (miod@) libssl ~ src/crypto/cms/cms_pwri.c ~ src/crypto/pem/pvkfmt.c ~ src/crypto/pkcs12/p12_decr.c ~ src/ssl/d1_srvr.c ~ src/ssl/s3_srvr.c > Fix return paths with missing EVP_CIPHER_CTX_cleanup() calls. > ok doug@ (jsg@) ~ ssl/Makefile ~ ssl/shlib_version > (kettenis@) libtls ~ Makefile > (kettenis@) == regress =========================================================== 04/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/regress sys ~ fifofs/fifotest.c > Race less. (millert@) usr.bin ~ sed/math.expected ~ sed/math.in ~ sed/math.sed > A new version from the original author of math.sed that does exponents > and lots of other neat things. Via FreeBSD (millert@) usr.sbin ~ relayd/LICENSE ~ relayd/Relayd.pm ~ relayd/funcs.pl + relayd/args-http-callback.pl + relayd/args-http-chunked-callback.pl + relayd/args-http-chunked-put.pl + relayd/args-http-headline-callback.pl + relayd/args-https-callback.pl + relayd/args-https-chunked-callback.pl + relayd/args-https-chunked-put.pl + relayd/args-https-headline-callback.pl > Add test cases for a crash reported by Bertrand PROVOST. When a > HTTP client writes multiple requests or chunks in a single transfer, > relayd invokes the libevent callback manually for the next data. > If the callback closes the session, this results in an use after > free. > Test an invalid second request method, test an invalid header line > in the second PUT request, test an invalid second chunked length > for a PUT request. Also test multiple valid HTTP 1.1 PUT requests > with chunked body. > To detect crashes of relayd, start it with "prefork 1" and grep for > "lost child" log messages. Unfortunately only the first child is > monitored by the parent. (bluhm@) == sbin ============================================================== 05/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sbin bioctl ~ bioctl.c > explicit_bzero(3) on private data, in case of core dumps (and being a good > example) > ok florian jsing (pelikan@) route ~ route.8 ~ route.c > nope, ioctl has not been used for a while (deraadt@) == share ============================================================= 06/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/share man ~ man4/urtwn.4 > Add newly matched devices to urtwn(4) hardware list. > from Mikhail mp39590@gmail on tech@ > Grammar fixes by sthen@. Made mandoc -T lint clean by me (must escape > 'D1'). (stsp@) ~ man8/autoinstall.8 > Extend autoinstall(8) to allow for <hostname>-<mode>.conf response files > and to put response files in a subdir of the webserver's document root. > Based on diffs from Nathanael Rensen, thanks! > While here fix a buglet introduced by the $_server -> $AI_SERVER change. > OK krw, halex (rpe@) ~ man4/urtwn.4 > sort; (jmc@) ~ man8/autoinstall.8 > spelling; (jmc@) mk ~ bsd.lib.mk ~ bsd.prog.mk ~ sys.mk > Stop including CFLAGS in CXXFLAGS it creates problems when mixing > C and C++ when wanting to pass flags to only the C compiler. > Tested in a ports bulk build by naddy and the handful of problems > found were fixed by naddy, jca and pascal. (jsg@) == sys =============================================================== 07/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sys arch/amd64/amd64 ~ nvram.c > Convert from uiomovei() to uiomove() to prevent short tranfers > diff from natano@bitrig, with some additional format and cast tweaks > (guenther@) arch/amd64/include ~ asm.h > Define END() to set ELF symbol size. > OK miod@ (uebayasi@) arch/arm/include ~ vmparam.h > VM_MAX_KERNEL_ADDRESS is the end of the kva range not the end of > addressable virtual memory space but arm ports were using a value of > 0xffffffff for this. > Instead of using a shared VM_MAX_KERNEL_ADDRESS define add md > VM_KERNEL_SPACE_SIZE defines based on the KERNEL_VM_SIZE values > from the respective machdep.c files. > djm's novena was hitting "panic: bufinit: can't reserve VM for buffers" > without a similiar change. > ok miod@ (jsg@) arch/arm/xscale ~ pxa27x_udc.c > add some missing splx() calls > ok deraadt@ kettenis@ krw@ (jsg@) arch/armish/armish ~ armish_machdep.c > VM_MAX_KERNEL_ADDRESS is the end of the kva range not the end of > addressable virtual memory space but arm ports were using a value of > 0xffffffff for this. > Instead of using a shared VM_MAX_KERNEL_ADDRESS define add md > VM_KERNEL_SPACE_SIZE defines based on the KERNEL_VM_SIZE values > from the respective machdep.c files. > djm's novena was hitting "panic: bufinit: can't reserve VM for buffers" > without a similiar change. > ok miod@ (jsg@) arch/armish/include ~ vmparam.h > VM_MAX_KERNEL_ADDRESS is the end of the kva range not the end of > addressable virtual memory space but arm ports were using a value of > 0xffffffff for this. > Instead of using a shared VM_MAX_KERNEL_ADDRESS define add md > VM_KERNEL_SPACE_SIZE defines based on the KERNEL_VM_SIZE values > from the respective machdep.c files. > djm's novena was hitting "panic: bufinit: can't reserve VM for buffers" > without a similiar change. > ok miod@ (jsg@) arch/armv7/armv7 ~ armv7_machdep.c > limit physical memory to (paddr_t)-PAGE_SIZE (0xfffff000) > novena has 4GB of physical memory and it's u-boot tells us > memstart: 0x10000000 > memsize: 0xf0000000 > which would previously cause an overflow leading to > "panic: initarm: out of memory" > tweak from and ok miod@ (jsg@) ~ armv7_machdep.c > VM_MAX_KERNEL_ADDRESS is the end of the kva range not the end of > addressable virtual memory space but arm ports were using a value of > 0xffffffff for this. > Instead of using a shared VM_MAX_KERNEL_ADDRESS define add md > VM_KERNEL_SPACE_SIZE defines based on the KERNEL_VM_SIZE values > from the respective machdep.c files. > djm's novena was hitting "panic: bufinit: can't reserve VM for buffers" > without a similiar change. > ok miod@ (jsg@) ~ armv7_machdep.c > Raise VM_PHYSSEG_MAX to two and load an additional physical memory > segment if u-boot reports it. Needed for the utilite where u-boot > reports two 1GB segments of physical memory. > From Patrick Wildt in bitrig with some additional sanity checks added. > (jsg@) ~ armv7.c > imxiic is known to be broken don't try attaching it on utilite (jsg@) ~ armv7.c ~ armv7_machdep.c ~ armv7_machdep.h ~ armv7var.h ~ autoconf.c > Make board attaching table driven and move it out into the soc > directories. Move the device tables while here as was done in bitrig. > With these changes the only use of the board id defines is in the soc > directories. > Tested by matthieu and djm on imx and myself on omap and sunxi (qemu). > ok djm@, ok jasper@ on an earlier version (jsg@) arch/armv7/exynos ~ exynos.c ~ exynos_machdep.c > Make board attaching table driven and move it out into the soc > directories. Move the device tables while here as was done in bitrig. > With these changes the only use of the board id defines is in the soc > directories. > Tested by matthieu and djm on imx and myself on omap and sunxi (qemu). > ok djm@, ok jasper@ on an earlier version (jsg@) arch/armv7/imx ~ imxuart.c > Add missing braces. > Tested by and ok djm@ (jsg@) ~ imxenet.c > Having the same settings on the same phy across multiple boards is just > asking too much in the arm world. > Translate the fec parameters from the novena dtb to set a different clock > skew to the same micrel phy used on sabre lite. The novena dtb sets > txd[0-3]-skew-ps to 3000, the sabre lite sets them to 0. When run through > the shifting/oring process from the micrel phy driver in FreeBSD this ends > up being a write of 0xffff on the novena and 0x0000 on sabre lite when > writing to TX_DATA_PAD_SKEW. > This change resolves the stability problems djm was seeing with imxenet > on novena. > ok djm@ (jsg@) ~ imxenet.c > rework imxenet hardware address setup: if COTP doesn't provide an > address, check whether the firmware/bootloader has already programmed > one. If all else fails, use ether_fakeaddr(); feedback and ok jsg@ > (committed from Novena) (djm@) ~ imxgpio.c > Remove BOARD_ID* tests that do the same thing for every imx board. > Tested by and ok djm@ (jsg@) ~ files.imx ~ imx.c ~ imx_machdep.c > Make board attaching table driven and move it out into the soc > directories. Move the device tables while here as was done in bitrig. > With these changes the only use of the board id defines is in the soc > directories. > Tested by matthieu and djm on imx and myself on omap and sunxi (qemu). > ok djm@, ok jasper@ on an earlier version (jsg@) ~ imx.c ~ imxesdhc.c > Decrement the unit numbers on the novena sdhc devices. The imx6 dts > files seem to have labels numbered one higher than the nodes they are > assigned to. > Tested by djm (jsg@) ~ imxesdhc.c > add missing calls to set the gpio direction before reading the > card detect gpio on phyflex/sabre lite/wandboard (jsg@) ~ imx.c ~ imxesdhc.c > Checking the dts files turned up some more imx sdhc problems. > - use the correct CD gpios on phyflex and wandboard usdhc3 (unit 2) > - udoo has just the one sd slot with no CD (jsg@) arch/armv7/include ~ vmparam.h > VM_MAX_KERNEL_ADDRESS is the end of the kva range not the end of > addressable virtual memory space but arm ports were using a value of > 0xffffffff for this. > Instead of using a shared VM_MAX_KERNEL_ADDRESS define add md > VM_KERNEL_SPACE_SIZE defines based on the KERNEL_VM_SIZE values > from the respective machdep.c files. > djm's novena was hitting "panic: bufinit: can't reserve VM for buffers" > without a similiar change. > ok miod@ (jsg@) ~ vmparam.h > Raise VM_PHYSSEG_MAX to two and load an additional physical memory > segment if u-boot reports it. Needed for the utilite where u-boot > reports two 1GB segments of physical memory. > From Patrick Wildt in bitrig with some additional sanity checks added. > (jsg@) arch/armv7/omap ~ files.omap ~ omap.c ~ omap_machdep.c > Make board attaching table driven and move it out into the soc > directories. Move the device tables while here as was done in bitrig. > With these changes the only use of the board id defines is in the soc > directories. > Tested by matthieu and djm on imx and myself on omap and sunxi (qemu). > ok djm@, ok jasper@ on an earlier version (jsg@) arch/armv7/sunxi ~ files.sunxi ~ sunxi.c ~ sunxi_machdep.c > Make board attaching table driven and move it out into the soc > directories. Move the device tables while here as was done in bitrig. > With these changes the only use of the board id defines is in the soc > directories. > Tested by matthieu and djm on imx and myself on omap and sunxi (qemu). > ok djm@, ok jasper@ on an earlier version (jsg@) arch/hppa/conf ~ files.hppa > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) arch/hppa/gsc ~ harmony.c > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) arch/luna88k/cbus ~ nec86hw.c > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) arch/luna88k/conf ~ files.luna88k > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) arch/macppc/conf ~ files.macppc > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) arch/macppc/dev ~ awacs.c ~ i2s.c > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) ~ if_bm.c > test mbuf pointers against NULL not 0 > ok krw@ miod@ (jsg@) arch/sgi/dev ~ mavb.c > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) arch/sgi/hpc ~ files.hpc > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) arch/sh/sh ~ sh_machdep.c > still needs kcore.h (deraadt@) arch/sparc/conf ~ files.sparc > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) arch/sparc/dev ~ audioamd.c ~ cs4231.c > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) ~ cs4231.c > Remove support for ADPCM encoding which isn't used nowadays and > not available to programs anyway. (ratchov@) arch/sparc64/conf ~ files.sparc64 > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) arch/sparc64/dev ~ ce4231.c > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) ~ ce4231.c > Remove support for ADPCM encoding which isn't used nowadays and > not available to programs anyway. (ratchov@) arch/vax/conf ~ files.vax > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) arch/vax/if ~ if_de.c ~ if_qe.c ~ sgec.c > test mbuf pointers against NULL not 0 > ok krw@ miod@ (jsg@) arch/vax/vax ~ machdep.c > Remove useless "bridge.h" include. > ok miod@ (mpi@) arch/vax/vsa ~ vsaudio.c > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) arch/zaurus/conf ~ files.zaurus > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) arch/zaurus/dev ~ zaurus_audio.c > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) arch/zaurus/include ~ vmparam.h > VM_MAX_KERNEL_ADDRESS is the end of the kva range not the end of > addressable virtual memory space but arm ports were using a value of > 0xffffffff for this. > Instead of using a shared VM_MAX_KERNEL_ADDRESS define add md > VM_KERNEL_SPACE_SIZE defines based on the KERNEL_VM_SIZE values > from the respective machdep.c files. > djm's novena was hitting "panic: bufinit: can't reserve VM for buffers" > without a similiar change. > ok miod@ (jsg@) arch/zaurus/zaurus ~ zaurus_machdep.c > VM_MAX_KERNEL_ADDRESS is the end of the kva range not the end of > addressable virtual memory space but arm ports were using a value of > 0xffffffff for this. > Instead of using a shared VM_MAX_KERNEL_ADDRESS define add md > VM_KERNEL_SPACE_SIZE defines based on the KERNEL_VM_SIZE values > from the respective machdep.c files. > djm's novena was hitting "panic: bufinit: can't reserve VM for buffers" > without a similiar change. > ok miod@ (jsg@) conf ~ files > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) dev - auconv.c - auconv.h ~ audio.c ~ audio_if.h ~ mulaw.c ~ mulaw.h > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) ~ softraid.c > Make softraid(4) compile with SR_DEBUG by fixing __kprintf__ specifiers. > ok jsing krw (pelikan@) ~ midi.c > Don't hold the audio mutex when calling uiomove(), as uiomove() > may sleep in case of a page fault (ratchov@) ~ midi.c ~ midi_if.h > Remove unused code & definitions: midi_attach(), midi_unit_count(), > midi_getinfo(), midi_writebytes(). They were "used" by the sequencer > interface which we removed years ago. (ratchov@) ~ midi.c > store byte count in a size_t and switch from uiomovei() to uiomove() > (ratchov@) ~ midi.c ~ midivar.h > remove the sc->isopen flag, as sc->flags already hold the > same information. (ratchov@) ~ midi.c ~ midivar.h > Use device_lookup() instead of digging into midi_cd.cd_devs[] and > maintaining a "dying" flag which is already present in the device > structure. As a side-effect, this adds the missing refcounting > that mididetach() was missing. With from mpi@ and dlg@ > ok mpi (ratchov@) ~ midi_if.h > hide private midi_softc strucure (ratchov@) dev/ata ~ atascsi.c ~ atascsi.h > rename the probe and free members of atascsi_methods to ata_probe > and ata_free. > this makes them consistent with the rest of the members, and lets > me #define free to weird debug things in the kernel without screwing > these files up. (dlg@) dev/hil ~ hil.c > add some missing splx() calls > ok deraadt@ kettenis@ krw@ (jsg@) dev/ic ~ lpt.c > Convert from uiomovei() to uiomove() to prevent short tranfers > diff from natano@bitrig (guenther@) ~ ac97.c ~ am7930.c ~ am7930var.h ~ arcofi.c > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) ~ am7990.c ~ am79900.c ~ dp8390.c ~ i82596.c > test mbuf pointers against NULL not 0 > ok krw@ miod@ (jsg@) dev/isa ~ ad1848.c ~ ess.c ~ files.isa ~ files.isapnp ~ gus.c ~ sbdsp.c > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) ~ ad1848.c > Remove support for ADPCM encoding which isn't used nowadays and > not available to programs anyway. (ratchov@) ~ gus_isa.c ~ gus_isapnp.c ~ sb_isa.c ~ sb_isapnp.c > Remove references to (deleted) auconv.h and mulaw.h files. Fixes > i386 build. Found by Mark Patruck <mark at wrapped.cx>, thanks! (ratchov@) ~ if_eg.c ~ if_el.c ~ if_ie.c > test mbuf pointers against NULL not 0 > ok krw@ miod@ (jsg@) dev/pci ~ auacer.c ~ auglx.c ~ auich.c ~ auixp.c ~ autri.c ~ auvia.c ~ azalia.c ~ cmpci.c ~ cs4280.c ~ cs4281.c ~ eap.c ~ emuxki.c ~ esa.c ~ eso.c ~ files.pci ~ fms.c ~ maestro.c ~ neo.c ~ sv.c ~ yds.c > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) ~ pcidevs > add id for virtio rng > ok sf@ (jasper@) ~ pcidevs.h ~ pcidevs_data.h > regen (jasper@) ~ if_em.c ~ if_em_hw.c ~ if_em_hw.h > The i211 does not support an external EEPROM only a OTP > Internal Non-Volatile Memory (iNVM). Add support for reading > words out of it instead of an EEPROM. > From Patrick Wildt with some more offsets added. (jsg@) ~ hifn7751.c > Fixup potential use after free and a memory leak. > Found by Maxime Villard <max at m00nbsd ! net> with the Brainy Code > Scanner, > thanks! (mikeb@) ~ if_em.c > Make sure the rx ring lwm is set to at least 4. As far as we know, all > hardware variants need at least 4 descriptors on the rx ring to be able to > receive packets. Should fix the issue reported by Christian Schulte on > bugs@. > ok mikeb@, sthen@ (kettenis@) ~ if_de.c ~ if_ste.c > Convert to if_input(). > ok dlg@ (mpi@) ~ if_myx.c > We don't need KERNEL_LOCK() around if_input() anymore, as if_input() has > appropriate locking around bpf now. > ok dlg@ (chris@) dev/pcmcia ~ if_cnw.c ~ if_xe.c > test mbuf pointers against NULL not 0 > ok krw@ miod@ (jsg@) dev/sbus ~ cs4231.c ~ files.sbus > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) ~ cs4231.c > Remove support for ADPCM encoding which isn't used nowadays and > not available to programs anyway. (ratchov@) ~ be.c > test mbuf pointers against NULL not 0 > ok krw@ miod@ (jsg@) dev/tc ~ bba.c ~ files.tc > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) dev/usb ~ if_urtwn.c > My commit adding RTL8188EU did introduce a regression after all. > Repair firmware reset on RTL8188CUS devices (and perhaps others) by > restoring the proper register write order in the non-8188EU code path. > Fixes "urtwn0: timeout waiting for firmware readiness" after device reset. > (stsp@) ~ if_urtwn.c ~ if_urtwnreg.h > Fix urtwn(4) 11b performance for RTL8188EU devices. > With 11b APs the firmware retries excessively unless we force it into > 11b-only mode instead of mixed b/g. Perhaps a firmware bug, perhaps > a bug in our code elsewhere. For now, this makes it work. > ok kettenis (stsp@) ~ files.usb ~ uaudio.c > Remove all audio format conversion code from the kernel (btw holding > the kernel_lock), as we already do better conversions in > user-mode. Yet, no need for every single driver to fiddle with the > conversion code as they are done transparently by common MI code. With > help from armani and miod, support from mpi > ok armani@ (ratchov@) ~ upd.c > Refresh sensor values asynchronously. > From David Higgs. (mpi@) ~ usbdevs > Add 4 additional urtwn(4) device IDs. > from FreeBSD via Mikhail mp39590@gmail on tech@ (stsp@) ~ usbdevs.h > usbdevs_data.h (stsp@) ~ usbdevs_data.h > regen (stsp@) ~ if_urtwnreg.h > Fix definition of R92C_USTIME_TSF. > from FreeBSD r282623 via Mikhail mp39590@gmail on tech@ (stsp@) ~ if_urtwn.c > Merge various urtwn(4) fixes from FreeBSD: > Match additional devices (RTL8188CU_3, DWA123D1, DWA125D1). > Fix efuse reading (fixes potential error in MAC address read from efuse). > Replace some magic numbers with macros. > Based on FreeBSD r270191, r273589, r281918, r281592, r282119, and r282623 > via Mikhail mp39590@gmail on tech@ > ok sthen@ (stsp@) ~ upd.c > Make the "Battery Present" sensor a depency of all the battery-related > sensors only if it is present, From David Higgs. > Fix a regression reported by and ok halex@ (mpi@) ~ if_axe.c ~ if_axereg.h > Read ethernet address from EEPROM on AX88772B based on FreeBSD > ok djm@ jsg@ (canacar@) kern ~ sys_generic.c > Set POLLHUP even if no valid events were specified as per POSIX. > Since we use the poll backend for select(2), care must be taken not > to set the fd's bit in writefds in this case. A kernel-only flag, > POLLNOHUP, is used by selscan() to tell the poll backend not to > return POLLHUP on EOF. This is currently only used by fifo_poll(). > The fifofs regress now passes. OK guenther@ (millert@) ~ kern_synch.c ~ vfs_subr.c > Drop and reacquire the kernel lock in the vfs_shutdown and "cold" > portions of msleep and tsleep to give interrupts a chance to run > on other CPUs. > Tweak and OK kettenis (mikeb@) ~ sys_socket.c > test mbuf pointers against NULL not 0 > ok krw@ miod@ (jsg@) ~ kern_descrip.c > isatty() is used by stdio to determine the buffering mode. Add a F_ISATTY > option to fcntl(), so that isatty() can use this rather than than the > bloated > ioctl() interface. Reducing uses of ioctl() by libc makes it easier to > constrain programs with various kinds of systrace sandboxes. > ok guenther, previously discussed as a concept with nicm (deraadt@) miscfs/fifofs ~ fifo_vnops.c > Set POLLHUP even if no valid events were specified as per POSIX. > Since we use the poll backend for select(2), care must be taken not > to set the fd's bit in writefds in this case. A kernel-only flag, > POLLNOHUP, is used by selscan() to tell the poll backend not to > return POLLHUP on EOF. This is currently only used by fifo_poll(). > The fifofs regress now passes. OK guenther@ (millert@) net ~ if_ethersubr.c ~ if_trunk.c ~ if_trunk.h ~ trunklacp.c ~ trunklacp.h > Take trunk(4) out of ether_input(). > Each physical interface of a trunk now gets a specific ifih (interface > input handler) that runs before ether_input(). > Tested by sthen@, dlg@, Theo Buehler and <mxb AT alumni.chalmers DOT se> > ok sthen@, dlg@ (mpi@) ~ pf.c > fix a potential use-after-free in pf_state_rm_src_node > found by jsg; ok jsg mikeb (henning@) ~ bridgestp.c > This file is only compiled when "pseudo-device bridge" is included in > your kernel configuration, no need for a #if NBRIDGE > 0 dance. (mpi@) ~ if_ethersubr.c ~ if_trunk.c ~ if_trunk.h > Get rid of the last "#if NTRUNK" by overwriting trunk ports' output > function. > ok claudio@, reyk@ (mpi@) ~ bpf.c ~ bpf_filter.c ~ if_ppp.c ~ raw_usrreq.c ~ rtsock.c > test mbuf pointers against NULL not 0 > ok krw@ miod@ (jsg@) ~ if_trunk.c ~ if_trunk.h > Allocate the input packet handler as part of the trunk_port structure > since they have the same lifetime. > Requested by and ok dlg@ (mpi@) ~ bridgestp.c ~ if.c ~ if_bridge.c ~ if_ethersubr.c ~ if_gif.c ~ if_mpe.c ~ if_pppx.c ~ if_spppsubr.c ~ if_trunk.c ~ if_tun.c ~ if_var.h ~ if_vlan.c ~ trunklacp.c > Introduce if_output(), a function do to the last steps before enqueuing > a packet on the sending queue of an interface. > Tested by many, thanks a lot! > ok dlg@, claudio@ (mpi@) ~ if_var.h > Give carp(4) interfaces their own low priority. The change should not > change behaviour for now but will allow to share the same address with > the parent interface without major hacks. > OK mpi@ (claudio@) ~ route.c > Allow multiple connected/interface routes to exist at the same time. > Use the existing multipath code. Switch away from using the ifa address > when making the cloning route and instead put a dummy sockaddr_dl route > in. With this it is possible to use the same network on multiple interfaces > at the same time. So if wireless and ethernet share the same network > the system will use the wired connection as long as there is link. > Still missing is builtin proxy-arp for the other interface IPs to allow > hitless failover. > OK mpi@ (claudio@) ~ if_tun.c > Remove useless comments mentioning ether_output(). (mpi@) netinet ~ in_gif.c > MPLS also needs a definition for etherip_output(), fix build without > bridge. (mpi@) ~ ip_icmp.c ~ ip_input.c ~ ip_ipip.c ~ ip_output.c ~ ipsec_input.c ~ tcp_input.c ~ tcp_subr.c > test mbuf pointers against NULL not 0 > ok krw@ miod@ (jsg@) ~ ip_carp.c > Remove a NULL check in carp_set_ifp() as we always pass a valid ifp > pointer to this function. > ok claudio@ (mpi@) ~ ip_carp.c > Give carp(4) interfaces their own low priority. The change should not > change behaviour for now but will allow to share the same address with > the parent interface without major hacks. > OK mpi@ (claudio@) ~ if_ether.c ~ in.c > Allow multiple connected/interface routes to exist at the same time. > Use the existing multipath code. Switch away from using the ifa address > when making the cloning route and instead put a dummy sockaddr_dl route > in. With this it is possible to use the same network on multiple interfaces > at the same time. So if wireless and ethernet share the same network > the system will use the wired connection as long as there is link. > Still missing is builtin proxy-arp for the other interface IPs to allow > hitless failover. > OK mpi@ (claudio@) netinet6 ~ in6_gif.c > MPLS also needs a definition for etherip_output(), fix build without > bridge. (mpi@) ~ ip6_output.c ~ udp6_output.c > test mbuf pointers against NULL not 0 > ok krw@ miod@ (jsg@) ~ in6.c ~ nd6.c ~ nd6_rtr.c > Allow multiple connected/interface routes to exist at the same time. > Use the existing multipath code. Switch away from using the ifa address > when making the cloning route and instead put a dummy sockaddr_dl route > in. With this it is possible to use the same network on multiple interfaces > at the same time. So if wireless and ethernet share the same network > the system will use the wired connection as long as there is link. > Still missing is builtin proxy-arp for the other interface IPs to allow > hitless failover. > OK mpi@ (claudio@) sys ~ poll.h > Set POLLHUP even if no valid events were specified as per POSIX. > Since we use the poll backend for select(2), care must be taken not > to set the fd's bit in writefds in this case. A kernel-only flag, > POLLNOHUP, is used by selscan() to tell the poll backend not to > return POLLHUP on EOF. This is currently only used by fifo_poll(). > The fifofs regress now passes. OK guenther@ (millert@) ~ fcntl.h > isatty() is used by stdio to determine the buffering mode. Add a F_ISATTY > option to fcntl(), so that isatty() can use this rather than than the > bloated > ioctl() interface. Reducing uses of ioctl() by libc makes it easier to > constrain programs with various kinds of systrace sandboxes. > ok guenther, previously discussed as a concept with nicm (deraadt@) ~ conf.h > for decades, wsdisplay has acted in one way like it is not a tty (deraadt@) == usr.bin =========================================================== 08/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.bin aucat ~ aucat.c > Call slot_stop() after the play buffer is drained to properly cleanup > the slot. Fixes periodic glitches occurring after eof is reached, and > then playback restarted with a MMC-relocate and MMC-start. (ratchov@) audioctl ~ audioctl.c > remove reference to encodings the kernel doesn't expose anymore (ratchov@) ~ audioctl.c > Display simply "play" when play mode is set (furthermore, "playsync", > aka PLAY_ALL mode is not used anymore) (ratchov@) calendar ~ calendars/calendar.holiday > New Year's Day (czarkoff@) ~ calendars/calendar.holiday > Montenegrin holidays > OK jmc@, sthen@ (same for previous comment) (czarkoff@) kdump ~ mksubr > isatty() is used by stdio to determine the buffering mode. Add a F_ISATTY > option to fcntl(), so that isatty() can use this rather than than the > bloated > ioctl() interface. Reducing uses of ioctl() by libc makes it easier to > constrain programs with various kinds of systrace sandboxes. > ok guenther, previously discussed as a concept with nicm (deraadt@) nm ~ elf.c ~ nm.1 ~ nm.c ~ util.h > Add -D option for displaying the dynamic symbol table > ok miod@ (guenther@) ~ nm.c > Add 'D' to usage optstring. Prodded by jmc@ (guenther@) renice ~ renice.8 ~ renice.c > rework synopsis to be a bit less ugly; the result is quite close to the > posix > spec... (jmc@) ssh ~ ssh-agent.c > Use a salted hash of the lock passphrase instead of plain text and do > constant-time comparisons of it. Should prevent leaking any information > about > it via timing, pointed out by Ryan Castellucci. Add a 0.1s incrementing > delay > for each failed unlock attempt up to 10s. ok markus@ (earlier version), > djm@ (dtucker@) tmux ~ input.c ~ tmux.1 ~ tmux.h ~ window.c > Remove the c0-* options which never really worked satisfactorily. Going > to try something else... (nicm@) ~ options-table.c > Left the c0-* options behind in the table. (nicm@) ~ options-table.c ~ server-window.c ~ tmux.1 ~ tmux.h > Add bell-action "other" to pass through bells in all windows except the > current, suggested by Jan ONDREJ. (nicm@) ~ format.c ~ options-table.c ~ tmux.1 > Add a session_alerts format which is a list of all the alerts in the > current session in symbolic form (something like "0!,4~,5!"). Use this > in the default set-titles-string. Prompted by a request from Jan ONDREJ. > (nicm@) ~ cmd-show-messages.c > Tidy blank lines when outputting server info. (nicm@) ~ tmux.h ~ tty.c ~ window.c > To replace c0-*, add a high watermark to the pty event, and also backoff > when the any of the ttys the pane is going to write to has buffered > enough data. (nicm@) == usr.sbin ========================================================== 09/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.sbin npppd ~ npppd/log.c ~ npppd/log.h > Move the logit() prototype so parse.y will see it. > ok yasuoka@ (jsg@) ~ pptp/pptp_ctrl.c > Fix an uninitialized variable access when pptp.hostname is configured. > (yasuoka@) ntpd ~ constraint.c ~ ntpd.h ~ parse.y > When resolving the "constraint" (singular), store all returned IP > addresses and try one after another until the connection succeeded - > based on the existing mechanism of "server". "constraint" previously > only tried to connect to the first returned address, aborted and > skipped the constraint on failure. In difference to "constraints" > (plural), it still only connects to one address at a time and not to > all of them at once. > Pointed out by rpe@ > OK rpe@ deraadt@ (reyk@) pkg_add ~ OpenBSD/PkgAdd.pm > add missing placeholder so that the "UNSIGNED PACKAGES: " line actually > contains the list of packages. > ok espie@ (jasper@) relayd ~ relayd.conf.5 > client/server mox ip; > from trondd (jmc@) ~ relayd.conf.5 > Fix kill'n'yank error: the port is mandatory in relay listen on > statements. > Pointed out by Alex Greif > OK jmc@ (reyk@) smtpd ~ enqueue.c > The enqueuer should never encouter the "From " separator in its headers, as > it is added by the mda. If it sees one, it means that the enqueuer was used > as the mda and it needs to strip it otherwise the message will end with two > delimiters later down the road. Same applies to "Return-Path". > bug experienced by James Turner, confirmed by Giovanni. > fix suggested by Todd Miller, diff ok eric@ (gilles@) ~ smtp_session.c > the code to prevent AUTH PLAIN from logging credentials upon authentication > failure does not catch the AUTH LOGIN case. rework to use the session state > rather than using the session command. > spotted by [email protected] (gilles@) =============================================================================== _______________________________________________ owc mailing list [email protected] http://www.squish.net/mailman/listinfo/owc
