summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorThomas Gleixner <tglx@linutronix.de>2017-08-25 22:34:05 +0200
committerThomas Gleixner <tglx@linutronix.de>2017-08-25 22:40:26 +0200
commitb33394ba5c0974a578c24b2fecbb91a984da5e09 (patch)
tree48d38aafe0061b061d2e5e3bda974785aa63382e
parentirqdomain: Prevent potential NULL pointer dereference in irq_domain_push_irq() (diff)
downloadlinux-b33394ba5c0974a578c24b2fecbb91a984da5e09.tar.xz
linux-b33394ba5c0974a578c24b2fecbb91a984da5e09.zip
genirq/proc: Avoid uninitalized variable warning
kernel/irq/proc.c: In function ‘show_irq_affinity’: include/linux/cpumask.h:24:29: warning: ‘mask’ may be used uninitialized in this function [-Wmaybe-uninitialized] #define cpumask_bits(maskp) ((maskp)->bits) gcc is silly, but admittedly it can't know that this won't be called with anything else than the enumerated constants. Shut up the warning by creating a default clause. Fixes: 6bc6d4abd22e ("genirq/proc: Use the the accessor to report the effective affinity Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
-rw-r--r--kernel/irq/proc.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/irq/proc.c b/kernel/irq/proc.c
index 9f62f9a1a5e9..6376b4a598d3 100644
--- a/kernel/irq/proc.c
+++ b/kernel/irq/proc.c
@@ -63,9 +63,9 @@ static int show_irq_affinity(int type, struct seq_file *m)
#ifdef CONFIG_GENERIC_IRQ_EFFECTIVE_AFF_MASK
mask = irq_data_get_effective_affinity_mask(&desc->irq_data);
break;
-#else
- return -EINVAL;
#endif
+ default:
+ return -EINVAL;
}
switch (type) {