From owner-svn-src-head@FreeBSD.ORG Wed Mar 3 22:06:40 2010 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 67F271065677; Wed, 3 Mar 2010 22:06:40 +0000 (UTC) (envelope-from jilles@stack.nl) Received: from mx1.stack.nl (relay02.stack.nl [IPv6:2001:610:1108:5010::104]) by mx1.freebsd.org (Postfix) with ESMTP id 2C6648FC23; Wed, 3 Mar 2010 22:06:40 +0000 (UTC) Received: from turtle.stack.nl (turtle.stack.nl [IPv6:2001:610:1108:5010::132]) by mx1.stack.nl (Postfix) with ESMTP id 3791C35A831; Wed, 3 Mar 2010 23:06:39 +0100 (CET) Received: by turtle.stack.nl (Postfix, from userid 1677) id 2A3D517B48; Wed, 3 Mar 2010 23:06:39 +0100 (CET) Date: Wed, 3 Mar 2010 23:06:39 +0100 From: Jilles Tjoelker To: Kostik Belousov Message-ID: <20100303220639.GA52066@stack.nl> References: <201002271532.o1RFWnCp099462@svn.freebsd.org> <20100227220854.GB77656@stack.nl> <20100228111508.GN2489@deviant.kiev.zoral.com.ua> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20100228111508.GN2489@deviant.kiev.zoral.com.ua> User-Agent: Mutt/1.5.20 (2009-06-14) Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r204413 - head/sys/kern X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 03 Mar 2010 22:06:40 -0000 On Sun, Feb 28, 2010 at 01:15:08PM +0200, Kostik Belousov wrote: > On Sat, Feb 27, 2010 at 11:08:54PM +0100, Jilles Tjoelker wrote: > > On Sat, Feb 27, 2010 at 03:32:49PM +0000, Konstantin Belousov wrote: > > > Author: kib > > > Date: Sat Feb 27 15:32:49 2010 > > > New Revision: 204413 > > > URL: http://svn.freebsd.org/changeset/base/204413 > > > Log: > > > For kinfo_proc in kp->ki_siglist, return the set of the signals pending > > > in the process queue when gathering information for the process, and set > > > of signals pending for the thread, when gathering information for the > > > thread. Previously, the sysctl returned a union of the process and some > > > arbitrary thread pending set for the process, and union of the process > > > and the thread pending set for the thread. > > Although the new way provides maximum information and the old way was > > definitely broken for processes, I think the new way may not be what I > > expect. In particular, 'ps O pending' and 'ps HO pending' now give > > (usually) disjunct answers, even for single-threaded processes. I > > suppose these different answers can be useful for kernel debugging, but > > it should be documented. > Not only for the kernel debugging. Being able to see a pending signal in > the process queue means that signal delivery for the process is stopped. > Change provides a capability to start analyze such situation without > resorting to the kernel debugger. > More, I do not consider the change to be significant enough from the > interface stability point of view, thus planning to merge it to 8. > Where do you suggest to document the behaviour ? ps(1) ? Yes. Unfortunately the signal keywords are pretty poorly documented. > > Somewhat related, ki_sigmask could be the logical AND of all threads' > > td_sigmask when gathering information for the process, instead of the > > td_sigmask of the most recently created thread; fill_kinfo_aggregate() > > could handle this. > ki_sigmask arguably has no meaning in the process context. Do you > propose this to simplify handling of a single-threaded process ? To make non-threaded output of multi-threaded processes slightly more consistent. It is not that important, also given apparent plans to show signal information in procstat(1) which will be easier to use. -- Jilles Tjoelker