lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 8 May 2019 01:54:03 +0200 From: "Niklas Söderlund" <niklas.soderlund@...natech.se> To: Jiada Wang <jiada_wang@...tor.com> Cc: rui.zhang@...el.com, edubezval@...il.com, daniel.lezcano@...aro.org, linux-pm@...r.kernel.org, linux-kernel@...r.kernel.org, horms+renesas@...ge.net.au, geert+renesas@...der.be, sergei.shtylyov@...entembedded.com, marek.vasut+renesas@...il.com, kuninori.morimoto.gx@...esas.com, hien.dang.eb@...esas.com, fabrizio.castro@...renesas.com, dien.pham.ry@...esas.com, biju.das@...renesas.com, erosca@...adit-jv.com, george_davis@...tor.com, joshua_frkuska@...tor.com Subject: Re: [PATCH v4 0/2] thermal: rcar_gen3_thermal: fix IRQ issues Hi Jiada, Thanks for your patches. On 2019-04-24 14:11:43 +0900, Jiada Wang wrote: > There are issues with interrupt handling in rcar_gen3_thermal driver. > > Currently IRQ is remain enabled after .remove, later if device is probed, > IRQ is requested before .thermal_init, this may cause IRQ function be > triggered but not able to clear IRQ status, thus cause system to hang. > > Since the irq line isn't shared between different devices, > so the proper interrupt type flag should be IRQF_ONESHOT. > > This patch-set fix these interrupt handling retated issues. I really like this series, nice work. Tested-by: Niklas Söderlund <niklas.soderlund+renesas@...natech.se> Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@...natech.se> > > --- > v4: remove 'spinlock_t lock' > add Fixes tag in ("thermal: rcar_gen3_thermal: fix interrupt type") > fix typos in ("thermal: rcar_gen3_thermal: disable interrupt in .remove") > > v3: fix to use correct code base > remove unused "flag" variable in rcar_gen3_thermal_irq > > v2: use irq type IRQF_ONESHOT instead of IRQF_SHARED > disable interrupt in .remove > > v1: initial version > > Jiada Wang (2): > thermal: rcar_gen3_thermal: fix interrupt type > thermal: rcar_gen3_thermal: disable interrupt in .remove > > drivers/thermal/rcar_gen3_thermal.c | 41 +++++++---------------------- > 1 file changed, 9 insertions(+), 32 deletions(-) > > -- > 2.19.2 > -- Regards, Niklas Söderlund
Powered by blists - more mailing lists