Re: svn commit: r300718 - in head/sys: kern sys

2016-05-27 Thread John Baldwin
On Thursday, May 26, 2016 04:54:48 PM Ian Lepore wrote: > On Thu, 2016-05-26 at 14:37 -0700, Gleb Smirnoff wrote: > > Hans, > > > > On Thu, May 26, 2016 at 08:41:55AM +, Hans Petter Selasky wrote: > > H> Author: hselasky > > H> Date: Thu May 26 08:41:55 2016 > > H> New Revision: 300718 > > H

Re: svn commit: r300718 - in head/sys: kern sys

2016-05-27 Thread Slawa Olhovchenkov
On Thu, May 26, 2016 at 04:54:48PM -0600, Ian Lepore wrote: > On Thu, 2016-05-26 at 14:37 -0700, Gleb Smirnoff wrote: > > Hans, > > > > On Thu, May 26, 2016 at 08:41:55AM +, Hans Petter Selasky wrote: > > H> Author: hselasky > > H> Date: Thu May 26 08:41:55 2016 > > H> New Revision: 300718

Re: svn commit: r300718 - in head/sys: kern sys

2016-05-27 Thread Bruce Evans
On Thu, 26 May 2016, Ian Lepore wrote: On Thu, 2016-05-26 at 14:37 -0700, Gleb Smirnoff wrote: Hans, On Thu, May 26, 2016 at 08:41:55AM +, Hans Petter Selasky wrote: H> Author: hselasky H> Date: Thu May 26 08:41:55 2016 H> New Revision: 300718 H> URL: https://svnweb.freebsd.org/changeset

Re: svn commit: r300718 - in head/sys: kern sys

2016-05-27 Thread Konstantin Belousov
On Fri, May 27, 2016 at 10:22:55AM +0200, Hans Petter Selasky wrote: > On 05/27/16 10:11, Konstantin Belousov wrote: > > On Fri, May 27, 2016 at 09:42:24AM +0200, Hans Petter Selasky wrote: > >> We cannot use the "bool" type in userspace at least, because > >> sizeof(bool) depends on the compiler.

Re: svn commit: r300718 - in head/sys: kern sys

2016-05-27 Thread Hans Petter Selasky
On 05/27/16 10:11, Konstantin Belousov wrote: On Fri, May 27, 2016 at 09:42:24AM +0200, Hans Petter Selasky wrote: We cannot use the "bool" type in userspace at least, because sizeof(bool) depends on the compiler. So the interface must use an integer type at least, possibly uint8_t like now. W

Re: svn commit: r300718 - in head/sys: kern sys

2016-05-27 Thread Konstantin Belousov
On Fri, May 27, 2016 at 09:42:24AM +0200, Hans Petter Selasky wrote: > We cannot use the "bool" type in userspace at least, because > sizeof(bool) depends on the compiler. So the interface must use an > integer type at least, possibly uint8_t like now. What do you mean ? There are ABIs, and we

Re: svn commit: r300718 - in head/sys: kern sys

2016-05-27 Thread Hans Petter Selasky
On 05/27/16 00:54, Ian Lepore wrote: On Thu, 2016-05-26 at 14:37 -0700, Gleb Smirnoff wrote: Hans, On Thu, May 26, 2016 at 08:41:55AM +, Hans Petter Selasky wrote: H> Author: hselasky H> Date: Thu May 26 08:41:55 2016 H> New Revision: 300718 H> URL: https://svnweb.freebsd.org/changeset/ba

Re: svn commit: r300718 - in head/sys: kern sys

2016-05-27 Thread Hans Petter Selasky
On 05/26/16 23:37, Gleb Smirnoff wrote: Hans, On Thu, May 26, 2016 at 08:41:55AM +, Hans Petter Selasky wrote: H> Author: hselasky H> Date: Thu May 26 08:41:55 2016 H> New Revision: 300718 H> URL: https://svnweb.freebsd.org/changeset/base/300718 H> H> Log: H> Add support for boolean sysc

Re: svn commit: r300718 - in head/sys: kern sys

2016-05-26 Thread Ian Lepore
On Thu, 2016-05-26 at 14:37 -0700, Gleb Smirnoff wrote: > Hans, > > On Thu, May 26, 2016 at 08:41:55AM +, Hans Petter Selasky wrote: > H> Author: hselasky > H> Date: Thu May 26 08:41:55 2016 > H> New Revision: 300718 > H> URL: https://svnweb.freebsd.org/changeset/base/300718 > H> > H> Log:

Re: svn commit: r300718 - in head/sys: kern sys

2016-05-26 Thread Gleb Smirnoff
Hans, On Thu, May 26, 2016 at 08:41:55AM +, Hans Petter Selasky wrote: H> Author: hselasky H> Date: Thu May 26 08:41:55 2016 H> New Revision: 300718 H> URL: https://svnweb.freebsd.org/changeset/base/300718 H> H> Log: H> Add support for boolean sysctl's. H> H> Because the size of bool

svn commit: r300718 - in head/sys: kern sys

2016-05-26 Thread Hans Petter Selasky
Author: hselasky Date: Thu May 26 08:41:55 2016 New Revision: 300718 URL: https://svnweb.freebsd.org/changeset/base/300718 Log: Add support for boolean sysctl's. Because the size of bool can be implementation defined, make a bool sysctl handler which handle bools. Userspace sees the bools