From mboxrd@z Thu Jan 1 00:00:00 1970 From: Juergen Gross Subject: Re: [PATCH RFC v1] libxl: Introduce a template for devices with a controller Date: Fri, 22 May 2015 06:21:01 +0200 Message-ID: <555EAEAD.2040103@suse.com> References: <1432228052-15667-1-git-send-email-george.dunlap@eu.citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1432228052-15667-1-git-send-email-george.dunlap@eu.citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: George Dunlap , xen-devel@lists.xen.org Cc: Ian Jackson , Olaf Hering , Wei Liu , Ian Campbell , Chun Yan Liu List-Id: xen-devel@lists.xenproject.org On 05/21/2015 07:07 PM, George Dunlap wrote: > We have several outstanding patch series which add devices that have > two levels: a controller and individual devices attached to that > controller. > > In the interest of consistency, this patch introduces a section that > sketches out a template for interfaces for such devices. > > Signed-off-by: George Dunlap > --- > CC: Ian Campbell > CC: Ian Jackson > CC: Wei Liu > CC: Juergen Gross > CC: Chun Yan Liu > CC: Olaf Hering > > So, this is definitely RFC -- I tried to spec things out in a way that > made sense, but I often just chose something that I thought would be a > sensible starting point for discussion. > > This spec looks a lot more like the PVUSB spec than the PVSCSI spec, > in part because I think the PVUSB spec has already had a lot more > thought that's gone into it. > > A couple of random points to discuss: > > * Calling things "controllers", using ctrl for the device name, > and using "ctrl" as the field name for the devid of the controller > in the individual devices. Hmm, what about "device group" (devgoup)? In the scsi world "controller" would be one level higher in the hierarchy. And the scsi controller is at least visible in the configuration syntax "h:c:t:l". Using "controller" for the "c" in this item and for the "t" internally could lead to confusion. Juergen