Re: [PATCH v9] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-16 Thread Neil Horman
On Tue, Apr 16, 2013 at 06:37:05PM +0200, Joerg Roedel wrote: > On Tue, Apr 16, 2013 at 09:35:56AM -0400, Neil Horman wrote: > > Actually, hold on that last note, the intel iommu init code doesn't seem to > > create any direct relationship between the set of iommu's and the pci_dev's > > that > >

Re: [PATCH v9] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-16 Thread Joerg Roedel
On Tue, Apr 16, 2013 at 09:35:56AM -0400, Neil Horman wrote: > Actually, hold on that last note, the intel iommu init code doesn't seem to > create any direct relationship between the set of iommu's and the pci_dev's > that > implement them. In the intel_irq_remapping_supported path I can loop

Re: [PATCH v9] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-16 Thread Neil Horman
On Tue, Apr 16, 2013 at 12:24:54PM +0200, Joerg Roedel wrote: > On Mon, Apr 15, 2013 at 06:41:17PM -0400, Neil Horman wrote: > > +#ifdef CONFIG_IRQ_REMAP > > +static void __init intel_remapping_check(int num, int slot, int func) > > +{ > > + u8 revision; > > + > > + revision =

Re: [PATCH v9] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-16 Thread Neil Horman
On Tue, Apr 16, 2013 at 12:24:54PM +0200, Joerg Roedel wrote: > On Mon, Apr 15, 2013 at 06:41:17PM -0400, Neil Horman wrote: > > +#ifdef CONFIG_IRQ_REMAP > > +static void __init intel_remapping_check(int num, int slot, int func) > > +{ > > + u8 revision; > > + > > + revision =

Re: [PATCH v9] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-16 Thread Joerg Roedel
On Mon, Apr 15, 2013 at 06:41:17PM -0400, Neil Horman wrote: > +#ifdef CONFIG_IRQ_REMAP > +static void __init intel_remapping_check(int num, int slot, int func) > +{ > + u8 revision; > + > + revision = read_pci_config_byte(num, slot, func, PCI_REVISION_ID); > + > + /* > + *

Re: [PATCH v9] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-16 Thread Arkadiusz Miskiewicz
On Tuesday 16 of April 2013, Neil Horman wrote: > A few years back intel published a spec update: > http://www.intel.com/content/dam/doc/specification-update/5520-and-5500-chi > pset-ioh-specification-update.pdf > > For the 5520 and 5500 chipsets which contained an errata (specificially > errata

Re: [PATCH v9] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-16 Thread Arkadiusz Miskiewicz
On Tuesday 16 of April 2013, Neil Horman wrote: A few years back intel published a spec update: http://www.intel.com/content/dam/doc/specification-update/5520-and-5500-chi pset-ioh-specification-update.pdf For the 5520 and 5500 chipsets which contained an errata (specificially errata 53),

Re: [PATCH v9] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-16 Thread Joerg Roedel
On Mon, Apr 15, 2013 at 06:41:17PM -0400, Neil Horman wrote: +#ifdef CONFIG_IRQ_REMAP +static void __init intel_remapping_check(int num, int slot, int func) +{ + u8 revision; + + revision = read_pci_config_byte(num, slot, func, PCI_REVISION_ID); + + /* + * Revision 0x13

Re: [PATCH v9] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-16 Thread Neil Horman
On Tue, Apr 16, 2013 at 12:24:54PM +0200, Joerg Roedel wrote: On Mon, Apr 15, 2013 at 06:41:17PM -0400, Neil Horman wrote: +#ifdef CONFIG_IRQ_REMAP +static void __init intel_remapping_check(int num, int slot, int func) +{ + u8 revision; + + revision = read_pci_config_byte(num,

Re: [PATCH v9] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-16 Thread Neil Horman
On Tue, Apr 16, 2013 at 12:24:54PM +0200, Joerg Roedel wrote: On Mon, Apr 15, 2013 at 06:41:17PM -0400, Neil Horman wrote: +#ifdef CONFIG_IRQ_REMAP +static void __init intel_remapping_check(int num, int slot, int func) +{ + u8 revision; + + revision = read_pci_config_byte(num,

Re: [PATCH v9] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-16 Thread Joerg Roedel
On Tue, Apr 16, 2013 at 09:35:56AM -0400, Neil Horman wrote: Actually, hold on that last note, the intel iommu init code doesn't seem to create any direct relationship between the set of iommu's and the pci_dev's that implement them. In the intel_irq_remapping_supported path I can loop over

Re: [PATCH v9] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-16 Thread Neil Horman
On Tue, Apr 16, 2013 at 06:37:05PM +0200, Joerg Roedel wrote: On Tue, Apr 16, 2013 at 09:35:56AM -0400, Neil Horman wrote: Actually, hold on that last note, the intel iommu init code doesn't seem to create any direct relationship between the set of iommu's and the pci_dev's that

Re: [PATCH v9] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-15 Thread Neil Horman
On Mon, Apr 15, 2013 at 04:02:56PM -0700, Yinghai Lu wrote: > On Mon, Apr 15, 2013 at 3:41 PM, Neil Horman wrote: > > A few years back intel published a spec update: > > http://www.intel.com/content/dam/doc/specification-update/5520-and-5500-chipset-ioh-specification-update.pdf > > > > > diff

Re: [PATCH v9] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-15 Thread Yinghai Lu
On Mon, Apr 15, 2013 at 3:41 PM, Neil Horman wrote: > A few years back intel published a spec update: > http://www.intel.com/content/dam/doc/specification-update/5520-and-5500-chipset-ioh-specification-update.pdf > > For the 5520 and 5500 chipsets which contained an errata (specificially errata >

[PATCH v9] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-15 Thread Neil Horman
A few years back intel published a spec update: http://www.intel.com/content/dam/doc/specification-update/5520-and-5500-chipset-ioh-specification-update.pdf For the 5520 and 5500 chipsets which contained an errata (specificially errata 53), which noted that these chipsets can't properly do

[PATCH v9] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-15 Thread Neil Horman
A few years back intel published a spec update: http://www.intel.com/content/dam/doc/specification-update/5520-and-5500-chipset-ioh-specification-update.pdf For the 5520 and 5500 chipsets which contained an errata (specificially errata 53), which noted that these chipsets can't properly do

Re: [PATCH v9] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-15 Thread Yinghai Lu
On Mon, Apr 15, 2013 at 3:41 PM, Neil Horman nhor...@tuxdriver.com wrote: A few years back intel published a spec update: http://www.intel.com/content/dam/doc/specification-update/5520-and-5500-chipset-ioh-specification-update.pdf For the 5520 and 5500 chipsets which contained an errata

Re: [PATCH v9] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-15 Thread Neil Horman
On Mon, Apr 15, 2013 at 04:02:56PM -0700, Yinghai Lu wrote: On Mon, Apr 15, 2013 at 3:41 PM, Neil Horman nhor...@tuxdriver.com wrote: A few years back intel published a spec update: http://www.intel.com/content/dam/doc/specification-update/5520-and-5500-chipset-ioh-specification-update.pdf