Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754540AbZIPAL2 (ORCPT ); Tue, 15 Sep 2009 20:11:28 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753890AbZIPALX (ORCPT ); Tue, 15 Sep 2009 20:11:23 -0400 Received: from fifo99.com ([67.223.236.141]:47223 "EHLO fifo99.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753478AbZIPALW (ORCPT ); Tue, 15 Sep 2009 20:11:22 -0400 Subject: Re: [PATCH] enabling ACS P2P upstream forwarding From: Daniel Walker To: "Kay, Allen M" Cc: "linux-kernel@vger.kernel.org" , "linux-pci@vger.kernel.org" , "jbarnes@virtuousgeek.org" In-Reply-To: <57C9024A16AD2D4C97DC78E552063EA3E05DC218@orsmsx505.amr.corp.intel.com> References: <57C9024A16AD2D4C97DC78E552063EA3E05DC218@orsmsx505.amr.corp.intel.com> Content-Type: text/plain Date: Tue, 15 Sep 2009 17:12:17 -0700 Message-Id: <1253059937.11643.527.camel@desktop> Mime-Version: 1.0 X-Mailer: Evolution 2.24.3 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2282 Lines: 45 On Tue, 2009-09-15 at 16:44 -0700, Kay, Allen M wrote: > This patch enables P2P upstream forwarding in ACS capable PCIe switches. This solves two potential problems in virtualization environment where a PCIe device is assigned to a guest domain using a HW iommu such as VT-d: > > 1) Unintentional failure caused by guest physical address programmed into the device's DMA that happens to match the memory address range of other downstream ports in the same PCIe switch. This causes the PCI transaction to go to the matching downstream port instead of go to the root complex to get translated by VT-d as it should be. > > 2) Malicious guest software intentionally attacks another downstream PCIe device by programming the DMA address into the assigned device that matches memory address range of the downstream PCIe port. > > We are in process of implementing device filtering software in KVM/XEN management software to allow device assignment of PCIe devices behind a PCIe switch only if it has ACS capability and with the P2P upstream forwarding bits enabled. This patch is intended to work for both KVM and Xen environments. > > Signed-off-by: Allen Kay allen.m.kay@intel.com Both times that you've submitted this the patch has been corrupt.. Your mailer, or something is adding little "=20" or "=3D" all over. For instance, "Signed-off-by: Allen Kay allen.m.kay@intel.com --- drivers/pci/pci.c | 37 +++++++++++++++++++++++++++++++++++++ drivers/pci/pci.h | 1 + drivers/pci/probe.c | 3 +++ include/linux/pci_regs.h | 14 ++++++++++++++ 4 files changed, 55 insertions(+) =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c index 7b70312..e70abde 100644 ..." So there's no way to test or apply your patch .. Most reviewers are just going to walk away if they see stuff like this.. Daniel -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/