[ovs-dev] [patch v9 07/11] dpctl: Simplify opt_dpif_open().

Ben Pfaff blp at ovn.org
Tue Dec 11 16:28:17 UTC 2018


On Mon, Nov 26, 2018 at 11:08:08AM -0800, Yi-Hung Wei wrote:
> On Mon, Nov 19, 2018 at 11:11 AM Darrell Ball <dlu998 at gmail.com> wrote:
> >
> > The commonly used function, opt_dpif_open(), recently became more complex
> > to check for a datapath argument. Unnecessary dummy parameters for most users
> > were hence added.  Revert back and call the intended api, dp_arg_exists(), to
> > query for a datapath argument being supplied.
> >
> > Fixes: 4eeec031d4c4 ("dpctl: Implement dpctl commands for conntrack per zone limit")
> > CC: Yi-Hung Wei <yihung.wei at gmail.com>
> > Signed-off-by: Darrell Ball <dlu998 at gmail.com>
> > ---
> 
> Thanks for the simplification.  Looks good to me.
> 
> Acked-by: Yi-Hung Wei <yihung.wei at gmail.com>

Thanks Darrell and Yi-Hung.  I applied this to master.


More information about the dev mailing list