Messages in this thread | | | Subject | Re: [BK FBDEV] A few more updates. | From | Benjamin Herrenschmidt <> | Date | 25 Mar 2003 21:10:26 +0100 |
| |
On Tue, 2003-03-25 at 20:48, James Simmons wrote:
> > 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.
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).
Ben.
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |