RE: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu implementation.

2013-04-04 Thread Sethi Varun-B16395
.org; linux- > ker...@vger.kernel.org; ga...@kernel.crashing.org; > b...@kernel.crashing.org > Subject: Re: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu > implementation. > > On Thu, 2013-04-04 at 16:35 +, Sethi Varun-B16395 wrote: > > > > > ---

Re: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu implementation.

2013-04-04 Thread Alex Williamson
B08248; Wood Scott-B07421; > > io...@lists.linux-foundation.org; linuxppc-...@lists.ozlabs.org; linux- > > ker...@vger.kernel.org; ga...@kernel.crashing.org; > > b...@kernel.crashing.org > > Subject: Re: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu > > im

RE: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu implementation.

2013-04-04 Thread Yoder Stuart-B08248
sts.ozlabs.org; linux-kernel@vger.kernel.org; > ga...@kernel.crashing.org; b...@kernel.crashing.org > Subject: Re: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu > implementation. > > On Thu, 2013-04-04 at 13:00 +, Sethi Varun-B16395 wrote: > > > > > ---

RE: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu implementation.

2013-04-04 Thread Sethi Varun-B16395
.org; linux- > ker...@vger.kernel.org; ga...@kernel.crashing.org; > b...@kernel.crashing.org > Subject: Re: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu > implementation. > > On Thu, 2013-04-04 at 13:00 +, Sethi Varun-B16395 wrote: > > > > > ---

Re: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu implementation.

2013-04-04 Thread Alex Williamson
B08248; Wood Scott-B07421; > > io...@lists.linux-foundation.org; linuxppc-...@lists.ozlabs.org; linux- > > ker...@vger.kernel.org; ga...@kernel.crashing.org; > > b...@kernel.crashing.org > > Subject: Re: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu > > im

RE: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu implementation.

2013-04-04 Thread Sethi Varun-B16395
.org; linux- > ker...@vger.kernel.org; ga...@kernel.crashing.org; > b...@kernel.crashing.org > Subject: Re: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu > implementation. > > On Tue, 2013-04-02 at 18:18 +0200, Joerg Roedel wrote: > > Cc'ing Alex Williamso

Re: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu implementation.

2013-04-03 Thread Alex Williamson
On Tue, 2013-04-02 at 18:18 +0200, Joerg Roedel wrote: > Cc'ing Alex Williamson > > Alex, can you please review the iommu-group part of this patch? Sure, it looks pretty reasonable. AIUI, all PCI devices are below some kind of host bridge that is either new and supports partitioning or old and d

RE: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu implementation.

2013-04-03 Thread Yoder Stuart-B08248
.ozlabs.org > Subject: RE: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu > implementation. > > > > > -Original Message- > > From: Wood Scott-B07421 > > Sent: Wednesday, April 03, 2013 7:23 AM > > To: Timur Tabi > > Cc: Joerg Roed

RE: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu implementation.

2013-04-03 Thread Sethi Varun-B16395
.@vger.kernel.org; ga...@kernel.crashing.org; > b...@kernel.crashing.org; Alex Williamson > Subject: Re: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu > implementation. > > Cc'ing Alex Williamson > > Alex, can you please review the iommu-group part of this patch?

RE: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu implementation.

2013-04-02 Thread Sethi Varun-B16395
.ozlabs.org > Subject: Re: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu > implementation. > > On 04/02/2013 08:35:54 PM, Timur Tabi wrote: > > On Tue, Apr 2, 2013 at 11:18 AM, Joerg Roedel wrote: > > > > > > + panic("\n"); > >

Re: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu implementation.

2013-04-02 Thread Scott Wood
On 04/02/2013 08:35:54 PM, Timur Tabi wrote: On Tue, Apr 2, 2013 at 11:18 AM, Joerg Roedel wrote: > > + panic("\n"); > > A kernel panic seems like an over-reaction to an access violation. We have no way to determining what code caused the violation, so we can't just kill the process. I ag

Re: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu implementation.

2013-04-02 Thread Timur Tabi
On Tue, Apr 2, 2013 at 11:18 AM, Joerg Roedel wrote: > > + panic("\n"); > > A kernel panic seems like an over-reaction to an access violation. We have no way to determining what code caused the violation, so we can't just kill the process. I agree it seems like overkill, but what else shoul

Re: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu implementation.

2013-04-02 Thread Joerg Roedel
Cc'ing Alex Williamson Alex, can you please review the iommu-group part of this patch? My comments so far are below: On Fri, Mar 29, 2013 at 01:24:02AM +0530, Varun Sethi wrote: > +config FSL_PAMU > + bool "Freescale IOMMU support" > + depends on PPC_E500MC > + select IOMMU_API > +

RE: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu implementation.

2013-04-02 Thread Yoder Stuart-B08248
ernel.crashing.org; b...@kernel.crashing.org > Cc: Sethi Varun-B16395 > Subject: [PATCH 5/5 v11] iommu/fsl: Freescale PAMU driver and iommu > implementation. > > Following is a brief description of the PAMU hardware: > PAMU determines what action to take and whether to authorize