From: Greg Kroah-Hartman Date: Fri, 30 Mar 2018 08:56:12 +0000 (+0200) Subject: 4.4-stable patches X-Git-Tag: v4.15.15~2 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=f27308024a468f7990d14342c97d64e36beded86;p=thirdparty%2Fkernel%2Fstable-queue.git 4.4-stable patches added patches: revert-genirq-use-irqd_get_trigger_type-to-compare-the.patch --- diff --git a/queue-4.4/genirq-track-whether-the-trigger-type-has-been-set.patch b/queue-4.4/genirq-track-whether-the-trigger-type-has-been-set.patch deleted file mode 100644 index 3d85ed38004..00000000000 --- a/queue-4.4/genirq-track-whether-the-trigger-type-has-been-set.patch +++ /dev/null @@ -1,105 +0,0 @@ -From 4f8413a3a799c958f7a10a6310a451e6b8aef5ad Mon Sep 17 00:00:00 2001 -From: Marc Zyngier -Date: Thu, 9 Nov 2017 14:17:59 +0000 -Subject: genirq: Track whether the trigger type has been set - -From: Marc Zyngier - -commit 4f8413a3a799c958f7a10a6310a451e6b8aef5ad upstream. - -When requesting a shared interrupt, we assume that the firmware -support code (DT or ACPI) has called irqd_set_trigger_type -already, so that we can retrieve it and check that the requester -is being reasonnable. - -Unfortunately, we still have non-DT, non-ACPI systems around, -and these guys won't call irqd_set_trigger_type before requesting -the interrupt. The consequence is that we fail the request that -would have worked before. - -We can either chase all these use cases (boring), or address it -in core code (easier). Let's have a per-irq_desc flag that -indicates whether irqd_set_trigger_type has been called, and -let's just check it when checking for a shared interrupt. -If it hasn't been set, just take whatever the interrupt -requester asks. - -Fixes: 382bd4de6182 ("genirq: Use irqd_get_trigger_type to compare the trigger type for shared IRQs") -Cc: stable@vger.kernel.org -Reported-and-tested-by: Petr Cvek -Signed-off-by: Marc Zyngier -Cc: Guenter Roeck -Signed-off-by: Greg Kroah-Hartman - ---- - include/linux/irq.h | 11 ++++++++++- - kernel/irq/manage.c | 13 ++++++++++++- - 2 files changed, 22 insertions(+), 2 deletions(-) - ---- a/include/linux/irq.h -+++ b/include/linux/irq.h -@@ -191,6 +191,7 @@ struct irq_data { - * IRQD_IRQ_INPROGRESS - In progress state of the interrupt - * IRQD_WAKEUP_ARMED - Wakeup mode armed - * IRQD_FORWARDED_TO_VCPU - The interrupt is forwarded to a VCPU -+ * IRQD_DEFAULT_TRIGGER_SET - Expected trigger already been set - */ - enum { - IRQD_TRIGGER_MASK = 0xf, -@@ -206,6 +207,7 @@ enum { - IRQD_IRQ_INPROGRESS = (1 << 18), - IRQD_WAKEUP_ARMED = (1 << 19), - IRQD_FORWARDED_TO_VCPU = (1 << 20), -+ IRQD_DEFAULT_TRIGGER_SET = (1 << 25), - }; - - #define __irqd_to_state(d) ((d)->common->state_use_accessors) -@@ -235,18 +237,25 @@ static inline void irqd_mark_affinity_wa - __irqd_to_state(d) |= IRQD_AFFINITY_SET; - } - -+static inline bool irqd_trigger_type_was_set(struct irq_data *d) -+{ -+ return __irqd_to_state(d) & IRQD_DEFAULT_TRIGGER_SET; -+} -+ - static inline u32 irqd_get_trigger_type(struct irq_data *d) - { - return __irqd_to_state(d) & IRQD_TRIGGER_MASK; - } - - /* -- * Must only be called inside irq_chip.irq_set_type() functions. -+ * Must only be called inside irq_chip.irq_set_type() functions or -+ * from the DT/ACPI setup code. - */ - static inline void irqd_set_trigger_type(struct irq_data *d, u32 type) - { - __irqd_to_state(d) &= ~IRQD_TRIGGER_MASK; - __irqd_to_state(d) |= type & IRQD_TRIGGER_MASK; -+ __irqd_to_state(d) |= IRQD_DEFAULT_TRIGGER_SET; - } - - static inline bool irqd_is_level_type(struct irq_data *d) ---- a/kernel/irq/manage.c -+++ b/kernel/irq/manage.c -@@ -1189,7 +1189,18 @@ __setup_irq(unsigned int irq, struct irq - * set the trigger type must match. Also all must - * agree on ONESHOT. - */ -- unsigned int oldtype = irqd_get_trigger_type(&desc->irq_data); -+ unsigned int oldtype; -+ -+ /* -+ * If nobody did set the configuration before, inherit -+ * the one provided by the requester. -+ */ -+ if (irqd_trigger_type_was_set(&desc->irq_data)) { -+ oldtype = irqd_get_trigger_type(&desc->irq_data); -+ } else { -+ oldtype = new->flags & IRQF_TRIGGER_MASK; -+ irqd_set_trigger_type(&desc->irq_data, oldtype); -+ } - - if (!((old->flags & new->flags) & IRQF_SHARED) || - (oldtype != (new->flags & IRQF_TRIGGER_MASK)) || diff --git a/queue-4.4/revert-genirq-use-irqd_get_trigger_type-to-compare-the.patch b/queue-4.4/revert-genirq-use-irqd_get_trigger_type-to-compare-the.patch new file mode 100644 index 00000000000..aadc2cabe20 --- /dev/null +++ b/queue-4.4/revert-genirq-use-irqd_get_trigger_type-to-compare-the.patch @@ -0,0 +1,37 @@ +From 5512cca5c518c20037b10369a4725327202dd80b Mon Sep 17 00:00:00 2001 +From: Greg Kroah-Hartman +Date: Fri, 30 Mar 2018 10:53:44 +0200 +Subject: [PATCH] Revert "genirq: Use irqd_get_trigger_type to compare the + trigger type for shared IRQs" + +This reverts commit 9d0273bb1c4b645817eccfe5c5975ea29add3300 which is +commit 382bd4de61827dbaaf5fb4fb7b1f4be4a86505e7 upstream. + +It causes too many problems with the stable tree, and would require too +many other things to be backported, so just revert it. + +Reported-by: Guenter Roeck +Cc: Thomas Gleixner +Cc: Hans de Goede +Cc: Marc Zyngier +Cc: Thomas Gleixner +Cc: Sasha Levin +Signed-off-by: Greg Kroah-Hartman +--- + kernel/irq/manage.c | 4 +--- + 1 file changed, 1 insertion(+), 3 deletions(-) + +--- a/kernel/irq/manage.c ++++ b/kernel/irq/manage.c +@@ -1189,10 +1189,8 @@ __setup_irq(unsigned int irq, struct irq + * set the trigger type must match. Also all must + * agree on ONESHOT. + */ +- unsigned int oldtype = irqd_get_trigger_type(&desc->irq_data); +- + if (!((old->flags & new->flags) & IRQF_SHARED) || +- (oldtype != (new->flags & IRQF_TRIGGER_MASK)) || ++ ((old->flags ^ new->flags) & IRQF_TRIGGER_MASK) || + ((old->flags ^ new->flags) & IRQF_ONESHOT)) + goto mismatch; + diff --git a/queue-4.4/series b/queue-4.4/series index a8dcc0ede95..f4dd2bb06ed 100644 --- a/queue-4.4/series +++ b/queue-4.4/series @@ -1,5 +1,5 @@ scsi-sg-don-t-return-bogus-sg_requests.patch -genirq-track-whether-the-trigger-type-has-been-set.patch +revert-genirq-use-irqd_get_trigger_type-to-compare-the.patch net-fix-hlist-corruptions-in-inet_evict_bucket.patch dccp-check-sk-for-closed-state-in-dccp_sendmsg.patch ipv6-fix-access-to-non-linear-packet-in-ndisc_fill_redirect_hdr_option.patch