diff options
author | Gustavo A. R. Silva <gustavo@embeddedor.com> | 2019-02-28 16:37:14 -0500 |
---|---|---|
committer | Thomas Gleixner <tglx@linutronix.de> | 2019-03-23 07:32:01 -0400 |
commit | 93417a3fda2060f2a34e3341904024c5b6980d1f (patch) | |
tree | 1efe77ea33c7918b5917955038a9a0c5f757addc /kernel/irq/manage.c | |
parent | bb2e320565f997273fe04035bb6c17f643da6f8a (diff) |
genirq: Mark expected switch case fall-through
In preparation to enabling -Wimplicit-fallthrough, mark switch
cases where we are expecting to fall through.
With -Wimplicit-fallthrough added to CFLAGS:
kernel/irq/manage.c: In function ‘irq_do_set_affinity’:
kernel/irq/manage.c:198:3: warning: this statement may fall through [-Wimplicit-fallthrough=]
cpumask_copy(desc->irq_common_data.affinity, mask);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
kernel/irq/manage.c:199:2: note: here
case IRQ_SET_MASK_OK_NOCOPY:
^~~~
Annotate it.
Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Cc: Kees Cook <keescook@chromium.org>
Link: https://lkml.kernel.org/r/20190228213714.GA9246@embeddedor
Diffstat (limited to 'kernel/irq/manage.c')
-rw-r--r-- | kernel/irq/manage.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/kernel/irq/manage.c b/kernel/irq/manage.c index 9ec34a2a6638..1401afa0d58a 100644 --- a/kernel/irq/manage.c +++ b/kernel/irq/manage.c | |||
@@ -196,6 +196,7 @@ int irq_do_set_affinity(struct irq_data *data, const struct cpumask *mask, | |||
196 | case IRQ_SET_MASK_OK: | 196 | case IRQ_SET_MASK_OK: |
197 | case IRQ_SET_MASK_OK_DONE: | 197 | case IRQ_SET_MASK_OK_DONE: |
198 | cpumask_copy(desc->irq_common_data.affinity, mask); | 198 | cpumask_copy(desc->irq_common_data.affinity, mask); |
199 | /* fall through */ | ||
199 | case IRQ_SET_MASK_OK_NOCOPY: | 200 | case IRQ_SET_MASK_OK_NOCOPY: |
200 | irq_validate_effective_affinity(data); | 201 | irq_validate_effective_affinity(data); |
201 | irq_set_thread_affinity(desc); | 202 | irq_set_thread_affinity(desc); |