Subject: Re: sobj_changepri (Re: CVS commit: [newlock2] src/sys)
To: None <ad@netbsd.org>
From: YAMAMOTO Takashi <yamt@mwd.biglobe.ne.jp>
List: tech-kern
Date: 02/23/2007 12:37:10
> > On Thu, Feb 22, 2007 at 08:13:29PM +0900, YAMAMOTO Takashi wrote:
> > 
> > > > Log Message:
> > > > - Change syncobj_t::sobj_changepri() to alter both the user priority and
> > > >   the effective priority of LWPs. How the effective priority is adjusted
> > > >   depends on the type of object.
> > > 
> > > i'm not sure if it was a good idea.
> > > priority inheritance needs a way to change the effective priority only.
> > 
> > At the time, it seemed like a reasonable accommodation to make so we can
> > handle the different behaviour of tsleep and condition variables. I figured
> > we would need another method like sobj_lendpri later on. If you can see a
> > better way of doing it, please change it.
> 
> after some more thoughts, i've changed my mind.
> it's cleaner to use another dedicated priority member in struct lwp.
> (see the attached patch.)

i like the name _lendpri.  i'll rename _inheritpri in the patch to it.

btw, do you care if i remove mutex_owner(), which seems unused?
(see "XXX naming conflict" in the patch.)

YAMAMOTO Takashi