Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751078Ab3CAJJB (ORCPT ); Fri, 1 Mar 2013 04:09:01 -0500 Received: from mail-db8lp0184.outbound.messaging.microsoft.com ([213.199.154.184]:22007 "EHLO db8outboundpool.messaging.microsoft.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750902Ab3CAJI4 convert rfc822-to-8bit (ORCPT ); Fri, 1 Mar 2013 04:08:56 -0500 X-Greylist: delayed 905 seconds by postgrey-1.27 at vger.kernel.org; Fri, 01 Mar 2013 04:08:56 EST X-Forefront-Antispam-Report: CIP:70.37.183.190;KIP:(null);UIP:(null);IPV:NLI;H:mail.freescale.net;RD:none;EFVD:NLI X-SpamScore: -5 X-BigFish: VS-5(zzbb2dI98dI9371I542I1432I1447Izz1f42h1ee6h1de0h1202h1e76h1d1ah1d2ahzz8275dh8275bhz2dh2a8h668h839h8e2h8e3h944hd25hf0ah1220h1288h12a5h12a9h12bdh137ah13b6h1441h1504h1537h153bh15d0h162dh1631h1758h18e1h1946h19b5h1ad9hbe9i1155h) From: Sethi Varun-B16395 To: Alexey Kardashevskiy , Kumar Gala CC: Benjamin Herrenschmidt , "iommu@lists.linux-foundation.org" , "linuxppc-dev@lists.ozlabs.org list" , "linux-kernel@vger.kernel.org list" , Wood Scott-B07421 , Yoder Stuart-B08248 , Joerg Roedel , Paul Mackerras , David Gibson , Alex Williamson Subject: RE: [PATCH 1/6 v8] iommu/fsl: Store iommu domain information pointer in archdata. Thread-Topic: [PATCH 1/6 v8] iommu/fsl: Store iommu domain information pointer in archdata. Thread-Index: AQHODdfGismM9fsq90SfjKazZaDao5iNoBSAgAAIE4CAAdMTgIAAoD+AgAAiKoA= Date: Fri, 1 Mar 2013 08:55:32 +0000 Message-ID: References: <1361191939-21260-1-git-send-email-Varun.Sethi@freescale.com> <1361191939-21260-2-git-send-email-Varun.Sethi@freescale.com> <20130227113045.GH26252@8bytes.org> <51300367.6080300@ozlabs.ru> In-Reply-To: <51300367.6080300@ozlabs.ru> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.232.132.143] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 X-OriginatorOrg: freescale.com Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3414 Lines: 97 Thanks for the clarification Alexey. Kumar, We are using this new field (for PAMU) to store the iommu domain (for iommu API) information for a device. Regards Varun > -----Original Message----- > From: Alexey Kardashevskiy [mailto:aik@ozlabs.ru] > Sent: Friday, March 01, 2013 6:55 AM > To: Kumar Gala > Cc: Sethi Varun-B16395; Benjamin Herrenschmidt; iommu@lists.linux- > foundation.org; linuxppc-dev@lists.ozlabs.org list; linux- > kernel@vger.kernel.org list; Wood Scott-B07421; Yoder Stuart-B08248; > Joerg Roedel; Paul Mackerras; David Gibson; Alex Williamson > Subject: Re: [PATCH 1/6 v8] iommu/fsl: Store iommu domain information > pointer in archdata. > > Hi! > > On POWERNV we use only the part of IOMMU API which handles devices and > groups. We do not use IOMMU domains as VFIO containers do everything we > need for VFIO and we do not implement iommu_ops as it is not very > relevant to our architecture (does not give dma window properties, etc). > > So your work does not overlap with my work :) > > > On 01/03/13 02:51, Kumar Gala wrote: > > > > On Feb 27, 2013, at 6:04 AM, Sethi Varun-B16395 wrote: > > > >> Hi Kumar,Ben, > >> I am implementing the Freescale PAMU (IOMMU) driver using the Linux > IOMMU API. In this particular patch, I have added a new field to > dev_archdata structure to store the dma domain information. > >> This field is updated whenever we attach a device to an iommu domain. > >> > >> Regards > >> Varun > > > > Would be good to see if this overlaps with Alexey's work for IOMMU > driver for powernv. > > > > - k > > > >> > >>> -----Original Message----- > >>> From: Joerg Roedel [mailto:joro@8bytes.org] > >>> Sent: Wednesday, February 27, 2013 5:01 PM > >>> To: Sethi Varun-B16395 > >>> Cc: iommu@lists.linux-foundation.org; linuxppc-dev@lists.ozlabs.org; > >>> linux-kernel@vger.kernel.org; Wood Scott-B07421; Yoder Stuart-B08248 > >>> Subject: Re: [PATCH 1/6 v8] iommu/fsl: Store iommu domain > >>> information pointer in archdata. > >>> > >>> On Mon, Feb 18, 2013 at 06:22:14PM +0530, Varun Sethi wrote: > >>>> Add a new field in the device (powerpc) archdata structure for > >>>> storing iommu domain information pointer. This pointer is stored > >>>> when the device is attached to a particular domain. > >>>> > >>>> > >>>> Signed-off-by: Varun Sethi > >>>> --- > >>>> - no change. > >>>> arch/powerpc/include/asm/device.h | 4 ++++ > >>>> 1 files changed, 4 insertions(+), 0 deletions(-) > >>>> > >>>> diff --git a/arch/powerpc/include/asm/device.h > >>>> b/arch/powerpc/include/asm/device.h > >>>> index 77e97dd..6dc79fe 100644 > >>>> --- a/arch/powerpc/include/asm/device.h > >>>> +++ b/arch/powerpc/include/asm/device.h > >>>> @@ -28,6 +28,10 @@ struct dev_archdata { > >>>> void *iommu_table_base; > >>>> } dma_data; > >>>> > >>>> + /* IOMMU domain information pointer. This would be set > >>>> + * when this device is attached to an iommu_domain. > >>>> + */ > >>>> + void *iommu_domain; > >>> > >>> Please Cc the PowerPC Maintainers on this, so that they can have a > >>> look at it. This also must be put this into an #ifdef > CONFIG_IOMMU_API. > > > -- > Alexey -- 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/