Date: Wed, 24 Jul 2002 08:11:24 -0700 (PDT) From: Bosko Milekic <bmilekic@FreeBSD.org> To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/kern subr_mbuf.c uipc_mbuf.c Message-ID: <200207241511.g6OFBO0V036915@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200207241511.g6OFBO0V036915>