Re: [PATCH 1/3] iommu/msm: track a context master per device and IOMMU
Dmitry Baryshkov <[email protected]> Thu, 30 Jul 2026 17:30:57 +0300
| Newsgroups | dev.linux.lists.iommu,org.freedesktop.lists.dri-devel,org.infradead.lists.linux-arm-kernel,org.infradead.lists.linux-rockchip,org.kernel.vger.linux-arm-msm,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <ketiunovezpn76nxocpima27esjt66vve6tmqg3e4murz3yjga@pxl6kikpbk5l> |
On Thu, Jul 30, 2026 at 03:57:42PM +0200, Konrad Dybcio wrote: > On 7/30/26 3:29 PM, Dmitry Baryshkov wrote: > > insert_iommu_master() decided whether to allocate a struct > > msm_iommu_ctx_dev by testing whether the IOMMU's ctx_list was empty, > > caching the result in dev_iommu_priv, then unconditionally dereferenced > > the master: > > [...] > > > - dev_warn(dev, "Stream ID 0x%x repeated; ignoring\n", > > - sid); > > + dev_dbg(dev, "Stream ID 0x%x repeated; ignoring\n", > > + sid); > > Is this intentionally part of the patch? It can be dropped, yes. -- With best wishes Dmitry