Re: [ovs-dev] [RFC 1/2] dpdk: Convert initialization from cmdline to db

2015-12-04 Thread Traynor, Kevin
> -Original Message- > From: Aaron Conole [mailto:acon...@redhat.com] > Sent: Friday, December 4, 2015 3:37 PM > To: Traynor, Kevin > Cc: dev@openvswitch.org; Flavio Leitner > Subject: Re: [ovs-dev] [RFC 1/2] dpdk: Convert initialization from cmdline to > db > &g

Re: [ovs-dev] [RFC 1/2] dpdk: Convert initialization from cmdline to db

2015-12-04 Thread Aaron Conole
Zoltan Kiss writes: > On 03/12/15 04:23, Aaron Conole wrote: >> Existing DPDK integration is provided by use of command line options which >> must be split out and passed to librte in a special manner. However, this >> forces any configuration to be passed by way of a special DPDK flag, and >> int

Re: [ovs-dev] [RFC 1/2] dpdk: Convert initialization from cmdline to db

2015-12-04 Thread Zoltan Kiss
On 03/12/15 04:23, Aaron Conole wrote: Existing DPDK integration is provided by use of command line options which must be split out and passed to librte in a special manner. However, this forces any configuration to be passed by way of a special DPDK flag, and interferes with ovs+dpdk packaging

Re: [ovs-dev] [RFC 1/2] dpdk: Convert initialization from cmdline to db

2015-12-04 Thread Aaron Conole
"Traynor, Kevin" writes: >> -Original Message- <> >> +static int >> +get_dpdk_args(const struct ovsrec_open_vswitch *ovs_cfg, char ***argv) >> +{ >> +struct dpdk_options_map { >> +const char *ovs_configuration; >> +const char *dpdk_option; >> +bool default_enabl

Re: [ovs-dev] [RFC 1/2] dpdk: Convert initialization from cmdline to db

2015-12-04 Thread Aaron Conole
Panu Matilainen writes: > On 12/03/2015 06:23 AM, Aaron Conole wrote: >> Existing DPDK integration is provided by use of command line options which >> must be split out and passed to librte in a special manner. However, this >> forces any configuration to be passed by way of a special DPDK flag, a

Re: [ovs-dev] [RFC 1/2] dpdk: Convert initialization from cmdline to db

2015-12-04 Thread Aaron Conole
Mauricio Vásquez writes: > Dear Aaron, > > The general idea looks good to me. > > Just some comments inline > > On 3 December 2015 at 05:23, Aaron Conole wrote: > << snipped >> >> +if(lookup) { >> +char **newargv = grow_argv(argv, ret, 2); >> + >> +if (!newargv) >

Re: [ovs-dev] [RFC 1/2] dpdk: Convert initialization from cmdline to db

2015-12-04 Thread Traynor, Kevin
> -Original Message- > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Aaron Conole > Sent: Thursday, December 3, 2015 4:24 AM > To: dev@openvswitch.org > Cc: Flavio Leitner > Subject: [ovs-dev] [RFC 1/2] dpdk: Convert initialization from cmdline to db

Re: [ovs-dev] [RFC 1/2] dpdk: Convert initialization from cmdline to db

2015-12-04 Thread Panu Matilainen
On 12/03/2015 06:23 AM, Aaron Conole wrote: Existing DPDK integration is provided by use of command line options which must be split out and passed to librte in a special manner. However, this forces any configuration to be passed by way of a special DPDK flag, and interferes with ovs+dpdk packag

Re: [ovs-dev] [RFC 1/2] dpdk: Convert initialization from cmdline to db

2015-12-03 Thread Mauricio Vásquez
Dear Aaron, The general idea looks good to me. Just some comments inline On 3 December 2015 at 05:23, Aaron Conole wrote: > Existing DPDK integration is provided by use of command line options which > must be split out and passed to librte in a special manner. However, this > forces any confi

[ovs-dev] [RFC 1/2] dpdk: Convert initialization from cmdline to db

2015-12-02 Thread Aaron Conole
Existing DPDK integration is provided by use of command line options which must be split out and passed to librte in a special manner. However, this forces any configuration to be passed by way of a special DPDK flag, and interferes with ovs+dpdk packaging solutions. This commit delays dpdk initia