From owner-cvs-all Wed Jun 14 10:11:51 2000 Delivered-To: cvs-all@freebsd.org Received: from freefall.freebsd.org (freefall.FreeBSD.ORG [204.216.27.21]) by hub.freebsd.org (Postfix) with ESMTP id 3A86237B9D9; Wed, 14 Jun 2000 10:11:49 -0700 (PDT) (envelope-from bde@FreeBSD.org) Received: (from bde@localhost) by freefall.freebsd.org (8.9.3/8.9.2) id KAA97091; Wed, 14 Jun 2000 10:11:48 -0700 (PDT) (envelope-from bde@FreeBSD.org) Message-Id: <200006141711.KAA97091@freefall.freebsd.org> From: Bruce Evans Date: Wed, 14 Jun 2000 10:11:48 -0700 (PDT) To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/sys malloc.h Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG bde 2000/06/14 10:11:48 PDT Modified files: sys/sys malloc.h Log: Removed support for generating inline code for MALLOC() and FREE() in the dysfunctional !KMEMSTATS case. This hasn't compiled since rev.1.31 of kern_malloc.c quietly removed the core of the support for the !KMEMSTATS case. I fixed it to see if it was worth saving and found that (as usual) inlining just wasted space and increased complexity without significantly affecting time, at least for the lmbench2 micro-benchmark on a Celeron. The space bloat was surprisingly large - the text size increased from 1700K to 1840K for a version with the entire malloc() family inlined. Removed even older garbage (kmemxtob() and btokmemx() macros). Attempt to deprecate MALLOC() and FREE(). Given current compilers (gcc-2.x or C99), they don't do anything that (safe) function-like macros or inline functions named malloc() and free() couldn't do. Fixed missing casts of macro args in MALLOC() and FREE(). Revision Changes Path 1.49 +5 -46 src/sys/sys/malloc.h To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message