linux-kernel - Re: [PATCH v8 1/1] genirq/devres: Add dev_err_probe() in devm_request_threaded_irq() and devm_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: <87o6t3jr31.ffs@tglx>
Date: Tue, 29 Jul 2025 11:15:14 +0200
From: Thomas Gleixner <tglx@...utronix.de>
To: Ahmad Fatoum <a.fatoum@...gutronix.de>, Pan Chuang <panchuang@...o.com>,
 linux-kernel@...r.kernel.org
Cc: miquel.raynal@...tlin.com, Jonathan.Cameron@...wei.com,
 angeg.delregno@...labora.com, krzk@...nel.org, frank.li@...o.com
Subject: Re: [PATCH v8 1/1] genirq/devres: Add dev_err_probe() in
 devm_request_threaded_irq() and devm_request_any_context_irq()

On Tue, Jul 29 2025 at 10:20, Ahmad Fatoum wrote:
> On 29.07.25 10:14, Pan Chuang wrote:
>> +	return dev_err_probe(dev, rc, "request_irq(%u) %pS %pS %s\n",
>
> Compared to %ps, %pS also prints offset and size relative to the symbol.
> This makes sense when you have an arbitrary program counter, but for
> merely printing a function symbol, I'd suggest %ps as it's less noisy.

Good point