cvs commit: src/sys/net if.h
Andre Oppermann
andre at freebsd.org
Mon Jun 11 20:56:45 UTC 2007
Jack Vogel wrote:
> On 6/11/07, Andre Oppermann <andre at freebsd.org> wrote:
>> Jack Vogel wrote:
>> > On 6/11/07, Jack Vogel <jfvogel at gmail.com> wrote:
>> >> 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.
>> >
>> > Oh, one other useful tidbit, for Oplin when RSS is enabled you can't do
>> > CKSUM anyway, they reuse bits.
>>
>> What is Oplin?
>
> Our new 10G adapter, er, its code name, its really 82598.
OK, then what is this 'RSS' feature?
--
Andre
More information about the cvs-all
mailing list