[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openvswitch-dev
Subject:    [ovs-dev] [Single DP 14/15] dpif: Add new dpif_port_exists() function.
From:       blp () nicira ! com (Ben Pfaff)
Date:       2012-10-31 4:07:36
Message-ID: 20121031040736.GC743 () nicira ! com
[Download RAW message or body]

On Tue, Oct 30, 2012 at 05:58:02PM -0700, Justin Pettit wrote:
> On Oct 22, 2012, at 4:35 PM, Ben Pfaff <blp at nicira.com> wrote:
> 
> > On Thu, Oct 18, 2012 at 12:51:59PM -0700, Justin Pettit wrote:
> >> Provide the ability to determine whether a port exists in a datapath
> >> without having to deal with a "dpif_port" structure as with
> >> dpif_port_query_by_name().  A future patch will use this function.
> >> 
> >> Signed-off-by: Justin Pettit <jpettit at nicira.com>
> > 
> > Seems reasonable.
> > 
> > Possibly, dpif_port_exists() should log if ->port_query_by_name()
> > returns a value other than 0 or ENODEV.
> 
> 
> Okay.  I added the following incremental:

Thanks.

[prev in list] [next in list] [prev in thread] [next in thread]