Seems better as we no longer modify the flow. Signed-off-by: Jarno Rajahalme <jrajaha...@nicira.com> --- ofproto/ofproto-dpif-upcall.c | 12 ++++++------ ofproto/ofproto-dpif-xlate.c | 8 ++++---- ofproto/ofproto-dpif-xlate.h | 8 ++++---- 3 files changed, 14 insertions(+), 14 deletions(-)
diff --git a/ofproto/ofproto-dpif-upcall.c b/ofproto/ofproto-dpif-upcall.c index 418597c..49e541d 100644 --- a/ofproto/ofproto-dpif-upcall.c +++ b/ofproto/ofproto-dpif-upcall.c @@ -823,8 +823,8 @@ upcall_receive(struct upcall *upcall, const struct dpif_backer *backer, { int error; - error = xlate_receive(backer, flow, &upcall->ofproto, &upcall->ipfix, - &upcall->sflow, NULL, &upcall->in_port); + error = xlate_lookup(backer, flow, &upcall->ofproto, &upcall->ipfix, + &upcall->sflow, NULL, &upcall->in_port); if (error) { return error; } @@ -1301,8 +1301,8 @@ revalidate_ukey(struct udpif *udpif, struct udpif_key *ukey, goto exit; } - error = xlate_receive(udpif->backer, &flow, &ofproto, NULL, NULL, &netflow, - &ofp_in_port); + error = xlate_lookup(udpif->backer, &flow, &ofproto, NULL, NULL, &netflow, + &ofp_in_port); if (error) { goto exit; } @@ -1435,8 +1435,8 @@ push_dump_ops__(struct udpif *udpif, struct dump_op *ops, size_t n_ops) continue; } - error = xlate_receive(udpif->backer, &flow, &ofproto, - NULL, NULL, &netflow, &ofp_in_port); + error = xlate_lookup(udpif->backer, &flow, &ofproto, + NULL, NULL, &netflow, &ofp_in_port); if (!error) { struct xlate_in xin; diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c index ffb2580..6081041 100644 --- a/ofproto/ofproto-dpif-xlate.c +++ b/ofproto/ofproto-dpif-xlate.c @@ -978,10 +978,10 @@ xlate_lookup_ofproto(const struct dpif_backer *backer, const struct flow *flow, * Returns 0 if successful, ENODEV if the parsed flow has no associated ofport. */ int -xlate_receive(const struct dpif_backer *backer, const struct flow *flow, - struct ofproto_dpif **ofproto, struct dpif_ipfix **ipfix, - struct dpif_sflow **sflow, struct netflow **netflow, - ofp_port_t *ofp_in_port) +xlate_lookup(const struct dpif_backer *backer, const struct flow *flow, + struct ofproto_dpif **ofproto, struct dpif_ipfix **ipfix, + struct dpif_sflow **sflow, struct netflow **netflow, + ofp_port_t *ofp_in_port) { const struct xport *xport; diff --git a/ofproto/ofproto-dpif-xlate.h b/ofproto/ofproto-dpif-xlate.h index 1864980..0f4c0b1 100644 --- a/ofproto/ofproto-dpif-xlate.h +++ b/ofproto/ofproto-dpif-xlate.h @@ -176,10 +176,10 @@ void xlate_ofport_remove(struct ofport_dpif *); struct ofproto_dpif * xlate_lookup_ofproto(const struct dpif_backer *, const struct flow *, ofp_port_t *ofp_in_port); -int xlate_receive(const struct dpif_backer *, const struct flow *, - struct ofproto_dpif **, struct dpif_ipfix **, - struct dpif_sflow **, struct netflow **, - ofp_port_t *ofp_in_port); +int xlate_lookup(const struct dpif_backer *, const struct flow *, + struct ofproto_dpif **, struct dpif_ipfix **, + struct dpif_sflow **, struct netflow **, + ofp_port_t *ofp_in_port); void xlate_actions(struct xlate_in *, struct xlate_out *); void xlate_in_init(struct xlate_in *, struct ofproto_dpif *, -- 1.7.10.4 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev