cvs commit: src/sys/conf options src/sys/netinet ip_output.c

Maxime Henrion mux at freebsd.org
Tue Mar 25 23:51:23 PST 2003


Mike Silbersack wrote:
> 
> On Tue, 25 Mar 2003, Maxime Henrion wrote:
> 
> > Looks like there is a bug in this code.  I tried to understand what's
> > wrong in the if_xl code that tries to deal with mbuf chains containing
> > more than XL_MAXFRAGS mbufs, and noticed that m_head->m_pkthdr.len isn't
> > set properly.  The length of an mbuf chain shouldn't change when we're
> > just splitting it into more mbufs, so this is kinda weird.  Using
> > m_fixhdr() just after the splitting code solves it, but that's probably
> > just a workaround, and either m_split() or the MBUF_FRAG_TEST code needs
> > to be fixed.
> 
> Hm, the bug's probably in the frag test code.  I'll take a look at it
> later tonight, but you're welcome to fix it in the meantime.

I committed a few fixes of mine.  I hope the commit log explains the
changes clearly enough.

Cheers,
Maxime


More information about the cvs-src mailing list