On Fri, Dec 30, 2011 at 03:08:19PM +0530, Amit Shah wrote:

quoted

quoted

+	if (!try_fill_recv(vi, GFP_KERNEL))
+		schedule_delayed_work(&vi->refill, 0);

This needs to be switched to non reentrant wq too?

Subsequent invocations could create problems?  Note that this will be
the first item to be queued in the workqueue for the refill work.

		Amit

See recent patches on net-next.
We switched other calls to a non reentrant wq so I think same
reasoning applies there.