cvs commit: src/sys/net if_ethersubr.c

Bruce M. Simpson bms at
Thu Feb 22 15:42:42 UTC 2007

Christian S.J. Peron wrote:
>   Use ETHER_BPF_MTAP() instead of BPF_MTAP() here.  It's possible
>   incoming packets have had their 802.1Q tags processed by the
>   hardware, resulting in them being stripped from the packets, and
>   placed on the mbuf.  This fixes the processing of 802.1Q tags when
>   hardware offload of 802.1Q tags is enabled.
Thanks for this change.

I have integrated it into my local checkout containing the 802.1p/q 
rewrite as it is needed there to do the right thing when M_VLANTAG is 
*always* used even when the cards themselves do not do VLAN hwtagging.

I have moved the 802.1p/q decapsulation to ether_input() itself as per 
thompsa@'s suggestion.

I should really use p4 (or Hg) for this next time around...


More information about the cvs-src mailing list