On Sat, Jul 28, 2012 at 11:54:09AM -0700, Justin Pettit wrote:
> The datapath allows requesting a specific port number for a port, but
> the dpif interface didn't expose it.  This commit adds that support.
> 
> Signed-off-by: Justin Pettit <jpet...@nicira.com>

One oddity is that dpif_port_add() uses 0 as an out-of-band value for
input and UINT16_MAX as an out-of-band value for output.  I think we
could use UINT16_MAX (or even 0) in both places, for a possibly less
confusing interface.

In dpif_port_add() don't we need to initialize port_no in the case
where port_nop is NULL?

Otherwise looks good, thanks.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to