[libvirt] [PATCH v4 0/3] Add support for Veritas HyperScale (VxHS) block device protocol

Peter Krempa pkrempa at redhat.com
Fri Jun 30 07:41:18 UTC 2017

On Thu, Jun 29, 2017 at 19:02:38 -0700, Ashish Mittal wrote:
> From: Ashish Mittal <ashish.mittal at veritas.com>
> QEMU changes for VxHS (including TLS support) are already upstream.
> This series of patches adds support for VxHS block devices in libvirt.
> Patch 1 adds the base functionality for supporting VxHS protocol.
> Patch 2 adds two new configuration options in qemu.conf to enable TLS
> for VxHS devices.
> Patch 3 implements the main TLS functionality.

This ordering is wrong and also your patches have a lot of stuff going
on at once.

I suggest you add the TLS support (Which should be designed to be
generic with other protocols which may start using TLS in mind) first
and then start adding the rest of the stuff.

I'll reply to some parts of the patches separately, but in this state
it's kind of a mess to go through, so please re-send the patch split
into reasonable chunks.

Note that each patch needs to make sense and can be compiled and tested

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20170630/31f39893/attachment-0001.sig>

More information about the libvir-list mailing list