From owner-freebsd-hackers@FreeBSD.ORG Wed Nov 28 12:03:22 2007 Return-Path: Delivered-To: hackers@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 307B416A46D for ; Wed, 28 Nov 2007 12:03:22 +0000 (UTC) (envelope-from skip@menantico.com) Received: from vms173001pub.verizon.net (vms173001pub.verizon.net [206.46.173.1]) by mx1.freebsd.org (Postfix) with ESMTP id 109FC13C45B for ; Wed, 28 Nov 2007 12:03:22 +0000 (UTC) (envelope-from skip@menantico.com) Received: from mx.menantico.com ([71.188.11.206]) by vms173001.mailsrvcs.net (Sun Java System Messaging Server 6.2-6.01 (built Apr 3 2006)) with ESMTPA id <0JS7004FWQ9XMCB0@vms173001.mailsrvcs.net>; Wed, 28 Nov 2007 04:53:58 -0600 (CST) Date: Wed, 28 Nov 2007 06:05:50 -0500 From: Skip Ford In-reply-to: <20071128092434.J94692@fledge.watson.org> To: Robert Watson Message-id: <20071128110550.GA2216@menantico.com> MIME-version: 1.0 Content-type: text/plain; charset=us-ascii Content-disposition: inline References: <20071127171228.N94692@fledge.watson.org> <20071128054208.GA813@menantico.com> <20071128092434.J94692@fledge.watson.org> User-Agent: Mutt/1.4.2.3i Cc: hackers@FreeBSD.org Subject: Re: Updated procstat(1) X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 28 Nov 2007 12:03:22 -0000 Robert Watson wrote: > On Wed, 28 Nov 2007, Skip Ford wrote: > >>>- "-a" now means "all processes", >> >>Thanks. :-) I'm a little surprised. You seemed pretty dedicated to a >>per-process tool. > > I was, but then I read your e-mail and became convinced that the first > patch that would be submitted against procstat(1) would be a "-a" patch. :-) Yep, would've happened. Now the first patch submitted will be a "-w interval" patch... :-) >>>- A new "-k" has been added, which prints the kernel thread stacks for >>>threads >>> in a process (although not swapped out or actively running threads). >>> This >>> is extremely useful for answering questions of the sort "But *why* is >>> the >>> process blocked in UMA". It has both a simple mode (-k_, which lists >>> just >>> kernel function names, and a slightly more detailed mode (-kk), which >>> adds >>> the offset into the function. >> >>This is excellent. Does this absolutely have to depend on DDB and KDB? > > Currently, yes, as stack(9) is conditional on DDB, and the MD bits of > stack(9) are defined in db_trace.c (and in some cases, depend on DDB > definitions, such as DDB types, although I think not critically so). I've > also been pondering breaking out stack(9) from DDB but haven't done that > yet. Maybe that will be today's task, as I'd like -k to work without the > kernel debugger, as it has use significantly beyond kernel debugging. That'd be great if it worked without DDB. It just "feels" like it should. This tool is a very nice addition. Thanks for writing it and for asking for feedback, then putting up with the responses. -- Skip