[ovs-dev] [Single DP 07/15] tests: Define new ADD_OF_PORTS macro for ofproto tests.

Justin Pettit jpettit at nicira.com
Mon Oct 29 22:32:41 UTC 2012


On Oct 19, 2012, at 11:05 AM, Ben Pfaff <blp at nicira.com> wrote:

> On Thu, Oct 18, 2012 at 12:51:52PM -0700, Justin Pettit wrote:
>> A future commit will break the relation between OpenFlow and datapath
>> port numbers.  The new ADD_OF_PORTS macro adds an interface to a bridge
>> and configures it such that both the OpenFlow and datapath port numbers
>> are the same.  This is important in tests that deal with both OpenFlow
>> and datapath port numbers (e.g., ones that use ofproto/trace).
>> 
>> Signed-off-by: Justin Pettit <jpettit at nicira.com>
> 
> This looks good, but the following definition of ADD_OF_PORTS should
> be faster because it will run ovs-vsctl only once regardless of the
> number of ports to add:
> 
> m4_define([ADD_OF_PORTS],
>  [ovs-vsctl m4_foreach([of_port], m4_cdr($@),
>    [ \
> -- add-port $1 p[]of_port -- set Interface p[]of_port type=dummy ofport_request=of_port])])

Awesome!  Much better.  Thank you.

--Justin





More information about the dev mailing list