[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20250924192246.GK2617119@nvidia.com>
Date: Wed, 24 Sep 2025 16:22:46 -0300
From: Jason Gunthorpe <jgg@...dia.com>
To: Nicolin Chen <nicolinc@...dia.com>
Cc: joro@...tes.org, bhelgaas@...gle.com, suravee.suthikulpanit@....com,
will@...nel.org, robin.murphy@....com, sven@...nel.org,
j@...nau.net, alyssa@...enzweig.io, neal@...pa.dev,
robin.clark@....qualcomm.com, m.szyprowski@...sung.com,
krzk@...nel.org, alim.akhtar@...sung.com, dwmw2@...radead.org,
baolu.lu@...ux.intel.com, kevin.tian@...el.com,
yong.wu@...iatek.com, matthias.bgg@...il.com,
angelogioacchino.delregno@...labora.com, tjeznach@...osinc.com,
paul.walmsley@...ive.com, palmer@...belt.com, aou@...s.berkeley.edu,
alex@...ti.fr, heiko@...ech.de, schnelle@...ux.ibm.com,
mjrosato@...ux.ibm.com, gerald.schaefer@...ux.ibm.com,
orsonzhai@...il.com, baolin.wang@...ux.alibaba.com,
zhang.lyra@...il.com, wens@...e.org, jernej.skrabec@...il.com,
samuel@...lland.org, jean-philippe@...aro.org, rafael@...nel.org,
lenb@...nel.org, yi.l.liu@...el.com, cwabbott0@...il.com,
quic_pbrahma@...cinc.com, iommu@...ts.linux.dev,
linux-kernel@...r.kernel.org, asahi@...ts.linux.dev,
linux-arm-kernel@...ts.infradead.org, linux-arm-msm@...r.kernel.org,
linux-samsung-soc@...r.kernel.org,
linux-mediatek@...ts.infradead.org, linux-riscv@...ts.infradead.org,
linux-rockchip@...ts.infradead.org, linux-s390@...r.kernel.org,
linux-sunxi@...ts.linux.dev, linux-tegra@...r.kernel.org,
virtualization@...ts.linux.dev, linux-acpi@...r.kernel.org,
linux-pci@...r.kernel.org, patches@...ts.linux.dev,
vsethi@...dia.com, helgaas@...nel.org, etzhao1900@...il.com
Subject: Re: [PATCH v4 4/7] iommu: Pass in old domain to attach_dev callback
functions
On Wed, Sep 24, 2025 at 12:18:43PM -0700, Nicolin Chen wrote:
> On Wed, Sep 24, 2025 at 03:43:46PM -0300, Jason Gunthorpe wrote:
> > I've split things like this into more patches before, but this
> > actually isn't too bad, so I wouldn't push for it.
>
> Thanks. I will keep the flow in mind.
>
> For working more on the new test attach op, I'd probably keep it
> as-is.
>
> > A series version would be:
> > - add a new op 'attach_dev2' or whatever
> > - Convert all drivers that just change the signature
> > - Convert drivers that have a trivial iommu_get_domain_for_dev()
> > - N patches to convert more complex drivers one by one
> > - Remove old op attach_dev
>
> Does this mean that drivers would live with the 'attach_dev2' op
> renamed from 'attach_dev'? Or the last could rename it back?
Could rename back, often better to select a name that doesn't need a
rename back :)
Jason
Powered by blists - more mailing lists