diff options
author | Jarno Rajahalme <jrajahalme@nicira.com> | 2014-11-03 14:40:22 -0800 |
---|---|---|
committer | Jarno Rajahalme <jrajahalme@nicira.com> | 2014-11-03 14:41:34 -0800 |
commit | 5e9a47719dd02a099939f5aaada5ca3ae3b47dc3 (patch) | |
tree | 6a74631775fae7059aba32f2fe566d881eae5b85 /ofproto | |
parent | 12e4681cfb4a7f0c85a41b347a8a89922a3e0ac2 (diff) | |
download | openvswitch-5e9a47719dd02a099939f5aaada5ca3ae3b47dc3.tar.gz |
ofproto-dpif-xlate: Do not clear ports with frags in xlate_actions().
Transport ports are already cleared for lookup if need be, and they
should be available for action processing, so do not clear them in
xlate_actions().
A drop flow takes care of dropping the packet, so no special case is
needed in xlate_actions().
Signed-off-by: Jarno Rajahalme <jrajahalme@nicira.com>
Acked-by: Ben Pfaff <blp@nicira.com>
Diffstat (limited to 'ofproto')
-rw-r--r-- | ofproto/ofproto-dpif-xlate.c | 23 |
1 files changed, 0 insertions, 23 deletions
diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c index e9bb4ff0c..12bfb9f91 100644 --- a/ofproto/ofproto-dpif-xlate.c +++ b/ofproto/ofproto-dpif-xlate.c @@ -4255,29 +4255,6 @@ xlate_actions(struct xlate_in *xin, struct xlate_out *xout) orig_flow = *flow; } - if (flow->nw_frag & FLOW_NW_FRAG_ANY) { - switch (ctx.xbridge->frag) { - case OFPC_FRAG_NORMAL: - /* We must pretend that transport ports are unavailable. */ - flow->tp_src = ctx.base_flow.tp_src = htons(0); - flow->tp_dst = ctx.base_flow.tp_dst = htons(0); - break; - - case OFPC_FRAG_DROP: - return; - - case OFPC_FRAG_REASM: - OVS_NOT_REACHED(); - - case OFPC_FRAG_NX_MATCH: - /* Nothing to do. */ - break; - - case OFPC_INVALID_TTL_TO_CONTROLLER: - OVS_NOT_REACHED(); - } - } - in_port = get_ofp_port(ctx.xbridge, flow->in_port.ofp_port); if (in_port && in_port->is_tunnel) { if (ctx.xin->resubmit_stats) { |