On 12/5/15 1:27 PM, David Ahern wrote: > 2. Why not leave the macros: > > +#define BR_VLAN_MAX 4096 > +#define BR_VLAN_BITMAP_LEN (BR_VLAN_MAX / sizeof(uint32_t)) oops, I meant (BR_VLAN_MAX / (sizeof(uint32_t) * 8))