[ovs-discuss] [PATCH] [ovs-dev] [PATCH] QinQ: support more vlan headers.

[Ben Pfaff]

On Fri, Sep 07, 2018 at 09:43:09AM -0400, Eric Garver wrote:
> On Fri, Sep 07, 2018 at 09:58:16AM +0000, Lilijun (Jerry, Cloud Networking) wrote:
> > We can remove the unlimited words.  Maybe like this:
> > 
> >               Value  0  means  the max number of VLANs  supported  by  Open vSwitch userspace (currently 2), or the number
> >               supported by the datapath.
> 
> This is still tricky wording. It may be read as the maximum value
> supported by OVS userspace or the maximum value supported by the
> datapath. Really it's min( max(OVS userspace), max(datapath) ).
> 
> I suggest:
> 
>     Value 0 means the least common value of maximum supported VLANs in;
>     the Open vSwitch userpsace (currently 2), and the datapath.

I sent a patch:
        https://patchwork.ozlabs.org/patch/967444/