Re: [Xen-devel] [PATCH 6/6] xl/libxl: implement QDISK libxl_device_disk_local_attach

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

Stefano Stabellini writes ("[PATCH 6/6] xl/libxl: implement QDISK 
libxl_device_disk_local_attach"):
> - Spawn a QEMU instance at boot time to handle disk local attach
> requests.

This is a bit strange.  Why does this need to be a single daemon ?
Can't we have one qemu per disk ?

> - Implement libxl_device_disk_local_attach for QDISKs in terms of a
> regular disk attach with the frontend and backend running in the same
> domain.
...
> +                    LIBXL__LOG(ctx, LIBXL__LOG_ERROR, "libxl__alloc_vdev 
> failed\n");

Too many long lines for me to comfortably read I'm afraid.

Ian.

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