OpenBSD src changes summary for 2016-01-27 ==========================================
distrib/sets distrib/special etc/MAKEDEV.common etc/Makefile etc/etc.amd64/MAKEDEV etc/etc.amd64/MAKEDEV.md etc/etc.i386/MAKEDEV etc/etc.i386/MAKEDEV.md etc/examples/pkg.conf games/arithmetic lib/librthread lib/libssl regress/lib sbin/iked sbin/pdisk share/man sys/arch/amd64/amd64 sys/arch/i386/i386 sys/ddb sys/dev/pv sys/msdosfs sys/net sys/sys usr.bin/less usr.bin/ssh usr.bin/vi usr.sbin/hostctl usr.sbin/lpr usr.sbin/ntpd usr.sbin/pkg_add usr.sbin/smtpd == distrib =========================================================== 01/10 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/distrib sets ~ lists/base/mi > sync (sthen@) special ~ pdisk/Makefile > Check block 0 signature, physical block size and physical block count when > reading partition map. > Check for unmapped physical blocks and overlapping partitions when reading > partition map. > No need for duplicate checks in validate_map(), so remove validate.c from > build and 'v' command from code and docs. (krw@) == etc =============================================================== 02/10 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/etc MAKEDEV.common ~ MAKEDEV.common > Add /dev/pvbus0 (mode 0640) to MAKEDEV on i386 and amd64. > Needed for the key-value interface that has been added to pvbus(4). > OK mikeb@ (reyk@) Makefile ~ Makefile > Add /etc/examples/pkg.conf, generated from mirrors.dat over in www/. > Positive feedback and/or OKs from benno millert jcs aja jasper (sthen@) etc.amd64/MAKEDEV ~ etc.amd64/MAKEDEV > sync (reyk@) etc.amd64/MAKEDEV.md ~ etc.amd64/MAKEDEV.md > Add /dev/pvbus0 (mode 0640) to MAKEDEV on i386 and amd64. > Needed for the key-value interface that has been added to pvbus(4). > OK mikeb@ (reyk@) etc.i386/MAKEDEV ~ etc.i386/MAKEDEV > sync (reyk@) etc.i386/MAKEDEV.md ~ etc.i386/MAKEDEV.md > Add /dev/pvbus0 (mode 0640) to MAKEDEV on i386 and amd64. > Needed for the key-value interface that has been added to pvbus(4). > OK mikeb@ (reyk@) examples/pkg.conf + examples/pkg.conf > Add /etc/examples/pkg.conf, generated from mirrors.dat over in www/. > Positive feedback and/or OKs from benno millert jcs aja jasper (sthen@) == games ============================================================= 03/10 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/games arithmetic ~ arithmetic.c > - optarg and optind are declared by unistd.h, so remove them... > - use strtonum rather than atoi > - zap case '?' in getopt(3) switch > - use _exit(2) in signal handler > - use __progname in usage() instead of hardcoded name > OK tb@ mestre@ (gsoares@) == lib =============================================================== 04/10 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/lib librthread ~ rthread.h ~ rthread_fork.c ~ rthread_libc.c > Replace the malloc spinlock with a mutex. This lock is held over system > calls > which run for many cycles and may even sleep. This leads to other threads > spinning for a long time waiting on the lock. Using a mutex means those > threads go to sleep and get woken up when the lock is released, which > results > in a lot less CPU usage. More work is needed to improve the performance of > threaded code that suffers from malloc lock contention, but this diff makes > ports like Firefox significantly more usable. > Tested by many. > ok mpi@, guenther@, tedu@, jca@ (kettenis@) libssl ~ src/ssl/s3_lib.c ~ src/ssl/s3_srvr.c > deprecate SSL_OP_SINGLE_DH_USE > ok jsing@ (beck@) ~ src/ssl/s3_lib.c ~ src/ssl/s3_srvr.c TAGGED OPENBSD_5_8 > deprecate SSL_OP_SINGLE_DH_USE > ok jsing@ (beck@) ~ src/ssl/s3_lib.c ~ src/ssl/s3_srvr.c TAGGED OPENBSD_5_7 > deprecate SSL_OP_SINGLE_DH_USE > ok jsing@ (beck@) == regress =========================================================== 05/10 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/regress lib ~ libpthread/restart/accept/accept.c ~ libpthread/restart/recvfrom/recvfrom.c ~ libpthread/restart/recvmsg/recvmsg.c > Properly initialize sin_family. (jca@) == sbin ============================================================== 06/10 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sbin iked ~ types.h > fyx typo. s,dynanic,dynamic, > OK reyk (gsoares@) pdisk ~ dump.c ~ io.c ~ partition_map.c > Nuke another (and I hope final) batch of superfluous '{}' around single > statements. (krw@) ~ io.c > Nope. Missed a couple of '{}' that could go. (krw@) ~ Makefile ~ dpme.h ~ partition_map.c ~ pdisk.8 ~ pdisk.c > Check block 0 signature, physical block size and physical block count when > reading partition map. > Check for unmapped physical blocks and overlapping partitions when reading > partition map. > No need for duplicate checks in validate_map(), so remove validate.c from > build and 'v' command from code and docs. (krw@) - validate.c - validate.h > Two more for the attic. (krw@) ~ dump.c ~ partition_map.c ~ partition_map.h ~ pdisk.c > We do not support logical_block != physical_block. Remove logical_block > field and left over code that checks for logical != physical. > Removes confusion with dmpe fields lblock_start and lblocks, which > have nothing to do with block sizes! lblock_start is the block > offset within the partition where the data actually starts. and > lblocks is the number of blocks of data within the partition. Both > are in units of *physical blocks*, a.k.a. disk sectors. (krw@) ~ pdisk.8 > Remove some lies and pointless verbiage. Tweak HISTORY. (krw@) ~ dpme.h ~ io.c ~ pdisk.c > On-disk partition name/types/processorid may or may not be NUL > terminated after all. So add an extra byte for a NUL in the in-memory > struct's. (krw@) ~ pdisk.8 > tweak previous; ok krw (jmc@) ~ dump.c > make sure to check malloc(3) return pointer against NULL. > OK krw@ (gsoares@) ~ pdisk.8 ~ pdisk.c > Tweak some verbiage. (krw@) ~ pdisk.8 > Clear out some repetitive and pointless verbiage. Shuffle other > verbiage around to make things clearer. (krw@) ~ dump.c > No need to copy dmpe_name to a buffer before printing it. (krw@) ~ partition_map.c > Traverse next_by_base when checking maps coverage, not next_on_disk! (krw@) == share ============================================================= 07/10 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/share man ~ man8/man8.amd64/MAKEDEV.8 ~ man8/man8.i386/MAKEDEV.8 > sync (reyk@) == sys =============================================================== 08/10 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sys arch/amd64/amd64 ~ conf.c > Add a key-value interface to pvbus(4) that allows to get or set values > in the underlying information store of the host from the OpenBSD-VM's > userspace. OpenBSD did not provide access to these stores before, > mostly because we did not want to add a custom tool and interface for > each hypervisor. The pvbus(4) interface provides backends for > xen(4)'s XenStore and vmt(4)'s VMware Tools "guestinfo". These > information stores are fairly different, XenStore is a "filesystem" > while vmt is a RPC, and the key-value abstraction limits them a bit > but provides the most wanted functionality. > Discussed with many > OK mikeb@ (reyk@) arch/i386/i386 ~ conf.c > Add a key-value interface to pvbus(4) that allows to get or set values > in the underlying information store of the host from the OpenBSD-VM's > userspace. OpenBSD did not provide access to these stores before, > mostly because we did not want to add a custom tool and interface for > each hypervisor. The pvbus(4) interface provides backends for > xen(4)'s XenStore and vmt(4)'s VMware Tools "guestinfo". These > information stores are fairly different, XenStore is a "filesystem" > while vmt is a RPC, and the key-value abstraction limits them a bit > but provides the most wanted functionality. > Discussed with many > OK mikeb@ (reyk@) ddb ~ db_sym.c ~ db_sym.h > Remove unused db_sifting(). (mpi@) ~ db_elf.c ~ db_hangman.c ~ db_sym.c ~ db_sym.h > Removes the abstraction layer to support multiple executable binaries. > Simplifies the API to manipulate symbols now that all our architectures > are ELF. > ok tb@ (mpi@) dev/pv ~ pvbus.c ~ pvvar.h ~ vmt.c ~ xen.c ~ xenstore.c ~ xenvar.h > Add a key-value interface to pvbus(4) that allows to get or set values > in the underlying information store of the host from the OpenBSD-VM's > userspace. OpenBSD did not provide access to these stores before, > mostly because we did not want to add a custom tool and interface for > each hypervisor. The pvbus(4) interface provides backends for > xen(4)'s XenStore and vmt(4)'s VMware Tools "guestinfo". These > information stores are fairly different, XenStore is a "filesystem" > while vmt is a RPC, and the key-value abstraction limits them a bit > but provides the most wanted functionality. > Discussed with many > OK mikeb@ (reyk@) ~ xen.c > shorten a few long lines (mikeb@) ~ xen.c > xen_bus_dma_init turned out to be unnecessary (mikeb@) ~ xen.c ~ xenvar.h > Reimplement Grant Table metadata linking and enable dynamic allocation > Instead of pre-allocating maximum number of Grant Table frames allotted by > the hypervisor we switch over to allocating them dynamically when the need > arises. At the same time we no longer link metadata entries representing > individual Grant Table frames as a list and use a table instead to speed > up reference lookups when establishing and removing mappings. (mikeb@) ~ xen.c > Tighten up next reference calculation, printing fixes (mikeb@) msdosfs ~ msdosfs_vnops.c > Convert min + uiomovei to ulmin + uiomove. > From Martin Natano. (stefan@) net ~ if_pfsync.c > white space tweaks. no functional change. (dlg@) sys ~ conf.h > Add a key-value interface to pvbus(4) that allows to get or set values > in the underlying information store of the host from the OpenBSD-VM's > userspace. OpenBSD did not provide access to these stores before, > mostly because we did not want to add a custom tool and interface for > each hypervisor. The pvbus(4) interface provides backends for > xen(4)'s XenStore and vmt(4)'s VMware Tools "guestinfo". These > information stores are fairly different, XenStore is a "filesystem" > while vmt is a RPC, and the key-value abstraction limits them a bit > but provides the most wanted functionality. > Discussed with many > OK mikeb@ (reyk@) ~ videoio.h > Include sys/ioccom.h since _IO* is used. > Based on the number and type of other system headers which pull in > ioccom.h, and > the fact that videoio.h isn't involved in any massive chain of includes in > the > kernel, I think it's reasonable and makes porting simpler. -- sthen@ > from Brad > very reasonable feinerer@, ok sthen@ (ajacoutot@) == usr.bin =========================================================== 09/10 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.bin less ~ lesskey.c > missing fclose(3). (silence cppcheck warnings). > OK mmcc@ (gsoares@) ssh ~ auth2-pubkey.c > make the debug messages a bit more useful here (djm@) ~ auth-krb5.c > change old $FreeBSD version string in comment so it doesn't become an > RCS ident downstream; requested by des AT des.no (djm@) ~ clientloop.c TAGGED OPENBSD_5_8 > When backporting the xauth fix, I had corrected for the malloc/stack > difference for xauth*dir* but missed the xauth*file* case. Fix from > Matthias Pitzl at genua and confirmed by markus@ (sthen@) ~ clientloop.c TAGGED OPENBSD_5_7 > When backporting the xauth fix, I had corrected for the malloc/stack > difference for xauth*dir* but missed the xauth*file* case. Fix from > Matthias Pitzl at genua and confirmed by markus@ (sthen@) vi ~ cl/cl_main.c > Replace fprintf+exit with errx. No functional change > OK zhuk@ and schwarze@ > fine with me tb@ (martijn@) ~ common/main.c > remove v_estr in favor of warn and warnx > OK schwarze@, zhuk@ > fine with me tb@ (martijn@) == usr.sbin ========================================================== 10/10 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.sbin hostctl + Makefile + hostctl.8 + hostctl.c > Add hostctl(8), a tool to access key-value stores on the host, > currently for hypervisor information stores on pvbus(4). > As discussed with deraadt@, the generic name is used to potentially > extend it for other use cases where the host or machine firmware > provides a key-value store, hypervisors or things like openprom. > Not enabled yet. > OK mikeb@ (reyk@) ~ hostctl.c > Use "VIS_SAFE | VIS_CSTYLE" instead of "VIS_OCTAL" as the value can > contain text files with whitespace, newlines and carriage returns. (reyk@) ~ hostctl.8 > appease mandoc lint, no difference in output > ok reyk@ (jasper@) ~ hostctl.8 ~ hostctl.c > - sort options > - add -q to usage() (jmc@) lpr ~ lpd/lpdchar.c > Fix typo in comment. From Chris Bennett, thanks! (tb@) ntpd ~ ntpd.c ~ ntpd.h > update ntpd log initialization to work like relayd, fix debug log levels > ok reyk@ (bcook@) ~ constraint.c ~ ntpd.c ~ ntpd.h > Don't attempt to kill() the constraint in the wrong process. The > process management of the contraint processes has been moved from ntp > to the parent, for better privsep and pledge, but the ntp process > still attempted to kill the constraints on timeout directly. Fix this > regression by introducing a new imsg from ntp to the parent and the > related logic to kill a constraint at the right place. > Reported & tested by bcook@ > Ok bcook@ (reyk@) pkg_add ~ pkg.conf.5 ~ OpenBSD/PackageLocator.pm > Allow pkg.conf's installpath to be set to a bare hostname (no /), which > implies http://hostname/pub/OpenBSD/[snapshots-or-version]/packages/[arch]. > This could be handled at a different location in the code to support > PKG_PATH > but this approach is pretty simple and I have something else to make use of > it. > ok espie@ for now. (sthen@) smtpd ~ filter.c > send events to filters in a simple loop, rather than using the > query-chaining machinery. > ok jung@ gilles@ (eric@) ~ queue.c ~ smtpd.c ~ smtpd.h > Check imsg data length before use. > Ok jung@ gilles@ eric@ (sunil@) =============================================================================== _______________________________________________ odc mailing list [email protected] http://www.squish.net/mailman/listinfo/odc
