[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1] block/vxhs: Add Veritas HyperScale VxHS bloc
From: |
Daniel P. Berrange |
Subject: |
Re: [Qemu-devel] [PATCH v1] block/vxhs: Add Veritas HyperScale VxHS block device support |
Date: |
Wed, 26 Oct 2016 08:30:18 +0200 |
User-agent: |
Mutt/1.7.1 (2016-10-04) |
On Wed, Oct 26, 2016 at 12:41:28AM -0400, Jeff Cody wrote:
> On Tue, Oct 25, 2016 at 03:02:07PM -0700, Ashish Mittal wrote:
> > This patch adds support for a new block device type called "vxhs".
> > Source code for the library that this code loads can be downloaded from:
> > https://github.com/MittalAshish/libqnio.git
> >
>
> I grabbed the latest of libqnio, compiled it (had to disable -Werror), and
> tried it out. I was able to do a qemu-img info on a raw file, but it would
> just hang when trying a format such as qcow2. I am assuming
> this is a limitation of test_server, and not libqnio.
>
> This will make qemu-iotests more difficult however.
>
> I haven't looked at the latest qnio code yet (other than compiling the
> test-server to test), so the rest of this review is on the qemu driver.
>
> > Sample command line using JSON syntax:
> > ./qemu-system-x86_64 -name instance-00000008 -S -vnc 0.0.0.0:0 -k en-us
> > -vga cirrus -device virtio-balloon-pci,id=balloon0,bus=pci.0,addr=0x5 -msg
> > timestamp=on
> > 'json:{"driver":"vxhs","vdisk_id":"{c3e9095a-a5ee-4dce-afeb-2a59fb387410}","server":[{"host":"172.172.17.4","port":"9999"}]}'
> >
> > Sample command line using URI syntax:
> > qemu-img convert -f raw -O raw -n
> > /var/lib/nova/instances/_base/0c5eacd5ebea5ed914b6a3e7b18f1ce734c386ad
> > vxhs://192.168.0.1:9999/%7Bc6718f6b-0401-441d-a8c3-1f0064d75ee0%7D
> >
> > Signed-off-by: Ashish Mittal <address@hidden>
> > ---
> > block/Makefile.objs | 2 +
> > block/trace-events | 22 ++
> > block/vxhs.c | 736
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++
> > configure | 41 +++jj
> > 4 files changed, 801 insertions(+)
> > create mode 100644 block/vxhs.c
>
> I think this version still does not address Daniel's concerns regarding a
> QAPI schema for vxhs.
Yes, indeed, which makes me question what level of testing this patch
has had - hotplugging for devices is not testable without having the
QAPI schema there.
Regards,
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://entangle-photo.org -o- http://search.cpan.org/~danberr/ :|