Subject: Re: yamt-idlelwp: setting curlwp in MI code.
To: Andrew Doran <ad@netbsd.org>
From: Matt Thomas <matt@3am-software.com>
List: tech-kern
Date: 04/05/2007 15:44:45
Andrew Doran wrote:
> YAMAMOTO Takashi wrote:
> 
>> how about having set_curlwp() which can be overridden by MD definition?
> 
> Did you envision a new place where it would be used?
> 
> On Thu, Apr 05, 2007 at 03:13:37PM -0700, Matt Thomas wrote:
> 
>> While I think the goal of setting curlwp in MI code is laudable, it's 
>> incomplete.  For PowerPC, I also need to set "curpcb" and "curpm" (which
>> I currently do in cpu_switchto).
> 
> Without looking at the ppc code, elsewhere curpcb is obsolete since the
> seperate idle PCB disappears.

ppc wants curpcb for copy*.  It could get it from curlwp but it doesn't do
that right now.

> I think there should be only two places curlwp needs to be set: before
> main() is called and when switching to another LWP. Personally, I think
> it's nicer to set them in the MD code.

I don't care.  But the change to i386 should have been discussed on a list
so that other MD code owners would know to update their code.