Hi,
On 17.03.2017 22:55, Ben Pfaff wrote:
Previously, this function could modify the pkt_mark field as part of IPsec
integration. It no longer does that, so there's no longer any need for it
to save and restore pkt_mark, and this commit removes that.
Does it mean that now there is no way to send a bit of information
across a pair of patch ports, that is, mark a packet on bridge A and
check the mark on bridge B?
Thanks,
Valentine
CC: Ansis Atteka <aatt...@ovn.org>
Signed-off-by: Ben Pfaff <b...@ovn.org>
---
ofproto/ofproto-dpif-xlate.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c
index 1a82b8d569be..9fe778a32857 100644
--- a/ofproto/ofproto-dpif-xlate.c
+++ b/ofproto/ofproto-dpif-xlate.c
@@ -3265,7 +3265,6 @@ compose_output_action__(struct xlate_ctx *ctx, ofp_port_t
ofp_port,
struct flow *flow = &ctx->xin->flow;
struct flow_tnl flow_tnl;
union flow_vlan_hdr flow_vlans[FLOW_MAX_VLAN_HEADERS];
- uint32_t flow_pkt_mark;
uint8_t flow_nw_tos;
odp_port_t out_port, odp_port;
bool tnl_push_pop_send = false;
@@ -3460,7 +3459,6 @@ compose_output_action__(struct xlate_ctx *ctx, ofp_port_t
ofp_port,
}
memcpy(flow_vlans, flow->vlans, sizeof flow_vlans);
- flow_pkt_mark = flow->pkt_mark;
flow_nw_tos = flow->nw_tos;
if (count_skb_priorities(xport)) {
@@ -3588,7 +3586,6 @@ compose_output_action__(struct xlate_ctx *ctx, ofp_port_t
ofp_port,
out:
/* Restore flow */
memcpy(flow->vlans, flow_vlans, sizeof flow->vlans);
- flow->pkt_mark = flow_pkt_mark;
flow->nw_tos = flow_nw_tos;
}
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev