On Fri, 2006-06-02 at 19:26 -0700, Mike Ditto wrote:
> In dls_header():
> >       if (vid != VLAN_ID_NONE) {
> 
> Don't you also want to insert a vlan header when pri > 0?  I don't know
> if anybody really uses priority on VID 0, but it seems trivial to change
> this test (and the one a few lines later) and have it work.

Yes, and this is an existing bug (the bug is in i_dls_ether_header() in
Nevada).  I'd rather have a bug filed and fixed separately as my testing
matrix is already quite large.  I've filed:

6434130 i_dls_ether_header() doesn't generate VLAN header when priority is 
non-zero

Thanks Mike,
-Seb


_______________________________________________
networking-discuss mailing list
[email protected]

Reply via email to