From: "Xia, Chenbo" <chenbo.xia@intel.com>
To: Maxime Coquelin <maxime.coquelin@redhat.com>,
"dev@dpdk.org" <dev@dpdk.org>,
"olivier.matz@6wind.com" <olivier.matz@6wind.com>,
"amorenoz@redhat.com" <amorenoz@redhat.com>,
"david.marchand@redhat.com" <david.marchand@redhat.com>
Subject: Re: [dpdk-dev] [PATCH v2 38/44] net/virtio: move protocol features to Vhost-user
Date: Fri, 22 Jan 2021 08:20:39 +0000 [thread overview]
Message-ID: <MN2PR11MB4063F18877814B6DA6823CA99CA00@MN2PR11MB4063.namprd11.prod.outlook.com> (raw)
In-Reply-To: <20210119212507.1043636-39-maxime.coquelin@redhat.com>
> -----Original Message-----
> From: Maxime Coquelin <maxime.coquelin@redhat.com>
> Sent: Wednesday, January 20, 2021 5:25 AM
> To: dev@dpdk.org; Xia, Chenbo <chenbo.xia@intel.com>; olivier.matz@6wind.com;
> amorenoz@redhat.com; david.marchand@redhat.com
> Cc: Maxime Coquelin <maxime.coquelin@redhat.com>
> Subject: [PATCH v2 38/44] net/virtio: move protocol features to Vhost-user
>
> Since only protocol features are specific to Vhost-user
> backend, this patch moves all related code to Vhost-user
> file.
>
> Signed-off-by: Maxime Coquelin <maxime.coquelin@redhat.com>
> ---
> drivers/net/virtio/virtio_user/vhost.h | 20 +----
> drivers/net/virtio/virtio_user/vhost_kernel.c | 9 ++
> drivers/net/virtio/virtio_user/vhost_user.c | 83 +++++++++++++++----
> drivers/net/virtio/virtio_user/vhost_vdpa.c | 34 +++++++-
> .../net/virtio/virtio_user/virtio_user_dev.c | 47 ++---------
> drivers/net/virtio/virtio_user_ethdev.c | 17 ----
> 6 files changed, 113 insertions(+), 97 deletions(-)
>
> diff --git a/drivers/net/virtio/virtio_user/vhost.h
> b/drivers/net/virtio/virtio_user/vhost.h
> index 2aa6b2cb70..dfa2735a20 100644
> --- a/drivers/net/virtio/virtio_user/vhost.h
> +++ b/drivers/net/virtio/virtio_user/vhost.h
> @@ -46,23 +46,6 @@ struct vhost_vring_addr {
> uint64_t log_guest_addr;
> };
>
> -#ifndef VHOST_USER_F_PROTOCOL_FEATURES
> -#define VHOST_USER_F_PROTOCOL_FEATURES 30
> -#endif
> -
> -/** Protocol features. */
> -#ifndef VHOST_USER_PROTOCOL_F_MQ
> -#define VHOST_USER_PROTOCOL_F_MQ 0
> -#endif
> -
> -#ifndef VHOST_USER_PROTOCOL_F_REPLY_ACK
> -#define VHOST_USER_PROTOCOL_F_REPLY_ACK 3
> -#endif
> -
> -#ifndef VHOST_USER_PROTOCOL_F_STATUS
> -#define VHOST_USER_PROTOCOL_F_STATUS 16
> -#endif
> -
> #ifndef VHOST_BACKEND_F_IOTLB_MSG_V2
> #define VHOST_BACKEND_F_IOTLB_MSG_V2 1
> #endif
> @@ -82,11 +65,10 @@ struct virtio_user_dev;
>
> struct virtio_user_backend_ops {
> int (*setup)(struct virtio_user_dev *dev);
> + int (*get_backend_features)(uint64_t *features);
> int (*set_owner)(struct virtio_user_dev *dev);
> int (*get_features)(struct virtio_user_dev *dev, uint64_t *features);
> int (*set_features)(struct virtio_user_dev *dev, uint64_t features);
> - int (*get_protocol_features)(struct virtio_user_dev *dev, uint64_t
> *features);
> - int (*set_protocol_features)(struct virtio_user_dev *dev, uint64_t
> features);
> int (*set_memory_table)(struct virtio_user_dev *dev);
> int (*set_vring_num)(struct virtio_user_dev *dev, struct
> vhost_vring_state *state);
> int (*set_vring_base)(struct virtio_user_dev *dev, struct
> vhost_vring_state *state);
> diff --git a/drivers/net/virtio/virtio_user/vhost_kernel.c
> b/drivers/net/virtio/virtio_user/vhost_kernel.c
> index 68f0c0b83d..5477eb7015 100644
> --- a/drivers/net/virtio/virtio_user/vhost_kernel.c
> +++ b/drivers/net/virtio/virtio_user/vhost_kernel.c
> @@ -445,8 +445,17 @@ vhost_kernel_enable_queue_pair(struct virtio_user_dev
> *dev,
> return 0;
> }
>
> +static int
> +vhost_kernel_get_backend_features(uint64_t *features)
> +{
> + *features = 0;
> +
> + return 0;
> +}
> +
> struct virtio_user_backend_ops virtio_ops_kernel = {
> .setup = vhost_kernel_setup,
> + .get_backend_features = vhost_kernel_get_backend_features,
> .set_owner = vhost_kernel_set_owner,
> .get_features = vhost_kernel_get_features,
> .set_features = vhost_kernel_set_features,
> diff --git a/drivers/net/virtio/virtio_user/vhost_user.c
> b/drivers/net/virtio/virtio_user/vhost_user.c
> index a48fadd8c9..986d8ac1ac 100644
> --- a/drivers/net/virtio/virtio_user/vhost_user.c
> +++ b/drivers/net/virtio/virtio_user/vhost_user.c
> @@ -17,6 +17,29 @@
> #include "vhost.h"
> #include "virtio_user_dev.h"
>
> +
> +#ifndef VHOST_USER_F_PROTOCOL_FEATURES
> +#define VHOST_USER_F_PROTOCOL_FEATURES 30
> +#endif
> +
> +/** Protocol features. */
> +#ifndef VHOST_USER_PROTOCOL_F_MQ
> +#define VHOST_USER_PROTOCOL_F_MQ 0
> +#endif
> +
> +#ifndef VHOST_USER_PROTOCOL_F_REPLY_ACK
> +#define VHOST_USER_PROTOCOL_F_REPLY_ACK 3
> +#endif
> +
> +#ifndef VHOST_USER_PROTOCOL_F_STATUS
> +#define VHOST_USER_PROTOCOL_F_STATUS 16
> +#endif
> +
> +#define VHOST_USER_SUPPORTED_PROTOCOL_FEATURES \
> + (1ULL << VHOST_USER_PROTOCOL_F_MQ | \
> + 1ULL << VHOST_USER_PROTOCOL_F_REPLY_ACK | \
> + 1ULL << VHOST_USER_PROTOCOL_F_STATUS)
> +
> /* The version of the protocol we support */
> #define VHOST_USER_VERSION 0x1
>
> @@ -205,11 +228,11 @@ vhost_user_set_owner(struct virtio_user_dev *dev)
> }
>
> static int
> -vhost_user_get_features(struct virtio_user_dev *dev, uint64_t *features)
> +vhost_user_get_protocol_features(struct virtio_user_dev *dev, uint64_t
> *features)
> {
> int ret;
> struct vhost_user_msg msg = {
> - .request = VHOST_USER_GET_FEATURES,
> + .request = VHOST_USER_GET_PROTOCOL_FEATURES,
> .flags = VHOST_USER_VERSION,
> };
>
> @@ -221,7 +244,7 @@ vhost_user_get_features(struct virtio_user_dev *dev,
> uint64_t *features)
> if (ret < 0)
> goto err;
>
> - if (msg.request != VHOST_USER_GET_FEATURES) {
> + if (msg.request != VHOST_USER_GET_PROTOCOL_FEATURES) {
> PMD_DRV_LOG(ERR, "Unexpected request type (%d)", msg.request);
> goto err;
> }
> @@ -235,27 +258,25 @@ vhost_user_get_features(struct virtio_user_dev *dev,
> uint64_t *features)
>
> return 0;
> err:
> - PMD_DRV_LOG(ERR, "Failed to get backend features");
> + PMD_DRV_LOG(ERR, "Failed to get backend protocol features");
>
> return -1;
> }
>
> static int
> -vhost_user_set_features(struct virtio_user_dev *dev, uint64_t features)
> +vhost_user_set_protocol_features(struct virtio_user_dev *dev, uint64_t
> features)
> {
> int ret;
> struct vhost_user_msg msg = {
> - .request = VHOST_USER_SET_FEATURES,
> + .request = VHOST_USER_SET_PROTOCOL_FEATURES,
> .flags = VHOST_USER_VERSION,
> .size = sizeof(features),
> .payload.u64 = features,
> };
>
> - msg.payload.u64 |= dev->device_features & (1ULL <<
> VHOST_USER_F_PROTOCOL_FEATURES);
> -
> ret = vhost_user_write(dev->vhostfd, &msg, NULL, 0);
> if (ret < 0) {
> - PMD_DRV_LOG(ERR, "Failed to set features");
> + PMD_DRV_LOG(ERR, "Failed to set protocol features");
> return -1;
> }
>
> @@ -263,11 +284,11 @@ vhost_user_set_features(struct virtio_user_dev *dev,
> uint64_t features)
> }
>
> static int
> -vhost_user_get_protocol_features(struct virtio_user_dev *dev, uint64_t
> *features)
> +vhost_user_get_features(struct virtio_user_dev *dev, uint64_t *features)
> {
> int ret;
> struct vhost_user_msg msg = {
> - .request = VHOST_USER_GET_PROTOCOL_FEATURES,
> + .request = VHOST_USER_GET_FEATURES,
> .flags = VHOST_USER_VERSION,
> };
>
> @@ -279,7 +300,7 @@ vhost_user_get_protocol_features(struct virtio_user_dev
> *dev, uint64_t *features
> if (ret < 0)
> goto err;
>
> - if (msg.request != VHOST_USER_GET_PROTOCOL_FEATURES) {
> + if (msg.request != VHOST_USER_GET_FEATURES) {
> PMD_DRV_LOG(ERR, "Unexpected request type (%d)", msg.request);
> goto err;
> }
> @@ -291,27 +312,46 @@ vhost_user_get_protocol_features(struct virtio_user_dev
> *dev, uint64_t *features
>
> *features = msg.payload.u64;
>
> + if (!(*features & (1ULL << VHOST_USER_F_PROTOCOL_FEATURES)))
> + return 0;
> +
> + /* Negotiate protocol features */
> + ret = vhost_user_get_protocol_features(dev, &dev->protocol_features);
> + if (ret < 0)
> + goto err;
> +
> + dev->protocol_features &= VHOST_USER_SUPPORTED_PROTOCOL_FEATURES;
> +
> + ret = vhost_user_set_protocol_features(dev, dev->protocol_features);
> + if (ret < 0)
> + goto err;
> +
> + if (!(dev->protocol_features & (1ULL << VHOST_USER_PROTOCOL_F_MQ)))
> + dev->unsupported_features |= (1ull << VIRTIO_NET_F_MQ);
> +
> return 0;
> err:
> - PMD_DRV_LOG(ERR, "Failed to get backend protocol features");
> + PMD_DRV_LOG(ERR, "Failed to get backend features");
>
> return -1;
> }
>
> static int
> -vhost_user_set_protocol_features(struct virtio_user_dev *dev, uint64_t
> features)
> +vhost_user_set_features(struct virtio_user_dev *dev, uint64_t features)
> {
> int ret;
> struct vhost_user_msg msg = {
> - .request = VHOST_USER_SET_PROTOCOL_FEATURES,
> + .request = VHOST_USER_SET_FEATURES,
> .flags = VHOST_USER_VERSION,
> .size = sizeof(features),
> .payload.u64 = features,
> };
>
> + msg.payload.u64 |= dev->device_features & (1ULL <<
> VHOST_USER_F_PROTOCOL_FEATURES);
> +
> ret = vhost_user_write(dev->vhostfd, &msg, NULL, 0);
> if (ret < 0) {
> - PMD_DRV_LOG(ERR, "Failed to set protocol features");
> + PMD_DRV_LOG(ERR, "Failed to set features");
> return -1;
> }
>
> @@ -780,13 +820,20 @@ vhost_user_enable_queue_pair(struct virtio_user_dev *dev,
> return 0;
> }
>
> +static int
> +vhost_user_get_backend_features(uint64_t *features)
> +{
> + *features = 1ULL << VHOST_USER_F_PROTOCOL_FEATURES;
> +
> + return 0;
> +}
> +
> struct virtio_user_backend_ops virtio_ops_user = {
> .setup = vhost_user_setup,
> + .get_backend_features = vhost_user_get_backend_features,
> .set_owner = vhost_user_set_owner,
> .get_features = vhost_user_get_features,
> .set_features = vhost_user_set_features,
> - .get_protocol_features = vhost_user_get_protocol_features,
> - .set_protocol_features = vhost_user_set_protocol_features,
> .set_memory_table = vhost_user_set_memory_table,
> .set_vring_num = vhost_user_set_vring_num,
> .set_vring_base = vhost_user_set_vring_base,
> diff --git a/drivers/net/virtio/virtio_user/vhost_vdpa.c
> b/drivers/net/virtio/virtio_user/vhost_vdpa.c
> index 7fb1249569..8d354b298e 100644
> --- a/drivers/net/virtio/virtio_user/vhost_vdpa.c
> +++ b/drivers/net/virtio/virtio_user/vhost_vdpa.c
> @@ -13,6 +13,10 @@
> #include "vhost.h"
> #include "virtio_user_dev.h"
>
> +#define VHOST_VDPA_SUPPORTED_BACKEND_FEATURES \
> + (1ULL << VHOST_BACKEND_F_IOTLB_MSG_V2 | \
> + 1ULL << VHOST_BACKEND_F_IOTLB_BATCH)
> +
> /* vhost kernel & vdpa ioctls */
> #define VHOST_VIRTIO 0xAF
> #define VHOST_GET_FEATURES _IOR(VHOST_VIRTIO, 0x00, __u64)
> @@ -88,13 +92,13 @@ vhost_vdpa_set_owner(struct virtio_user_dev *dev)
> }
>
> static int
> -vhost_vdpa_get_backend_features(struct virtio_user_dev *dev, uint64_t
> *features)
> +vhost_vdpa_get_protocol_features(struct virtio_user_dev *dev, uint64_t
> *features)
> {
> return vhost_vdpa_ioctl(dev->vhostfd, VHOST_GET_BACKEND_FEATURES,
> features);
> }
>
> static int
> -vhost_vdpa_set_backend_features(struct virtio_user_dev *dev, uint64_t
> features)
> +vhost_vdpa_set_protocol_features(struct virtio_user_dev *dev, uint64_t
> features)
> {
> return vhost_vdpa_ioctl(dev->vhostfd, VHOST_SET_BACKEND_FEATURES,
> &features);
> }
> @@ -113,6 +117,21 @@ vhost_vdpa_get_features(struct virtio_user_dev *dev,
> uint64_t *features)
> /* Multiqueue not supported for now */
> *features &= ~(1ULL << VIRTIO_NET_F_MQ);
>
> + /* Negotiated vDPA backend features */
> + ret = vhost_vdpa_get_protocol_features(dev, &dev->protocol_features);
> + if (ret < 0) {
> + PMD_DRV_LOG(ERR, "Failed to get backend features");
> + return -1;
> + }
> +
> + dev->protocol_features &= VHOST_VDPA_SUPPORTED_BACKEND_FEATURES;
> +
> + ret = vhost_vdpa_set_protocol_features(dev, dev->protocol_features);
> + if (ret < 0) {
> + PMD_DRV_LOG(ERR, "Failed to set backend features");
> + return -1;
> + }
> +
> return 0;
> }
>
> @@ -438,13 +457,20 @@ vhost_vdpa_enable_queue_pair(struct virtio_user_dev *dev,
> return 0;
> }
>
> +static int
> +vhost_vdpa_get_backend_features(uint64_t *features)
> +{
> + *features = 0;
> +
> + return 0;
> +}
> +
> struct virtio_user_backend_ops virtio_ops_vdpa = {
> .setup = vhost_vdpa_setup,
> + .get_backend_features = vhost_vdpa_get_backend_features,
> .set_owner = vhost_vdpa_set_owner,
> .get_features = vhost_vdpa_get_features,
> .set_features = vhost_vdpa_set_features,
> - .get_protocol_features = vhost_vdpa_get_backend_features,
> - .set_protocol_features = vhost_vdpa_set_backend_features,
> .set_memory_table = vhost_vdpa_set_memory_table,
> .set_vring_num = vhost_vdpa_set_vring_num,
> .set_vring_base = vhost_vdpa_set_vring_base,
> diff --git a/drivers/net/virtio/virtio_user/virtio_user_dev.c
> b/drivers/net/virtio/virtio_user/virtio_user_dev.c
> index c2a41fe3a0..2614bcb349 100644
> --- a/drivers/net/virtio/virtio_user/virtio_user_dev.c
> +++ b/drivers/net/virtio/virtio_user/virtio_user_dev.c
> @@ -480,24 +480,15 @@ virtio_user_dev_setup(struct virtio_user_dev *dev)
> 1ULL << VIRTIO_NET_F_GUEST_TSO6 | \
> 1ULL << VIRTIO_F_IN_ORDER | \
> 1ULL << VIRTIO_F_VERSION_1 | \
> - 1ULL << VIRTIO_F_RING_PACKED | \
> - 1ULL << VHOST_USER_F_PROTOCOL_FEATURES)
> + 1ULL << VIRTIO_F_RING_PACKED)
>
> -#define VHOST_USER_SUPPORTED_PROTOCOL_FEATURES \
> - (1ULL << VHOST_USER_PROTOCOL_F_MQ | \
> - 1ULL << VHOST_USER_PROTOCOL_F_REPLY_ACK | \
> - 1ULL << VHOST_USER_PROTOCOL_F_STATUS)
> -
> -#define VHOST_VDPA_SUPPORTED_PROTOCOL_FEATURES \
> - (1ULL << VHOST_BACKEND_F_IOTLB_MSG_V2 | \
> - 1ULL << VHOST_BACKEND_F_IOTLB_BATCH)
> int
> virtio_user_dev_init(struct virtio_user_dev *dev, char *path, int queues,
> int cq, int queue_size, const char *mac, char **ifname,
> int server, int mrg_rxbuf, int in_order, int packed_vq,
> enum virtio_user_backend_type backend_type)
> {
> - uint64_t protocol_features = 0;
> + uint64_t backend_features;
>
> pthread_mutex_init(&dev->mutex, NULL);
> strlcpy(dev->path, path, PATH_MAX);
> @@ -508,14 +499,9 @@ virtio_user_dev_init(struct virtio_user_dev *dev, char
> *path, int queues,
> dev->is_server = server;
> dev->mac_specified = 0;
> dev->frontend_features = 0;
> - dev->unsupported_features = ~VIRTIO_USER_SUPPORTED_FEATURES;
> + dev->unsupported_features = 0;
> dev->backend_type = backend_type;
>
> - if (dev->backend_type == VIRTIO_USER_BACKEND_VHOST_USER)
> - dev->protocol_features = VHOST_USER_SUPPORTED_PROTOCOL_FEATURES;
> - else if (dev->backend_type == VIRTIO_USER_BACKEND_VHOST_VDPA)
> - dev->protocol_features = VHOST_VDPA_SUPPORTED_PROTOCOL_FEATURES;
> -
> parse_mac(dev, mac);
>
> if (*ifname) {
> @@ -528,38 +514,21 @@ virtio_user_dev_init(struct virtio_user_dev *dev, char
> *path, int queues,
> return -1;
> }
>
> - if (dev->backend_type != VIRTIO_USER_BACKEND_VHOST_USER)
> - dev->unsupported_features |=
> - (1ULL << VHOST_USER_F_PROTOCOL_FEATURES);
> -
> if (dev->ops->set_owner(dev) < 0) {
> PMD_INIT_LOG(ERR, "(%s) Failed to set backend owner", dev->path);
> return -1;
> }
>
> - if (dev->ops->get_features(dev, &dev->device_features) < 0) {
> + if (dev->ops->get_backend_features(&backend_features) < 0) {
> PMD_INIT_LOG(ERR, "(%s) Failed to get backend features", dev-
> >path);
> return -1;
> }
>
> - if ((dev->device_features & (1ULL << VHOST_USER_F_PROTOCOL_FEATURES)) ||
> - dev->backend_type == VIRTIO_USER_BACKEND_VHOST_VDPA) {
> - if (dev->ops->get_protocol_features(dev, &protocol_features)) {
> - PMD_INIT_LOG(ERR, "(%s) Failed to get backend protocol
> features",
> - dev->path);
> - return -1;
> - }
> -
> - dev->protocol_features &= protocol_features;
> -
> - if (dev->ops->set_protocol_features(dev, dev->protocol_features))
> {
> - PMD_INIT_LOG(ERR, "(%s) Failed to set backend protocol
> features",
> - dev->path);
> - return -1;
> - }
> + dev->unsupported_features = ~(VIRTIO_USER_SUPPORTED_FEATURES |
> backend_features);
>
> - if (!(dev->protocol_features & (1ULL << VHOST_USER_PROTOCOL_F_MQ)))
> - dev->unsupported_features |= (1ull << VIRTIO_NET_F_MQ);
> + if (dev->ops->get_features(dev, &dev->device_features) < 0) {
> + PMD_INIT_LOG(ERR, "(%s) Failed to get device features", dev->path);
> + return -1;
> }
>
> if (!mrg_rxbuf)
> diff --git a/drivers/net/virtio/virtio_user_ethdev.c
> b/drivers/net/virtio/virtio_user_ethdev.c
> index e822d93690..e2c606a838 100644
> --- a/drivers/net/virtio/virtio_user_ethdev.c
> +++ b/drivers/net/virtio/virtio_user_ethdev.c
> @@ -70,7 +70,6 @@ virtio_user_server_reconnect(struct virtio_user_dev *dev)
> int ret, connectfd, old_status;
> struct rte_eth_dev *eth_dev = &rte_eth_devices[dev->port_id];
> struct virtio_hw *hw = &dev->hw;
> - uint64_t protocol_features;
>
> connectfd = accept(dev->listenfd, NULL, NULL);
> if (connectfd < 0)
> @@ -91,22 +90,6 @@ virtio_user_server_reconnect(struct virtio_user_dev *dev)
> return -1;
> }
>
> - if (dev->device_features &
> - (1ULL << VHOST_USER_F_PROTOCOL_FEATURES)) {
> - if (dev->ops->get_protocol_features(dev, &protocol_features))
> - return -1;
> -
> - /* Offer VHOST_USER_PROTOCOL_F_STATUS */
> - dev->protocol_features |= (1ULL << VHOST_USER_PROTOCOL_F_STATUS);
> - dev->protocol_features &= protocol_features;
> -
> - if (dev->ops->set_protocol_features(dev, dev->protocol_features))
> - return -1;
> -
> - if (!(dev->protocol_features & (1ULL << VHOST_USER_PROTOCOL_F_MQ)))
> - dev->unsupported_features |= (1ull << VIRTIO_NET_F_MQ);
> - }
> -
> dev->device_features |= dev->frontend_features;
>
> /* umask vhost-user unsupported features */
> --
> 2.29.2
Reviewed-by: Chenbo Xia <chenbo.xia@intel.com>
next prev parent reply other threads:[~2021-01-22 8:20 UTC|newest]
Thread overview: 86+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-01-19 21:24 [dpdk-dev] [PATCH v2 00/44] net/virtio: Virtio PMD rework Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 01/44] bus/vdev: add helper to get vdev from eth dev Maxime Coquelin
2021-01-20 0:56 ` Thomas Monjalon
2021-01-25 10:53 ` Maxime Coquelin
2021-01-25 11:04 ` Thomas Monjalon
2021-01-21 8:58 ` Xia, Chenbo
2021-01-25 14:51 ` Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 02/44] bus/vdev: add driver IOVA VA mode requirement Maxime Coquelin
2021-01-20 15:32 ` David Marchand
2021-01-20 17:47 ` Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 03/44] net/virtio: fix getting old status on reconnect Maxime Coquelin
2021-01-21 7:12 ` Xia, Chenbo
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 04/44] net/virtio: introduce Virtio bus type Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 05/44] net/virtio: refactor virtio-user device Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 06/44] net/virtio: introduce PCI device metadata Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 07/44] net/virtio: move PCI device init in dedicated file Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 08/44] net/virtio: move PCI specific dev init to PCI ethdev init Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 09/44] net/virtio: move MSIX detection to PCI ethdev Maxime Coquelin
2021-01-21 7:12 ` Xia, Chenbo
2021-01-25 12:41 ` Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 10/44] net/virtio: force IOVA as VA mode for Virtio-user Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 11/44] net/virtio: store PCI type in Virtio device metadata Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 12/44] net/virtio: add callback for device closing Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 13/44] net/virtio: validate features at bus level Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 14/44] net/virtio: remove bus type enum Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 15/44] net/virtio: move PCI-specific fields to PCI device Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 16/44] net/virtio: pack virtio HW struct Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 17/44] net/virtio: move legacy IO to Virtio PCI Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 18/44] net/virtio: introduce generic virtio header Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 19/44] net/virtio: move features definition to generic header Maxime Coquelin
2021-01-21 6:47 ` Xia, Chenbo
2021-01-25 12:35 ` Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 20/44] net/virtio: move virtqueue defines in " Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 21/44] net/virtio: move config definitions to " Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 22/44] net/virtio: make interrupt handling more generic Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 23/44] net/virtio: move vring alignment to generic header Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 24/44] net/virtio: remove last PCI refs in non-PCI code Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 25/44] net/virtio: make Vhost-user req sender consistent Maxime Coquelin
2021-01-21 8:50 ` Xia, Chenbo
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 26/44] net/virtio: add Virtio-user ops to set owner Maxime Coquelin
2021-01-22 3:27 ` Xia, Chenbo
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 27/44] net/virtio: add Virtio-user features ops Maxime Coquelin
2021-01-22 7:25 ` Xia, Chenbo
2021-01-25 14:53 ` Maxime Coquelin
2021-01-22 8:46 ` Xia, Chenbo
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 28/44] net/virtio: add Virtio-user protocol " Maxime Coquelin
2021-01-22 7:27 ` Xia, Chenbo
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 29/44] net/virtio: add Virtio-user memory tables ops Maxime Coquelin
2021-01-22 7:34 ` Xia, Chenbo
2021-01-25 14:56 ` Maxime Coquelin
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 30/44] net/virtio: add Virtio-user vring setting ops Maxime Coquelin
2021-01-22 7:49 ` Xia, Chenbo
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 31/44] net/virtio: add Virtio-user vring file ops Maxime Coquelin
2021-01-22 8:01 ` Xia, Chenbo
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 32/44] net/virtio: add Virtio-user vring address ops Maxime Coquelin
2021-01-22 8:02 ` Xia, Chenbo
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 33/44] net/virtio: add Virtio-user status ops Maxime Coquelin
2021-01-22 8:02 ` Xia, Chenbo
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 34/44] net/virtio: remove useless request ops Maxime Coquelin
2021-01-22 8:10 ` Xia, Chenbo
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 35/44] net/virtio: improve Virtio-user errors handling Maxime Coquelin
2021-01-22 8:10 ` Xia, Chenbo
2021-01-19 21:24 ` [dpdk-dev] [PATCH v2 36/44] net/virtio: move Vhost-user reqs to Vhost-user backend Maxime Coquelin
2021-01-21 8:56 ` Xia, Chenbo
2021-01-25 14:50 ` Maxime Coquelin
2021-01-19 21:25 ` [dpdk-dev] [PATCH v2 37/44] net/virtio: make server mode blocking Maxime Coquelin
2021-01-22 8:19 ` Xia, Chenbo
2021-01-19 21:25 ` [dpdk-dev] [PATCH v2 38/44] net/virtio: move protocol features to Vhost-user Maxime Coquelin
2021-01-22 8:20 ` Xia, Chenbo [this message]
2021-01-19 21:25 ` [dpdk-dev] [PATCH v2 39/44] net/virtio: introduce backend data Maxime Coquelin
2021-01-22 8:26 ` Xia, Chenbo
2021-01-19 21:25 ` [dpdk-dev] [PATCH v2 40/44] net/virtio: move Vhost-user specifics to its backend Maxime Coquelin
2021-01-22 8:49 ` Xia, Chenbo
2021-01-19 21:25 ` [dpdk-dev] [PATCH v2 41/44] net/virtio: move Vhost-kernel data " Maxime Coquelin
2021-01-22 8:55 ` Xia, Chenbo
2021-01-25 14:59 ` Maxime Coquelin
2021-01-19 21:25 ` [dpdk-dev] [PATCH v2 42/44] net/virtio: move Vhost-vDPA " Maxime Coquelin
2021-01-22 9:06 ` Xia, Chenbo
2021-01-25 15:02 ` Maxime Coquelin
2021-01-19 21:25 ` [dpdk-dev] [PATCH v2 43/44] net/virtio: improve Vhost-user error logging Maxime Coquelin
2021-01-22 9:11 ` Xia, Chenbo
2021-01-25 15:04 ` Maxime Coquelin
2021-01-26 6:10 ` Xia, Chenbo
2021-01-19 21:25 ` [dpdk-dev] [PATCH v2 44/44] net/virtio: handle Virtio-user setup failure properly Maxime Coquelin
2021-01-22 9:24 ` Xia, Chenbo
2021-01-25 16:16 ` Maxime Coquelin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=MN2PR11MB4063F18877814B6DA6823CA99CA00@MN2PR11MB4063.namprd11.prod.outlook.com \
--to=chenbo.xia@intel.com \
--cc=amorenoz@redhat.com \
--cc=david.marchand@redhat.com \
--cc=dev@dpdk.org \
--cc=maxime.coquelin@redhat.com \
--cc=olivier.matz@6wind.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.