Re: [BK FBDEV] A few more updates.
From: James Simmons <hidden>
Date: 2003-03-25 20:03:07
Also in:
lkml
quoted
quoted
Well, actually, creating a workqueue would be overhead since it involves one kernel thread per CPU. After more thinking & discussion, I beleive you shall rather use keventd existing workqueue (schedule_work() will do that)Done. Can you look over this patch and test it. I tested it and it worked fine.I don't have a test config at hand right now. The patch looks better, though you didn't remove the spinlock and replace it with some "softer" sync. primitives.
I didn't get around to removing the spinlock. That is next on the list. I just wanted to fix the big problem.
Note that if fbcon is ever to be rmmod'ed, you need to properly remove the timer and make sure all pending work queues have completed (and make sure the timer won't be re-scheduled by one).
Note support for rmmod fbcon is incomplete. The function giveup_console is really sad but none one ever imagine that the console system would become powerful enough to switch from one driver to another.