Re: [PATCH v4 1/2] tools/hotplug/Linux: Add bridge VLAN support

[Anthony PERARD <anthony.perard@xxxxxxxxxx>]

On Sat, May 18, 2024 at 08:03:13AM -0400, Jason Andryuk wrote:
> On Fri, May 17, 2024 at 10:05 AM Leigh Brown <leigh@xxxxxxxxxxxxx> wrote:
> >
> > Update add_to_bridge shell function to read the vlan parameter from
> > xenstore and set the bridge VLAN configuration for the VID.
> >
> > Add additional helper functions to parse the vlan specification,
> > which consists of one or more of the following:
> >
> > a) single VLAN (e.g. 10).
> > b) contiguous range of VLANs (e.g. 10-15).
> > c) discontiguous range with base, increment and count
> >    (e.g. 100+10x9 which gives VLAN IDs 100, 110, ... 190).
> >
> > A single VLAN can be suffixed with "p" to indicate the PVID, or
> > "u" to indicate untagged. A range of VLANs can be suffixed with
> > "u" to indicate untagged.  A complex example would be:
> >
> >    vlan=1p/10-15/20-25u
> >
> > This capability requires the iproute2 bridge command to be
> > installed.  An error will be generated if the vlan parameter is
> > set and the bridge command is not available.
> >
> > Signed-off-by: Leigh Brown <leigh@xxxxxxxxxxxxx>
> 
> Reviewed-by: Jason Andryuk <jason.andryuk@xxxxxxx>

Acked-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>

Thanks,


--
Anthony Perard | Vates XCP-ng Developer

XCP-ng & Xen Orchestra - Vates solutions

web: https://vates.tech