ovs-dpctl: Do not report pmd info for 'dpif-netlink' datapath.
authorAlex Wang <alexw@nicira.com>
Mon, 2 Feb 2015 22:50:47 +0000 (14:50 -0800)
committerAlex Wang <alexw@nicira.com>
Wed, 4 Feb 2015 01:59:16 +0000 (17:59 -0800)
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 <alexw@nicira.com>
Acked-by: Ben Pfaff <blp@nicira.com>
lib/dpif-netlink.c

index de43d80..1d8abdf 100644 (file)
@@ -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,