From owner-cvs-all@FreeBSD.ORG Mon Sep 13 17:48:19 2004 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id D498316A4CE; Mon, 13 Sep 2004 17:48:19 +0000 (GMT) Received: from mail.vicor-nb.com (bigwoop.vicor-nb.com [208.206.78.2]) by mx1.FreeBSD.org (Postfix) with ESMTP id 1ED1443D1D; Mon, 13 Sep 2004 17:48:19 +0000 (GMT) (envelope-from julian@elischer.org) Received: from elischer.org (julian.vicor-nb.com [208.206.78.97]) by mail.vicor-nb.com (Postfix) with ESMTP id C91317A3D2; Mon, 13 Sep 2004 10:48:18 -0700 (PDT) Message-ID: <4145DD62.8010203@elischer.org> Date: Mon, 13 Sep 2004 10:48:18 -0700 From: Julian Elischer User-Agent: Mozilla/5.0 (X11; U; FreeBSD i386; en-US; rv:1.3.1) Gecko/20030516 X-Accept-Language: en, hu MIME-Version: 1.0 To: John Baldwin References: <200409111007.i8BA7MeA090701@repoman.freebsd.org> <200409131135.48194.jhb@FreeBSD.org> In-Reply-To: <200409131135.48194.jhb@FreeBSD.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: Scott Long cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/i386/i386 pmap.c src/sys/kern sched_4bsd.c sched_ule.c src/sys/sys proc.h sched.h X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 13 Sep 2004 17:48:20 -0000 John Baldwin wrote: >On Saturday 11 September 2004 06:07 am, Scott Long wrote: > >>scottl 2004-09-11 10:07:22 UTC >> >> FreeBSD src repository >> >> Modified files: >> sys/i386/i386 pmap.c >> sys/kern sched_4bsd.c sched_ule.c >> sys/sys proc.h sched.h >> Log: >> Revert the previous round of changes to td_pinned. The scheduler isn't >> fully initialed when the pmap layer tries to call sched_pini() early in >>the boot and results in an quick panic. Use ke_pinned instead as was >> td_pinned >> >>originally done with Tor's patch. >> >> Approved by: julian >> > >Thanks, I think td_pinned belongs in struct thread just as td_critnest. Not >all parts of the scheduler purely live in sched_foo.c. Arguably things sleep >queues are also part of the scheduler since they deal with queues of threads >and manipulating thread state machines, so I think that while some things >should be private to sched_foo, not everything has to be completely private. > Actually I think that the pinned stuff should move to either proc.h or to smp.h sleep queues for example are not private, and do have some connection as you said to the scheduler, but are not in sched.h. It is really part of the world in which a scheduler needs to exist rather than part of the sceduler interface.