[PATCH 1/2 V2] iommu/amd: Add workaround for ERBT1312

2013-04-15 Thread suravee.suthikulpanit
From: Suravee Suthikulpanit suravee.suthikulpa...@amd.com The IOMMU interrupt handling in bottom half must clear the PPR log interrupt and event log interrupt bits to re-enable the interrupt. This is done by writing 1 to the memory mapped register to clear the bit. Due to hardware bug, if the

[PATCH 2/2 V2] iommu/AMD: Per-thread IOMMU Interrupt Handling

2013-04-15 Thread suravee.suthikulpanit
From: Suravee Suthikulpanit suravee.suthikulpa...@amd.com In the current interrupt handling scheme, there are as many threads as the number of IOMMUs. Each thread is created and assigned to an IOMMU at the time of registering interrupt handlers (request_threaded_irq). When an IOMMU HW generates

Re: [PATCH] intel-iommu: Synchronize gcmd value with global command register

2013-04-15 Thread Takao Indoh
(2013/04/10 13:47), Takao Indoh wrote: (2013/04/05 20:06), Joerg Roedel wrote: On Wed, Apr 03, 2013 at 09:24:39AM +0100, David Woodhouse wrote: On Wed, 2013-04-03 at 16:11 +0900, Takao Indoh wrote: Yeah, you are right. I forgot such a case. If you disable translation and there's some device

Re: [PATCH] intel-iommu: Synchronize gcmd value with global command register

2013-04-15 Thread Joerg Roedel
On Mon, Apr 15, 2013 at 06:00:13PM +0900, Takao Indoh wrote: On DMAR initialization during kdump boot, do you guys agree to change order like this? Current order: (1) Disable translation (2) PCI initialization (3) Make pgtable and enable translation. Order I'm proposing: (1) PCI

Re: [PATCH 1/3] iommu: Move swap_pci_ref function to pci.h.

2013-04-15 Thread Joerg Roedel
On Mon, Apr 15, 2013 at 12:42:00AM +0530, Varun Sethi wrote: swap_pci_ref function is used by the IOMMU API code for swapping pci device pointers, while determining the iommu group for the device. Currently this function was being implemented for different IOMMU drivers. This patch moves the

Re: [PATCH 1/3] iommu: Move swap_pci_ref function to pci.h.

2013-04-15 Thread Bjorn Helgaas
On Mon, Apr 15, 2013 at 8:58 AM, Joerg Roedel j...@8bytes.org wrote: On Mon, Apr 15, 2013 at 12:42:00AM +0530, Varun Sethi wrote: swap_pci_ref function is used by the IOMMU API code for swapping pci device pointers, while determining the iommu group for the device. Currently this function was

Re: PCI warning on boot 3.8.0-rc1

2013-04-15 Thread Alex Williamson
On Thu, 2013-04-11 at 11:23 -0600, Bjorn Helgaas wrote: On Wed, Apr 10, 2013 at 6:01 PM, Alex Williamson alex.william...@redhat.com wrote: On Wed, 2013-04-10 at 16:36 -0600, Bjorn Helgaas wrote: On Wed, Feb 06, 2013 at 08:58:41AM -0700, Alex Williamson wrote: On Wed, 2013-02-06 at 07:49

Re: PCI warning on boot 3.8.0-rc1

2013-04-15 Thread Bjorn Helgaas
On Mon, Apr 15, 2013 at 1:12 PM, Alex Williamson alex.william...@redhat.com wrote: On Thu, 2013-04-11 at 11:23 -0600, Bjorn Helgaas wrote: On Wed, Apr 10, 2013 at 6:01 PM, Alex Williamson alex.william...@redhat.com wrote: On Wed, 2013-04-10 at 16:36 -0600, Bjorn Helgaas wrote: On Wed, Feb