cvs commit: src/sys/net if.h

Jack Vogel jfvogel at gmail.com
Mon Jun 11 20:30:48 UTC 2007


On 6/11/07, Sam Leffler <sam at errno.com> wrote:
> Andre Oppermann wrote:
> > andre       2007-06-11 20:08:12 UTC
> >
> >   FreeBSD src repository
> >
> >   Modified files:
> >     sys/net              if.h
> >   Log:
> >   Add IFCAP_LRO flag for drivers to announce their TCP Large Receive Offload
> >   capabilities.
> >
> >   Revision  Changes    Path
> >   1.108     +1 -0      src/sys/net/if.h
> > http://cvsweb.FreeBSD.org/src/sys/net/if.h.diff?r1=1.107&r2=1.108
> >
> >
> There are many offload capabilities defined that are not well thought
> out.  In particular we do not distinguish between ipv4 and ipv6 for
> things like cksum and tso so there's no way to disable individual
> features.  The ability to tweak LRO is clearly needed and clearly belong
> as an ifnet capability but unilateraly deciding this is the wrong approach.
>
>         Sam

We do distinguish between TSO4 and TSO6, its just that all the pieces
for 6 arent actually there yet, you are right about the cksum granularity,
but I'm not convinced its needed. You might be right on the need to
fine tune the functionality, but having a big ON/OFF doesnt seem a
bad thing to me.

Jack


More information about the cvs-src mailing list