Subject: Re: configure() with valid context
To: Jason Thorpe <thorpej@nas.nasa.gov>
From: Eduardo E. Horvath <eeh@one-o.com>
List: tech-kern
Date: 09/16/1999 15:07:27
On Thu, 16 Sep 1999, Jason Thorpe wrote:
> On Thu, 16 Sep 1999 09:53:26 +0200
> Ignatios Souvatzis <ignatios@cs.uni-bonn.de> wrote:
>
> > > initclocks();
> > > +
> > > + /* Configure the system hardware. This will enable interrupts. */
> > > + configure();
> > >
> > > #ifdef SYSVSHM
> > > /* Initialize System V style shared memory. */
> > >
> > > ...essentially, this allows autoconfiguration to run in a valid thread
> > > context.
> > >
> > > Objections?
> >
> > I don't remember... where are the battery backed clocks read?
> > After autoconfiguration?
>
> Yes, actually, I realized there is a slight bug in my patch when I tried to
> boot it on an Alpha :-)
>
> I've since moved the initclocks() call into configure(), immediately after
> cpu_configure() returns. So, those semantics are now preserved.
Looks more reasonable now. I don't think it should cause any great
problems.
=========================================================================
Eduardo Horvath eeh@one-o.com
"I need to find a pithy new quote." -- me