Re: [PATCH 1/1] IOMMU: Save pci device id instead of pci_dev* pointer for DMAR devices

2013-11-20 Thread Yijing Wang
On 2013/11/20 23:59, David Woodhouse wrote: > On Fri, 2013-11-08 at 08:46 -0700, Bjorn Helgaas wrote: >> >> I don't know the IOMMU drivers well either, but it seems like they >> rely on notifications of device addition and removal (see >> iommu_bus_notifier()). It doesn't seem right for them to al

Re: [RFC PATCH] Crashdump Accepting Active IOMMU

2013-11-20 Thread Baoquan He
Hi Bill, I have tested this prototype patch on hp z420, it works very well. Looking forward to your formal patch set. I can help test and review. Baoquan Thanks On 11/18/13 at 11:30pm, Sumner, William wrote: > Thank you for testing this RFC patch. It is great to have confirmation that > the co

Re: Report from 2013 ARM kernel summit

2013-11-20 Thread Stephen Warren
On 11/20/2013 01:02 PM, Rob Herring wrote: > On Wed, Nov 20, 2013 at 12:39 AM, Hiroshi Doyu wrote: >> Hi Rob, >> >> Rob Herring wrote @ Tue, 19 Nov 2013 21:45:02 +0100: >> >>> On 11/19/2013 11:35 AM, Will Deacon wrote: On Tue, Nov 19, 2013 at 09:40:54AM +, Hiroshi Doyu wrote: > Grant

Re: Report from 2013 ARM kernel summit

2013-11-20 Thread Rob Herring
On Wed, Nov 20, 2013 at 4:31 AM, Will Deacon wrote: > On Tue, Nov 19, 2013 at 08:45:02PM +, Rob Herring wrote: >> On 11/19/2013 11:35 AM, Will Deacon wrote: >> > Adding Andreas and Rob for input on potential binding additions to the >> > SMMU. >> >> The above proposal would be an incompatible

Re: Report from 2013 ARM kernel summit

2013-11-20 Thread Rob Herring
On Wed, Nov 20, 2013 at 12:39 AM, Hiroshi Doyu wrote: > Hi Rob, > > Rob Herring wrote @ Tue, 19 Nov 2013 21:45:02 +0100: > >> On 11/19/2013 11:35 AM, Will Deacon wrote: >> > On Tue, Nov 19, 2013 at 09:40:54AM +, Hiroshi Doyu wrote: >> >> Grant Likely wrote @ Fri, 15 Nov 2013 >> >> 08:06:27

Re: Report from 2013 ARM kernel summit

2013-11-20 Thread Rob Herring
On Wed, Nov 20, 2013 at 7:53 AM, Thierry Reding wrote: > On Wed, Nov 20, 2013 at 10:31:11AM +, Will Deacon wrote: >> On Tue, Nov 19, 2013 at 08:45:02PM +, Rob Herring wrote: >> > On 11/19/2013 11:35 AM, Will Deacon wrote: >> > > Adding Andreas and Rob for input on potential binding additio

Re: [PATCH 0/9 v2] vfio-pci: add support for Freescale IOMMU (PAMU)

2013-11-20 Thread Alex Williamson
On Tue, 2013-11-19 at 10:47 +0530, Bharat Bhushan wrote: > From: Bharat Bhushan > > PAMU (FSL IOMMU) has a concept of primary window and subwindows. > Primary window corresponds to the complete guest iova address space > (including MSI space), with respect to IOMMU_API this is termed as > geometr

Re: [PATCHv5 2/9] driver/core: populate devices in order for IOMMUs

2013-11-20 Thread Stephen Warren
On 11/20/2013 07:03 AM, Hiroshi Doyu wrote: > Thierry Reding wrote @ Wed, 20 Nov 2013 14:14:48 > +0100: (Yes, what Thierry said) > >>> Does the above mean the following? >>> >>> int of_iommu_attach(struct device *dev) >>> { >>> int i; >>> struct of_phandle_args args; >>> >>> of

Re: [PATCH 1/1] IOMMU: Save pci device id instead of pci_dev* pointer for DMAR devices

2013-11-20 Thread David Woodhouse
On Fri, 2013-11-08 at 08:46 -0700, Bjorn Helgaas wrote: > > I don't know the IOMMU drivers well either, but it seems like they > rely on notifications of device addition and removal (see > iommu_bus_notifier()). It doesn't seem right for them to also use the > generic PCI interfaces like pci_get_

Re: [PATCH] iommu: Split iommu_unmaps

2013-11-20 Thread David Woodhouse
On Mon, 2013-11-11 at 16:09 -0700, Alex Williamson wrote: > On Thu, 2013-11-07 at 16:37 +, David Woodhouse wrote: > > On Fri, 2013-05-24 at 11:14 -0600, Alex Williamson wrote: > > > iommu_map splits requests into pages that the iommu driver reports > > > that it can handle. The iommu_unmap pat

Re: [PATCHv5 2/9] driver/core: populate devices in order for IOMMUs

2013-11-20 Thread Hiroshi Doyu
Thierry Reding wrote @ Wed, 20 Nov 2013 14:14:48 +0100: > > Does the above mean the following? > > > > int of_iommu_attach(struct device *dev) > > { > > int i; > > struct of_phandle_args args; > > > > of_property_for_each_phandle_with_args(dev->of_node, "iommus", > >

Re: Report from 2013 ARM kernel summit

2013-11-20 Thread Thierry Reding
On Wed, Nov 20, 2013 at 10:31:11AM +, Will Deacon wrote: > On Tue, Nov 19, 2013 at 08:45:02PM +, Rob Herring wrote: > > On 11/19/2013 11:35 AM, Will Deacon wrote: > > > Adding Andreas and Rob for input on potential binding additions to the > > > SMMU. > > > > The above proposal would be a

Re: [PATCHv5 2/9] driver/core: populate devices in order for IOMMUs

2013-11-20 Thread Thierry Reding
On Wed, Nov 20, 2013 at 04:17:08AM +0100, Hiroshi Doyu wrote: > Stephen Warren wrote @ Tue, 19 Nov 2013 22:22:47 > +0100: > > > On 11/19/2013 05:03 AM, Hiroshi Doyu wrote: > > > Hi Thierry, > > > > > > Thierry Reding wrote @ Tue, 19 Nov 2013 > > > 11:25:07 +0100: > > > > > >> From earlier di

Re: Report from 2013 ARM kernel summit

2013-11-20 Thread Will Deacon
On Tue, Nov 19, 2013 at 08:45:02PM +, Rob Herring wrote: > On 11/19/2013 11:35 AM, Will Deacon wrote: > > Adding Andreas and Rob for input on potential binding additions to the SMMU. > > The above proposal would be an incompatible change. However, I think we > could still deal with a change in