Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 10 Sep 2000 18:11:25 -0700 (PDT)
From:      John Baldwin <jhb@pike.osd.bsdi.com>
To:        Greg Lehey <grog@lemis.com>
Cc:        Mark Murray <markm@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/conf files src/sys/sys random.h src/sys/dev/randomdev hash.c hash.h harvest.c randomdev.c yarrow.c yarro
Message-ID:  <200009110111.SAA29487@pike.osd.bsdi.com>
In-Reply-To: <20000911093457.H15703@wantadilla.lemis.com> from Greg Lehey at "Sep 11, 2000 09:34:57 am"

next in thread | previous in thread | raw e-mail | index | archive | help
Greg Lehey wrote:
> On Sunday, 10 September 2000 at  6:52:19 -0700, Mark Murray wrote:
> > markm       2000/09/10 06:52:19 PDT
> >
> >   Log:
> >   Large upgrade to the entropy device; mainly inspired by feedback
> >   from many folk.
> >
> >   o The reseed process is now a kthread. With SMPng, kthreads are
> >     pre-emptive, so the annoying jerkiness of the mouse is gone.
> >
> >   o The data structures are protected by mutexes now, not splfoo()/splx().
> 
> The last thing I heard was that we were getting worried about putting
> in too many mutexes.  How was this resolved?

IIRC, Mark's code doesn't tsleep() with a mutex, which was one of the
problems that the malloc() mutex patch had.  Although, in my opinion, we'd
probably be better off starting with large subsystem locks as the first
step of the lock pushdown, and then successively push down the locks within
each subsystem and sub-subsystem.  I think trying to add a bunch of small
locks in the beginning will just give us massive amounts of headaches.

> Greg
> --
> Finger grog@lemis.com for PGP public key
> See complete headers for address and phone numbers

-- 

John Baldwin <jhb@bsdi.com> -- http://www.FreeBSD.org/~jhb/
PGP Key: http://www.cslab.vt.edu/~jobaldwi/pgpkey.asc
"Power Users Use the Power to Serve!"  -  http://www.FreeBSD.org/


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200009110111.SAA29487>