Re: svn commit: r273784 - in head/sys: amd64/ia32 compat/freebsd32 i386/i386 kern net

2014-10-29 Thread Konstantin Belousov
On Wed, Oct 29, 2014 at 12:00:49PM -0400, John Baldwin wrote: > On Wednesday, October 29, 2014 11:52:30 am Konstantin Belousov wrote: > > On Wed, Oct 29, 2014 at 06:26:42AM +1100, Bruce Evans wrote: > > > I just noticed some more API design errors. The pointer type for new > > > APIs should be [qu

Re: svn commit: r273784 - in head/sys: amd64/ia32 compat/freebsd32 i386/i386 kern net

2014-10-29 Thread John Baldwin
On Wednesday, October 29, 2014 11:52:30 am Konstantin Belousov wrote: > On Wed, Oct 29, 2014 at 06:26:42AM +1100, Bruce Evans wrote: > > On Tue, 28 Oct 2014, Konstantin Belousov wrote: > > > > > Log: > > > Replace some calls to fuword() by fueword() with proper error checking. > > > > I just not

Re: svn commit: r273784 - in head/sys: amd64/ia32 compat/freebsd32 i386/i386 kern net

2014-10-29 Thread Konstantin Belousov
On Wed, Oct 29, 2014 at 06:26:42AM +1100, Bruce Evans wrote: > On Tue, 28 Oct 2014, Konstantin Belousov wrote: > > > Log: > > Replace some calls to fuword() by fueword() with proper error checking. > > I just noticed some more API design errors. The pointer type for new > APIs should be [qualif

Re: svn commit: r273784 - in head/sys: amd64/ia32 compat/freebsd32 i386/i386 kern net

2014-10-28 Thread Bruce Evans
On Tue, 28 Oct 2014, Konstantin Belousov wrote: Log: Replace some calls to fuword() by fueword() with proper error checking. I just noticed some more API design errors. The pointer type for new APIs should be [qualifed] wordsize_t *, not [qualified] void *. Using void * reduces type safety

svn commit: r273784 - in head/sys: amd64/ia32 compat/freebsd32 i386/i386 kern net

2014-10-28 Thread Konstantin Belousov
Author: kib Date: Tue Oct 28 15:28:20 2014 New Revision: 273784 URL: https://svnweb.freebsd.org/changeset/base/273784 Log: Replace some calls to fuword() by fueword() with proper error checking. Sponsored by: The FreeBSD Foundation Tested by:pho MFC after:3 weeks Modified: he