This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
Home Products Support Community News


[Xen-devel] Re: [RFC PATCH] libxl: basic virtio disk / nic configuration

To: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
Subject: [Xen-devel] Re: [RFC PATCH] libxl: basic virtio disk / nic configuration support.
From: Wei Liu <liuw@xxxxxxxxx>
Date: Fri, 29 Jul 2011 14:35:07 +0800
Cc: Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Delivery-date: Thu, 28 Jul 2011 23:34:52 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <alpine.DEB.2.00.1107281258100.12963@kaball-desktop>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <1311818929.1800.21.camel@limbo> <alpine.DEB.2.00.1107281258100.12963@kaball-desktop>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.20 (2009-06-14)
On Thu, Jul 28, 2011 at 02:05:08PM +0100, Stefano Stabellini wrote:
> On Thu, 28 Jul 2011, Wei Liu wrote:
> > This is a prototype patch for libxl to write virtio disk and nic
> > configurations in xenstore.
> > 
> > I have following questions on this patch:
> > 
> > 1. VBD encoding for virtio disk.
> > In fact, virtio disk driver doesn't care about this encoding, so
> > arbitrary number is OK. I also discover a strange bug in xenstore -- if
> > I use (2<<28) | (disk<<4) | partition (reserved encoding), it will
> > trigger a xenstore parsing bug which affects existing protocol (VIF, VFB
> > etc.), causing them fail to initialize frontend and backend.
> Apart from the strange bug, I think that the encoding is reasonable.

A second thought comes to me. Since virtio frontend does not care
about this encoding, can we just use 0,1,...,n for it? Just like VIF
> > 2. Configuration syntax and implementation for virtio disk and nic.
> > Currently some hacks are needed to distinguish virtio devices. I hope we
> > can get a cleaner implementation.
> I think you need to add a new field in libxl_device_disk to specify the
> protocol, something like libxl_disk_protocol, that can be:
> then parse_disk_config can be modified to distinguish xen disks from
> virtio disks and set the field accordingly.

Good idea. Working on this.


Xen-devel mailing list