[ovs-dev] [next 33/35] ofproto: Improve abstraction by using OpenFlow port numbers in interface.

Ben Pfaff blp at nicira.com
Mon May 9 16:25:14 UTC 2011


On Fri, May 06, 2011 at 05:36:09PM -0700, Ethan Jackson wrote:
> I like this patch, it always seemed strange to me that the bridge
> cared about dp_ifidxes.
> 
> One thing I've been wondering about in the past couple of patches I've
> reviewed.  It seems a bit strange to me that "ofport" is indexed based
> on odp_port not ofp_port.  It' makes the ofproto module a bit awkward
> in places because you always have to convert in calls to get_port().
> I'm not sure that this patch is the place to change it, or if it
> should be changed.  Just thought I would through that out there.

I agree.  This gets changed later in the series.

> > - ? ?/* Obtain the correct "dp_ifidx"es from ofproto. */
> > + ? ?/* Obtain the correct "ofp_port"es from ofproto. */
> "es" => "s"

Thanks, fixed.



More information about the dev mailing list