From owner-cvs-all Wed Jul 24 8:11:32 2002 Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.FreeBSD.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 043F337B400; Wed, 24 Jul 2002 08:11:25 -0700 (PDT) Received: from freefall.freebsd.org (freefall.FreeBSD.org [216.136.204.21]) by mx1.FreeBSD.org (Postfix) with ESMTP id C05A243E75; Wed, 24 Jul 2002 08:11:24 -0700 (PDT) (envelope-from bmilekic@FreeBSD.org) Received: from freefall.freebsd.org (bmilekic@localhost [127.0.0.1]) by freefall.freebsd.org (8.12.4/8.12.4) with ESMTP id g6OFBOJU036916; Wed, 24 Jul 2002 08:11:24 -0700 (PDT) (envelope-from bmilekic@freefall.freebsd.org) Received: (from bmilekic@localhost) by freefall.freebsd.org (8.12.4/8.12.4/Submit) id g6OFBO0V036915; Wed, 24 Jul 2002 08:11:24 -0700 (PDT) Message-Id: <200207241511.g6OFBO0V036915@freefall.freebsd.org> From: Bosko Milekic Date: Wed, 24 Jul 2002 08:11:24 -0700 (PDT) To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/kern subr_mbuf.c uipc_mbuf.c X-FreeBSD-CVS-Branch: HEAD Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG bmilekic 2002/07/24 08:11:24 PDT Modified files: sys/kern subr_mbuf.c uipc_mbuf.c Log: Move m_freem() from uipc_mbuf.c to subr_mbuf.c so it can take advantage of the inlines, like its cousin, m_free(). Also, make a small (first step?) optimisation of m_free() to use the MBP_PERSIST{,ENT} interface to hold the lock across frees when possible. The thing is that right now, we can only do this easily for at most across one mbuf + one cluster free, as the comment mentions (it also explains why). Anyway, some basic tests revealed a 5-10% overall improvement. Some of the results can be found here: http://people.freebsd.org/~bmilekic/code/measure.txt Revision Changes Path 1.23 +48 -0 src/sys/kern/subr_mbuf.c 1.94 +0 -8 src/sys/kern/uipc_mbuf.c To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message