In 'ovs-dpctl dump-flows' output, we should only report the pmd
related info for 'dpif-netdev' datapath.  However, current
implementation also reports uninitialized pmd info for
'dpif-netlink' datapath, which is very confusing to users.

This commit fixes it.

Signed-off-by: Alex Wang <al...@nicira.com>

---
PATCH->V2:
- fix it using the correct way.
---
 lib/dpif-netlink.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/dpif-netlink.c b/lib/dpif-netlink.c
index de43d80..1d8abdf 100644
--- a/lib/dpif-netlink.c
+++ b/lib/dpif-netlink.c
@@ -1429,9 +1429,9 @@ dpif_netlink_flow_to_dpif_flow(struct dpif *dpif, struct 
dpif_flow *dpif_flow,
     dpif_flow->actions = datapath_flow->actions;
     dpif_flow->actions_len = datapath_flow->actions_len;
     dpif_flow->ufid_present = datapath_flow->ufid_present;
+    dpif_flow->pmd_id = PMD_ID_NULL;
     if (datapath_flow->ufid_present) {
         dpif_flow->ufid = datapath_flow->ufid;
-    dpif_flow->pmd_id = PMD_ID_NULL;
     } else {
         ovs_assert(datapath_flow->key && datapath_flow->key_len);
         dpif_flow_hash(dpif, datapath_flow->key, datapath_flow->key_len,
-- 
1.7.9.5

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to