mbox series

[iproute2-next,v4,0/5] Add vdpa device management tool

Message ID 20210205181029.365461-1-parav@nvidia.com
Headers show
Series Add vdpa device management tool | expand

Message

Parav Pandit Feb. 5, 2021, 6:10 p.m. UTC
Linux vdpa interface allows vdpa device management functionality.
This includes adding, removing, querying vdpa devices.

vdpa interface also includes showing supported management devices
which support such operations.

This patchset includes kernel uapi headers and a vdpa tool.

examples:

$ vdpa mgmtdev show
vdpasim:
  supported_classes net

$ vdpa mgmtdev show -jp
{
    "show": {
        "vdpasim": {
            "supported_classes": [ "net" ]
        }
    }
}

Create a vdpa device of type networking named as "foo2" from
the management device vdpasim_net:

$ vdpa dev add mgmtdev vdpasim_net name foo2

Show the newly created vdpa device by its name:
$ vdpa dev show foo2
foo2: type network mgmtdev vdpasim_net vendor_id 0 max_vqs 2 max_vq_size 256

$ vdpa dev show foo2 -jp
{
    "dev": {
        "foo2": {
            "type": "network",
            "mgmtdev": "vdpasim_net",
            "vendor_id": 0,
            "max_vqs": 2,
            "max_vq_size": 256
        }
    }
}

Delete the vdpa device after its use:
$ vdpa dev del foo2

An example of PCI PF, VF and SF management device:
pci/0000:03.00:0
  supported_classes
    net
pci/0000:03.00:4
  supported_classes
    net
auxiliary/mlx5_core.sf.8
  supported_classes
    net

Patch summary:
Patch-1 adds kernel headers for vdpa subsystem
Patch-2 adds library routines for indent handling
Patch-3 adds library routines for generic socket communication
PAtch-4 adds library routine for number to string mapping
Patch-5 adds vdpa tool

Kernel headers are from the vhost kernel tree [1] from branch linux-next.

[1] https://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git

---
changelog:
v3->v4:
 - relocated vdpa headers to vdpa/include/uapi/linux
 - added new headers path to CFLAGS of vdpa tool
v2->v3:
 - addressed David's comment to split patch for utils and other parts
 - rebased
 - using newly added helper routine for number to string mapping
v1->v2:
 - addressed below comments from David
 - added and used library helpers for socket communication
 - added and used library functions for string processing helpers
 - added and used library functions indent processing helpers

Parav Pandit (5):
  Add kernel headers
  utils: Add helper routines for indent handling
  utils: Add generic socket helpers
  utils: Add helper to map string to unsigned int
  vdpa: Add vdpa tool

 Makefile                             |   2 +-
 include/mnl_utils.h                  |  16 +
 include/utils.h                      |  20 +-
 lib/mnl_utils.c                      | 121 +++++
 lib/utils.c                          |  83 +++-
 man/man8/vdpa-dev.8                  |  96 ++++
 man/man8/vdpa-mgmtdev.8              |  53 +++
 man/man8/vdpa.8                      |  76 +++
 vdpa/Makefile                        |  25 +
 vdpa/include/uapi/linux/vdpa.h       |  40 ++
 vdpa/include/uapi/linux/virtio_ids.h |  58 +++
 vdpa/vdpa.c                          | 675 +++++++++++++++++++++++++++
 12 files changed, 1261 insertions(+), 4 deletions(-)
 create mode 100644 man/man8/vdpa-dev.8
 create mode 100644 man/man8/vdpa-mgmtdev.8
 create mode 100644 man/man8/vdpa.8
 create mode 100644 vdpa/Makefile
 create mode 100644 vdpa/include/uapi/linux/vdpa.h
 create mode 100644 vdpa/include/uapi/linux/virtio_ids.h
 create mode 100644 vdpa/vdpa.c

Comments

David Ahern Feb. 8, 2021, 3:47 p.m. UTC | #1
On 2/5/21 11:10 AM, Parav Pandit wrote:
> diff --git a/vdpa/include/uapi/linux/vdpa.h b/vdpa/include/uapi/linux/vdpa.h

> new file mode 100644

> index 00000000..66a41e4e

> --- /dev/null

> +++ b/vdpa/include/uapi/linux/vdpa.h

> @@ -0,0 +1,40 @@

> +/* SPDX-License-Identifier: GPL-2.0+ WITH Linux-syscall-note */

> +/*

> + * vdpa device management interface

> + * Copyright (c) 2020 Mellanox Technologies Ltd. All rights reserved.

> + */

> +

> +#ifndef _UAPI_LINUX_VDPA_H_

> +#define _UAPI_LINUX_VDPA_H_

> +


you should grab this header after a 'make headers_install' which cleans
up the _UAPI prefix.
Parav Pandit Feb. 10, 2021, 6:28 p.m. UTC | #2
> From: David Ahern <dsahern@gmail.com>

> Sent: Monday, February 8, 2021 9:18 PM

> 

> On 2/5/21 11:10 AM, Parav Pandit wrote:

> > diff --git a/vdpa/include/uapi/linux/vdpa.h

> > b/vdpa/include/uapi/linux/vdpa.h new file mode 100644 index

> > 00000000..66a41e4e

> > --- /dev/null

> > +++ b/vdpa/include/uapi/linux/vdpa.h

> > @@ -0,0 +1,40 @@

> > +/* SPDX-License-Identifier: GPL-2.0+ WITH Linux-syscall-note */

> > +/*

> > + * vdpa device management interface

> > + * Copyright (c) 2020 Mellanox Technologies Ltd. All rights reserved.

> > + */

> > +

> > +#ifndef _UAPI_LINUX_VDPA_H_

> > +#define _UAPI_LINUX_VDPA_H_

> > +

> 

> you should grab this header after a 'make headers_install' which cleans up

> the _UAPI prefix.

Ah right.
Fixing it now.
Also updating the pointer to linux-next which has this patchset instead of Michaels tree.