From owner-cvs-all Mon Jan 22 10:17:55 2001 Delivered-To: cvs-all@freebsd.org Received: from ringworld.nanolink.com (ringworld.nanolink.com [195.24.48.189]) by hub.freebsd.org (Postfix) with SMTP id E6C0E37B402 for ; Mon, 22 Jan 2001 10:17:28 -0800 (PST) Received: (qmail 12204 invoked by uid 1000); 22 Jan 2001 18:15:59 -0000 Date: Mon, 22 Jan 2001 20:15:59 +0200 From: Peter Pentchev To: Dag-Erling Smorgrav Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/vm vm_zone.c vm_zone.h Message-ID: <20010122201559.B11979@ringworld.oblivion.bg> Mail-Followup-To: Dag-Erling Smorgrav , cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org References: <200101220701.f0M71pv82114@freefall.freebsd.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5i In-Reply-To: <200101220701.f0M71pv82114@freefall.freebsd.org>; from des@FreeBSD.org on Sun, Jan 21, 2001 at 11:01:51PM -0800 Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG Is this the commit that broke usr.bin/vmstat/vmstat.c, in the dozmem() function? It would seem vmstat is still looking for the znext pointer :( G'luck, Peter -- If this sentence didn't exist, somebody would have invented it. On Sun, Jan 21, 2001 at 11:01:51PM -0800, Dag-Erling Smorgrav wrote: > des 2001/01/21 23:01:51 PST > > Modified files: > sys/vm vm_zone.c vm_zone.h > Log: > Give this code a major facelift: > > - replace the simplelock in struct vm_zone with a mutex. > > - use a proper SLIST rather than a hand-rolled job for the zone list. > > - add a subsystem lock that protects the zone list and the statistics > counters. > > - merge _zalloc() into zalloc() and _zfree() into zfree(), and > move them below _zget() so there's no need for a prototype. > > - add two initialization functions: one which initializes the > subsystem mutex and the zone list, and one that currently doesn't > do anything. > > - zap zerror(); use KASSERTs instead. > > - dike out half of sysctl_vm_zone(), which was mostly trying to do > manually what the snprintf() call could do better. > > Reviewed by: jhb, jasone > > Revision Changes Path > 1.38 +151 -222 src/sys/vm/vm_zone.c > 1.16 +16 -13 src/sys/vm/vm_zone.h To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message