From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751399AbWCDGtD (ORCPT ); Sat, 4 Mar 2006 01:49:03 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751400AbWCDGtD (ORCPT ); Sat, 4 Mar 2006 01:49:03 -0500 Received: from mail.gmx.net ([213.165.64.20]:46788 "HELO mail.gmx.net") by vger.kernel.org with SMTP id S1751399AbWCDGtC (ORCPT ); Sat, 4 Mar 2006 01:49:02 -0500 X-Authenticated: #14349625 Subject: Re: [patch 2.6.16-rc5-mm2] sched_cleanup-V17 - task throttling patch 1 of 2 From: Mike Galbraith To: Con Kolivas Cc: "Randy.Dunlap" , pwil3058@bigpond.net.au, linux-kernel@vger.kernel.org, mingo@elte.hu, nickpiggin@yahoo.com.au, kenneth.w.chen@intel.com, akpm@osdl.org In-Reply-To: <200603041654.59480.kernel@kolivas.org> References: <1140183903.14128.77.camel@homer> <1141450187.7703.40.camel@homer> <20060303214002.f36ce0b4.rdunlap@xenotime.net> <200603041654.59480.kernel@kolivas.org> Content-Type: text/plain Date: Sat, 04 Mar 2006 07:50:48 +0100 Message-Id: <1141455048.9482.13.camel@homer> Mime-Version: 1.0 X-Mailer: Evolution 2.4.0 Content-Transfer-Encoding: 7bit X-Y-GMX-Trusted: 0 Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Sat, 2006-03-04 at 16:54 +1100, Con Kolivas wrote: > On Saturday 04 March 2006 16:40, Randy.Dunlap wrote: > > On Sat, 04 Mar 2006 06:29:47 +0100 Mike Galbraith wrote: > > > On Sat, 2006-03-04 at 16:24 +1100, Con Kolivas wrote: > > > > On Saturday 04 March 2006 16:20, Mike Galbraith wrote: > > > > > On Sat, 2006-03-04 at 13:33 +1100, Peter Williams wrote: > > > > > > > include/linux/sched.h | 3 - > > > > > > > kernel/sched.c | 136 > > > > > > > +++++++++++++++++++++++++++++--------------------- 2 files > > > > > > > changed, 82 insertions(+), 57 deletions(-) > > > > > > > > > > > > > > --- linux-2.6.16-rc5-mm2/include/linux/sched.h.org 2006-03-01 > > > > > > > 15:06:22.000000000 +0100 +++ > > > > > > > linux-2.6.16-rc5-mm2/include/linux/sched.h 2006-03-02 > > > > > > > 08:33:12.000000000 +0100 @@ -720,7 +720,8 @@ > > > > > > > > > > > > > > unsigned long policy; > > > > > > > cpumask_t cpus_allowed; > > > > > > > - unsigned int time_slice, first_time_slice; > > > > > > > + int time_slice; > > > > > > > > > > > > Can you guarantee that int is big enough to hold a time slice in > > > > > > nanoseconds on all systems? I think that you'll need more than 16 > > > > > > bits. > > > > > > > > > > Nope, that's a big fat bug. > > > > > > > > Most ints are 32bit anyway, but even a 32 bit unsigned int overflows > > > > with nanoseconds at 4.2 seconds. A signed one at about half that. Our > > > > timeslices are never that large, but then int isn't always 32bit > > > > either. > > > > > > Yup. I just didn't realize that there were 16 bit integers out there. > > > > LDD 3rd ed. doesn't know about them either. Same for me. > > Alright I made that up, but it might not be one day :P Well Fudgecicles. Now you guys have gotten me aaaaall confused. Are there cpus out there (in generic linux land) that have 16 bit integers or not? 16 bit integers existing in a 32 bit cpu OS seems like an alien concept to me, but I'm not a twisted cpu designer... I'll just go with the flow ;-) -Mike