[ovs-dev] [PATCH 2/3] netdev-dpdk: Print port name in offload API messages.
Ilya Maximets
i.maximets at samsung.com
Fri Oct 19 13:51:14 UTC 2018
This is useful for understanding which flows offloaded to
which ports.
Code refactored a bit to reduce number of casts.
Signed-off-by: Ilya Maximets <i.maximets at samsung.com>
---
lib/netdev-dpdk.c | 26 +++++++++++++-------------
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c
index d2b392f61..d106a2fbd 100644
--- a/lib/netdev-dpdk.c
+++ b/lib/netdev-dpdk.c
@@ -4528,14 +4528,14 @@ end_proto_check:
free(rss);
if (!flow) {
- VLOG_ERR("rte flow creat error: %u : message : %s\n",
- error.type, error.message);
+ VLOG_ERR("%s: rte flow creat error: %u : message : %s\n",
+ netdev_get_name(netdev), error.type, error.message);
ret = -1;
goto out;
}
ufid_to_rte_flow_associate(ufid, flow);
- VLOG_DBG("installed flow %p by ufid "UUID_FMT"\n",
- flow, UUID_ARGS((struct uuid *)ufid));
+ VLOG_DBG("%s: installed flow %p by ufid "UUID_FMT"\n",
+ netdev_get_name(netdev), flow, UUID_ARGS((struct uuid *)ufid));
out:
free(patterns.items);
@@ -4639,9 +4639,10 @@ err:
}
static int
-netdev_dpdk_destroy_rte_flow(struct netdev_dpdk *dev,
+netdev_dpdk_destroy_rte_flow(struct netdev *netdev,
const ovs_u128 *ufid,
struct rte_flow *rte_flow) {
+ struct netdev_dpdk *dev = netdev_dpdk_cast(netdev);
struct rte_flow_error error;
int ret;
@@ -4650,11 +4651,12 @@ netdev_dpdk_destroy_rte_flow(struct netdev_dpdk *dev,
ret = rte_flow_destroy(dev->port_id, rte_flow, &error);
if (ret == 0) {
ufid_to_rte_flow_disassociate(ufid);
- VLOG_DBG("removed rte flow %p associated with ufid " UUID_FMT "\n",
- rte_flow, UUID_ARGS((struct uuid *)ufid));
+ VLOG_DBG("%s: removed rte flow %p associated with ufid " UUID_FMT "\n",
+ netdev_get_name(netdev), rte_flow,
+ UUID_ARGS((struct uuid *)ufid));
} else {
- VLOG_ERR("rte flow destroy error: %u : message : %s\n",
- error.type, error.message);
+ VLOG_ERR("%s: rte flow destroy error: %u : message : %s\n",
+ netdev_get_name(netdev), error.type, error.message);
}
ovs_mutex_unlock(&dev->mutex);
@@ -4675,8 +4677,7 @@ netdev_dpdk_flow_put(struct netdev *netdev, struct match *match,
*/
rte_flow = ufid_to_rte_flow_find(ufid);
if (rte_flow) {
- ret = netdev_dpdk_destroy_rte_flow(netdev_dpdk_cast(netdev),
- ufid, rte_flow);
+ ret = netdev_dpdk_destroy_rte_flow(netdev, ufid, rte_flow);
if (ret < 0) {
return ret;
}
@@ -4701,8 +4702,7 @@ netdev_dpdk_flow_del(struct netdev *netdev, const ovs_u128 *ufid,
return -1;
}
- return netdev_dpdk_destroy_rte_flow(netdev_dpdk_cast(netdev),
- ufid, rte_flow);
+ return netdev_dpdk_destroy_rte_flow(netdev, ufid, rte_flow);
}
#define DPDK_FLOW_OFFLOAD_API \
--
2.17.1
More information about the dev
mailing list