[ovs-dev] [PATCH 05/17] nx-match: Encode dp_hash and recirc_id in OXM also.

[Ben Pfaff]

dp_hash and recirc_id are specific to OVS, but that doesn't mean that we
shouldn't encode them into flow matches when OXM is used in OpenFlow 1.2
and later.

Signed-off-by: Ben Pfaff <b...@nicira.com>
---
 lib/nx-match.c | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/lib/nx-match.c b/lib/nx-match.c
index b896450..6537601 100644
--- a/lib/nx-match.c
+++ b/lib/nx-match.c
@@ -621,17 +621,13 @@ nx_put_raw(struct ofpbuf *b, enum ofp_version oxm, const 
struct match *match,
 
     /* Metadata. */
     if (match->wc.masks.dp_hash) {
-        if (!oxm) {
-            nxm_put_32m(b, mf_oxm_header(MFF_DP_HASH, oxm),
-                        htonl(flow->dp_hash), htonl(match->wc.masks.dp_hash));
-        }
+        nxm_put_32m(b, mf_oxm_header(MFF_DP_HASH, oxm),
+                    htonl(flow->dp_hash), htonl(match->wc.masks.dp_hash));
     }
 
     if (match->wc.masks.recirc_id) {
-        if (!oxm) {
-            nxm_put_32(b, mf_oxm_header(MFF_RECIRC_ID, oxm),
-                       htonl(flow->recirc_id));
-        }
+        nxm_put_32(b, mf_oxm_header(MFF_RECIRC_ID, oxm),
+                   htonl(flow->recirc_id));
     }
 
     if (match->wc.masks.in_port.ofp_port) {
-- 
1.9.1

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