linux-kernel - Re: recommended use of request_any_context

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: <57E391E0.6020202@arm.com>
Date:   Thu, 22 Sep 2016 09:10:08 +0100
From:   Marc Zyngier <marc.zyngier@....com>
To:     Leo Li <pku.leo@...il.com>, Marc Zyngier <maz@...terjones.org>,
        Thomas Gleixner <tglx@...utronix.de>,
        lkml <linux-kernel@...r.kernel.org>
Subject: Re: recommended use of request_any_context_irq()

Leo,

On 21/09/16 22:14, Leo Li wrote:
> Hi Marc and Thomas,
> 
> With the introduction of request_any_context_irq() routine, driver can
> deal with interrupt controllers using either threaded irq or normal
> irq.  But I don't see many drivers that have been changed to use this
> function to request interrupt.  For on-board devices, the driver
> normally don't know which kind of interrupt controller they are
> connected to.  Why don't we make the request_any_context_irq()
> mandatory or recommended for all drivers?  Is there any drawback for
> changing all the request_irq() to the request_any_context_irq()?

In 99.99% of the cases, a device is integrated in one particular way,
always. For the 0.01% that is left, we have the above API. And if a
particular device moves from the first category to the second, whoever
designed the system will change the driver to use this API, and that
driver only.

There is strictly no reason to perform a blanket change of all the
drivers. What would be the reason to change them other than to cater for
a contrived use case that may never happen?

Thanks,

	M.
-- 
Jazz is not dead. It just smells funny...