OpenBSD src changes summary for 2016-03-13 to 2016-03-20 inclusive ==================================================================
bin/csh bin/df bin/ksh bin/ls bin/ps distrib/notes distrib/sets etc/mail/spamd.conf etc/mtree/BSD.x11.dist games/atc games/hack games/sail games/snake games/tetris include/math.h include/stdio.h include/tib.h lib/csu lib/libc lib/libedit lib/libevent lib/libexpat lib/librthread lib/libskey lib/libssl libexec/ftpd libexec/getty libexec/ld.so libexec/spamlogd libexec/talkd regress/lib regress/libexec regress/sys sbin/fsck_ext2fs sbin/growfs sbin/isakmpd sbin/newfs sbin/newfs_ext2fs share/locale share/man share/zoneinfo sys/arch/alpha/alpha sys/arch/amd64/amd64 sys/arch/amd64/include sys/arch/amd64/stand/pxeboot sys/arch/arm/arm sys/arch/arm/armv7 sys/arch/arm/conf sys/arch/arm/include sys/arch/hppa/hppa sys/arch/i386/i386 sys/arch/i386/include sys/arch/i386/stand/pxeboot sys/arch/m88k/m88k sys/arch/macppc/dev sys/arch/macppc/stand sys/arch/mips64/mips64 sys/arch/octeon/conf sys/arch/octeon/dev sys/arch/octeon/stand/boot sys/arch/powerpc/powerpc sys/arch/sgi/localbus sys/arch/sgi/stand/boot sys/arch/sparc/dev sys/arch/sparc/stand/common sys/arch/sparc64/dev sys/arch/sparc64/sparc64 sys/arch/sparc64/stand/ofwboot sys/arch/zaurus/stand/zboot sys/arch/zaurus/stand/zbsdmod sys/dev sys/dev/acpi sys/dev/ata sys/dev/ic sys/dev/isa sys/dev/ofw sys/dev/pci sys/dev/pcmcia sys/dev/sbus sys/dev/sdmmc sys/dev/sun sys/dev/usb sys/isofs/cd9660 sys/isofs/udf sys/kern sys/lib/libkern sys/lib/libsa sys/lib/libz sys/miscfs/fifofs sys/miscfs/fuse sys/msdosfs sys/net sys/netinet sys/netinet6 sys/nfs sys/ntfs sys/scsi sys/sys sys/tmpfs sys/ufs/ext2fs sys/ufs/ffs sys/ufs/mfs sys/ufs/ufs sys/uvm usr.bin/column usr.bin/deroff usr.bin/ftp usr.bin/head usr.bin/hexdump usr.bin/kdump usr.bin/less usr.bin/locale usr.bin/mandoc usr.bin/mg usr.bin/quota usr.bin/rusers usr.bin/sed usr.bin/showmount usr.bin/ssh usr.bin/talk usr.bin/telnet usr.bin/tftp usr.bin/tmux usr.bin/unifdef usr.bin/vi usr.bin/w usr.sbin/amd usr.sbin/config usr.sbin/edquota usr.sbin/fdformat usr.sbin/inetd usr.sbin/iscsid usr.sbin/ldapd usr.sbin/lpr usr.sbin/npppd usr.sbin/pkg_add usr.sbin/pppd usr.sbin/rbootd usr.sbin/repquota usr.sbin/tcpdump usr.sbin/vmctl usr.sbin/vmd usr.sbin/zdump usr.sbin/zic == bin =============================================================== 01/13 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/bin csh ~ csh.c > Replace ioctl(fd, FIOCLEX) with fcntl(fd, F_SETFD, FD_CLOEXEC) > No functional change. "I like the idea" from guenther@ (millert@) ~ csh.c ~ sem.c > Turn off the close-on-exec bit with fcntl() instead of ioctl(). > Remove some now useless "#include <ioctl.h>" lines. > Inspired by & ok millert@ (krw@) ~ lex.c > Use fcntl(fd, F_SETFL, flags) to clear O_NONBLOCK instead of > ioctl(fd, FIONBIO, ...). OK krw@ (millert@) df ~ df.c > replace __progname with getprogname(3) > from Michal Mazurek, ok tb@ (mmcc@) ksh ~ edit.c ~ jobs.c ~ trap.c > remove some ARGSUSED (mmcc@) ~ io.c ~ misc.c ~ shf.c > Currently we have about a 50/50 split over fcntl(n, F_GETFL [,0]) > idioms. > Adopt the more concise fcntl(n, F_GETFL) over fcntl(n, F_GETFL, 0) > where it is obvious further investigation will not yield and > even better way. > Obviousness evaluation and ok guenther@ (krw@) ls ~ ls.1 ~ ls.c > Switch (non-curses, non-ksh) programs that use COLUMNS to a single idiom. > Previously behaviors were all over the map. This changes them to > use COLUMNS first, and either terminal width or a hardcoded value > (typically 80) as appropriate. > ok deraadt@; man bits ok jmc@ (bentley@) ~ ls.1 > simplify previous; ok bentley (jmc@) ps ~ ps.1 ~ ps.c > Switch (non-curses, non-ksh) programs that use COLUMNS to a single idiom. > Previously behaviors were all over the map. This changes them to > use COLUMNS first, and either terminal width or a hardcoded value > (typically 80) as appropriate. > ok deraadt@; man bits ok jmc@ (bentley@) ~ ps.1 > simplify previous; ok bentley (jmc@) == distrib =========================================================== 02/13 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/distrib notes ~ i386/hardware > 'accomodate' -> 'accommodate' in comments. > Started by diff from Mical Mazurek. (krw@) sets ~ lists/comp/mi > Remove now unused legacy uiomovei() function. > All its callers got reviewed and converted to > use uiomove() properly. > ok deraadt@ (stefan@) ~ lists/base/mi > sync (deraadt@) ~ lists/base/md.alpha ~ lists/base/md.amd64 ~ lists/base/md.armish ~ lists/base/md.armv7 ~ 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.zaurus > sync (deraadt@) ~ lists/base/mi > sync (deraadt@) == etc =============================================================== 03/13 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/etc mail/spamd.conf ~ mail/spamd.conf > fix word issue, as reported by hans (jmc@) mtree/BSD.x11.dist ~ mtree/BSD.x11.dist > add include directories used by recent versions of libdrm (jsg@) == games ============================================================= 04/13 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/games atc ~ log.c > Prefer fseek(3) over rewind(3) since the latter although it also calls > fseek > then additionally it calls clearerr(3) deliberately but we want to catch > any > error that may happen and this way we couldn't catch it > OK tb@ (mestre@) hack ~ hack.pager.c ~ hack.unix.c > execl(3) calls must have the last argument casted to (char *)NULL > At the time when this was commited me and tb@ discussed that it shouldn't > be > changed, but still snucked in within a larger diff that we didn't notice. > OK tb@ (mestre@) ~ hack.rumors.c > Prefer fseek(3) over rewind(3) since the latter although it also calls > fseek > then additionally it calls clearerr(3) deliberately but we want to catch > any > error that may happen and this way we couldn't catch it > OK tb@ (mestre@) sail ~ misc.c > Prefer fseek(3) over rewind(3) since the latter although it also calls > fseek > then additionally it calls clearerr(3) deliberately but we want to catch > any > error that may happen and this way we couldn't catch it > OK tb@ (mestre@) snake ~ snake.c > Prefer fseek(3) over rewind(3) since the latter although it also calls > fseek > then additionally it calls clearerr(3) deliberately but we want to catch > any > error that may happen and this way we couldn't catch it > OK tb@ (mestre@) tetris ~ scores.c > Prefer fseek(3) over rewind(3) since the latter although it also calls > fseek > then additionally it calls clearerr(3) deliberately but we want to catch > any > error that may happen and this way we couldn't catch it > OK tb@ (mestre@) == include =========================================================== 05/13 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/include math.h ~ math.h > Remove #ifdef __vax__ bits > ok deraadt@ tb@ (who had the same diff) (jca@) stdio.h ~ stdio.h > Expose snprintf() and vsnprintf() for XPG 5. It predates C99 but > still specified snprintf() and vsnprintf() based on earlier drafts. > Allows snprintf() and vsnprintf() to be used when _XOPEN_SOURCE is > defined as 500. OK guenther@ (millert@) ~ stdio.h > remove cpp version of __sputc, since the inline version is all we have > ever used. > ok jsg (deraadt@) tib.h ~ tib.h > Prepare for future ld.so/libc bump: update <tib.h> with the definitions > that will be needed and make libpthread work when ld.so/libc.a provide an > initial TIB. (guenther@) == lib =============================================================== 06/13 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/lib csu ~ alpha/md_init.h ~ hppa/md_init.h ~ hppa64/md_init.h ~ mips64/md_init.h ~ sparc64/md_init.h > crt0.c is already setting environ, so don't set it in MD_START_SETUP > ok kettenis@ mpi@ (guenther@) ~ crt0.c ~ alpha/md_init.h ~ amd64/md_init.h ~ arm/md_init.h ~ hppa/md_init.h ~ hppa64/md_init.h ~ i386/md_init.h ~ m88k/md_init.h ~ mips64/md_init.h ~ powerpc/md_init.h ~ sh/md_init.h ~ sparc/md_init.h ~ sparc64/md_init.h > Rearrange C runtime bits: now that ld.so exports environ and __progname, > move their definitions and initialization in static links to libc.a > Make crt0 always invoke a new func _csu_finish() in libc to process the > auxv > and to either register the ld.so cleanup function (in dynamic links) or > initialize environ and __progname and do MC_DISABLE_KBIND (in static > links). > In libc, get pagesize from auxv; cache that between getpagesize() and > sysconf(_SC_PAGESIZE) > ok mpi@ "good time" deraadt@ (guenther@) libc ~ gen/exec.c ~ gen/getprogname.c ~ gen/posix_spawn.c ~ gen/setproctitle.c ~ gen/setprogname.c ~ gen/syslog_r.c ~ gen/verr.c ~ gen/verrc.c ~ gen/verrx.c ~ gen/vwarn.c ~ gen/vwarnc.c ~ gen/vwarnx.c ~ gmon/gmon.c ~ hidden/stdlib.h ~ stdlib/getenv.c ~ stdlib/malloc.c ~ stdlib/setenv.c ~ stdlib/system.c ~ sys/stack_protector.c > environ and __progname are not declared in a public header; declare them > in libc's hidden/stdlib.h instead of in each .c file that needs one > ok deraadt@ gsoares@ mpi@ (guenther@) ~ gmon/gmon.c > (char *)0 -> NULL (mmcc@) ~ time/localtime.c > Remove doaccess variable and access(2) call since this interfers with > applications like zdump(8) because pledge(2) doesn't allow access(2) to > /usr/share/zoneinfo. > millert@ better described why this call can go away: > "This looks like an attempt to do access checks based on the real uid > instead > of the effective uid. Basically for setuid programs we don't want to allow > a > user to set TZ to a path they should not be able to otherwise access. > However, we already have a check for issetugid() above so I think the > doaccess > bits can just be removed and we can rely on open()." > After discussion with tb@, deraadt@ and millert@, this was also OK'ed by > them (mestre@) ~ stdlib/malloc.c > small step towards multiple pools: move two globls into the struct dir_info > ok @stefan armani@ (otto@) ~ stdio/fseek.3 > remind people rewind is seldom a good choice > okay jmc@ (espie@) ~ stdlib/malloc.c > properly guard to macros > ok otto@ (mmcc@) ~ gen/fstab.c > remove NOTREACHED (mmcc@) ~ sys/thrkill.2 > This is thrkill, not __thrsigdivert (guenther@) ~ sys/socket.2 ~ sys/socketpair.2 > Add STANDARDS and HISTORY blurbs for SOCK_{NONBLOCK,CLOEXEC,DNS} > (guenther@) ~ stdio/fdopen.c > Currently we have about a 50/50 split over fcntl(n, F_GETFL [,0]) > idioms. > Adopt the more concise fcntl(n, F_GETFL) over fcntl(n, F_GETFL, 0) > where it is obvious further investigation will not yield and > even better way. > Obviousness evaluation and ok guenther@ (krw@) ~ Symbols.list ~ shlib_version ~ dlfcn/Makefile.inc ~ gen/getpagesize.c ~ gen/sysconf.c ~ hidden/unistd.h + arch/alpha/gdtoa/kbind.h + arch/amd64/gdtoa/kbind.h + arch/arm/gdtoa/kbind.h + arch/hppa/gdtoa/kbind.h + arch/hppa64/gdtoa/kbind.h + arch/i386/gdtoa/kbind.h + arch/m88k/gdtoa/kbind.h + arch/mips64/gdtoa/kbind.h + arch/powerpc/gdtoa/kbind.h + arch/sh/gdtoa/kbind.h + arch/sparc/gdtoa/kbind.h + arch/sparc64/gdtoa/kbind.h + dlfcn/init.c > Rearrange C runtime bits: now that ld.so exports environ and __progname, > move their definitions and initialization in static links to libc.a > Make crt0 always invoke a new func _csu_finish() in libc to process the > auxv > and to either register the ld.so cleanup function (in dynamic links) or > initialize environ and __progname and do MC_DISABLE_KBIND (in static > links). > In libc, get pagesize from auxv; cache that between getpagesize() and > sysconf(_SC_PAGESIZE) > ok mpi@ "good time" deraadt@ (guenther@) ~ dlfcn/init.c > Add line (guenther@) ~ sys/__get_tcb.2 > It's libpthread, not librthread, and __tfork(2) lets you set the TCB > address of the new thread (guenther@) ~ sys/__thrsleep.2 > __thrsleep.2 is contained in section 2 > ok jmc (natano@) ~ locale/Makefile.inc > LOCALE_CHARSETS has been unused for some time now > so no need to include share/locale/ctype/Makefile.inc to get it. > ok guenther@ stsp@ (who had the same diff) (jca@) libedit ~ readline.c > Currently we have about a 50/50 split over fcntl(n, F_GETFL [,0]) > idioms. > Adopt the more concise fcntl(n, F_GETFL) over fcntl(n, F_GETFL, 0) > where it is obvious further investigation will not yield and > even better way. > Obviousness evaluation and ok guenther@ (krw@) ~ chartype.h ~ read.c > Fix the CHARSET_IS_UTF8 case in read_char(). > For now, this mainly help programs explicitly using > wide-character functions like el_wgetc(3) and el_wgets(3). > 1. After reading an invalid byte sequence, do not throw away additional > valid bytes; fix by me using mbrtowc(3), obsoleting utf8_islead(). > 2. When read(2) returns EOF, return that information to the caller, > do not prod on and potentially access garbage data in the buffer; > from Linas Vepstas via NetBSD read.c rev. 1.70 2013/05/27. > 3. After read__fixio() failure, restore errno to the one set by read(); > from Steffen Nurpmeso via NetBSD read.c rev. 1.68 2012/09/10. > 4. After read__fixio() success, restore errno to the initial state > upon function entry; fix by me. > OK czarkoff@. Also committed to NetBSD. (schwarze@) ~ read.c > Fix read_char() for the non-UTF-8 case, in particular for systems > supporting other multibyte locales or having an internal representation > of wchar_t that doesn't match UCS-4. > No functional change on OpenBSD, but it makes the code less confusing. > OK czarkoff@. (schwarze@) ~ chartype.h ~ eln.c > Fix the public interface function el_getc(3). > On OpenBSD, the effects are to set the return argument to the NUL byte > in case of a read failure (for robustness) and to properly set errno > when the character is out of range and cannot be stored in a byte. > Once we enable UTF-8, this will be needed to avoid returning bogus > bytes for valid Unicode characters. > On systems where the internal representation of wchar_t doesn't > match UCS-4, breakage was potentially even worse. > OK czarkoff@. (schwarze@) ~ el.h ~ eln.c ~ read.c > Delete the weird IGNORE_EXTCHARS flag, simplifying the code > in the generic low-level function read_char(). > Until we fully enable UTF-8 support, instead filter out non-ASCII > characters in the more logical place in the high-level function > el_gets(3). > OK czarkoff@. (schwarze@) ~ chartype.c ~ chartype.h ~ read.c > Get rid of "#ifdef WIDECHAR" and one goto in read_char(), > making the code more readable. Instead, provide an mbrtowc(3) > replacement function in chartype.[hc], files that encapsulate > such system dependencies anyway. No functional change. > OK czarkoff@ (schwarze@) ~ chared.c ~ chared.h ~ chartype.h ~ common.c ~ emacs.c ~ keymacro.c ~ makelist ~ map.c ~ parse.c ~ refresh.c ~ refresh.h ~ search.c ~ search.h ~ terminal.c ~ terminal.h ~ tty.c ~ vi.c > Delete the useless Int datatype and always use the standard wint_t > directly. This is not a problem because <wchar_t> is required all > over the place anyway, even when WIDECHAR is not defined. > No functional change except that it fixes a few printf(3) > format string issues, %c vs. %lc. > OK czarkoff@ (schwarze@) ~ tokenizer.c > Fix the same bug again that was already fixed in: > OpenBSD tokenizer.c rev. 1.8 2003/08/11 18:21:40 deraadt > Don't increase amax on realloc() failure. > The original fix got lost in a merge along the way. > This fix from Christos Zoulas via NetBSD tokenizer.c rev. 1.23 2016/02/15. > OK czarkoff@ (schwarze@) ~ sig.c ~ sys.h ~ terminal.c > minor simplifactions, style fixes, and trivial syncs from NetBSD > (schwarze@) ~ config.h ~ el.c ~ history.c ~ sys.h > Use getline(3) rather than fgetln(3) because it is standardized > and simpler and safer to use. Implemented by Christos Zoulas > following my suggestion, with a bug fix by me. (schwarze@) ~ TEST/Makefile > support WIDECHAR; > OK czarkoff@ (schwarze@) ~ chared.c ~ chared.h ~ common.c ~ el.c ~ el.h ~ eln.c ~ emacs.c ~ filecomplete.c ~ hist.h ~ makelist ~ map.c ~ parse.c ~ prompt.h ~ readline.c ~ refresh.h ~ search.c ~ search.h ~ sig.c ~ sig.h ~ terminal.h ~ tty.c ~ tty.h ~ vi.c > Cleanup of private header inclusion: > 1. Do not include private headers from "chared.h", "hist.h", "prompt.h", > "refresh.h", "search.h", "sig.h", "terminal.h", "tty.h". > The only private header having to include other private headers is "el.h". > 2. Do not include "common.h", "parse.h", "help.h" from "el.h", > and do not include "emacs.h" and "vi.h" from "chared.h", > include them directly where needed. > 3. Do include "fcns.h" from "el.h" because el_func_t is needed for "map.h". > 4. Do not include private headers again that are already included by > "el.h". > 5. Include private headers after standard headers. > OK czarkoff@ (schwarze@) ~ common.c ~ el.h > delete useless "typedef int bool_t", use int directly; > OK czarkoff@ (schwarze@) ~ Makefile ~ chared.c ~ chared.h ~ chartype.c ~ chartype.h ~ common.c ~ el.c ~ el.h ~ emacs.c ~ filecomplete.c ~ hist.c ~ hist.h ~ histedit.h ~ history.c ~ keymacro.c ~ makelist ~ map.c ~ parse.c ~ prompt.h ~ read.c ~ read.h ~ readline.c ~ refresh.c ~ search.c ~ sig.c ~ sys.h ~ terminal.c ~ tokenizer.c ~ tty.c ~ vi.c ~ TEST/tc1.c ~ readline/Makefile ~ readline/readline.h > Cleanup of standard header inclusion: > 1. Add the missing <errno.h> to sig.c. > 2. Do not include standard headers from private headers "chared.h" > and "el.h", include them directly where needed. > 3. Delete a few needless inclusions of <ctype.h>. > 4. Sort the standard headers. > 5. Delete _GNU_SOURCE weirdness from histedit.h, that file doesn't even > need the access to wcsdup(3) mentioned in the comment. > 6. Delete some trailing blanks and blanks before tabs. > OK czarkoff@ (schwarze@) libevent ~ evutil.c > Currently we have about a 50/50 split over fcntl(n, F_GETFL [,0]) > idioms. > Adopt the more concise fcntl(n, F_GETFL) over fcntl(n, F_GETFL, 0) > where it is obvious further investigation will not yield and > even better way. > Obviousness evaluation and ok guenther@ (krw@) libexpat ~ doc/reference.html > " the the " -> " the ", or in a couple of cases replace the superfluous > "the" with the obviously intended word. > Started with a "the the" spotted by Mihal Mazurek. (krw@) librthread ~ rthread.c > Prepare for future ld.so/libc bump: update <tib.h> with the definitions > that will be needed and make libpthread work when ld.so/libc.a provide an > initial TIB. (guenther@) libskey ~ skeylogin.c > 'miliseconds' -> 'milliseconds' in comments. > if_atu.c noted by Michal Mazurek. (krw@) libssl ~ src/doc/crypto/EVP_PKEY_decrypt.pod ~ src/doc/crypto/EVP_PKEY_encrypt.pod > Fix examples for EVP_PKEY_CTX_set_rsa_padding. > Noted here, https://github.com/libressl-portable/portable/issues/161, we > document a non-existent constant in the examples for > EVP_PKEY_CTX_set_rsa_padding. > ok deraadt@ (bcook@) ~ src/crypto/bn/bn_lib.c ~ src/crypto/dso/dso.h ~ src/crypto/sha/sha_locl.h ~ src/crypto/sha/asm/sha1-mips.pl ~ src/crypto/whrlpool/wp_block.c > 'accomodate' -> 'accommodate' in comments. > Started by diff from Mical Mazurek. (krw@) ~ src/crypto/asn1/a_object.c > explicit_bzero for asn1 objects on free. Too often these contain sensitive > information > and they should not be a performance bottleneck > ok miod@ krw@ (beck@) ~ src/crypto/des/asm/des_enc.m4 ~ src/crypto/ec/ec_asn1.c ~ src/doc/apps/ca.pod ~ src/doc/apps/dgst.pod ~ src/doc/apps/x509.pod ~ src/ssl/s3_cbc.c > " the the " -> " the ", or in a couple of cases replace the superfluous > "the" with the obviously intended word. > Started with a "the the" spotted by Mihal Mazurek. (krw@) == libexec =========================================================== 07/13 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/libexec ftpd ~ ftpd.c > More "(<blah> *)0" -> NULL, avoiding any stdarg functions. > Feedback millert@ kettenis@ (krw@) getty ~ main.c ~ subr.c > More "(<blah> *)0" -> NULL, avoiding any stdarg functions. > Feedback millert@ kettenis@ (krw@) ld.so ~ dlfcn.c > More "(<blah> *)0" -> NULL, avoiding any stdarg functions. > Feedback millert@ kettenis@ (krw@) ~ Symbols.map ~ dlfcn.c ~ library.c ~ library_mquery.c ~ loader.c ~ resolve.c ~ resolve.h ~ util.c ~ alpha/rtld_machine.c ~ m88k/rtld_machine.c ~ mips64/rtld_machine.c ~ powerpc/rtld_machine.c > Export environ and __progname, making the latter a copy of just the > filename > portion like crt0 does. This is prep for eliminating _dl_fixup_user_env() > Mark almost everything in resolve.h as hidden, to improve code generation. > ok kettenis@ mpi@ "good time" deraadt@ (guenther@) spamlogd ~ spamlogd.c > - Check if user running spamlogd(8) has root privileges and if not then > stop > program early > - #define SPAMD_USER "_spamd" and use it on getpwnam(3) call > - Set usage() as __dead void > - Remove lint-style comments > OK beck@ (mestre@) talkd ~ process.c ~ table.c > More "(<blah> *)0" -> NULL, avoiding any stdarg functions. > Feedback millert@ kettenis@ (krw@) == regress =========================================================== 08/13 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/regress lib ~ libcrypto/bn/general/bntest.c > check return value for BN_hex2bn in regression tests (bcook@) ~ libpthread/stdfiles/stdfiles.c > Currently we have about a 50/50 split over fcntl(n, F_GETFL [,0]) > idioms. > Adopt the more concise fcntl(n, F_GETFL) over fcntl(n, F_GETFL, 0) > where it is obvious further investigation will not yield and > even better way. > Obviousness evaluation and ok guenther@ (krw@) ~ libedit/read/Makefile > Fix the CHARSET_IS_UTF8 case in read_char(). > For now, this mainly help programs explicitly using > wide-character functions like el_wgetc(3) and el_wgets(3). > 1. After reading an invalid byte sequence, do not throw away additional > valid bytes; fix by me using mbrtowc(3), obsoleting utf8_islead(). > 2. When read(2) returns EOF, return that information to the caller, > do not prod on and potentially access garbage data in the buffer; > from Linas Vepstas via NetBSD read.c rev. 1.70 2013/05/27. > 3. After read__fixio() failure, restore errno to the one set by read(); > from Steffen Nurpmeso via NetBSD read.c rev. 1.68 2012/09/10. > 4. After read__fixio() success, restore errno to the initial state > upon function entry; fix by me. > OK czarkoff@. Also committed to NetBSD. (schwarze@) libexec ~ ld.so/Makefile + ld.so/init-env/Makefile + ld.so/init-env/libaa/Makefile + ld.so/init-env/libaa/aa.C + ld.so/init-env/libaa/shlib_version + ld.so/init-env/prog/Makefile + ld.so/init-env/prog/prog.c > Add regress for environ and __progname vs load-time .init functions > (guenther@) sys + netinet6/pktinfo_addr/Makefile + netinet6/pktinfo_addr/runtest.c > Add regress tests for IPV6_PKTINFO with addr (vgross@) == sbin ============================================================== 09/13 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sbin fsck_ext2fs ~ main.c ~ utilities.c > More "(<blah> *)0" -> NULL, avoiding any stdarg functions. > Feedback millert@ kettenis@ (krw@) growfs ~ growfs.8 ~ growfs.c > Switch (non-curses, non-ksh) programs that use COLUMNS to a single idiom. > Previously behaviors were all over the map. This changes them to > use COLUMNS first, and either terminal width or a hardcoded value > (typically 80) as appropriate. > ok deraadt@; man bits ok jmc@ (bentley@) ~ growfs.8 > missing El in previous; (jmc@) ~ growfs.8 > simplify previous; ok bentley (jmc@) isakmpd ~ conf.c > More "(<blah> *)0" -> NULL, avoiding any stdarg functions. > Feedback millert@ kettenis@ (krw@) newfs ~ mkfs.c ~ newfs.8 > Switch (non-curses, non-ksh) programs that use COLUMNS to a single idiom. > Previously behaviors were all over the map. This changes them to > use COLUMNS first, and either terminal width or a hardcoded value > (typically 80) as appropriate. > ok deraadt@; man bits ok jmc@ (bentley@) ~ newfs.8 > simplify previous; ok bentley (jmc@) newfs_ext2fs ~ newfs_ext2fs.c > Add a missing "cpath" promise; newfs_ext2fs can create files when used > with the -F and -s flags. > ok stefan (natano@) ~ extern.h ~ mke2fs.c ~ newfs_ext2fs.c > Replace an ad-hoc implementation of opendev(), with said function and > open the device only once with the correct flags, instead of twice (one > file handle for reading and one for writing). Committing this at once > because the changes are interrelated. > ok stefan (natano@) ~ newfs_ext2fs.c > There's no reason to have execute permissions on a file system image. > ok millert (natano@) == share ============================================================= 10/13 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/share locale - ctype/Makefile.inc ~ ctype/Makefile > Kill LOCALECHARSETS machinery, we only support ASCII and UTF-8 LC_CTYPE > Remove Makefile.inc and inline rules to produce LC_CTYPE for UTF-8. > ok guenther@ stsp@ (jca@) man ~ man1/gcc-local.1 > Remove a reference to lint(1) in the description of the -CC option. The > option itself may go away sometime soon. > ok guenther@, deraadt@ (mmcc@) ~ man3/queue.3 ~ man3/tree.3 > some Xr; from michal mazurek (jmc@) ~ man9/Makefile ~ man9/uiomove.9 > Remove now unused legacy uiomovei() function. > All its callers got reviewed and converted to > use uiomove() properly. > ok deraadt@ (stefan@) ~ man9/uiomove.9 > spelling fix; (jmc@) ~ man4/audio.4 > Expose new audio ioctls that do one thing only: start and stop DMA, > set and get parameters. This is much simpler. > ok semarie, armani, tweaks from jmc (ratchov@) ~ man4/audio.4 > knock out unused Pp in previous; (jmc@) ~ man4/ipsec.4 > +.Xr ipsec.conf 5 , > from rob pierce (jmc@) ~ man9/VOP_LOOKUP.9 ~ man9/vinvalbuf.9 > Remove the unused flags argument from VOP_UNLOCK(). > torture tested on amd64, i386 and macppc > ok beck mpi stefan > "the change looks right" deraadt (natano@) ~ man9/VOP_LOOKUP.9 > uio -> uiomove; (jmc@) zoneinfo ~ datfiles/asia ~ datfiles/australasia ~ datfiles/backward ~ datfiles/europe ~ datfiles/iso3166.tab ~ datfiles/leapseconds ~ datfiles/northamerica ~ datfiles/zone.tab ~ datfiles/zone1970.tab > Update to tzdata2016b from ftp.iana.org. (millert@) == sys =============================================================== 11/13 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sys arch/alpha/alpha ~ lock_machdep.c > Reduces the noise around the global ``ticks'' variable by renaming > all the local ones to ``nticks''. > ok stefan@, deraadt@ (mpi@) arch/amd64/amd64 ~ vmm.c > Introduce memory ranges to support VMs with >= 4G RAM > Kernel bits: > - When creating a VM, a list of memory ranges has to be specified, > similar to the BIOS memory map. This is necessary for VMs with > RAM sizes approaching 4G because we'll need PCI MMIO space in > the higher parts of the 32 bit address space. > vmctl and vmd bits: > - Construct appropriate memory ranges to create a VM with a given > RAM size > - Construct a corresponding BIOS memory map from the memory ranges > and update the boot params page accordingly. > - Make sure that all variables that represent guest physical addresses > match the address width of the target CPU instead of using uint32_t. > - Fix some integer promotion glitches that actually restricted VM > RAM size to 2G. > This changes the VM create ioctl interface, so update your kernel, > vmd, and vmctl. > ok mlarkin@ (stefan@) ~ db_interface.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) ~ vmm_support.S > 'accomodate' -> 'accommodate' in comments. > Started by diff from Mical Mazurek. (krw@) ~ cpu.c > Replace curcpu_is_idle() by cpu_is_idle() and use it instead of rolling > our own. > From Michal Mazurek, ok mmcc@ (mpi@) ~ lock_machdep.c > Reduces the noise around the global ``ticks'' variable by renaming > all the local ones to ``nticks''. > ok stefan@, deraadt@ (mpi@) arch/amd64/include ~ vmmvar.h > Introduce memory ranges to support VMs with >= 4G RAM > Kernel bits: > - When creating a VM, a list of memory ranges has to be specified, > similar to the BIOS memory map. This is necessary for VMs with > RAM sizes approaching 4G because we'll need PCI MMIO space in > the higher parts of the 32 bit address space. > vmctl and vmd bits: > - Construct appropriate memory ranges to create a VM with a given > RAM size > - Construct a corresponding BIOS memory map from the memory ranges > and update the boot params page accordingly. > - Make sure that all variables that represent guest physical addresses > match the address width of the target CPU instead of using uint32_t. > - Fix some integer promotion glitches that actually restricted VM > RAM size to 2G. > This changes the VM create ioctl interface, so update your kernel, > vmd, and vmctl. > ok mlarkin@ (stefan@) arch/amd64/stand/pxeboot ~ open.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) arch/arm/arm ~ cpu.c ~ cpufunc.c ~ pmap.c > Remove support for ARM8, an old armv4 processor without thumb that was > never supported by any arm port and wouldn't have built due to a missing > cpufunc_asm_arm8.S file. > From Patrick Wildt. (jsg@) - cpufunc_asm_arm9.S ~ cpu.c ~ cpufunc.c ~ pmap.c > Remove support for ARM9T (armv4t). Not used by any of the arm platforms. > From Patrick Wildt. (jsg@) - cpufunc_asm_sa1.S ~ bus_space_asm_generic.S ~ cpu.c ~ cpufunc.c ~ pmap.c > Remove support for StrongARM (SA1) and IXP12x0. Both are ARMv4 and > are not used by any of the arm platforms. > ok jsg@ (patrick@) ~ cpu.c ~ cpufunc.c > Remove support for IXP425. This is another architecture that is not > used and has probably never been used at all. Some included headers > do not even exist. > ok jsg@ (patrick@) ~ cpu.c ~ cpufunc.c > Remove support for the XScale 80200. We don't use it, it didn't compile > and the included headers didn't even exist. > ok jsg@ (patrick@) arch/arm/armv7 ~ bus_space_asm_armv7.S > Remove support for StrongARM (SA1) and IXP12x0. Both are ARMv4 and > are not used by any of the arm platforms. > ok jsg@ (patrick@) arch/arm/conf ~ files.arm > Remove support for ARM8, an old armv4 processor without thumb that was > never supported by any arm port and wouldn't have built due to a missing > cpufunc_asm_arm8.S file. > From Patrick Wildt. (jsg@) ~ files.arm > Remove support for ARM9T (armv4t). Not used by any of the arm platforms. > From Patrick Wildt. (jsg@) ~ files.arm > Remove support for StrongARM (SA1) and IXP12x0. Both are ARMv4 and > are not used by any of the arm platforms. > ok jsg@ (patrick@) ~ files.arm > Remove support for IXP425. This is another architecture that is not > used and has probably never been used at all. Some included headers > do not even exist. > ok jsg@ (patrick@) ~ files.arm > Remove support for the XScale 80200. We don't use it, it didn't compile > and the included headers didn't even exist. > ok jsg@ (patrick@) arch/arm/include ~ armreg.h ~ cpuconf.h ~ cpufunc.h ~ pmap.h > Remove support for ARM8, an old armv4 processor without thumb that was > never supported by any arm port and wouldn't have built due to a missing > cpufunc_asm_arm8.S file. > From Patrick Wildt. (jsg@) ~ armreg.h ~ cpuconf.h ~ cpufunc.h ~ pmap.h > Remove support for ARM9T (armv4t). Not used by any of the arm platforms. > From Patrick Wildt. (jsg@) ~ cpuconf.h > IXP425 is v5 not v4. Same change by msaitoh in NetBSD rev 1.16. (jsg@) ~ armreg.h ~ cpuconf.h ~ cpufunc.h ~ pmap.h > Remove support for StrongARM (SA1) and IXP12x0. Both are ARMv4 and > are not used by any of the arm platforms. > ok jsg@ (patrick@) ~ armreg.h ~ cpuconf.h ~ cpufunc.h > Remove support for IXP425. This is another architecture that is not > used and has probably never been used at all. Some included headers > do not even exist. > ok jsg@ (patrick@) ~ armreg.h ~ cpuconf.h ~ cpufunc.h > Remove support for the XScale 80200. We don't use it, it didn't compile > and the included headers didn't even exist. > ok jsg@ (patrick@) arch/hppa/hppa ~ lock_machdep.c > Reduces the noise around the global ``ticks'' variable by renaming > all the local ones to ``nticks''. > ok stefan@, deraadt@ (mpi@) arch/i386/i386 ~ db_interface.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) ~ autoconf.c ~ cpu.c ~ genassym.cf ~ locore.s ~ machdep.c ~ pmap.c > Burn more LDT deadwood: stop allocating one for each idle thread, > load the ldt register with the null selector (disabling use of it), > stop reloading it on every context switch, and blow away the table > itself, as well as the pcb and pmap bits that were used to track > it (making sure to keep pcb_savefpu correctly aligned). > testing naddy@ > ok kettenis@ mpi@ mlarkin@ (guenther@) ~ cpu.c > Replace curcpu_is_idle() by cpu_is_idle() and use it instead of rolling > our own. > From Michal Mazurek, ok mmcc@ (mpi@) ~ lock_machdep.c > Reduces the noise around the global ``ticks'' variable by renaming > all the local ones to ``nticks''. > ok stefan@, deraadt@ (mpi@) arch/i386/include ~ cpu.h ~ pcb.h ~ pmap.h ~ segments.h > Burn more LDT deadwood: stop allocating one for each idle thread, > load the ldt register with the null selector (disabling use of it), > stop reloading it on every context switch, and blow away the table > itself, as well as the pcb and pmap bits that were used to track > it (making sure to keep pcb_savefpu correctly aligned). > testing naddy@ > ok kettenis@ mpi@ mlarkin@ (guenther@) arch/i386/stand/pxeboot ~ open.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) arch/m88k/m88k ~ mplock.c > Reduces the noise around the global ``ticks'' variable by renaming > all the local ones to ``nticks''. > ok stefan@, deraadt@ (mpi@) arch/macppc/dev ~ adb.c ~ pm_direct.c ~ zs.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) ~ pm_direct.c > not compiled before commit (deraadt@) arch/macppc/stand ~ alloc.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) arch/mips64/mips64 ~ lock_machdep.c ~ mutex.c > Reduces the noise around the global ``ticks'' variable by renaming > all the local ones to ``nticks''. > ok stefan@, deraadt@ (mpi@) arch/octeon/conf ~ GENERIC ~ RAMDISK ~ files.octeon > add octuctl, a driver for the Octeon II usb controller interface, and > attachments for ehci and ohci. > ok uebayasi@ jasper@ visa@ mpi@ (jmatthew@) arch/octeon/dev ~ octeon_iobus.c + octehci.c + octohci.c + octuctl.c + octuctlreg.h + octuctlvar.h > add octuctl, a driver for the Octeon II usb controller interface, and > attachments for ehci and ohci. > ok uebayasi@ jasper@ visa@ mpi@ (jmatthew@) ~ octdwctwo.c > Attach dwctwo(4) only on Octeon models that have a DWC2 controller. > OK uebayasi@ (visa@) arch/octeon/stand/boot ~ machdep.c > Reduces the noise around the global ``ticks'' variable by renaming > all the local ones to ``nticks''. > ok stefan@, deraadt@ (mpi@) arch/powerpc/powerpc ~ lock_machdep.c ~ mutex.c > Reduces the noise around the global ``ticks'' variable by renaming > all the local ones to ``nticks''. > ok stefan@, deraadt@ (mpi@) arch/sgi/localbus ~ int.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) arch/sgi/stand/boot ~ netfs.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) arch/sparc/dev ~ bpp.c ~ magma.c > Convert sparc drivers bpp and magma to uiomove > Compile test and ok tobiasu@ (stefan@) ~ fb.c > 'accomodate' -> 'accommodate' in comments. > Started by diff from Mical Mazurek. (krw@) ~ if_ie.c > More "(<blah> *)0" -> NULL, avoiding any stdarg functions. > Feedback millert@ kettenis@ (krw@) arch/sparc/stand/common ~ promdev.c > Reduces the noise around the global ``ticks'' variable by renaming > all the local ones to ``nticks''. > ok stefan@, deraadt@ (mpi@) arch/sparc64/dev ~ fb.c > 'accomodate' -> 'accommodate' in comments. > Started by diff from Mical Mazurek. (krw@) ~ beep.c ~ beeper.c > Reduces the noise around the global ``ticks'' variable by renaming > all the local ones to ``nticks''. > ok stefan@, deraadt@ (mpi@) ~ vdsp.c > Remove the unused flags argument from VOP_UNLOCK(). > torture tested on amd64, i386 and macppc > ok beck mpi stefan > "the change looks right" deraadt (natano@) arch/sparc64/sparc64 ~ db_interface.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) ~ lock_machdep.c ~ openfirm.c > Reduces the noise around the global ``ticks'' variable by renaming > all the local ones to ``nticks''. > ok stefan@, deraadt@ (mpi@) arch/sparc64/stand/ofwboot ~ alloc.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) arch/zaurus/stand/zboot ~ alloc.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) arch/zaurus/stand/zbsdmod ~ zbsdmod.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) dev ~ audio.c > Expose new audio ioctls that do one thing only: start and stop DMA, > set and get parameters. This is much simpler. > ok semarie, armani, tweaks from jmc (ratchov@) ~ diskmap.c ~ vnd.c > Remove the unused flags argument from VOP_UNLOCK(). > torture tested on amd64, i386 and macppc > ok beck mpi stefan > "the change looks right" deraadt (natano@) dev/acpi ~ acpicpu.c ~ dsdt.c ~ dsdt.h > Correct the value of SRT_ENDTAG: it was documented incorrectly in early > ACPI specs. With the correct value, it can now be used in a few places > instead of hardcoding 0x79. > ok kettenis@ (guenther@) ~ acpicpu.c > Replace curcpu_is_idle() by cpu_is_idle() and use it instead of rolling > our own. > From Michal Mazurek, ok mmcc@ (mpi@) dev/ata ~ atascsi.c > Fix size passed to free(), found the hard way by Nick Permyakov > ok deraadt@ (jca@) dev/ic ~ bha.c ~ i82596.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) ~ rtwn.c ~ rtwnvar.h > usbd_is_dying() can't be part of the common rtwn(4) driver code once we > merge code from urtwn(4). So timeouts must move back to the bus-specific > part of the driver. > ok mpi@ (stsp@) ~ aic79xx.c ~ atw.c ~ lm78var.h > 'accomodate' -> 'accommodate' in comments. > Started by diff from Mical Mazurek. (krw@) ~ aic79xx_openbsd.c ~ re.c > Reduces the noise around the global ``ticks'' variable by renaming > all the local ones to ``nticks''. > ok stefan@, deraadt@ (mpi@) dev/isa ~ aha.c ~ if_ie.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) ~ spkr.c > Reduces the noise around the global ``ticks'' variable by renaming > all the local ones to ``nticks''. > ok stefan@, deraadt@ (mpi@) dev/ofw ~ fdt.c > Do not iterate all over the tree if OF_parent() is called on the root > node. The root is the root, it has no parent. > From Patrick Wildt. (mpi@) dev/pci ~ ichiic.c > In ichiic(4), ignore the SMBALERT# interrupt. This interrupt has been > observed to fire for spurious reasons with buggy hardware/bios. > It is being ignored by both FreeBSD and Linux as well. > Fixes GENERIC kernel boot on ADI RCC-VE with buggy bios versions, where > ichiic(4) ended up stealing interrupts from ehci(4), rendering the internal > eMMC flash unusable. > Also, always ackknowledge all interrupts by writing status bits back > to avoid a potential SMBALERT# interrupt storm. > with and ok deraadt@ kettenis@ mpi@ (stsp@) ~ if_oce.c > Allocate statistics command from the heap, pointed out by deraadt@ (mikeb@) ~ bktr/bktr_core.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) ~ pcidevs > Add Exar 4 port PCIe serial port (puc) (mlarkin@) ~ pcidevs.h ~ pcidevs_data.h > sync (mlarkin@) ~ pcidevs_data.h ~ pcidevs.h > sync (mlarkin@) ~ pcidevs > Typo in part number in previous commit (fix Exar P/N from XR17C354 to > XR17V354) (mlarkin@) ~ pcidevs.h ~ pcidevs_data.h > sync (mlarkin@) ~ pucdata.c > Exar XR17V354 puc(4) device, "high speed quad uart". (mlarkin@) ~ if_rtwn.c > usbd_is_dying() can't be part of the common rtwn(4) driver code once we > merge code from urtwn(4). So timeouts must move back to the bus-specific > part of the driver. > ok mpi@ (stsp@) ~ if_age.c ~ if_alc.c ~ if_ale.c ~ if_jme.c ~ if_vio.c ~ if_vr.c ~ ixgbe.h > Ethernet drivers no longer need to include if_vlan_var.h for the VLAN > definitions; ok mpi@ (naddy@) ~ files.pci > Only attempt to build the pci sdhc attachment when it is configured. > Don't require it when building other sdhc attachments. > From Patrick Wildt. (jsg@) ~ pcidevs > add TXIC TX382B (vendor 0x4651: product 0x3273) > ok deraadt@ (uaa@) ~ pcidevs.h ~ pcidevs_data.h > regen (uaa@) ~ pucdata.c > add TXIC TX382B entry. > currtently TX/RX FIFO is not working. > ok by detraadt@ (uaa@) ~ ichwdt.c > Reduces the noise around the global ``ticks'' variable by renaming > all the local ones to ``nticks''. > ok stefan@, deraadt@ (mpi@) dev/pcmcia ~ gpr.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) dev/sbus ~ bpp.c ~ magma.c > Convert sparc drivers bpp and magma to uiomove > Compile test and ok tobiasu@ (stefan@) ~ stp4020.c > Reduces the noise around the global ``ticks'' variable by renaming > all the local ones to ``nticks''. > ok stefan@, deraadt@ (mpi@) dev/sdmmc ~ sdmmc.c > Reduces the noise around the global ``ticks'' variable by renaming all > local ones to ``nticks''. > (missed in previous) > ok stefan@, deraadt@ (mpi@) dev/sun ~ sunkbd.c > Reduces the noise around the global ``ticks'' variable by renaming all > local ones to ``nticks''. > (missed in previous) > ok stefan@, deraadt@ (mpi@) dev/usb ~ if_urndis.c > Make urndis(4) print tx/rx errors only if compiled in DEBUG mode. > These printfs were causing too much noise in dmesg during regular > operation. > And don't forget to count input errors. > ok sthen@ mpi@ (stsp@) ~ ugen.c ~ if_atu.c > 'miliseconds' -> 'milliseconds' in comments. > if_atu.c noted by Michal Mazurek. (krw@) ~ uthum.c > Reduces the noise around the global ``ticks'' variable by renaming > all the local ones to ``nticks''. > ok stefan@, deraadt@ (mpi@) ~ utrh.c > Reduces the noise around the global ``ticks'' variable by renaming all > local ones to ``nticks''. > (missed in previous) > ok stefan@, deraadt@ (mpi@) ~ uvideo.c > Remove the unused flags argument from VOP_UNLOCK(). > torture tested on amd64, i386 and macppc > ok beck mpi stefan > "the change looks right" deraadt (natano@) ~ if_axen.c > Update the axen driver to initialize the hardware on reset. Change was > suggested by Takahiro Hayashi. Also lock the mii before performing the > write, based on changes in NetBSD. While in there, remove the decaying > eeprom code that has been commented out. This allows my adapter to attach > reliably, pass traffic, and has not panic'd my machine when I unplug it. > Test feedback and OK stsp@, and OK mpi@ (bmercer@) isofs/cd9660 ~ cd9660_vfsops.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) ~ cd9660_vfsops.c > Set mnt_data to NULL after freeing the file system specific mount point. > OK krw@ natano@ as part of a larger diff (bluhm@) ~ cd9660_lookup.c ~ cd9660_node.c ~ cd9660_vfsops.c ~ cd9660_vnops.c > Remove the unused flags argument from VOP_UNLOCK(). > torture tested on amd64, i386 and macppc > ok beck mpi stefan > "the change looks right" deraadt (natano@) isofs/udf ~ udf_vfsops.c > Set mnt_data to NULL after freeing the file system specific mount point. > OK krw@ natano@ as part of a larger diff (bluhm@) ~ udf_vfsops.c ~ udf_vnops.c > Remove the unused flags argument from VOP_UNLOCK(). > torture tested on amd64, i386 and macppc > ok beck mpi stefan > "the change looks right" deraadt (natano@) kern ~ kern_pledge.c > corrects on off-by-one error in pledge_namei() > - rewrite canonpath() to not require extra byte before shrinking > - make canonpath() error not fatal for the caller (proposition from tedu@) > ok millert@ tedu@ deraadt@ (semarie@) ~ kern_pledge.c > pledge: let wl_paths works well with chroot > it factorizes path resolution in resolvpath() function, and use it in > sys_pledge() and pledge_namei(). > please note that wl_paths is still disabled. (semarie@) ~ kern_pledge.c > pledge: replace #if 0 printf with DNPRINTF macro (semarie@) ~ kern_pledge.c > reenable wl_paths (whitelisted-paths) in pledge(2) (semarie@) ~ kern_pledge.c > keep disabled wl_paths for now. the expensiveness impact need to be better > considered. > ok deraadt@ (semarie@) ~ kern_acct.c ~ subr_autoconf.c ~ tty_subr.c ~ uipc_socket.c ~ vfs_subr.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) ~ kern_subr.c > Remove now unused legacy uiomovei() function. > All its callers got reviewed and converted to > use uiomove() properly. > ok deraadt@ (stefan@) ~ kern_pledge.c > pledge: wl_paths: corrects a memory leak in error code path and an > evaluation > order ("a += b > c" is the same as "a += (b > c)" which is wrong here) > ok jca@ (semarie@) ~ kern_pledge.c > pledge: wl_paths: resolvpath() logic error > prepend chroot value *after* canonization and not before. (semarie@) ~ kern_pledge.c > Expose new audio ioctls that do one thing only: start and stop DMA, > set and get parameters. This is much simpler. > ok semarie, armani, tweaks from jmc (ratchov@) ~ kern_pledge.c TAGGED OPENBSD_5_9 > Incorrect path processing in pledge_namei() could result in unexpected > program termination of pledge(2)'d programs. > Expand path variable size to +1 in order to add extra byte for canonpath() > processing. > -current was fixed by a large rewrite of canonpath(). > ok deraadt@ tedu@ (semarie@) ~ vfs_bio.c ~ vfs_biomem.c TAGGED OPENBSD_5_9 > - add realloc_pages to move a buffer's physmem from one range to another. > - modify B_DMA handling to be in vfs_biomem.c > - change buffer allocation to try allocations with NOWAIT and to throw away > clean pages > if allocation fails - allocate with WAITOK only if you can't throw away > enough pages to > succeed > "probably sound" deraadt@ (beck@) ~ kern_timeout.c TAGGED OPENBSD_5_9 > KNF: Remove a blank line. (uebayasi@) ~ kern_sched.c TAGGED OPENBSD_5_9 > Replace curcpu_is_idle() by cpu_is_idle() and use it instead of rolling > our own. > From Michal Mazurek, ok mmcc@ (mpi@) ~ exec_script.c ~ init_main.c ~ kern_acct.c ~ kern_descrip.c ~ kern_exec.c ~ kern_ktrace.c ~ kern_sig.c ~ spec_vnops.c ~ tty.c ~ tty_pty.c ~ tty_tty.c ~ uipc_usrreq.c ~ vfs_cache.c ~ vfs_getcwd.c ~ vfs_lookup.c ~ vfs_subr.c ~ vfs_sync.c ~ vfs_syscalls.c ~ vfs_vnops.c ~ vfs_vops.c TAGGED OPENBSD_5_9 > Remove the unused flags argument from VOP_UNLOCK(). > torture tested on amd64, i386 and macppc > ok beck mpi stefan > "the change looks right" deraadt (natano@) ~ kern_clock.c ~ kern_timeout.c TAGGED OPENBSD_5_9 > Update ticks in hardclock(). > OK mikeb@ (uebayasi@) lib/libkern ~ arch/amd64/strlen.S > " the the " -> " the ", or in a couple of cases replace the superfluous > "the" with the obviously intended word. > Started with a "the the" spotted by Mihal Mazurek. (krw@) lib/libsa ~ alloc.c ~ netif.c ~ nfs.c ~ open.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) lib/libz ~ deflate.c ~ trees.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) miscfs/fifofs ~ fifo_vnops.c > Remove the unused flags argument from VOP_UNLOCK(). > torture tested on amd64, i386 and macppc > ok beck mpi stefan > "the change looks right" deraadt (natano@) miscfs/fuse ~ fuse_vfsops.c > Set mnt_data to NULL after freeing the file system specific mount point. > OK krw@ natano@ as part of a larger diff (bluhm@) ~ fuse_lookup.c ~ fuse_vnops.c > Remove the unused flags argument from VOP_UNLOCK(). > torture tested on amd64, i386 and macppc > ok beck mpi stefan > "the change looks right" deraadt (natano@) msdosfs ~ msdosfs_vnops.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) ~ msdosfs_vfsops.c > Set mnt_data to NULL after freeing the file system specific mount point. > OK krw@ natano@ as part of a larger diff (bluhm@) ~ msdosfs_denode.c ~ msdosfs_lookup.c ~ msdosfs_vfsops.c ~ msdosfs_vnops.c > Remove the unused flags argument from VOP_UNLOCK(). > torture tested on amd64, i386 and macppc > ok beck mpi stefan > "the change looks right" deraadt (natano@) net ~ if_vlan.c > consistencly name ifnet pointers like we did in carp > the vlan interface is ifp, the vlans parent interface is ifp0, the new > interface parent is newifp0. > requested by mpi@ > no binary change (dlg@) ~ if_vlan.c > i forgot to rename p to ifp0 in vlan_config > still no binary change. (dlg@) ~ if_vlan_var.h > move the vlan protocol definitions to the same places as the ethernet ones > ok mpi@ (dlg@) ~ if.c > if ticks diverge from ifq_congestion too far the diff will go negative > detect this and bump ifq_congestion forward rather than claim the > system is congested for a long period of time. > ok mpi@ henning@ jmatthew@ (dlg@) ~ if_vlan.c > refactor the vlan multicast list handling. > the previous code had vlan_ether_purgemulti and vlan_ether_resetmulti, > both of which did too many things. purgemulti would try and remove > the multicast entries from the parent, and then free the local > copies of the addresses. resetmulti would try to remove the address > from the parent, and then optionally try to add them to a new parent. > resetmulti in particular makes the overall vlan config steps fairly > twisty. > the refactor offers vlan_multi_apply, and vlan_multi_free. multi_apply > simply adds or removes the multicast addresses from a parent > interface. it is now up to the config steps to call them appropriately > when configuring a parent or a new parent. vlan_multi_free only > deletes the memory associated with the vlans multicast addresses. > vlan_multi_apply is called when a parent is configured (ie, ifconfig > vlan0 up), or unconfigured (ifconfig vlan0 down or a detach of the > parent). vlan_multi_free is called when a vlan interface is destroyed > (ifconfig vlan0 destroy). > ok mpi@ (dlg@) netinet ~ if_ether.h > move the vlan protocol definitions to the same places as the ethernet ones > ok mpi@ (dlg@) ~ if_ether.h > macros for the null, min, and max vlan ids. > ok mpi@ and tweaks from sthen@ (dlg@) netinet6 ~ ip6_input.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) ~ in6_pcb.c ~ udp6_output.c TAGGED OPENBSD_5_7 > Insufficient checks in IPv6 socket binding and UDP IPv6 option > processing allow a local user to send UDP packets with a source > (IPv6 address + port) already reserved by another user. > See Errata 024 (vgross@) ~ in6_pcb.c ~ udp6_output.c TAGGED OPENBSD_5_8 > Insufficient checks in IPv6 socket binding and UDP IPv6 option > processing allow a local user to send UDP packets with a source > (IPv6 address + port) already reserved by another user. > See Errata 012 (vgross@) ~ in6_pcb.c ~ udp6_output.c TAGGED OPENBSD_5_8 > Extract in6_pcbaddrisavail() from in6_pcbbind(), and use it when > checking for source availability in udp6_output(). > Ok jca@ bluhm@ (vgross@) ~ in6_pcb.c ~ udp6_output.c TAGGED OPENBSD_5_8 > Revert, missing decl for in6_pcbaddrisavail() breaks kernel build. > Spotted by deraadt@ (jca@) nfs ~ nfs_vfsops.c > Set mnt_data to NULL after freeing the file system specific mount point. > OK krw@ natano@ as part of a larger diff (bluhm@) ~ nfs_node.c ~ nfs_serv.c ~ nfs_subs.c ~ nfs_vnops.c > Remove the unused flags argument from VOP_UNLOCK(). > torture tested on amd64, i386 and macppc > ok beck mpi stefan > "the change looks right" deraadt (natano@) ntfs ~ ntfs_vfsops.c > Set mnt_data to NULL after freeing the file system specific mount point. > OK krw@ natano@ as part of a larger diff (bluhm@) ~ ntfs_vfsops.c ~ ntfs_vnops.c > Remove the unused flags argument from VOP_UNLOCK(). > torture tested on amd64, i386 and macppc > ok beck mpi stefan > "the change looks right" deraadt (natano@) scsi ~ sd.c > Add SDF_DYING checks to more functions in scsi disk, to prevent an > use after free of the scsi link structure during detach. > OK krw@ (bluhm@) ~ sd.c > Do not access the scsi link structure in sdclose() if the disk > device is already dying. Delete the sc_timeout when setting the > SDF_DYING flag as the timeout delete in sdclose() may not be reached. > OK krw@ (bluhm@) ~ sd.c > After sleeping and before accessing sc_link, check that scsi disk > is not dying. > OK krw@ (bluhm@) ~ sd.c > Do not run into sdgetdisklabel() when scsi disk is dying. Add > special error handling in sdopen() as temporary hack. > OK krw@ (bluhm@) sys ~ systm.h > Remove now unused legacy uiomovei() function. > All its callers got reviewed and converted to > use uiomove() properly. > ok deraadt@ (stefan@) ~ audioio.h > Expose new audio ioctls that do one thing only: start and stop DMA, > set and get parameters. This is much simpler. > ok semarie, armani, tweaks from jmc (ratchov@) ~ sched.h > Replace curcpu_is_idle() by cpu_is_idle() and use it instead of rolling > our own. > From Michal Mazurek, ok mmcc@ (mpi@) ~ sysctl.h > Fix a comment about ``p_usrpri'', from Michal Mazurek. (mpi@) ~ vnode.h > Remove the unused flags argument from VOP_UNLOCK(). > torture tested on amd64, i386 and macppc > ok beck mpi stefan > "the change looks right" deraadt (natano@) tmpfs ~ tmpfs_vnops.c > Remove the unused flags argument from VOP_UNLOCK(). > torture tested on amd64, i386 and macppc > ok beck mpi stefan > "the change looks right" deraadt (natano@) ufs/ext2fs ~ ext2fs_lookup.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) ~ ext2fs_vfsops.c > Set mnt_data to NULL after freeing the file system specific mount point. > OK krw@ natano@ as part of a larger diff (bluhm@) ~ ext2fs_inode.c ~ ext2fs_lookup.c ~ ext2fs_vfsops.c ~ ext2fs_vnops.c > Remove the unused flags argument from VOP_UNLOCK(). > torture tested on amd64, i386 and macppc > ok beck mpi stefan > "the change looks right" deraadt (natano@) ufs/ffs ~ ffs_vfsops.c > Set mnt_data to NULL after freeing the file system specific mount point. > OK krw@ natano@ as part of a larger diff (bluhm@) ~ ffs_softdep.c ~ ffs_vfsops.c > Remove the unused flags argument from VOP_UNLOCK(). > torture tested on amd64, i386 and macppc > ok beck mpi stefan > "the change looks right" deraadt (natano@) ufs/mfs ~ mfs_vfsops.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) ~ mfs_vnops.c > Remove the unused flags argument from VOP_UNLOCK(). > torture tested on amd64, i386 and macppc > ok beck mpi stefan > "the change looks right" deraadt (natano@) ufs/ufs ~ ufs_quota.c > Change a bunch of (<blah> *)0 to NULL. > ok beck@ deraadt@ (krw@) ~ ufs_inode.c ~ ufs_lookup.c ~ ufs_quota.c ~ ufs_vnops.c > Remove the unused flags argument from VOP_UNLOCK(). > torture tested on amd64, i386 and macppc > ok beck mpi stefan > "the change looks right" deraadt (natano@) uvm ~ uvm_amap.c > For amaps with only a few slots, allocate the slots via pool(9) > This saves some memory compared to using malloc, because there's > no roundup to the next bucket size. And it reduces kmem pressure > at least for some architectures (e.g. amd64). > Testing by sthen@ and tobiasu@, thanks! > ok sthen@ deraadt@ (stefan@) ~ uvm_unix.c > Allocate amap slots for a virtual memory range reserved with sbrk lazily. > This avoids wasting kernel memory if the user process does not make > use of the allocated memory. > Testing by sthen@ and tobiasu@, thanks! > ok deraadt@ (stefan@) ~ uvm_map.c > 'accomodate' -> 'accommodate' in comments. > Started by diff from Mical Mazurek. (krw@) ~ uvm_amap.c > Remove redundant check. > The compiler is also smart enough to recognize that this is redundant. > The resulting code on amd64 is basically equivalent (slightly different > register allocation and instruction scheduling). > ok mpi@ deraadt@ (stefan@) ~ uvm_vnode.c > Remove the unused flags argument from VOP_UNLOCK(). > torture tested on amd64, i386 and macppc > ok beck mpi stefan > "the change looks right" deraadt (natano@) == usr.bin =========================================================== 12/13 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.bin column ~ column.1 ~ column.c > Switch (non-curses, non-ksh) programs that use COLUMNS to a single idiom. > Previously behaviors were all over the map. This changes them to > use COLUMNS first, and either terminal width or a hardcoded value > (typically 80) as appropriate. > ok deraadt@; man bits ok jmc@ (bentley@) ~ column.1 > simplify previous; ok bentley (jmc@) deroff ~ deroff.c > More "(<blah> *)0" -> NULL, avoiding any stdarg functions. > Feedback millert@ kettenis@ (krw@) ftp ~ ftp.c ~ main.c ~ util.c > More "(<blah> *)0" -> NULL, avoiding any stdarg functions. > Feedback millert@ kettenis@ (krw@) head ~ head.c > whitespace: remove a space before a tab and a trailing tab. (tb@) hexdump ~ display.c ~ hexdump.h ~ hexsyntax.c ~ odsyntax.c ~ parse.c > o clean up a little whitespace and formatting > o move some declarations out of hexdump.h > o mark some declarations as __dead or static > from Michal Mazurek, with input from and ok tb@ (mmcc@) kdump ~ Makefile ~ mkioctls > Recognize ipmi, vscsi, pvbus, udl, fuse, trunk, pipex and memrange > ioctl requests. (natano@) less ~ ch.c > Wrap a macro's args in parentheses. No binary change. > I'm discussing with deraadt@ whether it's a good idea to convert some of > these to functions. The one changed by this commit probably isn't > eligible because it defines only a for loop's condition, but many others > in less(1) should probably be converted. > ok millert@ (mmcc@) ~ ch.c ~ charset.c ~ cmdbuf.c ~ command.c ~ cvt.c ~ decode.c ~ edit.c ~ filename.c ~ lesskey.c ~ line.c ~ lsystem.c ~ main.c ~ os.c ~ screen.c ~ search.c ~ signal.c > style(9) includes > ok nicm@ (mmcc@) locale ~ Makefile > LOCALE_CHARSETS has been unused for some time now > so no need to include share/locale/ctype/Makefile.inc to get it. > ok guenther@ stsp@ (who had the same diff) (jca@) mandoc ~ mandocdb.c > 'accomodate' -> 'accommodate' in comments. > Started by diff from Mical Mazurek. (krw@) ~ cgi.c > support short URIs for man.openbsd.org (schwarze@) ~ cgi.c > make man(1) mode the default rather than apropos(1) mode (schwarze@) ~ man.cgi.8 > document short URIs (schwarze@) ~ cgi.c ~ cgi.h.example ~ man.cgi.8 > Make the SCRIPT_NAME logic simpler, safer, and make it actually work; > in part based on ideas by bentley@. > While here, improve the documentation. (schwarze@) ~ man.cgi.8 > double word; (jmc@) ~ term.c > " the the " -> " the ", or in a couple of cases replace the superfluous > "the" with the obviously intended word. > Started with a "the the" spotted by Mihal Mazurek. (krw@) mg ~ mg.1 > notab mode is not enabled at the moment. (lum@) ~ mg.1 > Show modes for buffers more clearly. ok jasper@ (lum@) quota ~ quota.c > More "(<blah> *)0" -> NULL, avoiding any stdarg functions. > Feedback millert@ kettenis@ (krw@) rusers ~ rusers.1 ~ rusers.c > Switch (non-curses, non-ksh) programs that use COLUMNS to a single idiom. > Previously behaviors were all over the map. This changes them to > use COLUMNS first, and either terminal width or a hardcoded value > (typically 80) as appropriate. > ok deraadt@; man bits ok jmc@ (bentley@) ~ rusers.1 > simplify previous; ok bentley (jmc@) sed ~ main.c ~ sed.1 > Switch (non-curses, non-ksh) programs that use COLUMNS to a single idiom. > Previously behaviors were all over the map. This changes them to > use COLUMNS first, and either terminal width or a hardcoded value > (typically 80) as appropriate. > ok deraadt@; man bits ok jmc@ (bentley@) showmount ~ showmount.c > More "(<blah> *)0" -> NULL, avoiding any stdarg functions. > Feedback millert@ kettenis@ (krw@) ssh ~ sshconnect2.c > unbreak authentication using lone certificate keys in ssh-agent: > when attempting pubkey auth with a certificate, if no separate > private key is found among the keys then try with the certificate > key itself. > bz#2550 reported by Peter Moody (djm@) ~ sshd_config.5 > UseDNS affects ssh hostname processing in authorized_keys, not > known_hosts; bz#2554 reported by jjelen AT redhat.com (djm@) talk ~ invite.c > More "(<blah> *)0" -> NULL, avoiding any stdarg functions. > Feedback millert@ kettenis@ (krw@) telnet ~ commands.c ~ sys_bsd.c > More "(<blah> *)0" -> NULL, avoiding any stdarg functions. > Feedback millert@ kettenis@ (krw@) tftp ~ main.c > More "(<blah> *)0" -> NULL, avoiding any stdarg functions. > Feedback millert@ kettenis@ (krw@) tmux ~ key-string.c ~ mode-key.c ~ server-client.c ~ tmux.1 ~ tmux.h > Instead of reusing MouseUp at the finish of a drag, add a new key > MouseDragEnd. It can be useful to bind them separately in copy mode. > (nicm@) ~ window-copy.c > Make scrolling behaviour more sensible and maintain cursor position, as > if the same had been done line-by-line. From Michal Mazurek. (nicm@) unifdef ~ unifdef.c > Replace two malloc casts that deraadt@ and I removed over the past year > or two. Upstream wants to keep them for C++ support, and it isn't worth > maintaining a larger diff. No binary change. > Upstream commit: > https://github.com/fanf2/unifdef/commit/87dfd91 > I'll send more merging diffs to tech@ soon. (mmcc@) vi ~ vi/v_cmd.c > remove an extra space before ^\ help message. > Fixes allignment in viusage > By Raf Czlonka (martijn@) ~ cl/cl_term.c > Add error checking for COLUMNS/LINES environment variables. > It would be better to replace all the complicated existing code with a > simple idiom, and this is being worked on. But for the moment, > preventing vi from crashing is worthwhile. > ok jung@ (bentley@) ~ ex/ex.c > By issuing :e +something in vi(1) this uncovers a backwards memcpy with the > code because the 2 buffers overlap and in order to solve it then replace > memcpy(3) call by memmove(3) > Issue noticed by Michael Price <michael at ectospheno dot com> who also > tested > this diff > OK deraadt@ (mestre@) w ~ w.c > if -a is specified, allow "dns" in pledge requests. > problem noticed by Raf Czlonka (deraadt@) == usr.sbin ========================================================== 13/13 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.sbin amd ~ amd/sfs_ops.c > More "(<blah> *)0" -> NULL, avoiding any stdarg functions. > Feedback millert@ kettenis@ (krw@) config ~ config.8 > one more un-vax, which i fixed but somehow didn;t commit; > Xr fix while here... (jmc@) edquota ~ edquota.8 > Remove needless DIAGNOSTICS sections saying only "Various messages about > inaccessible files; self-explanatory." > ok jmc@ (mmcc@) fdformat ~ fdformat.c > More "(<blah> *)0" -> NULL, avoiding any stdarg functions. > Feedback millert@ kettenis@ (krw@) inetd ~ inetd.c > Replace atoi(3) calls for safer strtonum(3) > OK deeradt@ jca@ (mestre@) ~ inetd.c > Replace strncpy(3) calls for safer strlcpy(3) > OK deraadt@ jca@ (mestre@) ~ inetd.c > Remove a (now) obsolete comment about shortened IPv4 syntax > OK deraadt@ jca@ (mestre@) iscsid ~ util.c > Currently we have about a 50/50 split over fcntl(n, F_GETFL [,0]) > idioms. > Adopt the more concise fcntl(n, F_GETFL) over fcntl(n, F_GETFL, 0) > where it is obvious further investigation will not yield and > even better way. > Obviousness evaluation and ok guenther@ (krw@) ldapd ~ btree.c > Currently we have about a 50/50 split over fcntl(n, F_GETFL [,0]) > idioms. > Adopt the more concise fcntl(n, F_GETFL) over fcntl(n, F_GETFL, 0) > where it is obvious further investigation will not yield and > even better way. > Obviousness evaluation and ok guenther@ (krw@) lpr ~ lpc/lpc.c > More "(<blah> *)0" -> NULL, avoiding any stdarg functions. > Feedback millert@ kettenis@ (krw@) ~ common_source/displayq.c ~ lpq/lpq.1 > Switch (non-curses, non-ksh) programs that use COLUMNS to a single idiom. > Previously behaviors were all over the map. This changes them to > use COLUMNS first, and either terminal width or a hardcoded value > (typically 80) as appropriate. > ok deraadt@; man bits ok jmc@ (bentley@) ~ lpq/lpq.1 > simplify previous; ok bentley (jmc@) npppd ~ npppd/fsm.c > Transition to "Req-Sent" had been missing when RTA in "Opened". This > caused > a timer event leak. Diff from Yuuichi Someya. (yasuoka@) ~ npppd/fsm.c > Fix a typo in comment. Diff from Yuuichi Someya. (yasuoka@) ~ common/slist_test.c > More "(<blah> *)0" -> NULL, avoiding any stdarg functions. > Feedback millert@ kettenis@ (krw@) pkg_add ~ OpenBSD/PackageRepository.pm > use properly separated _pkgfetch user. > abort if you can't find it. if you somehow managed NOT to update your > users thru sysmerge or the normal build process, you deserve this. (espie@) pppd ~ chat/chat.c > More "(<blah> *)0" -> NULL, avoiding any stdarg functions. > Feedback millert@ kettenis@ (krw@) rbootd ~ rbootd.c ~ rmpproto.c > More "(<blah> *)0" -> NULL, avoiding any stdarg functions. > Feedback millert@ kettenis@ (krw@) repquota ~ repquota.c > More "(<blah> *)0" -> NULL, avoiding any stdarg functions. > Feedback millert@ kettenis@ (krw@) ~ repquota.8 > Remove needless DIAGNOSTICS sections saying only "Various messages about > inaccessible files; self-explanatory." > ok jmc@ (mmcc@) tcpdump ~ print-icmp6.c ~ print-snmp.c > o remove legacy code that defines abs(3) if it isn't available > o inline an ugly and potentially risky macro of the form: > #define x if (a) b; else > o fix a >21 y.o. bug resulting from someone writing: > puts("[nothing to parse], stdout"); > when they meant: > fputs("[nothing to parse]", stdout); > ok canacar@ (mmcc@) vmctl ~ vmctl.c > Introduce memory ranges to support VMs with >= 4G RAM > Kernel bits: > - When creating a VM, a list of memory ranges has to be specified, > similar to the BIOS memory map. This is necessary for VMs with > RAM sizes approaching 4G because we'll need PCI MMIO space in > the higher parts of the 32 bit address space. > vmctl and vmd bits: > - Construct appropriate memory ranges to create a VM with a given > RAM size > - Construct a corresponding BIOS memory map from the memory ranges > and update the boot params page accordingly. > - Make sure that all variables that represent guest physical addresses > match the address width of the target CPU instead of using uint32_t. > - Fix some integer promotion glitches that actually restricted VM > RAM size to 2G. > This changes the VM create ioctl interface, so update your kernel, > vmd, and vmctl. > ok mlarkin@ (stefan@) vmd ~ vmm.c > Rework an erroneous warning message about 16 bit i8253 PIT counter I/O. > The unimplemented mode is MSB/LSB (8 bit), not 16 bit mode. (mlarkin@) ~ loadfile.h ~ loadfile_elf.c ~ parse.y ~ virtio.c ~ vmd.h ~ vmm.c > Introduce memory ranges to support VMs with >= 4G RAM > Kernel bits: > - When creating a VM, a list of memory ranges has to be specified, > similar to the BIOS memory map. This is necessary for VMs with > RAM sizes approaching 4G because we'll need PCI MMIO space in > the higher parts of the 32 bit address space. > vmctl and vmd bits: > - Construct appropriate memory ranges to create a VM with a given > RAM size > - Construct a corresponding BIOS memory map from the memory ranges > and update the boot params page accordingly. > - Make sure that all variables that represent guest physical addresses > match the address width of the target CPU instead of using uint32_t. > - Fix some integer promotion glitches that actually restricted VM > RAM size to 2G. > This changes the VM create ioctl interface, so update your kernel, > vmd, and vmctl. > ok mlarkin@ (stefan@) zdump ~ zdump.c > zdump(8) needs to read files so it needs pledge(2) rpath promise > After discussion with tb@, deraadt@ and millert@, this was also OK'ed by > them (mestre@) ~ zdump.c > Don't warn about valid time zone abbreviations. POSIX through 2000 > says that an abbreviation cannot start with ':', and cannot contain > ',', '-', '+', NUL, or a digit. POSIX from 2001 on changes this > rule to say that an abbreviation can contain only '-', '+', and > alphanumeric characters from the portable character set in the > current locale. To be portable to both sets of rules, an abbreviation > must therefore use only ASCII letters." Adapted from tzcode2015f. > OK deraadt@ mestre@ (millert@) zic ~ zic.c > Don't warn about valid time zone abbreviations. POSIX through 2000 > says that an abbreviation cannot start with ':', and cannot contain > ',', '-', '+', NUL, or a digit. POSIX from 2001 on changes this > rule to say that an abbreviation can contain only '-', '+', and > alphanumeric characters from the portable character set in the > current locale. To be portable to both sets of rules, an abbreviation > must therefore use only ASCII letters." Adapted from tzcode2015f. > OK deraadt@ mestre@ (millert@) =============================================================================== _______________________________________________ owc mailing list [email protected] http://www.squish.net/mailman/listinfo/owc
