Re: [Xen-devel] [RFC] support more qdisk types

[Ian Campbell <ian.campbell@xxxxxxxxxx>]

On Tue, 2016-02-02 at 15:06 -0700, Jim Fehlig wrote:
> 
> > And extending
> > the structure seems to be the right thing to do.
> 
> So what do you think of the approach in the RFC patch? It adds discrete knobs 
> in
> the disk config and extends the disk structure similarly. Before I can make
> progress on this we need to agree on extending the config and 
> libxl_device_disk
> structure.

My main concern is that this approach requires us to update libxl for each
new possible backend type.

The intention of the target= in the disk spec is that it consumes the rest
of the line so it can be used to encode pretty much anything. Is it not
possible (modulo bugs) to pass all the necessary information to qdisk in
this form? I thought Dave S had made it possible to use qdisk in this way
back in:

    commit a8a1f236a2964506a22d1779648d8e1c8668cb1a
    Author: David Scott <    dave.scott@xxxxxxxxxxxxx    >
    Date:ÂÂÂTue Apr 23 10:59:26 2013 +0100

    ÂÂÂÂlibxl: Only call stat() when adding a disk if we expect a device to 
exist.
    ÂÂÂÂ
    ÂÂÂÂWe consider calling stat() a helpful error check in the following
    ÂÂÂÂcircumstances only:
    ÂÂÂÂÂ1. the disk backend type must be PHYsical
    ÂÂÂÂÂ2. the disk backend domain must be the same as the running libxl
    ÂÂÂÂÂÂÂÂcode (ie LIBXL_TOOLSTACK_DOMID)
    ÂÂÂÂÂ3. there must not be a hotplug script because this would imply that
    ÂÂÂÂÂÂÂÂthe device won't be created until after the hotplug script has
    ÂÂÂÂÂÂÂÂrun.
    ÂÂÂÂ
    ÂÂÂÂWith this fix, it is possible to use qemu's built-in block drivers
    ÂÂÂÂsuch as ceph/rbd, with a xl config disk spec like this:
    ÂÂÂÂ
    ÂÂÂÂdisk=[ 
'backendtype=qdisk,format=raw,vdev=hda,access=rw,target=rbd:rbd/ubuntu1204.img' 
]
    ÂÂÂÂ
    ÂÂÂÂSigned-off-by: David Scott <    dave.scott@xxxxxxxxxxxxx    >
    ÂÂÂÂAcked-by: Ian Campbell <    ian.campbell@xxxxxxxxxx    >
    ÂÂÂÂAcked-by: Roger Pau Monnà <    roger.pau@xxxxxxxxxx    >

Of course things may have regressed again since then.

Since target is basically passed as is to qdisk does libvirt not already
have some helpers in the QEMU backend for constructing such things?

Ian.

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