Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 27 Mar 2003 01:42:44 -0600 (CST)
From:      Mike Silbersack <silby@silby.com>
To:        Maxime Henrion <mux@freebsd.org>
Cc:        cvs-src@FreeBSD.org
Subject:   Re: cvs commit: src/sys/conf options src/sys/netinet ip_output.c
Message-ID:  <20030327014054.K7674@odysseus.silby.com>
In-Reply-To: <20030327013224.P7674@odysseus.silby.com>
References:  <200303260452.h2Q4quap015364@www.ambrisko.com>    <20030326183351.GJ57674@elvis.mu.org> <20030327013224.P7674@odysseus.silby.com>

next in thread | previous in thread | raw e-mail | index | archive | help

Er, and XL_MAXFRAGS won't change, that's part of the test code too.

We may have to enhance MBUF_FRAG_TEST to get a real test of some drivers;
as Sam informed me, bus_dmamap_load_mbuf coalesces adjacent fragments,
thereby defeating the testing ability of MBUF_FRAG_TEST.  I guess I'll
work on that next.

On Thu, 27 Mar 2003, Mike Silbersack wrote:

>
> Ok, I think I have m_defrag in a working state, please review.
>
> Changes from last time:
>
> - It supports infinitely long chains.
> - It has a "goal" argument which is supposed to be a hint to tell m_defrag
> how long the chain can be.  It is currently ignored; someone may want to
> honor this later for optimization purposes.
> - I fixed up the error case in if_xl, it only runs if needed now
>
> At the top of the if_loop and if_xl patches are debugging code I used to
> make it was working right, they're certainly not going in.
>
> m_defrag should be totally free of debug code.
>
> Dig in!
>
> Mike "Silby" Silbersack



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