> >Double word removed. Error message rewritten to avoid breaking of the >user-visible string. > >CC: Mark Kavanagh <mark.b.kavan...@intel.com> >Fixes: 58be5c0eecf3 ("netdev-dpdk: Fix coding style") >Signed-off-by: Ilya Maximets <i.maxim...@samsung.com> >--- > lib/netdev-dpdk.c | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > >diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c >index 3847b1f..6679f45 100644 >--- a/lib/netdev-dpdk.c >+++ b/lib/netdev-dpdk.c >@@ -495,7 +495,7 @@ dpdk_mp_get(int socket_id, int mtu) > - sizeof(struct rte_mbuf); > /* XXX: this is a really rough method of provisioning memory. > * It's impossible to determine what the exact memory requirements are > when >- * when the number of ports and rxqs that utilize a particular mempool can >+ * the number of ports and rxqs that utilize a particular mempool can
Thanks for the catch. > * change dynamically at runtime. For now, use this rough heurisitic. > */ > if (mtu >= ETHER_MTU) { >@@ -1024,8 +1024,9 @@ netdev_dpdk_vhost_destruct(struct netdev *netdev) > && !(dev->vhost_driver_flags & RTE_VHOST_USER_CLIENT)) { > VLOG_ERR("Removing port '%s' while vhost device still attached.", > netdev->name); >- VLOG_ERR("To restore connectivity after re-adding of port, VM on " >- "socket '%s' must be restarted.", dev->vhost_id); >+ VLOG_ERR("Restart the VM on socket '%s" >+ "' to restore connectivity after re-adding of port", >+ dev->vhost_id); Thanks for documenting this in the coding style doc. It's clear now how logs should be formatted across multiple lines. Acked-by: Mark Kavanagh <mark.b.kavan...@intel.com> > } > > free(ovsrcu_get_protected(struct ingress_policer *, >-- >2.7.4 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev