Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756864Ab3GWLAb (ORCPT ); Tue, 23 Jul 2013 07:00:31 -0400 Received: from mail-pd0-f175.google.com ([209.85.192.175]:34007 "EHLO mail-pd0-f175.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753059Ab3GWLAa (ORCPT ); Tue, 23 Jul 2013 07:00:30 -0400 MIME-Version: 1.0 X-Originating-IP: [2a01:e35:2434:4600:6116:785e:d2c6:30a] In-Reply-To: <00d701ce878f$da099fe0$8e1cdfa0$%dae@samsung.com> References: <1374573694-30595-1-git-send-email-a.motakis@virtualopensystems.com> <00d701ce878f$da099fe0$8e1cdfa0$%dae@samsung.com> Date: Tue, 23 Jul 2013 13:00:29 +0200 Message-ID: Subject: Re: [PATCH] iommu/exynos: add devices attached to the System MMU to an IOMMU group From: Antonios Motakis To: Inki Dae Cc: Linux ARM Kernel , Linux IOMMU , Linux Samsung SOC , kvm-arm , Cho KyongHo , Joerg Roedel , Sachin Kamat , Jiri Kosina , Wei Yongjun , open list Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4083 Lines: 111 On Tue, Jul 23, 2013 at 12:31 PM, Inki Dae wrote: > > > > > -----Original Message----- > > From: linux-samsung-soc-owner@vger.kernel.org [mailto:linux-samsung-soc- > > owner@vger.kernel.org] On Behalf Of Antonios Motakis > > Sent: Tuesday, July 23, 2013 7:02 PM > > To: linux-arm-kernel@lists.infradead.org; > iommu@lists.linux-foundation.org; > > linux-samsung-soc@vger.kernel.org > > Cc: kvmarm@lists.cs.columbia.edu; Antonios Motakis; Cho KyongHo; Joerg > > Roedel; Sachin Kamat; Jiri Kosina; Wei Yongjun; open list > > Subject: [PATCH] iommu/exynos: add devices attached to the System MMU to > > an IOMMU group > > > > IOMMU groups are expected by certain users of the IOMMU API, > > e.g. VFIO. Since each device is behind its own System MMU, we > > can allocate a new IOMMU group for each device. > > > > This patch depends on Cho KyongHo's patch series titled "[PATCH v7 00/12] > > iommu/exynos: Fixes and Enhancements of System MMU driver with DT", > > applied on a Linux 3.10.1 kernel. It has been tested on the Arndale board. > > > > Signed-off-by: Antonios Motakis > > --- > > drivers/iommu/exynos-iommu.c | 24 ++++++++++++++++++++++++ > > 1 file changed, 24 insertions(+) > > > > diff --git a/drivers/iommu/exynos-iommu.c b/drivers/iommu/exynos-iommu.c > > index 51d43bb..9f39eaa 100644 > > --- a/drivers/iommu/exynos-iommu.c > > +++ b/drivers/iommu/exynos-iommu.c > > @@ -1134,6 +1134,28 @@ static phys_addr_t exynos_iommu_iova_to_phys(struct > > iommu_domain *domain, > > return phys; > > } > > > > +static int exynos_iommu_add_device(struct device *dev) > > +{ > > + struct iommu_group *group; > > + int ret; > > + > > + group = iommu_group_alloc(); > > Is that correct? I don't see why you allocate a group object every time > add_device callback is called. That doesn't have any meaning we have to use > iommu group feature. I think the implementation should be one more devices > per a group. So I guess a given device object should be wrapped by higher > device object than the given device object. For a good example, you can > refer to intel-iommu.c file. With an Intel IOMMU it can be the case that 2 devices have to share the same IOMMU mappings (i.e. you can't program them separately). With the Exynos System MMU, there is always one System MMU per device, so there is nothing stopping you from programming any 2 devices' mappings differently. So yes, the right thing to do here is to have a one to one relationship between devices and IOMMU groups. (resending because of html mail) Cheers, Antonios > > > Thanks, > Inki Dae > > > + if (IS_ERR(group)) { > > + dev_err(dev, "Failed to allocate IOMMU group\n"); > > + return PTR_ERR(group); > > + } > > + > > + ret = iommu_group_add_device(group, dev); > > + iommu_group_put(group); > > + > > + return ret; > > +} > > + > > +static void exynos_iommu_remove_device(struct device *dev) > > +{ > > + iommu_group_remove_device(dev); > > +} > > + > > static struct iommu_ops exynos_iommu_ops = { > > .domain_init = &exynos_iommu_domain_init, > > .domain_destroy = &exynos_iommu_domain_destroy, > > @@ -1142,6 +1164,8 @@ static struct iommu_ops exynos_iommu_ops = { > > .map = &exynos_iommu_map, > > .unmap = &exynos_iommu_unmap, > > .iova_to_phys = &exynos_iommu_iova_to_phys, > > + .add_device = exynos_iommu_add_device, > > + .remove_device = exynos_iommu_remove_device, > > .pgsize_bitmap = SECT_SIZE | LPAGE_SIZE | SPAGE_SIZE, > > }; > > > > -- > > 1.8.1.2 > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-samsung- > > soc" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/