[ovs-dev] [PATCH] vswitch.xml: Better explain vlan-limit.

[Lilijun (Jerry, Cloud Networking)]

Yes, That's OK,  thanks.

-----Original Message-----
From: Ben Pfaff [mailto:blp at ovn.org] 
Sent: Saturday, September 08, 2018 1:59 AM
To: Eric Garver <eric at garver.life>; dev at openvswitch.org; Lilijun (Jerry, Cloud Networking) <jerry.lilijun at huawei.com>
Subject: Re: [PATCH] vswitch.xml: Better explain vlan-limit.

On Fri, Sep 07, 2018 at 01:49:03PM -0400, Eric Garver wrote:
> On Fri, Sep 07, 2018 at 10:39:18AM -0700, Ben Pfaff wrote:
> > CC: Eric Garver <e at erig.me>
> > Requested-by: Jerry Lilijun <jerry.lilijun at huawei.com>
> > Signed-off-by: Ben Pfaff <blp at ovn.org>
> > ---
> >  vswitchd/vswitch.xml | 8 ++++----
> >  1 file changed, 4 insertions(+), 4 deletions(-)
> > 
> > diff --git a/vswitchd/vswitch.xml b/vswitchd/vswitch.xml index 
> > e318151f0ce9..f05f616feed6 100644
> > --- a/vswitchd/vswitch.xml
> > +++ b/vswitchd/vswitch.xml
> > @@ -493,11 +493,11 @@
> >            e.g. a packet with more 802.1q headers will match Ethernet type
> >            0x8100.
> >          </p>
> > +
> >          <p>
> > -          Value <code>0</code> means unlimited.  The actual number of supported
> > -          VLAN headers is the smallest of <code>vlan-limit</code>, the number
> > -          of VLANs supported by Open vSwitch userspace (currently 2), and the
> > -          number supported by the datapath.
> > +          Open vSwitch userspace currently supports at most 2 VLANs, and each
> > +          datapath has its own limit.  If <code>vlan-limit</code> is nonzero,
> > +          it acts as a further limit.
> >          </p>
> >  
> >          <p>
> > --
> > 2.16.1
> 
> I think the simpler explanation helps. Thanks Ben!
> 
> Acked-by: Eric Garver <e at erig.me>

Thanks, applied to master.