Subject: Re: [PATCH 2/3] myri10ge: more Intel chipsets providing aligned PCIe completions

From: Jeff Garzik <[email-masked]>
Date: Wed, 11 Apr 2007 11:54:20 -0400

Brice Goglin wrote:
> Add the Intel 5000 southbridge (aka Intel 6310/6311/6321ESB) PCIe ports
> and the Intel E30x0 chipsets to the whitelist of aligned PCIe completion.
> 
> Signed-off-by: Brice Goglin <[email-masked]>
> ---
>  drivers/net/myri10ge/myri10ge.c |   17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
> 
> Index: linux-rc/drivers/net/myri10ge/myri10ge.c
> ===================================================================
> --- linux-rc.orig/drivers/net/myri10ge/myri10ge.c	2007-04-10 21:03:59.000000000 +0200
> +++ linux-rc/drivers/net/myri10ge/myri10ge.c	2007-04-10 21:04:35.000000000 +0200
> @@ -2487,6 +2487,10 @@
>  
>  #define PCI_DEVICE_ID_INTEL_E5000_PCIE23 0x25f7
>  #define PCI_DEVICE_ID_INTEL_E5000_PCIE47 0x25fa
> +#define PCI_DEVICE_ID_INTEL_6300ESB_PCIEE1 0x3510
> +#define PCI_DEVICE_ID_INTEL_6300ESB_PCIEE4 0x351b
> +#define PCI_DEVICE_ID_INTEL_E3000_PCIE	0x2779
> +#define PCI_DEVICE_ID_INTEL_E3010_PCIE	0x277a
>  #define PCI_DEVICE_ID_SERVERWORKS_HT2100_PCIE_FIRST 0x140
>  #define PCI_DEVICE_ID_SERVERWORKS_HT2100_PCIE_LAST 0x142
>  
> @@ -2526,6 +2530,18 @@
>  				PCI_DEVICE_ID_SERVERWORKS_HT2100_PCIE_FIRST
>  				&& bridge->device <=
>  				PCI_DEVICE_ID_SERVERWORKS_HT2100_PCIE_LAST)
> +			    /* All Intel E3000/E3010 PCIE ports */
> +			    || (bridge->vendor == PCI_VENDOR_ID_INTEL
> +				&& (bridge->device ==
> +				    PCI_DEVICE_ID_INTEL_E3000_PCIE
> +				    || bridge->device ==
> +				    PCI_DEVICE_ID_INTEL_E3010_PCIE))
> +			    /* All Intel 6310/6311/6321ESB PCIE ports */
> +			    || (bridge->vendor == PCI_VENDOR_ID_INTEL
> +				&& bridge->device >=
> +				PCI_DEVICE_ID_INTEL_6300ESB_PCIEE1
> +				&& bridge->device <=
> +				PCI_DEVICE_ID_INTEL_6300ESB_PCIEE4)
>  			    /* All Intel E5000 PCIE ports */
>  			    || (bridge->vendor == PCI_VENDOR_ID_INTEL
>  				&& bridge->device >=

though I'm applying this, long term this should probably move out of 
myri driver