cvs commit: src/sys/geom/eli g_eli.c

Scott Long scottl at samsco.org
Mon Jan 29 19:11:02 UTC 2007


Pawel Jakub Dawidek wrote:
> On Mon, Jan 29, 2007 at 10:07:24AM -0800, Nate Lawson wrote:
>> Pawel Jakub Dawidek wrote:
>>> Hmm, I thought that cold is zeroed before smp_started is set?
>> I don't think that's guaranteed.  Besides, there's an easier way to fix your problem.  Instead of calling kthread_create() from your geom create routine directly, queue a 
>> stub request on the thread taskqueue to call kthread_create().
>>
>> For example, see sys/dev/acpica/acpi_acad.c.  We call this from attach()    and then the system thread only runs and then calls acpi_acad_init() once all CPUs are 
>> initialized.  See sys/sys/taskqueue.h for how to add a task to the system thread taskqueue:
>>
>> taskqueue_enqueue(taskqueue_thread, &task);
> 
> If we want to fix it better, I think GEOM should start (tasting at
> least) after all CPUs are online.
> 

Why?  You're proposing yet another intrusive change to the kernel to
handle yet another one-off requirement of your code.  Why not do what I
suggested before with hooking the appropriate SYSINIT in your module?
Or why not follow Robert's suggestion and implement a simple event
mechanism so that any module can know when a CPU has come online or
offline.  Heck, you probably don't even need to implement a new
mechanism, just hook the existing EVENTHANLER mechanism.  That's what
it's designed for!!

Scott



More information about the cvs-src mailing list