2011-05-28 18:15:08

by Mike Travis

[permalink] [raw]
Subject: [PATCH 0/7] Intel pci: Fix various problems with Intel IOMMU code


Various problems exist in the Intel IOMMU PCI driver when
using DMA remapping in 1:1 identity mode on UV systems and
devices cannot address all of physical memory.

This patchset addresses those problems.

--


2011-05-29 00:12:22

by David Woodhouse

[permalink] [raw]
Subject: Re: [PATCH 0/7] Intel pci: Fix various problems with Intel IOMMU code

On Sat, 2011-05-28 at 13:15 -0500, Mike Travis wrote:
> Various problems exist in the Intel IOMMU PCI driver when
> using DMA remapping in 1:1 identity mode on UV systems and
> devices cannot address all of physical memory.
>
> This patchset addresses those problems.

Patches 1-6 applied to iommu-2.6.git. Please test. Thanks.

Please note for future reference that 'From:' lines such as the one in
your patch 1/7 should be *first* in the body of your email. The git-am
tool doesn't find them and set the authorship if you put them with the
Signed-off-bys. Also please avoid putting whitespace at the beginning of
every line of your commit comments.

I still hate patch 7, but the point was that I wouldn't have to care
about it. Once your patches 1-6 are included in known stable releases
that your SCU can recognise, it can just pass 'iommu=pt' for those.

It's only the "probably buggy" kernels with older release numbers that
you'll pass 'forcedac,pt64' to, so your mission is to get patch 7 or
some variant of it accepted into *those* kernels.

--
dwmw2

2011-05-30 20:57:16

by Mike Travis

[permalink] [raw]
Subject: Re: [PATCH 0/7] Intel pci: Fix various problems with Intel IOMMU code



On 05/28/2011 05:12 PM, David Woodhouse wrote:
> On Sat, 2011-05-28 at 13:15 -0500, Mike Travis wrote:
>> Various problems exist in the Intel IOMMU PCI driver when
>> using DMA remapping in 1:1 identity mode on UV systems and
>> devices cannot address all of physical memory.
>>
>> This patchset addresses those problems.
>
> Patches 1-6 applied to iommu-2.6.git. Please test. Thanks.
>
> Please note for future reference that 'From:' lines such as the one in
> your patch 1/7 should be *first* in the body of your email. The git-am
> tool doesn't find them and set the authorship if you put them with the
> Signed-off-bys. Also please avoid putting whitespace at the beginning of
> every line of your commit comments.
>
> I still hate patch 7, but the point was that I wouldn't have to care
> about it. Once your patches 1-6 are included in known stable releases
> that your SCU can recognise, it can just pass 'iommu=pt' for those.
>
> It's only the "probably buggy" kernels with older release numbers that
> you'll pass 'forcedac,pt64' to, so your mission is to get patch 7 or
> some variant of it accepted into *those* kernels.
>

Thanks David, all noted. The From: thing is swallowed by the
quilt mail command if it's first. I'll see if I can't fix that.

Cheers.

2011-06-03 14:04:04

by David Woodhouse

[permalink] [raw]
Subject: Re: [PATCH 0/7] Intel pci: Fix various problems with Intel IOMMU code

On Sun, 2011-05-29 at 01:12 +0100, David Woodhouse wrote:
> Patches 1-6 applied to iommu-2.6.git. Please test. Thanks.

This is all now in Linus' tree for 3.0-rc2, and since you included
'Cc: stable' in the commit comments it should magically turn up in a
stable tree near you.

I will leave you to deal with the question of how to mark 'old' kernels
as actually having had later bugs fixed.

--
dwmw2