From owner-cvs-all Fri Sep 21 18:26:33 2001 Delivered-To: cvs-all@freebsd.org Received: from mail6.speakeasy.net (mail6.speakeasy.net [216.254.0.206]) by hub.freebsd.org (Postfix) with ESMTP id A83CC37B40E for ; Fri, 21 Sep 2001 18:26:25 -0700 (PDT) Received: (qmail 33036 invoked from network); 22 Sep 2001 01:26:25 -0000 Received: from unknown (HELO laptop.baldwin.cx) ([64.81.54.73]) (envelope-sender ) by mail6.speakeasy.net (qmail-ldap-1.03) with SMTP for ; 22 Sep 2001 01:26:25 -0000 Message-ID: X-Mailer: XFMail 1.4.0 on FreeBSD X-Priority: 3 (Normal) Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 8bit MIME-Version: 1.0 In-Reply-To: Date: Fri, 21 Sep 2001 18:26:11 -0700 (PDT) From: John Baldwin To: Julian Elischer Subject: Re: cvs commit: src/sys/kern sys_generic.c Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org 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 22-Sep-01 Julian Elischer wrote: > You are right.. I meant p_sflag, > and in that context, The signal flag did move. > It is however the only one.. luckily.. > > It does suggest to me that maybe it was mislocated origially :-) > > I never 100% understood that flag... Nah, what determines a per-thrad flag and what determines a scheduler flag aren't necessarily the same thing, so it was right for it to be in p_flag (it's not scheduler related very much) and it was right for it to be in td_flag (definitely a thread property). I think that TDF_SELECT basically is used so that selrecord() won't try to wakeup a process during the active scan in select() and poll() and after we finish the scan we check that flag to see if any events were found that should trigger an immediate return. I'll tackle that flag when I get to it in my second sweep of proc locking. It will probably stay as it is, but if I always lock the proc as well as sched_lock when writing to the flag, than I can get away with only locking the proc when testing that flag to avoid sched_lock in a few cases. -- John Baldwin -- http://www.FreeBSD.org/~jhb/ PGP Key: http://www.baldwin.cx/~john/pgpkey.asc "Power Users Use the Power to Serve!" - http://www.FreeBSD.org/ To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message