Messages in this thread | | | From | Amit Shah <> | Subject | Re: [PATCH] taskqueue to workqueue update for riscom8 driver | Date | Mon, 3 Mar 2003 13:05:22 +0530 |
| |
On Sunday 02 Mar 2003 22:04, Matthew Wilcox wrote: > So it only compiles on UP. Not terribly interesting. > > BTW, I wouldn't necessarily expect it to work. Work queues run in > process context; the code you replaced ran in bottom half context. > If you're going to do this kind of lame hack, it should be converted > to a tasklet, not a work queue.
A simple grep for cli() in drivers/char reveals that many of those drivers still use cli(), which means even they haven't yet been converted to the new framework.
As for the patch, other drivers, like cyclades, for example, too have been modified in the same manner.... on digging, I found that the original patch that converted the taskqueues to workqueues had also done it the same way.
Amit.
-- Amit Shah http://amitshah.nav.to/
The most exciting phrase to hear in science, the one that heralds new discoveries, is not "Eureka!" (I found it!) but "That's funny ..." -- Isaac Asimov
- 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/
| |