From mboxrd@z Thu Jan  1 00:00:00 1970
From: George Dunlap <george.dunlap@eu.citrix.com>
Subject: Re: [PATCH V3 4/6] xl: add pvusb commands
Date: Thu, 21 May 2015 14:43:48 +0100
Message-ID: <555DE114.2020904@eu.citrix.com>
References: <1429415452-6161-1-git-send-email-cyliu@suse.com>
	<1429415452-6161-5-git-send-email-cyliu@suse.com>	<5534B4D8.4010109@suse.com>
	<CAFLBxZYRFHsUUJo60Gf2U3H_DN6wbsyZWPDTU1KGfGaLLeJP4Q@mail.gmail.com>
	<555C9B3A.6000906@suse.com>
	<CAFLBxZY5YTnXf_OaJT=WQgz9CHYQmLJZ3GXnJ9EdQfhVCZTOwQ@mail.gmail.com>
	<555CA04F.20705@suse.com> <555CA77F.9030509@eu.citrix.com>
	<555D526C.4090603@suse.com> <555DB578.8070902@eu.citrix.com>
	<555DB8DE.50903@suse.com> <555DBD59.1070201@eu.citrix.com>
	<555DC84E.9090302@suse.com> <555DD745.10908@eu.citrix.com>
	<555DD8DC.3050102@suse.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <555DD8DC.3050102@suse.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: Juergen Gross <jgross@suse.com>
Cc: Lars Kurth <lars.kurth@citrix.com>, Wei Liu <wei.liu2@citrix.com>, Ian Campbell <ian.campbell@citrix.com>, Chunyan Liu <cyliu@suse.com>, "xen-devel@lists.xen.org" <xen-devel@lists.xen.org>, Ian Jackson <ian.jackson@citrix.com>, Simon Cao <caobosimon@gmail.com>
List-Id: xen-devel@lists.xenproject.org

On 05/21/2015 02:08 PM, Juergen Gross wrote:
> On 05/21/2015 03:01 PM, George Dunlap wrote:
>> On 05/21/2015 12:58 PM, Juergen Gross wrote:
>>> There might be existing users who know about "xm usb-assignable-list".
>>
>> Yes -- unfortunately something has to give: either we confuse new users
>> with two names that sound similar but do something different, or we
>> confuse former xm users by renaming their function, or we confuse
>> current xl users by renaming their function.  There's badness to be had
>> whichever one we choose.  I think renaming the xm function is the least
>> bad of all the options.
>>
>>> OTOH I don't care giving it another name, as long as the functionality
>>> is available.
>>
>> OK -- what about "usb-available-list"?  Any objections / alternate
>> suggestions from anyone?
> 
> Hmm, just another idea:
> 
> xl usb-list -a
> 
> could list all domains with assigned USB-devices and the currently not
> assigned devices as well. This would avoid the need for another command
> name. It would even be possible to omit the "-a".

That works for me too, I guess.

Can I suggest, though, that work on that functionality be detached from
getting the core pvusb functionality in?  I can't really effectively do
the qemu side until it is in due to the cost of rebasing, and I would
really like to have both in for 4.6 if possible.

 -George