[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <aW5CIA1aCC8FVyFl@aspen.lan>
Date: Mon, 19 Jan 2026 14:39:28 +0000
From: Daniel Thompson <daniel@...cstar.com>
To: Markus Elfring <Markus.Elfring@....de>
Cc: linux-trace-kernel@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org,
Masami Hiramatsu <mhiramat@...nel.org>,
Mathieu Desnoyers <mathieu.desnoyers@...icios.com>,
Steven Rostedt <rostedt@...dmis.org>,
LKML <linux-kernel@...r.kernel.org>,
Robin Murphy <robin.murphy@....com>, Will Deacon <will@...nel.org>
Subject: Re: [PATCH] iommu: Fix NULL pointer deref when io_page_fault
tracepoint fires
On Fri, Jan 16, 2026 at 05:02:35PM +0100, Markus Elfring wrote:
> …
> > Fix this by adding logic to the tracepoint to safely propagate NULL.
>
> * How do you think about to add any tags (like “Fixes” and “Cc”) accordingly?
I could add a
Fixes: f8f934c180f6 ("iommu/arm-smmu: Add support for driver IOMMU fault handlers")
However, who do you think I neglected to Cc:?
> * Would a summary phrase like “Prevent null pointer dereference for a tracepoint”
> be a bit nicer?
I don't understand what is wrong with the original phrasing. Can you
explain why this change matters to you?
Daniel.
Powered by blists - more mailing lists