cvs commit: src/sys/net if_vlan.c

Garrett Wollman wollman at lcs.mit.edu
Mon Jul 7 18:27:51 PDT 2003


<<On Mon, 7 Jul 2003 16:45:11 -0700 (PDT), wpaul at FreeBSD.ORG (Bill Paul) said:

> 1) Change vlan_input() to apply EVL_VLANOFTAG() to all incoming tags
>    (in both hardware and non-hardware tagging cases) and change the
>    vlan_ioctl() function to only accept vlan tags from 0 to 4095
>    and return EINVAL if the caller tries to set any of the other
>    bits. (Prevent user from shooting self in foot.)

I believe that this is the correct choice.  (It's probably a result of
my stupidity that the vlan driver didn't do this already.)

At some point we should do something sensible with the other bits,
too.  In some applications it might be useful to allow the user to
configure the .1p priority bits, outside of a VLAN context.  (We
should be able to at least decode frames that arrive with .1p
encapsulation but .1Q tag 0.)

-GAWollman

_______________________________________________
cvs-all at freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe at freebsd.org"


More information about the cvs-src mailing list