[ovs-dev] [PATCH v4 2/3] netdev-dpdk: Consistent naming for vhost
Ciara Loftus
ciara.loftus at intel.com
Thu Aug 11 16:28:30 UTC 2016
A mix of vhost_user_ and vhost_ is used when naming vhost functions. The
'user_' has been dropped for consistency. Also remove empty init
functions for netdev dpdk classes.
Suggested-by: Daniele Di Proietto <diproiettod at vmware.com>
Acked-by: Daniele Di Proietto <diproiettod at vmware.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Ciara Loftus <ciara.loftus at intel.com>
---
lib/netdev-dpdk.c | 25 ++++++++-----------------
1 file changed, 8 insertions(+), 17 deletions(-)
diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c
index 83320b5..4e4c74e 100644
--- a/lib/netdev-dpdk.c
+++ b/lib/netdev-dpdk.c
@@ -825,7 +825,7 @@ dpdk_dev_parse_name(const char dev_name[], const char prefix[],
}
static int
-netdev_dpdk_vhost_user_construct(struct netdev *netdev)
+netdev_dpdk_vhost_construct(struct netdev *netdev)
{
struct netdev_dpdk *dev = netdev_dpdk_cast(netdev);
const char *name = netdev->name;
@@ -2448,12 +2448,6 @@ dpdk_vhost_class_init(void)
return 0;
}
-static int
-dpdk_vhost_user_class_init(void)
-{
- return 0;
-}
-
static void
dpdk_common_init(void)
{
@@ -2869,7 +2863,7 @@ out:
}
static int
-netdev_dpdk_vhost_user_reconfigure(struct netdev *netdev)
+netdev_dpdk_vhost_reconfigure(struct netdev *netdev)
{
struct netdev_dpdk *dev = netdev_dpdk_cast(netdev);
int err = 0;
@@ -2907,7 +2901,7 @@ netdev_dpdk_vhost_user_reconfigure(struct netdev *netdev)
return err;
}
-#define NETDEV_DPDK_CLASS(NAME, INIT, CONSTRUCT, DESTRUCT, \
+#define NETDEV_DPDK_CLASS(NAME, CONSTRUCT, DESTRUCT, \
SET_CONFIG, SET_TX_MULTIQ, SEND, \
GET_CARRIER, GET_STATS, \
GET_FEATURES, GET_STATUS, \
@@ -2915,7 +2909,7 @@ netdev_dpdk_vhost_user_reconfigure(struct netdev *netdev)
{ \
NAME, \
true, /* is_pmd */ \
- INIT, /* init */ \
+ NULL, /* init */ \
NULL, /* netdev_dpdk_run */ \
NULL, /* netdev_dpdk_wait */ \
\
@@ -3356,7 +3350,6 @@ dpdk_init(const struct smap *ovs_other_config)
static const struct netdev_class dpdk_class =
NETDEV_DPDK_CLASS(
"dpdk",
- NULL,
netdev_dpdk_construct,
netdev_dpdk_destruct,
netdev_dpdk_set_config,
@@ -3372,7 +3365,6 @@ static const struct netdev_class dpdk_class =
static const struct netdev_class dpdk_ring_class =
NETDEV_DPDK_CLASS(
"dpdkr",
- NULL,
netdev_dpdk_ring_construct,
netdev_dpdk_destruct,
netdev_dpdk_set_config,
@@ -3385,11 +3377,10 @@ static const struct netdev_class dpdk_ring_class =
netdev_dpdk_reconfigure,
netdev_dpdk_rxq_recv);
-static const struct netdev_class OVS_UNUSED dpdk_vhost_user_class =
+static const struct netdev_class OVS_UNUSED dpdk_vhost_class =
NETDEV_DPDK_CLASS(
"dpdkvhostuser",
- dpdk_vhost_user_class_init,
- netdev_dpdk_vhost_user_construct,
+ netdev_dpdk_vhost_construct,
netdev_dpdk_vhost_destruct,
NULL,
NULL,
@@ -3398,7 +3389,7 @@ static const struct netdev_class OVS_UNUSED dpdk_vhost_user_class =
netdev_dpdk_vhost_get_stats,
NULL,
NULL,
- netdev_dpdk_vhost_user_reconfigure,
+ netdev_dpdk_vhost_reconfigure,
netdev_dpdk_vhost_rxq_recv);
void
@@ -3407,7 +3398,7 @@ netdev_dpdk_register(void)
dpdk_common_init();
netdev_register_provider(&dpdk_class);
netdev_register_provider(&dpdk_ring_class);
- netdev_register_provider(&dpdk_vhost_user_class);
+ netdev_register_provider(&dpdk_vhost_class);
}
void
--
2.4.3
More information about the dev
mailing list