Messages in this thread |  | | | Subject | Re: use of preempt_count instead of in_atomic() at leds-gpio.c | | From | Tetsuo Handa <> | | Date | Sun, 23 Mar 2008 14:53:01 +0900 |
| |
Hello.
> >> Is "if (in_atomic()) return;" check a correct method for avoiding > >> deadlocks when my code was accidentally called from a context > >> where use of down()/mutex_lock()/kmalloc(GFP_KERNEL)/get_user_pages()/ > >> kmap() etc. are not permitted? > > No. Quoting Andrew: "in_atomic() returns false inside spinlock on > non-preemptible kernels."
So, just "if (in_atomic()) return;" check is insufficient for detecting all cases when it is not permitted to sleep. I see. Is "in_atomic() returns false inside spinlock on non-preemptible kernels." the only case that the in_atomic() can't tell whether it is permitted to sleep or not? If this is the only case, can't we somehow know it by remembering "how many spinlocks does this CPU is holding now" since "it is not permitted to sleep inside the spinlock" means "the CPU the current process is running will not change". Something like
#ifdef CONFIG_COUNT_SPINLOCKS_HELD atomic_t spinlock_held_counter[NR_CPUS]; #endif
void spin_lock(x) { /* obtain this spinlock. */ #ifdef CONFIG_COUNT_SPINLOCKS_HELD /* increment spinlock_held_counter[this_CPU]. */ #endif } void spin_unlock(x) { #ifdef CONFIG_COUNT_SPINLOCKS_HELD /* decrement spinlock_held_counter[this_CPU]. */ #endif /* release this spinlock. */ } bool in_spinlock() { #ifdef CONFIG_COUNT_SPINLOCKS_HELD /* return spinlock_held_counter[this_CPU] != 0. */ #else return false; #endif } and use "if (in_atomic() || in_spinlock()) return;" instead of "if (in_atomic()) return;" ?
|  |