[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Xen-devel] Re: [RFC PATCH] libxl: basic virtio disk / nic configuration support.
On Sat, 30 Jul 2011, Wei Liu wrote:
> On Thu, Jul 28, 2011 at 02:05:08PM +0100, Stefano Stabellini wrote:
> > 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:
> > LIBXL_DISK_PROTOCOL_XENVBD
> > LIBXL_DISK_PROTOCOL_VIRTIO
> > then parse_disk_config can be modified to distinguish xen disks from
> > virtio disks and set the field accordingly.
> Hmm... wait...
> Adding this field is easy. But I don't know if I fully understand your
> I am just about to modify the parser. But looking back your replay,
> you are not suggesting adding configuration syntax support in config
> file (sort of `disk=["...,protocol=virito"]`).
> So essentially the new patch will be of no difference to the original
> one. But one advantage is that your plan seems cleaner (not exposing
> hacks to other functions).
> Do I get your point?
Yes, that was my main point.
Xen-devel mailing list