From owner-cvs-all@FreeBSD.ORG  Mon Jun  4 16:26:00 2007
Return-Path: <owner-cvs-all@FreeBSD.ORG>
X-Original-To: cvs-all@FreeBSD.org
Delivered-To: cvs-all@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52])
	by hub.freebsd.org (Postfix) with ESMTP id E707B16A46B;
	Mon,  4 Jun 2007 16:26:00 +0000 (UTC) (envelope-from sam@errno.com)
Received: from ebb.errno.com (ebb.errno.com [69.12.149.25])
	by mx1.freebsd.org (Postfix) with ESMTP id 62DCD13C465;
	Mon,  4 Jun 2007 16:26:00 +0000 (UTC) (envelope-from sam@errno.com)
Received: from trouble.errno.com (trouble.errno.com [10.0.0.248])
	(authenticated bits=0)
	by ebb.errno.com (8.13.6/8.12.6) with ESMTP id l54GPwXZ020218
	(version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO);
	Mon, 4 Jun 2007 09:25:59 -0700 (PDT) (envelope-from sam@errno.com)
Message-ID: <46643E93.1000009@errno.com>
Date: Mon, 04 Jun 2007 09:32:19 -0700
From: Sam Leffler <sam@errno.com>
User-Agent: Thunderbird 2.0.0.0 (X11/20070530)
MIME-Version: 1.0
To: Pyun YongHyeon <yongari@FreeBSD.org>
References: <200706040601.l54615lN069185@repoman.freebsd.org>
In-Reply-To: <200706040601.l54615lN069185@repoman.freebsd.org>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject: Re: cvs commit: src/sys/dev/gem if_gem.c if_gemreg.h if_gemvar.h
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Mon, 04 Jun 2007 16:26:01 -0000

Pyun YongHyeon wrote:

>   o Borrowed gem_defrag() from ath(4) which is supposed to be much
>     faster than m_defrag(9) since it's not need to defrag all
>     mbuf chains.

We need to replace m_defrag w/ something that is not a performance 
bottleneck and has a sane calling api.  It's been discussed in the past 
but been dropped (multiple times).  The main issue is that the current 
m_defrag code is used for 2 totally different purposes and untangling 
them is a bunch of work.

	Sam