[ovs-dev] [PATCH] rhel: Add support for DPDK Bond

[Flavio Leitner]

On Tue, Jul 26, 2016 at 01:35:58PM -0400, Aaron Conole wrote:
> Flavio Leitner <fbl at redhat.com> writes:
> 
> > The current ifcfg file lacks support for OVS DPDK Bond.
> >
> > Signed-off-by: Flavio Leitner <fbl at redhat.com>
> > ---
> >  rhel/README.RHEL                            | 20 +++++++++++++++++---
> >  rhel/etc_sysconfig_network-scripts_ifup-ovs |  9 +++++++++
> >  2 files changed, 26 insertions(+), 3 deletions(-)
> >
> > diff --git a/rhel/README.RHEL b/rhel/README.RHEL
> > index 54c1b44..0f51be5 100644
> > --- a/rhel/README.RHEL
> > +++ b/rhel/README.RHEL
> > @@ -33,13 +33,16 @@ assignments.  The following OVS-specific variable names are supported:
> >          * "OVSDPDKPort", if <name> is a physical DPDK NIC port (name
> >            must start with "dpdk" and end with portid, eg "dpdk0")
> >  
> > -	* "OVSDPDKRPort", if <name> is a DPDK ring port (name must
> > +        * "OVSDPDKRPort", if <name> is a DPDK ring port (name must
> >            start with dpdkr and end with portid, eg "dpdkr0")
> >  
> >          * "OVSDPDKVhostPort" if <name> is a DPDK vhost-cuse port
> >  
> >          * "OVSDPDKVhostUserPort" if <name> is a DPDK vhost-user port
> >  
> > +        * "OVSDPDKBond" if <name> is an OVS DPDK bond.
> > +
> > +
> 
> Small nit - why insert two newlines here as opposed to one?

No good reason.  Will follow up with v2.

> Otherwise, this looks good to me.
> 
> Reviewed-by: Aaron Conole <aconole at redhat.com>
Thanks!
fbl