cvs commit: src/sys/net if_vlan.c
Qing Li
qingli at speakeasy.net
Wed Aug 2 03:37:19 UTC 2006
>
> Removing the "dead" assignment to evl->evl_encap_proto above and
> reducing the bcopy window by 2 bytes here is a nano-optimization.
> Which is worse, it smuggles hidden dot1q details in this otherwise
> encapsulation-neutral block of code. E.g., I could use the former
> code for both dot1q and ISL privately by just redefining
> ETHER_VLAN_ENCAP_LEN.
>
You are right.
>
> Grand total: I would greately appreciate if you backed out this
> change and then fixed the tag value of -1 alone, but let me review
> that first. Thanks!
>
Okay.
Thanks,
-- Qing
More information about the cvs-src
mailing list