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 <diproiet...@vmware.com> Acked-by: Daniele Di Proietto <diproiettod at vmware.com> Acked-by: Flavio Leitner <f...@sysclose.org> Signed-off-by: Ciara Loftus <ciara.lof...@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 6998452..9866c85 100644 --- a/lib/netdev-dpdk.c +++ b/lib/netdev-dpdk.c @@ -875,7 +875,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; @@ -2486,12 +2486,6 @@ dpdk_vhost_class_init(void) return 0; } -static int -dpdk_vhost_user_class_init(void) -{ - return 0; -} - static void dpdk_common_init(void) { @@ -2914,7 +2908,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); @@ -2948,7 +2942,7 @@ netdev_dpdk_vhost_user_reconfigure(struct netdev *netdev) return 0; } -#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, \ @@ -2956,7 +2950,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 */ \ \ @@ -3413,7 +3407,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, @@ -3429,7 +3422,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, @@ -3442,11 +3434,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, @@ -3455,7 +3446,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 @@ -3464,7 +3455,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 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev