[ovs-dev] [next 33/35] ofproto: Improve abstraction by using OpenFlow port numbers in interface.
Ethan Jackson
ethan at nicira.com
Sat May 7 00:36:09 UTC 2011
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.
> - /* Obtain the correct "dp_ifidx"es from ofproto. */
> + /* Obtain the correct "ofp_port"es from ofproto. */
"es" => "s"
Looks Good.
Ethan
More information about the dev
mailing list