[ovs-dev] [PATCH] ofproto-dpif: Fix argument in send_packet() call.

[Justin Pettit]

Looks good.

--Justin


On May 18, 2011, at 4:46 PM, Ben Pfaff wrote:

> The second argument to send_packet() is an ODP port, not an OFP port, so
> we need to use ODPP_LOCAL instead of OFPP_LOCAL.
> ---
> ofproto/ofproto-dpif.c |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c
> index 714cdf6..2f03dfd 100644
> --- a/ofproto/ofproto-dpif.c
> +++ b/ofproto/ofproto-dpif.c
> @@ -1591,7 +1591,7 @@ handle_miss_upcall(struct ofproto_dpif *ofproto, struct dpif_upcall *upcall)
>     /* Check with in-band control to see if this packet should be sent
>      * to the local port regardless of the flow table. */
>     if (connmgr_msg_in_hook(ofproto->up.connmgr, &flow, upcall->packet)) {
> -        send_packet(ofproto, OFPP_LOCAL, upcall->packet);
> +        send_packet(ofproto, ODPP_LOCAL, upcall->packet);
>     }
> 
>     facet = facet_lookup_valid(ofproto, &flow);
> -- 
> 1.7.4.4
> 
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev