Re: svn commit: r290475 - in head: share/man/man9 sys/kern sys/sys

2016-01-24 Thread Conrad Meyer
No. They break ABI. Best, Conrad On Sun, Jan 24, 2016 at 1:45 PM, Oliver Pinter wrote: > On 11/8/15, Hans Petter Selasky wrote: >> On 11/07/15 19:26, Conrad Meyer wrote: >>> Can do — fixed in r290505. >>> >>> Thanks, >>> Conrad >> >> Thank you. Looks good! >> >> --HPS > > Hi All! > > Is there

Re: svn commit: r290475 - in head: share/man/man9 sys/kern sys/sys

2016-01-24 Thread Oliver Pinter
On 11/8/15, Hans Petter Selasky wrote: > On 11/07/15 19:26, Conrad Meyer wrote: >> Can do — fixed in r290505. >> >> Thanks, >> Conrad > > Thank you. Looks good! > > --HPS Hi All! Is there any way to merge back to 10-STABLE these patches? > > ___ > svn

Re: svn commit: r290475 - in head: share/man/man9 sys/kern sys/sys

2015-11-08 Thread Hans Petter Selasky
On 11/07/15 19:26, Conrad Meyer wrote: Can do — fixed in r290505. Thanks, Conrad Thank you. Looks good! --HPS ___ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-sr

Re: svn commit: r290475 - in head: share/man/man9 sys/kern sys/sys

2015-11-07 Thread Conrad Meyer
Sorry about that — Ravi and I missed this. I see Konstantin has since fixed it (thanks!). There is also the intptr_t in SYSCTL_HANDLER_ARGS and oid_arg2 in struct sysctl_oid remaining. Best, Conrad On Sat, Nov 7, 2015 at 12:37 AM, Svatopluk Kraus wrote: > You broke buildkernel. The following p

Re: svn commit: r290475 - in head: share/man/man9 sys/kern sys/sys

2015-11-07 Thread Conrad Meyer
Can do — fixed in r290505. Thanks, Conrad On Sat, Nov 7, 2015 at 3:20 AM, Hans Petter Selasky wrote: > On 11/07/15 09:37, Svatopluk Kraus wrote: >> >> +#defineSYSCTL_NULL_U64_PTR ((unsigned *)NULL) > > > Hi Conrad, > > All the NULL macros must have correct type according to the size-chec

Re: svn commit: r290475 - in head: share/man/man9 sys/kern sys/sys

2015-11-07 Thread Bruce Evans
On Sat, 7 Nov 2015, Konstantin Belousov wrote: On Sat, Nov 07, 2015 at 09:25:32PM +1100, Bruce Evans wrote: ... I put intptr_t in long ago, since it was more needed and less magic than intmax_t. This was obfuscated by moving it to and including that in various places. intmax_t is still only

Re: svn commit: r290475 - in head: share/man/man9 sys/kern sys/sys

2015-11-07 Thread Konstantin Belousov
On Sat, Nov 07, 2015 at 09:25:32PM +1100, Bruce Evans wrote: > On Sat, 7 Nov 2015, Svatopluk Kraus wrote: > > > You broke buildkernel. The following patch helps: > > > > diff --git a/sys/sys/sysctl.h b/sys/sys/sysctl.h > > index 950e712..a34c890 100644 > > --- a/sys/sys/sysctl.h > > +++ b/sys/sys/

Re: svn commit: r290475 - in head: share/man/man9 sys/kern sys/sys

2015-11-07 Thread Hans Petter Selasky
On 11/07/15 09:37, Svatopluk Kraus wrote: +#defineSYSCTL_NULL_U64_PTR ((unsigned *)NULL) Hi Conrad, All the NULL macros must have correct type according to the size-check asserts. Else they won't have any purpose: #define SYSCTL_NULL_U64_PTR ((uint64_t *)NULL) #define SYSCTL_NULL_S6

Re: svn commit: r290475 - in head: share/man/man9 sys/kern sys/sys

2015-11-07 Thread Bruce Evans
On Sat, 7 Nov 2015, Svatopluk Kraus wrote: You broke buildkernel. The following patch helps: diff --git a/sys/sys/sysctl.h b/sys/sys/sysctl.h index 950e712..a34c890 100644 --- a/sys/sys/sysctl.h +++ b/sys/sys/sysctl.h @@ -37,6 +37,7 @@ #define _SYS_SYSCTL_H_ #include +#include struct thread

Re: svn commit: r290475 - in head: share/man/man9 sys/kern sys/sys

2015-11-07 Thread Bruce Evans
On Sat, 7 Nov 2015, Conrad E. Meyer wrote: Log: Round out SYSCTL macros to the full set of fixed-width types Add S8, S16, S32, and U32 types; add SYSCTL*() macros for them, as well as for the existing 64-bit types. (While SYSCTL*QUAD and UQUAD macros already exist, they do not take the sa

Re: svn commit: r290475 - in head: share/man/man9 sys/kern sys/sys

2015-11-07 Thread Svatopluk Kraus
You broke buildkernel. The following patch helps: diff --git a/sys/sys/sysctl.h b/sys/sys/sysctl.h index 950e712..a34c890 100644 --- a/sys/sys/sysctl.h +++ b/sys/sys/sysctl.h @@ -37,6 +37,7 @@ #define _SYS_SYSCTL_H_ #include +#include struct thread; /* @@ -949,7 +950,7 @@ extern char kern_