From owner-cvs-all Fri Jan 3 4:38:31 2003 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 1485037B401; Fri, 3 Jan 2003 04:38:30 -0800 (PST) Received: from mailman.zeta.org.au (mailman.zeta.org.au [203.26.10.16]) by mx1.FreeBSD.org (Postfix) with ESMTP id 9765343EC5; Fri, 3 Jan 2003 04:38:28 -0800 (PST) (envelope-from bde@zeta.org.au) Received: from katana.zip.com.au (katana.zip.com.au [61.8.7.246]) by mailman.zeta.org.au (8.9.3/8.8.7) with ESMTP id XAA26216; Fri, 3 Jan 2003 23:38:20 +1100 Date: Fri, 3 Jan 2003 23:38:29 +1100 (EST) From: Bruce Evans X-X-Sender: bde@gamplex.bde.org To: phk@freebsd.org Cc: cvs-committers@freebsd.org, Subject: Re: cvs commit: src/sys/sys _mutex.h In-Reply-To: <10907.1041590410@critter.freebsd.dk> Message-ID: <20030103230420.W3339-100000@gamplex.bde.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII 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 On Fri, 3 Jan 2003 phk@freebsd.org wrote: > In message <20030103212617.K2888-100000@gamplex.bde.org>, Bruce Evans writes: > >On Thu, 2 Jan 2003, Poul-Henning Kamp wrote: > >> MFC: Conditionalize 16 bytes of struct mutex on MUTEX_PROFILING > > > >Please back this out. It is contentious even in -current. Even its > >misleading comment hasn't been fixed. > > I'll await re@'s decision. It is necessary to prevent -current and > 5.0-RELENG machines from running out of kmem_map space and as I > read John, that takes precedence. Really? It should make more than a few percent difference. E.g., it bloats the size of a vnode from 284 to 300 on i386's (6%). This isn't all that small for a struct that has so many instances, but if we cared about the size of a vnode then we wouldn't be using so many mutexes and locks in it. Mutexes without profiling have size 52 on i386's, so using a mutex instead of a simplelock for v_interlock bloats the size of a vnode from 236 to 284 (20%). > I agree that we should probably fix the comment and remove > MUTEX_PROFILING from conf/options to prevent unintended use. That would make it a non-option. Several other options have similar problems. A quick grep in shows: - the non-option USE_BUFHASH modifies struct buf. - ... - the option DEBUG_LOCKS modifies struct lockmgr and struct vnode. Bruce To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message