linux-kernel - Re: [PATCH 4/7] net: ieee802154: mcr20a: Use IRQF_NO_AUTOEN flag in request

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20240910093258.358a6d85@xps-13>
Date: Tue, 10 Sep 2024 09:32:58 +0200
From: Miquel Raynal <miquel.raynal@...tlin.com>
To: Jinjie Ruan <ruanjinjie@...wei.com>
Cc: <davem@...emloft.net>, <edumazet@...gle.com>, <kuba@...nel.org>,
 <pabeni@...hat.com>, <claudiu.manoil@....com>, <vladimir.oltean@....com>,
 <louis.peens@...igine.com>, <stefan@...enfreihafen.org>,
 <alex.aring@...il.com>, <chunkeey@...glemail.com>, <kvalo@...nel.org>,
 <briannorris@...omium.org>, <francesco@...cini.it>,
 <set_pte_at@...look.com>, <damien.lemoal@...nsource.wdc.com>,
 <mpe@...erman.id.au>, <horms@...nel.org>, <yinjun.zhang@...igine.com>,
 <fei.qin@...igine.com>, <johannes.berg@...el.com>,
 <ryno.swart@...igine.com>, <krzysztof.kozlowski@...aro.org>,
 <leitao@...ian.org>, <liuxuenetmail@...il.com>, <netdev@...r.kernel.org>,
 <linux-kernel@...r.kernel.org>, <oss-drivers@...igine.com>,
 <linux-wpan@...r.kernel.org>, <linux-wireless@...r.kernel.org>
Subject: Re: [PATCH 4/7] net: ieee802154: mcr20a: Use IRQF_NO_AUTOEN flag in
 request_irq()

Hi Jinjie,

ruanjinjie@...wei.com wrote on Mon, 9 Sep 2024 21:30:31 +0800:

> disable_irq() after request_irq() still has a time gap in which
> interrupts can come. request_irq() with IRQF_NO_AUTOEN flag will
> disable IRQ auto-enable when request IRQ.
> 
> Fixes: 8c6ad9cc5157 ("ieee802154: Add NXP MCR20A IEEE 802.15.4 transceiver driver")
> Signed-off-by: Jinjie Ruan <ruanjinjie@...wei.com>

This one could go through wpan(-next), but otherwise:

Reviewed-by: Miquel Raynal <miquel.raynal@...tlin.com>

Thanks,
Miquèl