[ovs-dev] [PATCH] vswitchd: Also consider access port VLANs as "in use" for VLAN splinters.

Ben Pfaff blp at nicira.com
Tue Jan 3 21:37:18 UTC 2012


On Tue, Jan 03, 2012 at 01:28:39PM -0800, Justin Pettit wrote:
> On Dec 27, 2011, at 1:01 PM, Ben Pfaff wrote:
> 
> > @@ -3425,22 +3426,23 @@ collect_splinter_vlans(const struct ovsrec_open_vswitch *ovs_cfg)
> >             for (k = 0; k < port_cfg->n_interfaces; k++) {
> >                 struct ovsrec_interface *iface_cfg = port_cfg->interfaces[k];
> > 
> > +                vlan_splinters_enabled_anywhere = true;
> >                 if (vlan_splinters_is_enabled(iface_cfg)) {
> 
> I think you want that assignment inside this loop.  Otherwise, it
> looks good to me.

Thanks, I fixed that and pushed this to master and branch-1.4.



More information about the dev mailing list