From mboxrd@z Thu Jan  1 00:00:00 1970
From: Juergen Gross <jgross@suse.com>
Subject: Re: [PATCH V4 3/7] libxl: add pvusb API
Date: Thu, 18 Jun 2015 09:02:25 +0200
Message-ID: <55826D01.3080008@suse.com>
References: <1433906441-3280-1-git-send-email-cyliu@suse.com>	<1433906441-3280-4-git-send-email-cyliu@suse.com>	<CAFLBxZbVXjj-VM4KLdhqBkpC0qSR_sy3mNnSZAJQJmHuo4ZC0Q@mail.gmail.com>	<21887.64856.265751.921367@mariner.uk.xensource.com>	<558000E5.9000803@suse.com>	<21888.1058.681450.470806@mariner.uk.xensource.com>	<5580079D.6070007@suse.com>	<21888.3180.937801.173964@mariner.uk.xensource.com>	<55801F56.1090906@suse.com>
	<55802025.2020102@eu.citrix.com>	<55802339.40808@suse.com>
	<558024B3.2060303@eu.citrix.com>	<55802977.8060309@suse.com>
	<55802D6D.2030108@eu.citrix.com>	<558030BD.5050709@suse.com>
	<1434540857.13744.334.camel@citrix.com>
	<5582D39102000066000D8846@relay2.provo.novell.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"; Format="flowed"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <5582D39102000066000D8846@relay2.provo.novell.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Chun Yan Liu <cyliu@suse.com>, Ian Campbell <ian.campbell@citrix.com>
Cc: Wei Liu <wei.liu2@citrix.com>, George Dunlap <george.dunlap@eu.citrix.com>, Ian Jackson <Ian.Jackson@eu.citrix.com>, "xen-devel@lists.xen.org" <xen-devel@lists.xen.org>, Jim Fehlig <JFEHLIG@suse.com>, Simon Cao <caobosimon@gmail.com>
List-Id: xen-devel@lists.xenproject.org

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@citrix.com>,
> Ian Campbell <ian.campbell@citrix.com> 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