From owner-freebsd-arch@FreeBSD.ORG Tue Sep 18 08:26:05 2007 Return-Path: Delivered-To: arch@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id B50FB16A417 for ; Tue, 18 Sep 2007 08:26:05 +0000 (UTC) (envelope-from jroberson@chesapeake.net) Received: from webaccess-cl.virtdom.com (webaccess-cl.virtdom.com [216.240.101.25]) by mx1.freebsd.org (Postfix) with ESMTP id 68C5A13C491 for ; Tue, 18 Sep 2007 08:26:05 +0000 (UTC) (envelope-from jroberson@chesapeake.net) Received: from [192.168.1.103] (c-67-160-44-208.hsd1.wa.comcast.net [67.160.44.208]) (authenticated bits=0) by webaccess-cl.virtdom.com (8.13.6/8.13.6) with ESMTP id l8I8PXC4049976 (version=TLSv1/SSLv3 cipher=DHE-DSS-AES256-SHA bits=256 verify=NO); Tue, 18 Sep 2007 04:25:35 -0400 (EDT) (envelope-from jroberson@chesapeake.net) Date: Tue, 18 Sep 2007 01:28:27 -0700 (PDT) From: Jeff Roberson X-X-Sender: jroberson@10.0.0.1 To: Julian Elischer In-Reply-To: <46EF644E.9050207@elischer.org> Message-ID: <20070918012555.G558@10.0.0.1> References: <20070917165657.B558@10.0.0.1> <46EF644E.9050207@elischer.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Cc: arch@freebsd.org Subject: Re: Changing p_swtime and td_slptime to ticks X-BeenThere: freebsd-arch@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussion related to FreeBSD architecture List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 18 Sep 2007 08:26:05 -0000 On Mon, 17 Sep 2007, Julian Elischer wrote: > Jeff Roberson wrote: >> Enclosed is a patch that fixes swapping with ULE. ULE has never properly >> set p_swtime and td_slptime which are used by the swapout/swapin code to >> select the appropriate thread to swap. > > I have not looked at in the depth required, but 2 points that I was unable > to check to my satisfaction before I got called away for work.... > > 1/ the source of the ticks is a monotonically increasing count that never > goes backwards or changes? ticks is incremented each time hardclock() is called. That's it. > > 2/ nothing that used to be accounted in seconds becomes accounted for in > ticks? I scale back to seconds where it is required. Really I think ticks would be the better metric in vm_glue.c but didn't want to make any drastic changes. Thanks, Jeff > > >> >> In 4BSD these two variables are increment once per-second as schedcpu() >> iterates over all threads. ULE does not have a once per-second loop >> iterating over all threads. So I have changed p_swtime to p_swtick and >> td_slptime to td_slptick. These record the value of 'ticks' when the >> thread slept or was last swapped in or out. >> >> For backwards compatibility I leave the values in kinfo_proc with the >> legacy meaning by subtracting from ticks and dividing by hz. I perform a >> similar transformation in the swapout code to convert to seconds. This >> change does make it possible to use sub-second granular decisions in the >> swap code, however I'm not sure if that's really necessary. >> >> So that I did not disturb the 4BSD mechanism I kept the original td_slptime >> in the td_sched area. It should be possible to use td_slptick directly but >> especially this close to release I did not want to change 4BSD. >> >> Feedback and testing welcome. >> >> Thanks, >> Jeff >> >> >> ------------------------------------------------------------------------ >> >> _______________________________________________ >> freebsd-arch@freebsd.org mailing list >> http://lists.freebsd.org/mailman/listinfo/freebsd-arch >> To unsubscribe, send any mail to "freebsd-arch-unsubscribe@freebsd.org" >