From owner-cvs-all Wed Feb 20 21:53:11 2002 Delivered-To: cvs-all@freebsd.org Received: from freefall.freebsd.org (freefall.FreeBSD.org [216.136.204.21]) by hub.freebsd.org (Postfix) with ESMTP id CF15237B41E; Wed, 20 Feb 2002 21:52:49 -0800 (PST) Received: (from bde@localhost) by freefall.freebsd.org (8.11.6/8.11.6) id g1L5qn185235; Wed, 20 Feb 2002 21:52:49 -0800 (PST) (envelope-from bde) Message-Id: <200202210552.g1L5qn185235@freefall.freebsd.org> From: Bruce Evans Date: Wed, 20 Feb 2002 21:52:49 -0800 (PST) To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/sys gmon.h src/usr.sbin/kgmon kgmon.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 bde 2002/02/20 21:52:49 PST Modified files: sys/sys gmon.h usr.sbin/kgmon kgmon.c Log: Reserved one of the spare fields in struct gmon to record the history counter type, as threatened in rev.1.8 (the density doesn't need to be recorded since it can be derived from other fields). This doesn't affect binary compatibility, but new utilities won't be able to depend on the contents of this field because libc/gmon/gmon.c was broken -- it wrote garbage to the spare fields. Added a history counter type field to struct gmonparam. This breaks binary compatibility a little, since kgmon wanted to read the whole struct. Fixed kgmon to only depend on reading the critical earlier parts of the struct. This should also fix 6+ year old breakage of binary compatibility when the profrate field was added. Only initialize the new field in struct gmon for now, so that the compatibility code for this (in kgmon) gets tested. The compatibility code has to guesstimate the value. The new field in struct gmonparam is for the kernel to initialize so that kgmon doesn't have to guess. Revision Changes Path 1.16 +3 -2 src/sys/sys/gmon.h 1.10 +31 -3 src/usr.sbin/kgmon/kgmon.c To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message