Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <670b01819f6f875364abaafa4accad9fa92e6177.camel@linux.ibm.com>
Subject: Re: [PATCH v8 0/6] iommu/dma: s390 DMA API conversion and optimized
 IOTLB flushing
From:   Niklas Schnelle <schnelle@linux.ibm.com>
To:     Christoph Hellwig <hch@infradead.org>
Cc:     Joerg Roedel <joro@8bytes.org>,
        Matthew Rosato <mjrosato@linux.ibm.com>,
        Will Deacon <will@kernel.org>,
        Wenjia Zhang <wenjia@linux.ibm.com>,
        Robin Murphy <robin.murphy@arm.com>,
        Jason Gunthorpe <jgg@ziepe.ca>,
        Gerd Bayer <gbayer@linux.ibm.com>,
        Julian Ruess <julianr@linux.ibm.com>,
        Pierre Morel <pmorel@linux.ibm.com>,
        Alexandra Winter <wintera@linux.ibm.com>,
        Heiko Carstens <hca@linux.ibm.com>,
        Vasily Gorbik <gor@linux.ibm.com>,
        Alexander Gordeev <agordeev@linux.ibm.com>,
        Christian Borntraeger <borntraeger@linux.ibm.com>,
        Sven Schnelle <svens@linux.ibm.com>,
        Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>,
        Hector Martin <marcan@marcan.st>,
        Sven Peter <sven@svenpeter.dev>,
        Alyssa Rosenzweig <alyssa@rosenzweig.io>,
        David Woodhouse <dwmw2@infradead.org>,
        Lu Baolu <baolu.lu@linux.intel.com>,
        Andy Gross <agross@kernel.org>,
        Bjorn Andersson <andersson@kernel.org>,
        Konrad Dybcio <konrad.dybcio@linaro.org>,
        Yong Wu <yong.wu@mediatek.com>,
        Matthias Brugger <matthias.bgg@gmail.com>,
        AngeloGioacchino Del Regno 
        <angelogioacchino.delregno@collabora.com>,
        Gerald Schaefer <gerald.schaefer@linux.ibm.com>,
        Orson Zhai <orsonzhai@gmail.com>,
        Baolin Wang <baolin.wang@linux.alibaba.com>,
        Chunyan Zhang <zhang.lyra@gmail.com>,
        Chen-Yu Tsai <wens@csie.org>,
        Jernej Skrabec <jernej.skrabec@gmail.com>,
        Samuel Holland <samuel@sholland.org>,
        Thierry Reding <thierry.reding@gmail.com>,
        Krishna Reddy <vdumpa@nvidia.com>,
        Jonathan Hunter <jonathanh@nvidia.com>,
        Jonathan Corbet <corbet@lwn.net>, linux-s390@vger.kernel.org,
        netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
        iommu@lists.linux.dev, asahi@lists.linux.dev,
        linux-arm-kernel@lists.infradead.org,
        linux-arm-msm@vger.kernel.org, linux-mediatek@lists.infradead.org,
        linux-sunxi@lists.linux.dev, linux-tegra@vger.kernel.org,
        linux-doc@vger.kernel.org
Date:   Mon, 15 May 2023 11:33:25 +0200
In-Reply-To: <ZF5NRls92rhmzV7B@infradead.org>
References: <20230310-dma_iommu-v8-0-2347dfbed7af@linux.ibm.com>
         <ZF5NRls92rhmzV7B@infradead.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
User-Agent: Evolution 3.48.1 (3.48.1-1.fc38) 
MIME-Version: 1.0
Precedence: bulk

On Fri, 2023-05-12 at 07:29 -0700, Christoph Hellwig wrote:
> On Fri, Mar 10, 2023 at 05:07:45PM +0100, Niklas Schnelle wrote:
> > Hi All,
> >=20
> > This patch series converts s390's PCI support from its platform specifi=
c DMA
> > API implementation in arch/s390/pci/pci_dma.c to the common DMA IOMMU l=
ayer.
> > The conversion itself is done in patches 3-4 with patch 2 providing the=
 final
> > necessary IOMMU driver improvement to handle s390's special IOTLB flush
> > out-of-resource indication in virtualized environments. Patches 1-2 may=
 be
> > applied independently. The conversion itself only touches the s390 IOMM=
U driver
> > and s390 arch code moving over remaining functions from the s390 DMA AP=
I
> > implementation. No changes to common code are necessary.
>=20
> It looks like this still hasn't made it upstream as of 6.4-rc1.  What's
> holding this series up?
>=20

I think with all the IOMMUFD work going on this got starved out of
reviewer resources. I didn't have any open todos but I guess there is
still some review needed around how the IOMMU driver tells the dma-
iommu that it expects IOTLB flushes to be slow requiring a larger
single flush queue. There was also a small conflict with commit
49a22aae7d9c ("iommu: Replace device_lock() with group->mutex") that
prevented this from applying to current upstream so I've just now sent
out a v9 rebased on v6.4-rc2.

Thanks,
Niklas