Re: [Patch]net/xfrm/xfrm_policy.c: replace timer with delayed_work
From: Herbert Xu <herbert@gondor.apana.org.au>
Date: 2008-05-01 01:11:48
Also in:
lkml
On Thu, May 01, 2008 at 12:06:55AM +0800, WANG Cong wrote:
Hi, Herbert. I referenced the book Linux Device Drivers, it is said that the work function in workqueue can sleep, that is to say, it only works in process context, so we don't need the disable the lock in softirq, which is interrupt context, right?
Because it can sleep, you must disable BH for locks that can be taken on the softirq path as otherwise a softirq can come in, try to take the lock again and dead-lock. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmV>HI~} [off-list ref] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt