[ovs-dev] [next 1/2] ofproto-dpif: Avoid unnecessary ODP-to-OFP port conversion.

Justin Pettit jpettit at nicira.com
Wed May 18 01:03:48 UTC 2011


Looks good.

--Justin


On May 17, 2011, at 3:50 PM, Ben Pfaff wrote:

> ---
> 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 82bb0e0..0e4f8cb 100644
> --- a/ofproto/ofproto-dpif.c
> +++ b/ofproto/ofproto-dpif.c
> @@ -990,7 +990,7 @@ bundle_set(struct ofproto *ofproto_, void *aux,
> 
>         LIST_FOR_EACH_SAFE (port, next_port, bundle_node, &bundle->ports) {
>             for (i = 0; i < s->n_slaves; i++) {
> -                if (s->slaves[i] == odp_port_to_ofp_port(port->odp_port)) {
> +                if (s->slaves[i] == port->up.ofp_port) {
>                     goto found;
>                 }
>             }
> -- 
> 1.7.4.4
> 
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev




More information about the dev mailing list