Re: lockmgr_sleep() (was Re: ssleep() (was Re: mention msleep() in porting_drivers.txt))

["Simon 'corecode' Schubert" <corecode@xxxxxxxxxxxx>]

Aggelos Economopoulos wrote:
> On Saturday 26 January 2008, Aggelos Economopoulos wrote:
> > On Wednesday 16 January 2008, Aggelos Economopoulos wrote:
> > > On Wednesday 16 January 2008, Matthew Dillon wrote:
> > > >     lockmgr_sleep(ident, lock, slpflags, wmesg, timeo)
> > > >
> > > >     lockmgr_sleep can also figure out what kind of lock is held internally
> > > >     and deal with it, or it can just assume an exclusive lock.  Either way
> > > >     lkflags do not have to be passed to it.
> > > Assuming an exclusive lock is not intuitive IMO and differs from the FreeBSD
> > > semantics (even though it matches our msleep()/spin_sleep()). What do you
> > > think of the lockmgr change suggested in another mail in this thread?
> > >
> > > > :If we're going to rename msleep() to spin_sleep() anyway, I suggest changing
> > > > :the prototype to
> > > > :
> > > > :spin_sleep(void *ident, int flags, const char *wmesg, int timo,
> > > > :	struct spinlock *spin)
> > > >
> > > >     We want all the *sleep() procedures to use a similar prototype,
> > > >     and in this case being compatible with our own history as well
> > > >     as FreeBSD's will reduce confusion to a minimum.  That means putting
> > > >     the lock as the second argument.
> > > >
> > > >     spin_sleep(ident, spin, slpflags, wmesg, timeo)
> > > Nobody can disagree with the "compatible" part, but for a whatever_sleep()
> > > that does a "drop, tsleep() and reacquire whatever lock" it seems more
> > > natural to require the tsleep() args followed by the whatever args. It may
> > > be unlikely, but imagine having to add another flag. Would you put it after
> > > "spin" seperating the tsleep() args further?
> > >
> > > Also, I don't think FreeBSD compatibility is an issue, unless the two choices
> > > are otherwise on par.
> > So what's the verdict? Veto still on? Need to know in order to submit a patch :)
>
> Yet another ping.

Pick one and put it in :)

cheers
  simon

--
Serve - BSD     +++  RENT this banner advert  +++    ASCII Ribbon   /"\
Work - Mac      +++  space for low €€€ NOW!1  +++      Campaign     \ /
Party Enjoy Relax   |   http://dragonflybsd.org      Against  HTML   \
Dude 2c 2 the max   !   http://golden-apple.biz       Mail + News   / \