From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga03.intel.com ([134.134.136.65]:51858 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726614AbgCTHGi (ORCPT ); Fri, 20 Mar 2020 03:06:38 -0400 Subject: Re: [PATCH 3/8] iommu/vt-d: Remove IOVA handling code from non-dma_ops path References: <20191221150402.13868-1-murphyt7@tcd.ie> <20191221150402.13868-4-murphyt7@tcd.ie> From: Lu Baolu Message-ID: <46bf21e2-bb3e-1c1e-8dae-2c5bd8c5274f@linux.intel.com> Date: Fri, 20 Mar 2020 15:06:24 +0800 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-s390-owner@vger.kernel.org List-ID: To: Tom Murphy , iommu@lists.linux-foundation.org Cc: baolu.lu@linux.intel.com, Jani Nikula , Joonas Lahtinen , Rodrigo Vivi , David Airlie , Daniel Vetter , Joerg Roedel , Will Deacon , Robin Murphy , Marek Szyprowski , Kukjin Kim , Krzysztof Kozlowski , David Woodhouse , Andy Gross , Bjorn Andersson , Matthias Brugger , Rob Clark , Heiko Stuebner , Gerald Schaefer , Thierry Reding , Jonathan Hunter , Jean-Philippe Brucker , Alex Williamson , Cornelia Huck , Eric Auger , Marc Zyngier , Julien Grall , Thomas Gleixner , intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, Linux Kernel Mailing List , linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-s390@vger.kernel.org, linux-tegra@vger.kernel.org, virtualization@lists.linux-foundation.org, kvm@vger.kernel.org On 2020/3/20 14:30, Tom Murphy wrote: > Could we merge patch 1-3 from this series? it just cleans up weird > code and merging these patches will cover some of the work needed to > move the intel iommu driver to the dma-iommu api in the future. Can you please take a look at this patch series? https://lkml.org/lkml/2020/3/13/1162 It probably makes this series easier. Best regards, baolu > > On Sat, 21 Dec 2019 at 07:04, Tom Murphy wrote: >> Remove all IOVA handling code from the non-dma_ops path in the intel >> iommu driver. >> >> There's no need for the non-dma_ops path to keep track of IOVAs. The >> whole point of the non-dma_ops path is that it allows the IOVAs to be >> handled separately. The IOVA handling code removed in this patch is >> pointless. >> >> Signed-off-by: Tom Murphy