Date: Sat, 16 Oct 2010 11:12:37 +0300 From: Kostik Belousov <kostikbel@gmail.com> To: Hans Petter Selasky <hselasky@freebsd.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r213852 - in head: lib/libusb sys/dev/usb Message-ID: <20101016081237.GR2392@deviant.kiev.zoral.com.ua> In-Reply-To: <201010142038.o9EKcImV036360@svn.freebsd.org> References: <201010142038.o9EKcImV036360@svn.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--jGE3xONbfNfY0RQX Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Oct 14, 2010 at 08:38:18PM +0000, Hans Petter Selasky wrote: > Author: hselasky > Date: Thu Oct 14 20:38:18 2010 > New Revision: 213852 > URL: http://svn.freebsd.org/changeset/base/213852 >=20 > Log: > - Add support for LibUSB in 32-bit compatibility mode. > =20 > Approved by: thompsa (mentor) >=20 > Modified: > head/lib/libusb/Makefile > head/lib/libusb/libusb20.c > head/lib/libusb/libusb20_int.h > head/lib/libusb/libusb20_ugen20.c > head/sys/dev/usb/usb_ioctl.h >=20 > Modified: head/lib/libusb/Makefile > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D > --- head/lib/libusb/Makefile Thu Oct 14 20:31:07 2010 (r213851) > +++ head/lib/libusb/Makefile Thu Oct 14 20:38:18 2010 (r213852) > @@ -30,5 +30,9 @@ SRCS+=3D libusb10.c > SRCS+=3D libusb10_desc.c > SRCS+=3D libusb10_io.c > =20 > +.if defined(COMPAT_32BIT) > +CFLAGS+=3D -DCOMPAT_32BIT > +.endif > + > .include <bsd.lib.mk> The support is provided in a way that contradicts the established practice of doing 32-bit compat. Very nice that the support is provided, thank you for care about it. But, can it be changed so that the kernel emulates 32-bit ABI instead of library conforming to the kernel ABI ? For COMPAT32, we aim in making the system where 32bit binaries and libraries just work on the 64bit host. Your change does not allow to take 32bit host into jail and run it on 64bit kernel, as example. Please see numerous examples of ioctl translations under #ifdef COMPAT_FREEBSD32 on how it is done. --jGE3xONbfNfY0RQX Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (FreeBSD) iEYEARECAAYFAky5XnQACgkQC3+MBN1Mb4i8fwCeLIBdtrj7uuq95LBmLD8NJDVC cFAAnRFvYgdJHwSHj4YsGdYtSjLA33lj =OQhf -----END PGP SIGNATURE----- --jGE3xONbfNfY0RQX--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20101016081237.GR2392>