Thread (14 messages) flat view 14 messages, 4 authors, 2012-03-11

Re: [PATCH] Video : Amba: Use in_interrupt() in clcdfb_sleep().

From: richard -rw- weinberger <hidden>
Date: 2012-03-11 13:23:00
Also in: kernel-janitors, lkml

On Sun, Mar 11, 2012 at 2:14 PM, santosh nayak
[off-list ref] wrote:
From: Santosh Nayak <redacted>

Instead of "in_atomic()", we can use in_interrupt() to check whether
its an interrupt context.
What is the benefit?

Your change turns clcdfb_sleep() into a ticking bomb.
If clcdfb_sleep() is called from an atomic section (e.g. under a
spinlock) it will call msleep()
and the whole thing explodes...

-- 
Thanks,
//richard
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help