upcall: Remove redundant xlate_actions_for_side_effects().
authorEthan Jackson <ethan@nicira.com>
Thu, 22 May 2014 17:53:27 +0000 (10:53 -0700)
committerEthan Jackson <ethan@nicira.com>
Fri, 23 May 2014 17:58:19 +0000 (10:58 -0700)
commit691d39b2f93bc6d6339de82d4aeca337ab6aed86
tree4eac0ab1b0b7ca8aff4d34e99a3d5c5b0bbd8173
parent3b3f885732a8cc7424812dd65be0ed8755b70438
upcall: Remove redundant xlate_actions_for_side_effects().

As a result of commit a0bab87 (ofproto: Remove per-flow miss hash
table from upcall handler.) we're guaranteed that every packet has had
xlate_actions() called on it at least once.  Therefore, there's no
need to re-xlate slow path flows just to shove their packets through
the system.

This also may fix a bug discussed here:
http://openvswitch.org/pipermail/discuss/2014-April/013670.html

Signed-off-by: Ethan Jackson <ethan@nicira.com>
Reported-by: Murphy McCauley <murphy.mccauley@gmail.com>
Acked-by: Ben Pfaff <blp@nicira.com>
ofproto/ofproto-dpif-upcall.c