cvs commit: src/sys/net if_vlan.c

Yar Tikhiy yar at comp.chem.msu.su
Fri Sep 16 05:34:28 PDT 2005


On Fri, Sep 16, 2005 at 03:53:12PM +0400, Gleb Smirnoff wrote:
> On Fri, Sep 16, 2005 at 11:44:43AM +0000, Yar Tikhiy wrote:
> Y> yar         2005-09-16 11:44:43 UTC
> Y> 
> Y>   FreeBSD src repository
> Y> 
> Y>   Modified files:
> Y>     sys/net              if_vlan.c 
> Y>   Log:
> Y>   Test the new M_VLANTAG packet flag before calling
> Y>   m_tag_locate().  This adds little overhead of a simple
> Y>   bitwise operation in case hardware VLAN acceleration
> Y>   is on, yet saves the more expensive function call if
> Y>   the acceleration is off.
> Y>   
> Y>   Reviewed by:    ru, glebius
> Y>   X-MFC-after:    6.0
>      ^^^^^^^^^^^^^^^^^^^
> 			Why?

'Cause it's more about code consistency than stability, or performance,
or features.  So I'd rather not bother re@ with this when they are
being flooded with really important bugfixes.  It shouldn't be the
extra m_tag_locate() call that is likely to be a bottleneck in 6.0 ;)

-- 
Yar


More information about the cvs-src mailing list