Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 12 Jun 2007 19:38:20 +0900
From:      Pyun YongHyeon <pyunyh@gmail.com>
To:        Andrew Gallatin <gallatin@cs.duke.edu>
Cc:        cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org, Pyun YongHyeon <yongari@FreeBSD.org>
Subject:   Re: cvs commit: src/sys/dev/nfe if_nfe.c if_nfereg.h
Message-ID:  <20070612103820.GG44998@cdnetworks.co.kr>
In-Reply-To: <20070612060521.D84335@grasshopper.cs.duke.edu>
References:  <200706120235.l5C2Z1iJ022543@repoman.freebsd.org> <20070612060521.D84335@grasshopper.cs.duke.edu>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Jun 12, 2007 at 06:05:21AM -0400, Andrew Gallatin wrote:
 > Pyun YongHyeon [yongari@FreeBSD.org] wrote:
 > > yongari     2007-06-12 02:35:01 UTC
 > > 
 > >   FreeBSD src repository
 > > 
 > >   Modified files:
 > >     sys/dev/nfe          if_nfe.c if_nfereg.h 
 > >   Log:
 > >   Increase a maximum segment size of DMA to 4096. Previously it used
 > >   MCLBYTES for the segment size but it used too many Tx descriptors in
 > >   TSO case.
 > >   While I'm here adjust maximum size of the sum of all segment lengths
 > >   in a given DMA mapping to 65535, the maximum size, in bytes, of a IP
 > 
 > Can't it actually be 65535 + link layer headers?  Eg, up to
 > 65535 + 14 or 65535 + 18?
 > 

Oops, you're right. It should be 65535 + sizeof(struct ether_vlan_header).
Thanks for pointing out! :)

 > Drew

-- 
Regards,
Pyun YongHyeon



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20070612103820.GG44998>