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

_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "[EMAIL PROTECTED]"

Reply via email to