Re: AMD-Vi: Event logged [INVALID_DEVICE_REQUEST device=00:14.0 address=0x000000fdf9103300 flags=0x0600]

2015-10-16 Thread G. Richard Bellamy
"Works with..." On Fri, Oct 16, 2015 at 3:02 PM, G. Richard Bellamy wrote: > With with virtualization enabled and running. > > On Fri, Oct 16, 2015 at 2:02 AM, Joerg Roedel wrote: > >> On Thu, Oct 15, 2015 at 11:28:03PM -0700, G. Richard Bellamy wrote:

Re: AMD-Vi: Event logged [INVALID_DEVICE_REQUEST device=00:14.0 address=0x000000fdf9103300 flags=0x0600]

2015-10-16 Thread G. Richard Bellamy
With with virtualization enabled and running. On Fri, Oct 16, 2015 at 2:02 AM, Joerg Roedel wrote: > On Thu, Oct 15, 2015 at 11:28:03PM -0700, G. Richard Bellamy wrote: > > Applied the patch, and no longer receiving the kernel error messages. > Haven't > > tried to do anything

Re: [PATCH 2/2] arm64: Use gfpflags_allow_blocking()

2015-10-16 Thread Andrew Morton
On Fri, 16 Oct 2015 16:33:42 +0100 Robin Murphy wrote: > __GFP_WAIT is going away to live its life under a new identity; convert > __iommu_alloc_attrs() to the new helper function instead. > > ... > > --- a/arch/arm64/mm/dma-mapping.c > +++ b/arch/arm64/mm/dma-mapping.c >

Re: AMD-Vi: Event logged [INVALID_DEVICE_REQUEST device=00:14.0 address=0x000000fdf9103300 flags=0x0600]

2015-10-16 Thread G. Richard Bellamy
Applied the patch, and no longer receiving the kernel error messages. Haven't tried to do anything with my VMs yet, but thought you'd want to know sooner than later. On Wed, Oct 14, 2015 at 4:47 PM, G. Richard Bellamy wrote: > I don't think I'm explicitly trying to

Re: AMD-Vi: Event logged [INVALID_DEVICE_REQUEST device=00:14.0 address=0x000000fdf9103300 flags=0x0600]

2015-10-16 Thread Joerg Roedel
On Thu, Oct 15, 2015 at 11:28:03PM -0700, G. Richard Bellamy wrote: > Applied the patch, and no longer receiving the kernel error messages. Haven't > tried to do anything with my VMs yet, but thought you'd want to know sooner > than later. Okay, thanks. It is possible that the issue only happens

Re: [GIT PULL] iommu/arm-smmu: Updates for 4.4

2015-10-16 Thread Joerg Roedel
On Thu, Oct 15, 2015 at 05:16:22PM +0100, Will Deacon wrote: > are available in the git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/will/linux.git > for-joerg/arm-smmu/updates > > for you to fetch changes up to 166bdbd23161160f2abcea70621adba179050bee: Pulled, thanks

[PATCH 2/2] arm64: Use gfpflags_allow_blocking()

2015-10-16 Thread Robin Murphy
__GFP_WAIT is going away to live its life under a new identity; convert __iommu_alloc_attrs() to the new helper function instead. CC: Mel Gorman CC: Andrew Morton Reported-by: Sudeep Holla Signed-off-by: Robin Murphy

[PATCH 1/2] arm64: Workaround renaming of __GFP_WAIT

2015-10-16 Thread Robin Murphy
The imminent renaming of __GFP_WAIT in the mm tree conflicts with its use in the new IOMMU DMA ops; introduce a temporary local version of its replacement to smooth over the transition. This patch should be reverted at 4.4-rc1. CC: Mel Gorman CC: Andrew Morton

Re: [PATCH 1/2] arm64: Workaround renaming of __GFP_WAIT

2015-10-16 Thread Catalin Marinas
On Fri, Oct 16, 2015 at 04:33:41PM +0100, Robin Murphy wrote: > The imminent renaming of __GFP_WAIT in the mm tree conflicts with its > use in the new IOMMU DMA ops; introduce a temporary local version of > its replacement to smooth over the transition. > > This patch should be reverted at

Re: [PATCH 2/2] arm64: Use gfpflags_allow_blocking()

2015-10-16 Thread Catalin Marinas
On Fri, Oct 16, 2015 at 04:33:42PM +0100, Robin Murphy wrote: > __GFP_WAIT is going away to live its life under a new identity; convert > __iommu_alloc_attrs() to the new helper function instead. > > CC: Mel Gorman > CC: Andrew Morton >

[patch] iommu/vt-d: fix a loop in prq_event_thread()

2015-10-16 Thread Dan Carpenter
There is an extra semi-colon on this if statement so we always break on the first iteration. Fixes: 0204a4960982 ('iommu/vt-d: Add callback to device driver on page faults') Signed-off-by: Dan Carpenter diff --git a/drivers/iommu/intel-svm.c b/drivers/iommu/intel-svm.c

Re: [patch] iommu/vt-d: shift wrapping bug in prq_event_thread()

2015-10-16 Thread Woodhouse, David
On Thu, 2015-10-15 at 21:25 +0300, Dan Carpenter wrote: > The "req->addr" variable is a bit field declared as "u64 addr:52;". > The "address" variable is a u64. We need to cast "req->addr" to a u64 > before the shift or the result is truncated to 52 bits. > > Fixes: 0b9252a34858 ('iommu/vt-d: