summaryrefslogtreecommitdiffstats
path: root/include/linux/irq.h
diff options
context:
space:
mode:
authorKOSAKI Motohiro2008-12-17 11:40:33 +0100
committerIngo Molnar2008-12-18 14:35:53 +0100
commit74c8a6130486bed224e960790f4aa72dd09c061e (patch)
tree0d6da9737c66ca2f4b2a20b7eb34e6c2d04ba4cc /include/linux/irq.h
parentsparseirq, xen: make sure irq_desc is allocated for interrupts (diff)
downloadkernel-qcow2-linux-74c8a6130486bed224e960790f4aa72dd09c061e.tar.gz
kernel-qcow2-linux-74c8a6130486bed224e960790f4aa72dd09c061e.tar.xz
kernel-qcow2-linux-74c8a6130486bed224e960790f4aa72dd09c061e.zip
locking, irq: enclose irq_desc_lock_class in CONFIG_LOCKDEP
Impact: simplify code commit "08678b0: generic: sparse irqs: use irq_desc() [...]" introduced the irq_desc_lock_class variable. But it is used only if CONFIG_SPARSE_IRQ=Y or CONFIG_TRACE_IRQFLAGS=Y. Otherwise, following warnings happen: CC kernel/irq/handle.o kernel/irq/handle.c:26: warning: 'irq_desc_lock_class' defined but not used Actually, current early_init_irq_lock_class has a bit strange and messy ifdef. In addition, it is not valueable. 1. this function is protected by !CONFIG_SPARSE_IRQ, but that is not necessary. if CONFIG_SPARSE_IRQ=Y, desc of all irq number are initialized by NULL at first - then this function calling is safe. 2. this function protected by CONFIG_TRACE_IRQFLAGS too. but it is not necessary either, because lockdep_set_class() doesn't have bad side effect even if CONFIG_TRACE_IRQFLAGS=n. This patch bloat kernel size a bit on CONFIG_TRACE_IRQFLAGS=n and CONFIG_SPARSE_IRQ=Y - but that's ok. early_init_irq_lock_class() is not a fastpatch at all. To avoid messy ifdefs is more important than a few bytes diet. Signed-off-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com> Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'include/linux/irq.h')
0 files changed, 0 insertions, 0 deletions