Re: [PATCH 0/10] iommu/vt-d: Fix intel vt-d faults in kdump kernel

2015-01-05 Thread Li, ZhenHua
According to Takao Indoh's testing, seems adding flush after loading old irte and updating old irte does not fix the dmar faults. According to Takao Indoh's log and your log, the faults happens while and after driver is loaded. Maybe I am using incorrect code in 08/10. On 01/06/2015 02:37 P

Re: Hang (due to HW?) in qi_submit_sync()

2015-01-05 Thread Alex Williamson
On Mon, 2015-01-05 at 20:39 -0800, Roland Dreier wrote: > On Mon, Jan 5, 2015 at 6:54 PM, Alex Williamson > wrote: > > Try disabling CONFIG_NET_DMA > > We already have that disabled (well, in 3.10 it depends on BROKEN, and > we don't have BROKEN enabled :). Only since v3.10.26 is it marked BROKE

Re: [PATCH 0/10] iommu/vt-d: Fix intel vt-d faults in kdump kernel

2015-01-05 Thread Baoquan He
Hi Zhenhua, I just tested your patchset based on 3.19.0-rc2+, and found several dmar fault and intr-remap fault, it seems not the same as Takao's, please check the attachment. Thanks Baoquan [root@dhcp-16-105 ~]# kdumpctl restart kexec: failed to unloaded kdump kernel Stopping kdump: [FAILED] +

Re: Hang (due to HW?) in qi_submit_sync()

2015-01-05 Thread Roland Dreier
On Mon, Jan 5, 2015 at 6:54 PM, Alex Williamson wrote: > Try disabling CONFIG_NET_DMA We already have that disabled (well, in 3.10 it depends on BROKEN, and we don't have BROKEN enabled :). However I'm curious why you suggest that. Because some of the devices we're accessing via vfio are in fac

Re: Hang (due to HW?) in qi_submit_sync()

2015-01-05 Thread Alex Williamson
On Mon, 2015-01-05 at 16:57 -0800, Roland Dreier wrote: > From: Roland Dreier > > Hi, we're running kernel 3.10.59 (pretty recent long-term kernel) on a > 2-socket Xeon E5 v3 (Haswell) system. We're using vfio to access some > PCI devices from userspace, and occasionally when we kill a process,

Re: [PATCH 0/10] iommu/vt-d: Fix intel vt-d faults in kdump kernel

2015-01-05 Thread Li, ZhenHua
Thank you very much for your help. I have found there are several places need flush, and I will send a new version of this patchset with the flush functions. Regards Zhenhua On 01/06/2015 08:18 AM, Takao Indoh wrote: > On 2014/12/29 12:15, Li, ZhenHua wrote: >> Hi Takao Indoh, >> >> Happy New Ye

RE: [v3 00/26] Add VT-d Posted-Interrupts support

2015-01-05 Thread Wu, Feng
Ping... Hi Joerg & David, Could you please have a look at the IOMMU part of this series (patch 02 - 04, patch 06 - 09 , patch 26)? Hi Thomas, Ingo, & Peter, Could you please have a look at this series, especially for patch 01, 05, 21? Thanks, Feng > -Original Message- > From: Wu, Fen

Hang (due to HW?) in qi_submit_sync()

2015-01-05 Thread Roland Dreier
From: Roland Dreier Hi, we're running kernel 3.10.59 (pretty recent long-term kernel) on a 2-socket Xeon E5 v3 (Haswell) system. We're using vfio to access some PCI devices from userspace, and occasionally when we kill a process, we see the system hang in qi_submit_sync(). Based on a very old p

Re: [PATCH 0/10] iommu/vt-d: Fix intel vt-d faults in kdump kernel

2015-01-05 Thread Takao Indoh
On 2014/12/29 12:15, Li, ZhenHua wrote: > Hi Takao Indoh, > > Happy New Year, and thank you very much for you help. The flush is quite Happy new year! > a problem, as there are several places the flush function should be called, > I think the flush should be placed in functions like __iommu_

Remove Code in ipmmu-vmsa.c

2015-01-05 Thread nick
Greetings Joreg, I am wondering if you can remove this comment: TODO: Is TLB flush really needed ? as it's rather clear we are needing to flush the TLB cache based on the above comment in the function,ipmmu_domain_destroy_context in ipmmu_vmsa.c. In addition can you put a suggested by Nicholas Kr

Re: [PATCH 0/2] Change order of linkage in kernel makefiles for amdkfd

2015-01-05 Thread Thierry Reding
On Mon, Dec 29, 2014 at 10:34:32AM +0100, Christian König wrote: > Am 29.12.2014 um 09:16 schrieb Laurent Pinchart: > >Hi Oded, > > > >On Sunday 28 December 2014 13:36:50 Oded Gabbay wrote: > >>On 12/26/2014 11:19 AM, Laurent Pinchart wrote: > >>>On Thursday 25 December 2014 14:20:59 Thierry Reding

Re: [PATCH] iommu: Fix build of allmod/allyes config on ARMv8 architecture

2015-01-05 Thread Joerg Roedel
On Mon, Jan 05, 2015 at 12:09:03PM +0100, Krzysztof Kozlowski wrote: > Thanks, but... Few days after my patch Mark Brown posted something > similar: > http://lists.infradead.org/pipermail/linux-arm-kernel/2014-December/310743.html > > It was merged by Arnd Bergman and sent to Linus around 3.19-rc1

Re: [PATCH] iommu: Fix build of allmod/allyes config on ARMv8 architecture

2015-01-05 Thread Krzysztof Kozlowski
On pon, 2015-01-05 at 12:03 +0100, Joerg Roedel wrote: > On Fri, Dec 05, 2014 at 02:47:37PM +0100, Krzysztof Kozlowski wrote: > > drivers/iommu/Kconfig | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > Applied to iommu/fixes, thanks. Thanks, but... Few days after my patch Mark Brown

Re: [PATCH 09/28] iommu: drop owner assignment from platform_drivers

2015-01-05 Thread Joerg Roedel
On Sun, Dec 21, 2014 at 10:14:30PM +0100, Wolfram Sang wrote: > This platform_driver does not need to set an owner, it will be populated by > the > driver core. > > Signed-off-by: Wolfram Sang > --- > Generated with coccinelle. SmPL file is in the introductory msg. The big > cleanup was pulled i

Re: [PATCH] iommu: Fix build of allmod/allyes config on ARMv8 architecture

2015-01-05 Thread Joerg Roedel
On Fri, Dec 05, 2014 at 02:47:37PM +0100, Krzysztof Kozlowski wrote: > drivers/iommu/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Applied to iommu/fixes, thanks. ___ iommu mailing list iommu@lists.linux-foundation.org https://lists.l

Re: [PATCH 1/2] iommu/vt-d: Fix dmar_domain leak in iommu_attach_device

2015-01-05 Thread Joerg Roedel
Hi Jerry, On Wed, Dec 17, 2014 at 02:37:47PM -0700, Jerry Hoemann wrote: > Before applying this change on a 3.18-rc7 kernel layered on a > RHEL 7.0 root disk, I was able to reproduce the memory > leak that Alex reported when powering on/off a VM w/ a PCI > device assigned to it. > > After applyin

Re: [PATCH FOR v3.19 - v2] iommu/ipmmu-vmsa: change IOMMU_EXEC to IOMMU_NOEXEC

2015-01-05 Thread Joerg Roedel
On Mon, Dec 15, 2014 at 08:41:13PM +0200, Laurent Pinchart wrote: > Joerg, this is a compilation breakage fix for v3.19. Could you please take it > in your tree and submit it early during the -rc cycle ? Patch applied, sorry for the delay. ___ iommu mai