svn commit: r271735 - head/sys/net
Kevin Lo
kevlo at FreeBSD.org
Thu Sep 18 06:13:31 UTC 2014
On Thu, Sep 18, 2014 at 09:18:34AM +0400, Gleb Smirnoff wrote:
>
> On Thu, Sep 18, 2014 at 03:09:35AM +0000, Marcelo Araujo wrote:
> M> Author: araujo (ports committer)
> M> Date: Thu Sep 18 03:09:34 2014
> M> New Revision: 271735
> M> URL: http://svnweb.freebsd.org/changeset/base/271735
> M>
> M> Log:
> M> Remove old comment, we already do 802.1q tagging.
>
> But the comment speaks about 802.1p priority tagging, which
> we do not support. So the comment was absolutely correct.
>
> I'd prefer to see patch reverted.
Sorry, I overlooked that. Marcelo has a patch with 802.1p tagging support,
I thought the patch was committed.
> M> Phabric: D797
> M> Reviewed by: kevlo
> M> Approved by: kevlo
> M> Sponsored by: QNAP Systems Inc.
> M>
> M> Modified:
> M> head/sys/net/if_vlan.c
> M>
> M> Modified: head/sys/net/if_vlan.c
> M> ==============================================================================
> M> --- head/sys/net/if_vlan.c Thu Sep 18 03:02:00 2014 (r271734)
> M> +++ head/sys/net/if_vlan.c Thu Sep 18 03:09:34 2014 (r271735)
> M> @@ -29,8 +29,7 @@
> M>
> M> /*
> M> * if_vlan.c - pseudo-device driver for IEEE 802.1Q virtual LANs.
> M> - * Might be extended some day to also handle IEEE 802.1p priority
> M> - * tagging. This is sort of sneaky in the implementation, since
> M> + * This is sort of sneaky in the implementation, since
> M> * we need to pretend to be enough of an Ethernet implementation
> M> * to make arp work. The way we do this is by telling everyone
> M> * that we are an Ethernet, and then catch the packets that
> M>
>
> --
> Totus tuus, Glebius.
Kevin
More information about the svn-src-all
mailing list