diff options
author | Thomas Gleixner <tglx@linutronix.de> | 2017-08-25 22:34:05 +0200 |
---|---|---|
committer | Thomas Gleixner <tglx@linutronix.de> | 2017-08-25 22:40:26 +0200 |
commit | b33394ba5c0974a578c24b2fecbb91a984da5e09 (patch) | |
tree | 48d38aafe0061b061d2e5e3bda974785aa63382e | |
parent | irqdomain: Prevent potential NULL pointer dereference in irq_domain_push_irq() (diff) | |
download | linux-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.c | 4 |
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) { |