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

2013-04-09 Thread Joerg Roedel
On Mon, Apr 08, 2013 at 01:42:57PM -0400, Neil Horman wrote: > On Mon, Apr 08, 2013 at 11:17:23AM -0600, Bjorn Helgaas wrote: > > I don't care whether the "if (irq_remap_broken)" test is in > > irq_remapping.c or intel_irq_remapping.c. The quirk itself, where we > > actually look at config

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

2013-04-09 Thread Joerg Roedel
On Mon, Apr 08, 2013 at 01:42:57PM -0400, Neil Horman wrote: On Mon, Apr 08, 2013 at 11:17:23AM -0600, Bjorn Helgaas wrote: I don't care whether the if (irq_remap_broken) test is in irq_remapping.c or intel_irq_remapping.c. The quirk itself, where we actually look at config space, is

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

2013-04-08 Thread Neil Horman
On Mon, Apr 08, 2013 at 11:17:23AM -0600, Bjorn Helgaas wrote: > [+cc Joerg, Konrad] > > On Mon, Apr 8, 2013 at 9:29 AM, Don Dutile wrote: > > On 04/05/2013 09:55 PM, Bjorn Helgaas wrote: > >> > >> On Fri, Apr 5, 2013 at 1:31 PM, Neil Horman wrote: > >>> > >>> A few years back intel published a

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

2013-04-08 Thread Bjorn Helgaas
[+cc Joerg, Konrad] On Mon, Apr 8, 2013 at 9:29 AM, Don Dutile wrote: > On 04/05/2013 09:55 PM, Bjorn Helgaas wrote: >> >> On Fri, Apr 5, 2013 at 1:31 PM, Neil Horman wrote: >>> >>> A few years back intel published a spec update: >>> >>>

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

2013-04-08 Thread Don Dutile
On 04/05/2013 09:55 PM, Bjorn Helgaas wrote: On Fri, Apr 5, 2013 at 1:31 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

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

2013-04-08 Thread Don Dutile
On 04/05/2013 09:55 PM, Bjorn Helgaas wrote: On Fri, Apr 5, 2013 at 1:31 PM, Neil Hormannhor...@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

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

2013-04-08 Thread Bjorn Helgaas
[+cc Joerg, Konrad] On Mon, Apr 8, 2013 at 9:29 AM, Don Dutile ddut...@redhat.com wrote: On 04/05/2013 09:55 PM, Bjorn Helgaas wrote: On Fri, Apr 5, 2013 at 1:31 PM, Neil Hormannhor...@tuxdriver.com wrote: A few years back intel published a spec update:

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

2013-04-08 Thread Neil Horman
On Mon, Apr 08, 2013 at 11:17:23AM -0600, Bjorn Helgaas wrote: [+cc Joerg, Konrad] On Mon, Apr 8, 2013 at 9:29 AM, Don Dutile ddut...@redhat.com wrote: On 04/05/2013 09:55 PM, Bjorn Helgaas wrote: On Fri, Apr 5, 2013 at 1:31 PM, Neil Hormannhor...@tuxdriver.com wrote: A few years

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

2013-04-05 Thread Yinghai Lu
On Fri, Apr 5, 2013 at 6:25 PM, Neil Horman wrote: > I'm sorry. Forgot to change the wording of the error for the new model that > I'm following here. Although the message is mostly right as bios is > responsible for setting and clearing the IRQ remapping feature bit in the > chips

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

2013-04-05 Thread Bjorn Helgaas
On Fri, Apr 5, 2013 at 1:31 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 >

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

2013-04-05 Thread Neil Horman
I'm sorry. Forgot to change the wording of the error for the new model that I'm following here. Although the message is mostly right as bios is responsible for setting and clearing the IRQ remapping feature bit in the chips capabilities register. I'll fix and repost Monday Neil Yinghai Lu

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

2013-04-05 Thread Yinghai Lu
On Fri, Apr 5, 2013 at 12:31 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 v6] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-05 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 v6] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-05 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 v6] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-05 Thread Yinghai Lu
On Fri, Apr 5, 2013 at 12:31 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 v6] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-05 Thread Neil Horman
I'm sorry. Forgot to change the wording of the error for the new model that I'm following here. Although the message is mostly right as bios is responsible for setting and clearing the IRQ remapping feature bit in the chips capabilities register. I'll fix and repost Monday Neil Yinghai Lu

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

2013-04-05 Thread Bjorn Helgaas
On Fri, Apr 5, 2013 at 1:31 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 v6] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-05 Thread Yinghai Lu
On Fri, Apr 5, 2013 at 6:25 PM, Neil Horman nhor...@tuxdriver.com wrote: I'm sorry. Forgot to change the wording of the error for the new model that I'm following here. Although the message is mostly right as bios is responsible for setting and clearing the IRQ remapping feature bit in the