From owner-freebsd-current@FreeBSD.ORG Thu Apr 17 17:19:58 2003 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id A904537B401 for ; Thu, 17 Apr 2003 17:19:58 -0700 (PDT) Received: from sccrmhc02.attbi.com (sccrmhc02.attbi.com [204.127.202.62]) by mx1.FreeBSD.org (Postfix) with ESMTP id E583543FA3 for ; Thu, 17 Apr 2003 17:19:57 -0700 (PDT) (envelope-from julian@elischer.org) Received: from interjet.elischer.org (12-232-168-4.client.attbi.com[12.232.168.4]) by sccrmhc02.attbi.com (sccrmhc02) with ESMTP id <200304180019560020028h0ue>; Fri, 18 Apr 2003 00:19:56 +0000 Received: from localhost (localhost.elischer.org [127.0.0.1]) by InterJet.elischer.org (8.9.1a/8.9.1) with ESMTP id RAA56429; Thu, 17 Apr 2003 17:19:54 -0700 (PDT) Date: Thu, 17 Apr 2003 17:19:51 -0700 (PDT) From: Julian Elischer To: "Andrew R. Reiter" In-Reply-To: <20030417200446.N54973@fledge.watson.org> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII cc: Jeff Roberson cc: FreeBSD current users Subject: Re: some small patches X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 18 Apr 2003 00:19:58 -0000 On Thu, 17 Apr 2003, Andrew R. Reiter wrote: > On Thu, 17 Apr 2003, Julian Elischer wrote: > > : > :Here are two small patches they are pretty non-controversial in my > :opinion. > : > :they are part of a bigger patch, but I'd like to get them in separatly > :to simplify the bigger one. > :the first patch: > :http://www.freebsd.org/~julian/IDLETD.diff > :moves the flag that identifies a thread as being one of the idle threads > :from the KSE to the thread. The code that wants to know already > :has a thread pointer, but not the KSE pointer so this makes more sense. > > Is this correct? > > Index: sys/proc.h > =================================================================== > RCS file: /repos/projects/mirrored/freebsd/src/sys/sys/proc.h,v > retrieving revision 1.313 > diff -u -r1.313 proc.h > --- sys/proc.h 2003/04/13 21:29:11 1.313 > +++ sys/proc.h 2003/04/17 22:52:07 > @@ -349,6 +323,7 @@ > #define TDF_CAN_UNBIND 0x000004 /* Only temporarily bound. */ > #define TDF_SINTR 0x000008 /* Sleep is interruptible. */ > #define TDF_TIMEOUT 0x000010 /* Timing out during sleep. */ > +#define TDF_IDLETD 0x000040 /* This is an idle thread */ > #define TDF_SELECT 0x000040 /* Selecting; wakeup/waiting > danger. */ > #define TDF_CVWAITQ 0x000080 /* Thread is on a cv_waitq (not > slpq). */ > #define TDF_UPCALLING 0x000100 /* This thread is doing an > upcall. */ > > > Both TDF_IDLETD and TD_SELECT have the same value. uh, no, TDF_IDLETD should be 0x20 I copied that line by hand from my test system because the diff from there is different (there are more changes frm which this was extracted) and flubbed it.. good catch. (That's why we do reviews right?) :-) > > Just curious. > > Cheers, > Andrew > > > : > : > :The second patch: > :http://www.freebsd.org/~julian/sched_clock.diff > :makes the sched_ API entrypoint sched_clock() > :take a thread argument instead of a KSE. > :Once again, the callers have the thread pointer and not the KSE pointer, > :and in fact they probably should not have the KSE pointer. > : > : > :anyone object to these patches? > : > : > :_______________________________________________ > :freebsd-current@freebsd.org mailing list > :http://lists.freebsd.org/mailman/listinfo/freebsd-current > :To unsubscribe, send any mail to "freebsd-current-unsubscribe@freebsd.org" > : > > -- > Andrew R. Reiter > arr@watson.org > arr@FreeBSD.org >