[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH V4 3/7] libxl: add pvusb API



On 06/18/2015 08:20 AM, Chun Yan Liu wrote:


On 6/17/2015 at 07:34 PM, in message <1434540857.13744.334.camel@xxxxxxxxxx>,
Ian Campbell <ian.campbell@xxxxxxxxxx> wrote:
On Tue, 2015-06-16 at 16:20 +0200, Juergen Gross wrote:
My point was to avoid the sysfs accesses in libxl in order to support
BSD as well and to reduce the complexity.

As a slight aside to this, can't libxl use libusb for a lot of this
stuff and therefore avoid being Linux specific?

http://libusb.info/ claims to support "Linux, OS X, Windows, Windows CE,
Android, OpenBSD/NetBSD, Haiku.". Interestingly FreeBSD is missing there
but I don't think that need to be a blocker.

I don't see a problem with adding libusb to our set of dependencies, and
it's certainly got to be better than (re)implementing a bunch of sysfs
stuff (which I presume is what libusb does under the hood anyway).

Using libusb is certainly good and can save effort to implement things by
ourself. Only one concern: some functions require newer version of libusb.
For example, functions to get port (quite like busid) information
libusb_get_port_numbers or libusb_get_port_path need
libusb version >= 1.0.12. If libusb version is not satisfied, the whole pvusb
work might be blocked.

Is this really a problem? lubusb version 1.0.12 is out for three years
now. Would we really need to support pvUSB on dom0 not being capable
to install a more recent libusb?


Juergen


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.