Possible logic error in 'ioapic0: misconfigured as apic' fix?

2017-08-28 Thread Andrew Daugherity
I was looking at post-6.0 commits to sys/arch/i386/i386 to see if there was anything obvious that would explain why 6.1 won't boot on a 486 while 6.0 does. Nothing jumped out, but I did a double-take at this unrelated commit: Stop printing the "ioapic0: misconfigured as apic..." messages. Ba

Fix OF_getbodebyname()

2017-08-28 Thread Mark Kettenis
The idea is that it will look for children of the passed node with a specific name. But it doesn't actually work because it looks for nodes at the same level of the passed node. This diff fixes it. ok? Index: dev/ofw/fdt.c === RCS

Re: CVS: cvs.openbsd.org: src

2017-08-28 Thread lists
Mon, 28 Aug 2017 10:16:58 -0600 (MDT) Ted Unangst > CVSROOT: /cvs > Module name: src > Changes by: t...@cvs.openbsd.org2017/08/28 10:16:58 > > Modified files: > usr.sbin/apmd : apmd.8 apmd.c > > Log message: > add -z and -Z options to auto suspend or hibernate when low on bat

Re: strtok_r(3): mention standards

2017-08-28 Thread Jeremie Courreges-Anglas
On Mon, Aug 28 2017, "Todd C. Miller" wrote: > On Mon, 28 Aug 2017 14:29:18 +0200, Jeremie Courreges-Anglas wrote: > >> IIUC strtok_r appeared in IEEE Std 1003.1c-1995, the diff below >> documents that. That would be the first use of .St -p1003.1c-95 in the >> tree. (Is this one available online

Re: [PATCH] add initial set of regress tests for vmd

2017-08-28 Thread Carlos Cardenas
On Sun, Aug 27, 2017 at 11:44 PM, Mike Larkin wrote: > On Sun, Aug 27, 2017 at 02:24:08PM -0700, Carlos Cardenas wrote: > > Howdy. > > > > Below is patch for some initial parsing tests for vmd: > > * Memory string parsing (too small, invalid size, and rounding) > > * Max disk path > > * Max vm na

Re: [patch] Add -z and -Z to apmd for automatic suspend/hibernate

2017-08-28 Thread Ted Unangst
Jesper Wallin wrote: > On Tue, Aug 15, 2017 at 07:24:47PM -0400, Ted Unangst wrote: > > this looks good. will commit soon. > > > > (one nit, i'll correct, is man page options are upper case first.) > > Sorry if this comes on as a bit pushy, but any news here? I don't know > how 6.2-beta affects t

Re: [PATCH 0/2] SMALL_TIME_T follow-ups (was Re: [PATCH] allow notAfter after 2038 with 32-bit time_t)

2017-08-28 Thread Kyle J. McKay
On Aug 26, 2017, at 18:52, Bob Beck wrote: With the new define (SMALL_TIME_T) enabled, a 32-bit time_t build using "openssl s_client -connect" can successfully connect to a server and verify its certificate chain when one or more notAfter dates after 2038 are present. However, using "nc -c

Re: ksh: walking valid history entries

2017-08-28 Thread Todd C. Miller
On Mon, 28 Aug 2017 11:13:03 +0200, Jeremie Courreges-Anglas wrote: > Right now the code uses two methods to access "valid" entries in the > history array. One method only looks at entries in [history;histptr], > the other method walks the array until it finds a NULL pointer. > > The latter meth

Re: strtok_r(3): mention standards

2017-08-28 Thread Todd C. Miller
On Mon, 28 Aug 2017 14:29:18 +0200, Jeremie Courreges-Anglas wrote: > IIUC strtok_r appeared in IEEE Std 1003.1c-1995, the diff below > documents that. That would be the first use of .St -p1003.1c-95 in the > tree. (Is this one available online? My search skillz have failed me > so far...) The

Re: smtpd: tweak static table parser

2017-08-28 Thread Gilles Chehade
On Mon, Aug 28, 2017 at 05:27:03PM +0200, Eric Faurot wrote: > Hi, > > The current static (file) table parser is a bit clumsy. It tries to > determine the type (mapping or list entry) of each line independently > by splitting on allowed separators (" \t:") without using any context, > then fails

Re: [PATCH] diff exit status

2017-08-28 Thread Todd C. Miller
On Sun, 27 Aug 2017 00:47:26 +0200, Ibrahim Khalifa wrote: > Bumping this to tech@ since I think it would be nice to get this into the > stock diff. Committed, thanks. - todd

smtpd: tweak static table parser

2017-08-28 Thread Eric Faurot
Hi, The current static (file) table parser is a bit clumsy. It tries to determine the type (mapping or list entry) of each line independently by splitting on allowed separators (" \t:") without using any context, then fails if the type is not what's expected. It's impossible to define a list of i

strtok_r(3): mention standards

2017-08-28 Thread Jeremie Courreges-Anglas
IIUC strtok_r appeared in IEEE Std 1003.1c-1995, the diff below documents that. That would be the first use of .St -p1003.1c-95 in the tree. (Is this one available online? My search skillz have failed me so far...) Other systems: - linux, "POSIX.1-2001, POSIX.1-2008", FreeBSD - freebsd, "IEEE

Re: [patch] Add -z and -Z to apmd for automatic suspend/hibernate

2017-08-28 Thread Jesper Wallin
On Tue, Aug 15, 2017 at 07:24:47PM -0400, Ted Unangst wrote: > this looks good. will commit soon. > > (one nit, i'll correct, is man page options are upper case first.) Sorry if this comes on as a bit pushy, but any news here? I don't know how 6.2-beta affects this, but maybe it would be nice to

Re: include pinebook firmware in the install media

2017-08-28 Thread Jonathan Gray
On Mon, Aug 28, 2017 at 11:54:48AM +1000, Jonathan Gray wrote: > On Sun, Aug 27, 2017 at 09:54:07PM +0200, Peter Hessler wrote: > > (this depends on the patch to u-boot I just sent to ports@) > > > > If we should manage to boot on a Pinebook machine, we should install the > > correct firmware for

ksh: walking valid history entries

2017-08-28 Thread Jeremie Courreges-Anglas
Right now the code uses two methods to access "valid" entries in the history array. One method only looks at entries in [history;histptr], the other method walks the array until it finds a NULL pointer. The latter method appears fragile to me. *IIUC* the test at the beginning of history_write()

iked const ** warning

2017-08-28 Thread Otto Moerbeek
Hi, this one is a bit tricky (cee http://c-faq.com/ansi/constmismatch.html), it fixes the warning: /usr/src/sbin/iked/crypto.c:854:44: warning: cast from 'uint8_t **' (aka 'unsigned char **') to 'const uint8_t **' (aka 'const unsigned char **') must have all intermediate pointers con

csh(1): paste long lines

2017-08-28 Thread Anton Lindqvist
Hi, deraadt@ discovered that pasting long lines in csh with filec enabled is currently broken. NUL-terminating the input buffer from tenex() instructs csh that the buffer contains a complete command. This is wrong and should only happen when buffer is not full, otherwise more data has to be read in