Re: [PATCH nfs-utils v3 00/14] add NFS over AF_VSOCK support

[NeilBrown <neilb@suse.com>]

[-- Attachment #1: Type: text/plain, Size: 3919 bytes --]


Please don't send these patches to me.  You know what I think of the
whole project.

NeilBrown


On Wed, Sep 13 2017, Stefan Hajnoczi wrote:

> v3:
>  * Documented vsock syntax in exports.man, nfs.man, and nfsd.man
>  * Added clientaddr autodetection in mount.nfs(8)
>  * Replaced #ifdefs with a single vsock.h header file
>  * Tested nfsd serving both IPv4 and vsock at the same time
>
> Status:
>
>  * The last revision was somewhat controversial because it's already possible
>    to share files between a hypervisor and virtual machine using TCP/IP, so why
>    add AF_VSOCK support to the stack?  TCP/IP based solutions require the
>    virtual machine administrator to be involved in the configuration and are
>    therefore not suitable for automatic management by OpenStack, oVirt, etc.
>    Maintainers, is this feature acceptable?
>
>  * Need advice on netid: is there agreement to use "tcpv" instead of "vsock" as
>    Chuck Lever suggested and how to ask IESG to assign it?
>
> The AF_VSOCK address family allows virtual machines to communicate with the
> hypervisor using a zero-configuration transport.  KVM, VMware, and Hyper-V
> hypervisors support AF_VSOCK and it was first introduced in Linux 3.9.
>
> This patch series adds AF_VSOCK support to mount.nfs(8) and rpc.nfsd(8).  To
> mount an export from the hypervisor (CID 2):
>
>   # mount.nfs 2:/srv/vm01 /mnt -o proto=vsock
>
> To serve exports over vsock port 2049:
>
>   # nfsd ... --vsock 2049
>
> This series extends exports(5) syntax to handle vsock:<CID> or vsock:*.  For
> example, the guest with CID 3 can be given access using vsock:3.
>
> nfsd can export over IPv4/IPv6 and vsock at the same time.  See the changes to
> exports.man, nfs.man, and nfsd.man in the patches for syntax details.
>
> NFSv4 and later are supported.
>
> The code is also available here:
> https://github.com/stefanha/nfs-utils/tree/vsock-nfsd
>
> The latest kernel patches are available here:
> https://github.com/stefanha/linux/tree/vsock-nfsd
>
> Stefan Hajnoczi (14):
>   mount: don't use IPPROTO_UDP for address resolution
>   nfs-utils: add vsock.h
>   nfs-utils: add AF_VSOCK support to sockaddr.h
>   mount: present AF_VSOCK addresses
>   mount: accept AF_VSOCK in nfs_verify_family()
>   mount: generate AF_VSOCK clientaddr
>   getport: recognize "vsock" netid
>   mount: AF_VSOCK address parsing
>   exportfs: introduce host_freeaddrinfo()
>   exportfs: add AF_VSOCK address parsing and printing
>   exportfs: add AF_VSOCK support to set_addrlist()
>   exportfs: add support for "vsock:" exports(5) syntax
>   nfsd: add --vsock (-v) option to nfsd
>   tests: add "vsock:" exports(5) test case
>
>  tests/Makefile.am          |   3 +-
>  support/include/exportfs.h |   4 ++
>  support/include/sockaddr.h |  18 +++++
>  support/include/vsock.h    |  59 +++++++++++++++++
>  utils/nfsd/nfssvc.h        |   1 +
>  support/export/client.c    |   8 +--
>  support/export/hostname.c  | 161 +++++++++++++++++++++++++++++++++++++++++++--
>  support/nfs/getport.c      |  16 +++--
>  utils/exportfs/exportfs.c  |  42 ++++++++++--
>  utils/mount/network.c      |  37 ++++++++++-
>  utils/mount/stropts.c      |  61 ++++++++++++++---
>  utils/mountd/auth.c        |   2 +-
>  utils/mountd/cache.c       |  10 +--
>  utils/mountd/mountd.c      |   4 +-
>  utils/mountd/rmtab.c       |   2 +-
>  utils/nfsd/nfsd.c          |  18 ++++-
>  utils/nfsd/nfssvc.c        |  62 +++++++++++++++++
>  configure.ac               |   3 +
>  tests/t0002-vsock-basic.sh |  53 +++++++++++++++
>  utils/exportfs/exports.man |  12 +++-
>  utils/mount/nfs.man        |  20 ++++--
>  utils/nfsd/nfsd.man        |   4 ++
>  22 files changed, 552 insertions(+), 48 deletions(-)
>  create mode 100644 support/include/vsock.h
>  create mode 100755 tests/t0002-vsock-basic.sh
>
> -- 
> 2.13.5

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]