Message ID | 1510060648-30484-2-git-send-email-i.maximets@samsung.com |
---|---|
State | Superseded |
Delegated to: | Ian Stokes |
Headers | show |
Series | netdev-dpdk: Fix and document variables naming. | expand |
> Function 'netdev_dpdk_set_admin_state()' was missed while fixing variables > naming according to the following convention: > > 'struct netdev':'netdev' > 'struct netdev_dpdk':'dev' > 'struct netdev_rxq':'rxq' > 'struct netdev_rxq_dpdk':'rx' > LGTM, will add this to the DPDK merge branch. Thanks Ian > Fixes: d46285a2206f ("netdev-dpdk: Consistent variable naming.") > Signed-off-by: Ilya Maximets <i.maximets@samsung.com> > --- > lib/netdev-dpdk.c | 19 ++++++++++--------- > 1 file changed, 10 insertions(+), 9 deletions(-) > > diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c index dc210cc..872b133 > 100644 > --- a/lib/netdev-dpdk.c > +++ b/lib/netdev-dpdk.c > @@ -2486,12 +2486,13 @@ netdev_dpdk_set_admin_state(struct unixctl_conn > *conn, int argc, > > if (argc > 2) { > struct netdev *netdev = netdev_from_name(argv[1]); > + > if (netdev && is_dpdk_class(netdev->netdev_class)) { > - struct netdev_dpdk *dpdk_dev = netdev_dpdk_cast(netdev); > + struct netdev_dpdk *dev = netdev_dpdk_cast(netdev); > > - ovs_mutex_lock(&dpdk_dev->mutex); > - netdev_dpdk_set_admin_state__(dpdk_dev, up); > - ovs_mutex_unlock(&dpdk_dev->mutex); > + ovs_mutex_lock(&dev->mutex); > + netdev_dpdk_set_admin_state__(dev, up); > + ovs_mutex_unlock(&dev->mutex); > > netdev_close(netdev); > } else { > @@ -2500,13 +2501,13 @@ netdev_dpdk_set_admin_state(struct unixctl_conn > *conn, int argc, > return; > } > } else { > - struct netdev_dpdk *netdev; > + struct netdev_dpdk *dev; > > ovs_mutex_lock(&dpdk_mutex); > - LIST_FOR_EACH (netdev, list_node, &dpdk_list) { > - ovs_mutex_lock(&netdev->mutex); > - netdev_dpdk_set_admin_state__(netdev, up); > - ovs_mutex_unlock(&netdev->mutex); > + LIST_FOR_EACH (dev, list_node, &dpdk_list) { > + ovs_mutex_lock(&dev->mutex); > + netdev_dpdk_set_admin_state__(dev, up); > + ovs_mutex_unlock(&dev->mutex); > } > ovs_mutex_unlock(&dpdk_mutex); > } > -- > 2.7.4 > > _______________________________________________ > dev mailing list > dev@openvswitch.org > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c index dc210cc..872b133 100644 --- a/lib/netdev-dpdk.c +++ b/lib/netdev-dpdk.c @@ -2486,12 +2486,13 @@ netdev_dpdk_set_admin_state(struct unixctl_conn *conn, int argc, if (argc > 2) { struct netdev *netdev = netdev_from_name(argv[1]); + if (netdev && is_dpdk_class(netdev->netdev_class)) { - struct netdev_dpdk *dpdk_dev = netdev_dpdk_cast(netdev); + struct netdev_dpdk *dev = netdev_dpdk_cast(netdev); - ovs_mutex_lock(&dpdk_dev->mutex); - netdev_dpdk_set_admin_state__(dpdk_dev, up); - ovs_mutex_unlock(&dpdk_dev->mutex); + ovs_mutex_lock(&dev->mutex); + netdev_dpdk_set_admin_state__(dev, up); + ovs_mutex_unlock(&dev->mutex); netdev_close(netdev); } else { @@ -2500,13 +2501,13 @@ netdev_dpdk_set_admin_state(struct unixctl_conn *conn, int argc, return; } } else { - struct netdev_dpdk *netdev; + struct netdev_dpdk *dev; ovs_mutex_lock(&dpdk_mutex); - LIST_FOR_EACH (netdev, list_node, &dpdk_list) { - ovs_mutex_lock(&netdev->mutex); - netdev_dpdk_set_admin_state__(netdev, up); - ovs_mutex_unlock(&netdev->mutex); + LIST_FOR_EACH (dev, list_node, &dpdk_list) { + ovs_mutex_lock(&dev->mutex); + netdev_dpdk_set_admin_state__(dev, up); + ovs_mutex_unlock(&dev->mutex); } ovs_mutex_unlock(&dpdk_mutex); }
Function 'netdev_dpdk_set_admin_state()' was missed while fixing variables naming according to the following convention: 'struct netdev':'netdev' 'struct netdev_dpdk':'dev' 'struct netdev_rxq':'rxq' 'struct netdev_rxq_dpdk':'rx' Fixes: d46285a2206f ("netdev-dpdk: Consistent variable naming.") Signed-off-by: Ilya Maximets <i.maximets@samsung.com> --- lib/netdev-dpdk.c | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-)