Re: [PATCH] taskqueue to workqueue update for riscom8 driver

Amit Shah (shahamit@gmx.net)
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/