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-all
mailing list