[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v3] block/vxhs: Add Veritas HyperScale VxHS bloc

From: Kevin Wolf
Subject: Re: [Qemu-devel] [PATCH v3] block/vxhs: Add Veritas HyperScale VxHS block device support
Date: Wed, 2 Nov 2016 16:02:07 +0100
User-agent: Mutt/1.5.21 (2010-09-15)

Am 02.11.2016 um 15:13 hat Eric Blake geschrieben:
> On 11/02/2016 04:57 AM, Kevin Wolf wrote:
> >>> IMHO it should allow use of UNIX sockets, as its possible to have
> >>> SSH setup a tunnel to a IP server, and expose the endpoint via a
> >>> UNIX socket. So even if your reference server only supports IPv4,
> >>> users can conceivably connect with any sockets protocol.
> >>>
> >>
> >> This is not a use-case that we have in mind right now, but a very fair
> >> point! Kind of like accessing the audio/video streams remotely from my
> >> Raspberry Pi over ssh. Is it OK if we target this in a future patch
> >> after proper review/testing?
> > 
> > No, going from InetSocketAddress to SocketAddress changes the API in an
> > incompatible way (previously working blockdev-add commands would stop
> > working), so we must decide now before the API is introduced.
> Going from InetSocketAddress to SocketAddress is a pain because
> SocketAddress is a 'simple union' (which really means extra nesting on
> the wire).  I'd really rather that blockdev-add move towards the
> GlusterServer style of a 'flat union', which CAN be done in a
> backwards-compatible manner.

However, the actual networking functions in qemu take a SocketAddress or
InetSocketAddress pointer, not whatever else we would invent for the
block layer API, so we would have to add conversion functions. The whole
point of using these types was to have a consistent API with other
subsystems. :-/

gluster and NFS can use something different only because they don't use
the qemu networking functions, but whatever limited set of options their
external library supports.

> [side note: Uggh - why did we let gluster go into 2.7 with 'tcp' instead
> of 'inet' :( I'm wondering if we want gluster in 2.8 to recognize 'inet'
> as a synonym for 'tcp'; I could propose a patch if we think it is
> worthwhile]

Fine with me. Remove 'tcp' from the schema then as long as blockdev-add
is still experimental. We only need to keep it working on the command


Attachment: pgpySgI6LxqVR.pgp
Description: PGP signature

reply via email to

[Prev in Thread] Current Thread [Next in Thread]