From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43334) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bpDd4-0007Sk-EN for qemu-devel@nongnu.org; Wed, 28 Sep 2016 08:06:30 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bpDcw-0002zh-Tt for qemu-devel@nongnu.org; Wed, 28 Sep 2016 08:06:25 -0400 Received: from mx1.redhat.com ([209.132.183.28]:42798) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bpDcw-0002zT-N4 for qemu-devel@nongnu.org; Wed, 28 Sep 2016 08:06:18 -0400 Date: Wed, 28 Sep 2016 13:06:13 +0100 From: "Daniel P. Berrange" Message-ID: <20160928120613.GA15510@redhat.com> Reply-To: "Daniel P. Berrange" References: <1475035789-685-1-git-send-email-ashish.mittal@veritas.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <1475035789-685-1-git-send-email-ashish.mittal@veritas.com> Subject: Re: [Qemu-devel] [PATCH v7 RFC] block/vxhs: Initial commit to add Veritas HyperScale VxHS block device support List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Ashish Mittal Cc: qemu-devel@nongnu.org, pbonzini@redhat.com, kwolf@redhat.com, armbru@redhat.com, jcody@redhat.com, famz@redhat.com, ashish.mittal@veritas.com, stefanha@gmail.com, Ketan.Nilangekar@veritas.com, Abhijit.Dey@veritas.com On Tue, Sep 27, 2016 at 09:09:49PM -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 > > 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"},{"host":"172.172.17.2","port":"9999"}]}' Please line wrap the text here > > 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 and here. > Signed-off-by: Ashish Mittal > block/Makefile.objs | 2 + > block/trace-events | 47 ++ > block/vxhs.c | 1645 +++++++++++++++++++++++++++++++++++++++++++++++++++ > configure | 41 ++ > 4 files changed, 1735 insertions(+) > create mode 100644 block/vxhs.c This has lost the QAPI schema definition in qapi/block-core.json that was in earlier versions. We should not be adding new block drivers without having a QAPI schema defined for them. I would like to see this use the exact same syntax for specifying the server as is used for Gluster, as it will simplify life for libvirt to only have one format to generate. This would simply renaming the current 'GlusterServer' QAPI struct to be something more generic perhaps "BlockServer" so that it can be shared between both. It also means that the JSON example above must include the 'type' discriminator. 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/ :|