Re: [PATCH v7 00/22] Generic DT bindings for PCI IOMMUs and ARM SMMU

2016-09-16 Thread Robin Murphy
Hi Eric, On 15/09/16 17:46, Auger Eric wrote: [...] > Hum OK; thanks for the explanation. With that implementation however, > don't we face back the issue we encountered in early stage of default > domain implementation: > > With this sample config (AMD overdrive + I350-T2 + 2VFs per PF) I fill >

Re: [PATCHv9 0/6] dmaengine: rcar-dmac: add iommu support for slave transfers

2016-09-16 Thread Laurent Pinchart
Hi Robin, On Friday 16 Sep 2016 13:49:21 Robin Murphy wrote: > On 16/09/16 13:05, Laurent Pinchart wrote: > [...] > > One concern I have is that we might get an awkward situation if we ever > encounter one DMA engine hardware that is used in different systems > that all have an IOM

Re: [PATCHv9 0/6] dmaengine: rcar-dmac: add iommu support for slave transfers

2016-09-16 Thread Laurent Pinchart
Hi Arnd, On Friday 16 Sep 2016 14:22:31 Arnd Bergmann wrote: > On Friday, September 16, 2016 3:09:29 PM CEST Laurent Pinchart wrote: > >> I wasn't thinking quite that far, though that is also a theoretical > >> problem. However, the simple solution would be to have a bit in the DMA > >> specifier

Re: [PATCHv9 0/6] dmaengine: rcar-dmac: add iommu support for slave transfers

2016-09-16 Thread Robin Murphy
On 16/09/16 13:05, Laurent Pinchart wrote: [...] One concern I have is that we might get an awkward situation if we ever encounter one DMA engine hardware that is used in different systems that all have an IOMMU, but on some of them the connection between the DMA master and the

Re: [PATCHv9 0/6] dmaengine: rcar-dmac: add iommu support for slave transfers

2016-09-16 Thread Arnd Bergmann
On Friday, September 16, 2016 3:09:29 PM CEST Laurent Pinchart wrote: > > I wasn't thinking quite that far, though that is also a theoretical > > problem. However, the simple solution would be to have a bit in the DMA > > specifier let the driver know whether translation is needed or not. > > > >

Re: [PATCHv9 0/6] dmaengine: rcar-dmac: add iommu support for slave transfers

2016-09-16 Thread Laurent Pinchart
Hi Arnd, On Friday 16 Sep 2016 14:02:35 Arnd Bergmann wrote: > On Friday, September 16, 2016 12:48:23 PM CEST Laurent Pinchart wrote: > > On Friday 16 Sep 2016 11:07:48 Arnd Bergmann wrote: > >> On Thursday, September 15, 2016 9:56:51 PM CEST Vinod Koul wrote: > >>> On Wed, Aug 10, 2016 at 11:07:1

Re: [PATCHv9 0/6] dmaengine: rcar-dmac: add iommu support for slave transfers

2016-09-16 Thread Laurent Pinchart
Hi Rubin, On Friday 16 Sep 2016 11:36:29 Robin Murphy wrote: > On 16/09/16 10:48, Laurent Pinchart wrote: > > On Friday 16 Sep 2016 11:07:48 Arnd Bergmann wrote: > >> On Thursday, September 15, 2016 9:56:51 PM CEST Vinod Koul wrote: > >>> On Wed, Aug 10, 2016 at 11:07:10PM +0530, Vinod Koul wrote:

Re: [PATCHv9 0/6] dmaengine: rcar-dmac: add iommu support for slave transfers

2016-09-16 Thread Arnd Bergmann
On Friday, September 16, 2016 12:48:23 PM CEST Laurent Pinchart wrote: > On Friday 16 Sep 2016 11:07:48 Arnd Bergmann wrote: > > On Thursday, September 15, 2016 9:56:51 PM CEST Vinod Koul wrote: > > > On Wed, Aug 10, 2016 at 11:07:10PM +0530, Vinod Koul wrote: > > >> On Wed, Aug 10, 2016 at 01:22:1

Re: [PATCHv9 0/6] dmaengine: rcar-dmac: add iommu support for slave transfers

2016-09-16 Thread Robin Murphy
On 16/09/16 10:48, Laurent Pinchart wrote: > Hi Arnd, > > On Friday 16 Sep 2016 11:07:48 Arnd Bergmann wrote: >> On Thursday, September 15, 2016 9:56:51 PM CEST Vinod Koul wrote: >>> On Wed, Aug 10, 2016 at 11:07:10PM +0530, Vinod Koul wrote: On Wed, Aug 10, 2016 at 01:22:13PM +0200, Niklas S

Re: [PATCHv9 0/6] dmaengine: rcar-dmac: add iommu support for slave transfers

2016-09-16 Thread Laurent Pinchart
Hi Arnd, On Friday 16 Sep 2016 11:07:48 Arnd Bergmann wrote: > On Thursday, September 15, 2016 9:56:51 PM CEST Vinod Koul wrote: > > On Wed, Aug 10, 2016 at 11:07:10PM +0530, Vinod Koul wrote: > >> On Wed, Aug 10, 2016 at 01:22:13PM +0200, Niklas Söderlund wrote: > >>> Hi, > >>> > >>> This series

Re: [PATCHv9 0/6] dmaengine: rcar-dmac: add iommu support for slave transfers

2016-09-16 Thread Arnd Bergmann
On Thursday, September 15, 2016 9:56:51 PM CEST Vinod Koul wrote: > On Wed, Aug 10, 2016 at 11:07:10PM +0530, Vinod Koul wrote: > > On Wed, Aug 10, 2016 at 01:22:13PM +0200, Niklas Söderlund wrote: > > > Hi, > > > > > > This series tries to solve the problem with DMA with device registers > > > (M

Re: [RFC PATCH v2 19/20] x86: Access the setup data through debugfs un-encrypted

2016-09-16 Thread Borislav Petkov
On Thu, Sep 15, 2016 at 12:08:04PM -0500, Tom Lendacky wrote: > The problem is that this physical address does not contain the > encryption bit, and even if it did, it wouldn't matter. The __va() > define creates a virtual address that will be mapped as encrypted given > the current approach (whic

Re: [RFC PATCH v2 15/20] iommu/amd: AMD IOMMU support for memory encryption

2016-09-16 Thread Borislav Petkov
On Thu, Sep 15, 2016 at 11:57:41AM -0500, Tom Lendacky wrote: > If I do that, then I could put an #ifdef in the header to include the > asm/mem_encrypt.h if the memory encryption is configured, else set the > value to zero. Yeah, something along those lines... > I'll look into this. One immediate