* [PATCH v2] PCI: Mark broken INTx masking for Mellanox devices @ 2014-08-01 4:54 Gavin Shan 2014-08-03 7:51 ` Amir Vadai 2014-08-11 13:48 ` Wei Yang 0 siblings, 2 replies; 13+ messages in thread From: Gavin Shan @ 2014-08-01 4:54 UTC (permalink / raw) To: linux-pci; +Cc: amirv, alex.williamson, Gavin Shan The VFIO driver is routing LSI interrupts by capturing, masking, and then delivering. When passing though Mallanox adapters from host to guest, interrupt storm was reported from host and guest. That's because we can't mask the LSI interrupt with help of PCI command register. [root@ncc-1701 ~]# lspci | grep Mellanox 0001:05:00.0 Ethernet controller: Mellanox Technologies MT27500 \ Family [ConnectX-3] 0005:01:00.0 Ethernet controller: Mellanox Technologies MT26448 \ [ConnectX EN 10GigE, PCIe 2.0 5GT/s] (rev b0) The patch marks broken INTx masking for Mellanox devices so that the VFIO driver will always mask the interrupt from interrupt controller side to avoid interrupt storm. Cc: Amir Vadai <amirv@mellanox.com> Suggested-by: Benjamin Herrenschmidt <benh@kernel.crashing.org> Signed-off-by: Gavin Shan <gwshan@linux.vnet.ibm.com> --- drivers/pci/quirks.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c index d0f6926..8c2b96f 100644 --- a/drivers/pci/quirks.c +++ b/drivers/pci/quirks.c @@ -2977,6 +2977,10 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_CHELSIO, 0x0030, quirk_broken_intx_masking); DECLARE_PCI_FIXUP_HEADER(0x1814, 0x0601, /* Ralink RT2800 802.11n PCI */ quirk_broken_intx_masking); +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MELLANOX, 0x1003, + quirk_broken_intx_masking); +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MELLANOX, 0x6750, + quirk_broken_intx_masking); /* * Realtek RTL8169 PCI Gigabit Ethernet Controller (rev 10) * Subsystem: Realtek RTL8169/8110 Family PCI Gigabit Ethernet NIC -- 1.8.3.2 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH v2] PCI: Mark broken INTx masking for Mellanox devices 2014-08-01 4:54 [PATCH v2] PCI: Mark broken INTx masking for Mellanox devices Gavin Shan @ 2014-08-03 7:51 ` Amir Vadai 2014-08-03 8:20 ` Gavin Shan 2014-08-11 13:48 ` Wei Yang 1 sibling, 1 reply; 13+ messages in thread From: Amir Vadai @ 2014-08-03 7:51 UTC (permalink / raw) To: Gavin Shan, linux-pci Cc: alex.williamson, Yevgeny Petrilin, Peter Paneah, Eli Cohen On 8/1/2014 7:54 AM, Gavin Shan wrote: > The VFIO driver is routing LSI interrupts by capturing, masking, > and then delivering. When passing though Mallanox adapters from > host to guest, interrupt storm was reported from host and guest. > That's because we can't mask the LSI interrupt with help of PCI > command register. Hi Gavin, What is the problem with masking the interrupts with the PCI command register? I'm asking because I want to understand in which devices we have the problem, and if it could be fixed by firmware guys. What are the implications of having the quirk? > > [root@ncc-1701 ~]# lspci | grep Mellanox > 0001:05:00.0 Ethernet controller: Mellanox Technologies MT27500 \ > Family [ConnectX-3] > 0005:01:00.0 Ethernet controller: Mellanox Technologies MT26448 \ > [ConnectX EN 10GigE, PCIe 2.0 5GT/s] (rev b0) > > The patch marks broken INTx masking for Mellanox devices so that > the VFIO driver will always mask the interrupt from interrupt > controller side to avoid interrupt storm. > > Cc: Amir Vadai <amirv@mellanox.com> > Suggested-by: Benjamin Herrenschmidt <benh@kernel.crashing.org> > Signed-off-by: Gavin Shan <gwshan@linux.vnet.ibm.com> > --- > drivers/pci/quirks.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c > index d0f6926..8c2b96f 100644 > --- a/drivers/pci/quirks.c > +++ b/drivers/pci/quirks.c > @@ -2977,6 +2977,10 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_CHELSIO, 0x0030, > quirk_broken_intx_masking); > DECLARE_PCI_FIXUP_HEADER(0x1814, 0x0601, /* Ralink RT2800 802.11n PCI */ > quirk_broken_intx_masking); > +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MELLANOX, 0x1003, > + quirk_broken_intx_masking); > +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MELLANOX, 0x6750, > + quirk_broken_intx_masking); I still don't understand what exactly is the problem, but I assume that there are other Mellanox devices that suffer from it. > /* > * Realtek RTL8169 PCI Gigabit Ethernet Controller (rev 10) > * Subsystem: Realtek RTL8169/8110 Family PCI Gigabit Ethernet NIC > Thanks, Amir ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v2] PCI: Mark broken INTx masking for Mellanox devices 2014-08-03 7:51 ` Amir Vadai @ 2014-08-03 8:20 ` Gavin Shan 2014-08-03 8:57 ` Eli Cohen 0 siblings, 1 reply; 13+ messages in thread From: Gavin Shan @ 2014-08-03 8:20 UTC (permalink / raw) To: Amir Vadai Cc: Gavin Shan, linux-pci, alex.williamson, Yevgeny Petrilin, Peter Paneah, Eli Cohen On Sun, Aug 03, 2014 at 10:51:08AM +0300, Amir Vadai wrote: >On 8/1/2014 7:54 AM, Gavin Shan wrote: >> The VFIO driver is routing LSI interrupts by capturing, masking, >> and then delivering. When passing though Mallanox adapters from >> host to guest, interrupt storm was reported from host and guest. >> That's because we can't mask the LSI interrupt with help of PCI >> command register. >Hi Gavin, > >What is the problem with masking the interrupts with the PCI command >register? I'm asking because I want to understand in which devices we >have the problem, and if it could be fixed by firmware guys. >What are the implications of having the quirk? > The way to mask the interrupt through PCI command register isn't taking effect on IBM power platform. So we have to have the quirk so that the interrupt could be masked from interrupt controller side with function disable_irq_nosync(). If the interrupt can't be masked properly, we detect interrupt storm reported from host/guest when passing through those devices via VFIO without suprise. >> >> [root@ncc-1701 ~]# lspci | grep Mellanox >> 0001:05:00.0 Ethernet controller: Mellanox Technologies MT27500 \ >> Family [ConnectX-3] >> 0005:01:00.0 Ethernet controller: Mellanox Technologies MT26448 \ >> [ConnectX EN 10GigE, PCIe 2.0 5GT/s] (rev b0) >> >> The patch marks broken INTx masking for Mellanox devices so that >> the VFIO driver will always mask the interrupt from interrupt >> controller side to avoid interrupt storm. >> >> Cc: Amir Vadai <amirv@mellanox.com> >> Suggested-by: Benjamin Herrenschmidt <benh@kernel.crashing.org> >> Signed-off-by: Gavin Shan <gwshan@linux.vnet.ibm.com> >> --- >> drivers/pci/quirks.c | 4 ++++ >> 1 file changed, 4 insertions(+) >> >> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c >> index d0f6926..8c2b96f 100644 >> --- a/drivers/pci/quirks.c >> +++ b/drivers/pci/quirks.c >> @@ -2977,6 +2977,10 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_CHELSIO, 0x0030, >> quirk_broken_intx_masking); >> DECLARE_PCI_FIXUP_HEADER(0x1814, 0x0601, /* Ralink RT2800 802.11n PCI */ >> quirk_broken_intx_masking); >> +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MELLANOX, 0x1003, >> + quirk_broken_intx_masking); >> +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MELLANOX, 0x6750, >> + quirk_broken_intx_masking); >I still don't understand what exactly is the problem, but I assume that >there are other Mellanox devices that suffer from it. > Yeah, that's possible. I didn't have chance to test other Mellanox devices except the above two. >> /* >> * Realtek RTL8169 PCI Gigabit Ethernet Controller (rev 10) >> * Subsystem: Realtek RTL8169/8110 Family PCI Gigabit Ethernet NIC >> > >Thanks, >Amir > Thanks, Gavin ^ permalink raw reply [flat|nested] 13+ messages in thread
* RE: [PATCH v2] PCI: Mark broken INTx masking for Mellanox devices 2014-08-03 8:20 ` Gavin Shan @ 2014-08-03 8:57 ` Eli Cohen 2014-08-03 14:30 ` Gavin Shan 0 siblings, 1 reply; 13+ messages in thread From: Eli Cohen @ 2014-08-03 8:57 UTC (permalink / raw) To: Gavin Shan, Amir Vadai Cc: linux-pci, alex.williamson, Yevgeny Petrilin, Peter Paneah >> >>What is the problem with masking the interrupts with the PCI command >>register? I'm asking because I want to understand in which devices we >>have the problem, and if it could be fixed by firmware guys. >>What are the implications of having the quirk? >> >The way to mask the interrupt through PCI command register isn't taking effect on IBM power platform. So we have to have the >quirk so that the interrupt could be masked from interrupt controller side with function disable_irq_nosync(). > >If the interrupt can't be masked properly, we detect interrupt storm reported from host/guest when passing through those devices >via VFIO without suprise. Hi Gavin, Does it have any effect on performance. Also, can you tell in which cases interrupts need to be masked? ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v2] PCI: Mark broken INTx masking for Mellanox devices 2014-08-03 8:57 ` Eli Cohen @ 2014-08-03 14:30 ` Gavin Shan 2014-08-03 15:08 ` Alex Williamson 0 siblings, 1 reply; 13+ messages in thread From: Gavin Shan @ 2014-08-03 14:30 UTC (permalink / raw) To: Eli Cohen Cc: Gavin Shan, Amir Vadai, linux-pci, alex.williamson, Yevgeny Petrilin, Peter Paneah On Sun, Aug 03, 2014 at 08:57:39AM +0000, Eli Cohen wrote: >>> >>>What is the problem with masking the interrupts with the PCI command >>>register? I'm asking because I want to understand in which devices we >>>have the problem, and if it could be fixed by firmware guys. >>>What are the implications of having the quirk? >>> > >>The way to mask the interrupt through PCI command register isn't taking effect on IBM power platform. So we have to have the >quirk so that the interrupt could be masked from interrupt controller side with function disable_irq_nosync(). >> >>If the interrupt can't be masked properly, we detect interrupt storm reported from host/guest when passing through those devices >via VFIO without suprise. > >Hi Gavin, >Does it have any effect on performance. Also, can you tell in which cases interrupts need to be masked? > Eli, more code needed to be run for masking the LSI from interrupt controller side than from PCI command register. I was passing through Mellanox devices from host to guest with VFIO, and I designated to use LSI in the guest side. More details could be found in drivers/vfio/pci/vfio_pci_intrs.c::vfio_intx_handler() Thanks, Gavin ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v2] PCI: Mark broken INTx masking for Mellanox devices 2014-08-03 14:30 ` Gavin Shan @ 2014-08-03 15:08 ` Alex Williamson 2014-08-04 0:34 ` Gavin Shan 0 siblings, 1 reply; 13+ messages in thread From: Alex Williamson @ 2014-08-03 15:08 UTC (permalink / raw) To: Gavin Shan Cc: Eli Cohen, Amir Vadai, linux-pci, Yevgeny Petrilin, Peter Paneah On Mon, 2014-08-04 at 00:30 +1000, Gavin Shan wrote: > On Sun, Aug 03, 2014 at 08:57:39AM +0000, Eli Cohen wrote: > >>> > >>>What is the problem with masking the interrupts with the PCI command > >>>register? I'm asking because I want to understand in which devices we > >>>have the problem, and if it could be fixed by firmware guys. > >>>What are the implications of having the quirk? > >>> > > > >>The way to mask the interrupt through PCI command register isn't taking effect on IBM power platform. So we have to have the >quirk so that the interrupt could be masked from interrupt controller side with function disable_irq_nosync(). > >> > >>If the interrupt can't be masked properly, we detect interrupt storm reported from host/guest when passing through those devices >via VFIO without suprise. > > > >Hi Gavin, > >Does it have any effect on performance. Also, can you tell in which cases interrupts need to be masked? > > > > Eli, more code needed to be run for masking the LSI from interrupt controller > side than from PCI command register. > > I was passing through Mellanox devices from host to guest with VFIO, and I > designated to use LSI in the guest side. More details could be found in > drivers/vfio/pci/vfio_pci_intrs.c::vfio_intx_handler() INTx is relatively high overhead already for device assignment since the interrupt is level triggered and needs to be masked on the host while the guest is processing it. The more important restriction imposed by marking broken INTx masking is that the device needs an exclusive interrupt line in order to be assigned to a guest. That may be common practice on IBM power, but on x86 it can make it much harder to configure the system for this use case. Thanks, Alex ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v2] PCI: Mark broken INTx masking for Mellanox devices 2014-08-03 15:08 ` Alex Williamson @ 2014-08-04 0:34 ` Gavin Shan 2014-08-07 3:09 ` Gavin Shan 0 siblings, 1 reply; 13+ messages in thread From: Gavin Shan @ 2014-08-04 0:34 UTC (permalink / raw) To: Alex Williamson Cc: Gavin Shan, Eli Cohen, Amir Vadai, linux-pci, Yevgeny Petrilin, Peter Paneah On Sun, Aug 03, 2014 at 09:08:06AM -0600, Alex Williamson wrote: >On Mon, 2014-08-04 at 00:30 +1000, Gavin Shan wrote: >> On Sun, Aug 03, 2014 at 08:57:39AM +0000, Eli Cohen wrote: >> >>> >> >>>What is the problem with masking the interrupts with the PCI command >> >>>register? I'm asking because I want to understand in which devices we >> >>>have the problem, and if it could be fixed by firmware guys. >> >>>What are the implications of having the quirk? >> >>> >> > >> >>The way to mask the interrupt through PCI command register isn't taking effect on IBM power platform. So we have to have the >quirk so that the interrupt could be masked from interrupt controller side with function disable_irq_nosync(). >> >> >> >>If the interrupt can't be masked properly, we detect interrupt storm reported from host/guest when passing through those devices >via VFIO without suprise. >> > >> >Hi Gavin, >> >Does it have any effect on performance. Also, can you tell in which cases interrupts need to be masked? >> > >> >> Eli, more code needed to be run for masking the LSI from interrupt controller >> side than from PCI command register. >> >> I was passing through Mellanox devices from host to guest with VFIO, and I >> designated to use LSI in the guest side. More details could be found in >> drivers/vfio/pci/vfio_pci_intrs.c::vfio_intx_handler() > >INTx is relatively high overhead already for device assignment since the >interrupt is level triggered and needs to be masked on the host while >the guest is processing it. The more important restriction imposed by >marking broken INTx masking is that the device needs an exclusive >interrupt line in order to be assigned to a guest. That may be common >practice on IBM power, but on x86 it can make it much harder to >configure the system for this use case. Thanks, > Power platform has the similar situation: Each PCI controller has 4 LSIs shared by all child devices attached to the PHB. It would be racy if one LSI is shared by 2 or more devices. So masking LSI with PCI command register is the preferred mechanism. Unfortunately, it doesn't work on those 2 Mellanox devices. With the quirk, it's workable at least. Thanks, Gavin >Alex > > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v2] PCI: Mark broken INTx masking for Mellanox devices 2014-08-04 0:34 ` Gavin Shan @ 2014-08-07 3:09 ` Gavin Shan 0 siblings, 0 replies; 13+ messages in thread From: Gavin Shan @ 2014-08-07 3:09 UTC (permalink / raw) To: Gavin Shan Cc: Alex Williamson, Eli Cohen, Amir Vadai, linux-pci, Yevgeny Petrilin, Peter Paneah On Mon, Aug 04, 2014 at 10:34:38AM +1000, Gavin Shan wrote: >On Sun, Aug 03, 2014 at 09:08:06AM -0600, Alex Williamson wrote: >>On Mon, 2014-08-04 at 00:30 +1000, Gavin Shan wrote: >>> On Sun, Aug 03, 2014 at 08:57:39AM +0000, Eli Cohen wrote: >>> >>> >>> >>>What is the problem with masking the interrupts with the PCI command >>> >>>register? I'm asking because I want to understand in which devices we >>> >>>have the problem, and if it could be fixed by firmware guys. >>> >>>What are the implications of having the quirk? >>> >>> >>> > >>> >>The way to mask the interrupt through PCI command register isn't taking effect on IBM power platform. So we have to have the >quirk so that the interrupt could be masked from interrupt controller side with function disable_irq_nosync(). >>> >> >>> >>If the interrupt can't be masked properly, we detect interrupt storm reported from host/guest when passing through those devices >via VFIO without suprise. >>> > >>> >Hi Gavin, >>> >Does it have any effect on performance. Also, can you tell in which cases interrupts need to be masked? >>> > >>> >>> Eli, more code needed to be run for masking the LSI from interrupt controller >>> side than from PCI command register. >>> >>> I was passing through Mellanox devices from host to guest with VFIO, and I >>> designated to use LSI in the guest side. More details could be found in >>> drivers/vfio/pci/vfio_pci_intrs.c::vfio_intx_handler() >> >>INTx is relatively high overhead already for device assignment since the >>interrupt is level triggered and needs to be masked on the host while >>the guest is processing it. The more important restriction imposed by >>marking broken INTx masking is that the device needs an exclusive >>interrupt line in order to be assigned to a guest. That may be common >>practice on IBM power, but on x86 it can make it much harder to >>configure the system for this use case. Thanks, >> > >Power platform has the similar situation: Each PCI controller has 4 >LSIs shared by all child devices attached to the PHB. It would be >racy if one LSI is shared by 2 or more devices. So masking LSI with >PCI command register is the preferred mechanism. Unfortunately, it >doesn't work on those 2 Mellanox devices. With the quirk, it's workable >at least. > ping, Any more comments on this? :-) Thanks, Gavin ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v2] PCI: Mark broken INTx masking for Mellanox devices 2014-08-01 4:54 [PATCH v2] PCI: Mark broken INTx masking for Mellanox devices Gavin Shan 2014-08-03 7:51 ` Amir Vadai @ 2014-08-11 13:48 ` Wei Yang 2014-08-12 3:52 ` Gavin Shan 1 sibling, 1 reply; 13+ messages in thread From: Wei Yang @ 2014-08-11 13:48 UTC (permalink / raw) To: Gavin Shan; +Cc: linux-pci, amirv, alex.williamson On Fri, Aug 01, 2014 at 02:54:31PM +1000, Gavin Shan wrote: >The VFIO driver is routing LSI interrupts by capturing, masking, >and then delivering. When passing though Mallanox adapters from one typo Mellanox Others, looks good to me. >host to guest, interrupt storm was reported from host and guest. >That's because we can't mask the LSI interrupt with help of PCI >command register. > >[root@ncc-1701 ~]# lspci | grep Mellanox >0001:05:00.0 Ethernet controller: Mellanox Technologies MT27500 \ > Family [ConnectX-3] >0005:01:00.0 Ethernet controller: Mellanox Technologies MT26448 \ > [ConnectX EN 10GigE, PCIe 2.0 5GT/s] (rev b0) > >The patch marks broken INTx masking for Mellanox devices so that >the VFIO driver will always mask the interrupt from interrupt >controller side to avoid interrupt storm. > >Cc: Amir Vadai <amirv@mellanox.com> >Suggested-by: Benjamin Herrenschmidt <benh@kernel.crashing.org> >Signed-off-by: Gavin Shan <gwshan@linux.vnet.ibm.com> >--- > drivers/pci/quirks.c | 4 ++++ > 1 file changed, 4 insertions(+) > >diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c >index d0f6926..8c2b96f 100644 >--- a/drivers/pci/quirks.c >+++ b/drivers/pci/quirks.c >@@ -2977,6 +2977,10 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_CHELSIO, 0x0030, > quirk_broken_intx_masking); > DECLARE_PCI_FIXUP_HEADER(0x1814, 0x0601, /* Ralink RT2800 802.11n PCI */ > quirk_broken_intx_masking); >+DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MELLANOX, 0x1003, >+ quirk_broken_intx_masking); >+DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MELLANOX, 0x6750, >+ quirk_broken_intx_masking); > /* > * Realtek RTL8169 PCI Gigabit Ethernet Controller (rev 10) > * Subsystem: Realtek RTL8169/8110 Family PCI Gigabit Ethernet NIC >-- >1.8.3.2 > >-- >To unsubscribe from this list: send the line "unsubscribe linux-pci" in >the body of a message to majordomo@vger.kernel.org >More majordomo info at http://vger.kernel.org/majordomo-info.html -- Richard Yang Help you, Help me ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v2] PCI: Mark broken INTx masking for Mellanox devices 2014-08-11 13:48 ` Wei Yang @ 2014-08-12 3:52 ` Gavin Shan 2014-08-12 7:51 ` Amir Vadai 0 siblings, 1 reply; 13+ messages in thread From: Gavin Shan @ 2014-08-12 3:52 UTC (permalink / raw) To: Wei Yang; +Cc: Gavin Shan, linux-pci, amirv, alex.williamson On Mon, Aug 11, 2014 at 09:48:54PM +0800, Wei Yang wrote: >On Fri, Aug 01, 2014 at 02:54:31PM +1000, Gavin Shan wrote: >>The VFIO driver is routing LSI interrupts by capturing, masking, >>and then delivering. When passing though Mallanox adapters from > >one typo Mellanox > >Others, looks good to me. > Thanks, Richard. I'll correct it in next revision. However, I'm still waiting for comments or ACK from Alex/Eli/Amir. Thanks, Gavin >>host to guest, interrupt storm was reported from host and guest. >>That's because we can't mask the LSI interrupt with help of PCI >>command register. >> >>[root@ncc-1701 ~]# lspci | grep Mellanox >>0001:05:00.0 Ethernet controller: Mellanox Technologies MT27500 \ >> Family [ConnectX-3] >>0005:01:00.0 Ethernet controller: Mellanox Technologies MT26448 \ >> [ConnectX EN 10GigE, PCIe 2.0 5GT/s] (rev b0) >> >>The patch marks broken INTx masking for Mellanox devices so that >>the VFIO driver will always mask the interrupt from interrupt >>controller side to avoid interrupt storm. >> >>Cc: Amir Vadai <amirv@mellanox.com> >>Suggested-by: Benjamin Herrenschmidt <benh@kernel.crashing.org> >>Signed-off-by: Gavin Shan <gwshan@linux.vnet.ibm.com> >>--- >> drivers/pci/quirks.c | 4 ++++ >> 1 file changed, 4 insertions(+) >> >>diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c >>index d0f6926..8c2b96f 100644 >>--- a/drivers/pci/quirks.c >>+++ b/drivers/pci/quirks.c >>@@ -2977,6 +2977,10 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_CHELSIO, 0x0030, >> quirk_broken_intx_masking); >> DECLARE_PCI_FIXUP_HEADER(0x1814, 0x0601, /* Ralink RT2800 802.11n PCI */ >> quirk_broken_intx_masking); >>+DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MELLANOX, 0x1003, >>+ quirk_broken_intx_masking); >>+DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MELLANOX, 0x6750, >>+ quirk_broken_intx_masking); >> /* >> * Realtek RTL8169 PCI Gigabit Ethernet Controller (rev 10) >> * Subsystem: Realtek RTL8169/8110 Family PCI Gigabit Ethernet NIC >>-- >>1.8.3.2 >> >>-- >>To unsubscribe from this list: send the line "unsubscribe linux-pci" in >>the body of a message to majordomo@vger.kernel.org >>More majordomo info at http://vger.kernel.org/majordomo-info.html > >-- >Richard Yang >Help you, Help me ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v2] PCI: Mark broken INTx masking for Mellanox devices 2014-08-12 3:52 ` Gavin Shan @ 2014-08-12 7:51 ` Amir Vadai 2014-08-12 8:57 ` Gavin Shan 0 siblings, 1 reply; 13+ messages in thread From: Amir Vadai @ 2014-08-12 7:51 UTC (permalink / raw) To: Gavin Shan, Wei Yang Cc: linux-pci, alex.williamson, Or Gerlitz, Yevgeny Petrilin, Eli Cohen, Peter Paneah On 8/12/2014 6:52 AM, Gavin Shan wrote: > On Mon, Aug 11, 2014 at 09:48:54PM +0800, Wei Yang wrote: >> >On Fri, Aug 01, 2014 at 02:54:31PM +1000, Gavin Shan wrote: >>> >>The VFIO driver is routing LSI interrupts by capturing, masking, >>> >>and then delivering. When passing though Mallanox adapters from >> > >> >one typo Mellanox >> > >> >Others, looks good to me. >> > > Thanks, Richard. I'll correct it in next revision. However, I'm > still waiting for comments or ACK from Alex/Eli/Amir. > > Thanks, > Gavin > Hi Gavin, Sorry for the delay - wanted to understand the exact limitations with our Hardware/Firmware guys. Please change the device to PCI_ANY_ID since all Mellanox devices currently need the quirk. The issue will be fixed in the next GA firmwares. Thanks, Amir ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v2] PCI: Mark broken INTx masking for Mellanox devices 2014-08-12 7:51 ` Amir Vadai @ 2014-08-12 8:57 ` Gavin Shan 2014-08-12 9:04 ` Amir Vadai 0 siblings, 1 reply; 13+ messages in thread From: Gavin Shan @ 2014-08-12 8:57 UTC (permalink / raw) To: Amir Vadai Cc: Gavin Shan, Wei Yang, linux-pci, alex.williamson, Or Gerlitz, Yevgeny Petrilin, Eli Cohen, Peter Paneah On Tue, Aug 12, 2014 at 10:51:57AM +0300, Amir Vadai wrote: >On 8/12/2014 6:52 AM, Gavin Shan wrote: >> On Mon, Aug 11, 2014 at 09:48:54PM +0800, Wei Yang wrote: >>> >On Fri, Aug 01, 2014 at 02:54:31PM +1000, Gavin Shan wrote: >>>> >>The VFIO driver is routing LSI interrupts by capturing, masking, >>>> >>and then delivering. When passing though Mallanox adapters from >>> > >>> >one typo Mellanox >>> > >>> >Others, looks good to me. >>> > >> Thanks, Richard. I'll correct it in next revision. However, I'm >> still waiting for comments or ACK from Alex/Eli/Amir. >> >> Thanks, >> Gavin >> > >Hi Gavin, > >Sorry for the delay - wanted to understand the exact limitations with >our Hardware/Firmware guys. > >Please change the device to PCI_ANY_ID since all Mellanox devices >currently need the quirk. >The issue will be fixed in the next GA firmwares. > Amir, thanks for confirm. I'll update in next revision accordingly. Please ack next revision if possible. Thanks, Gavin >Thanks, >Amir > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v2] PCI: Mark broken INTx masking for Mellanox devices 2014-08-12 8:57 ` Gavin Shan @ 2014-08-12 9:04 ` Amir Vadai 0 siblings, 0 replies; 13+ messages in thread From: Amir Vadai @ 2014-08-12 9:04 UTC (permalink / raw) To: Gavin Shan Cc: Wei Yang, linux-pci, alex.williamson, Or Gerlitz, Yevgeny Petrilin, Eli Cohen, Peter Paneah On 8/12/2014 11:57 AM, Gavin Shan wrote: > On Tue, Aug 12, 2014 at 10:51:57AM +0300, Amir Vadai wrote: >> On 8/12/2014 6:52 AM, Gavin Shan wrote: [..] > > Amir, thanks for confirm. I'll update in next revision accordingly. > Please ack next revision if possible. Sure. Amir ^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2014-08-12 9:04 UTC | newest] Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2014-08-01 4:54 [PATCH v2] PCI: Mark broken INTx masking for Mellanox devices Gavin Shan 2014-08-03 7:51 ` Amir Vadai 2014-08-03 8:20 ` Gavin Shan 2014-08-03 8:57 ` Eli Cohen 2014-08-03 14:30 ` Gavin Shan 2014-08-03 15:08 ` Alex Williamson 2014-08-04 0:34 ` Gavin Shan 2014-08-07 3:09 ` Gavin Shan 2014-08-11 13:48 ` Wei Yang 2014-08-12 3:52 ` Gavin Shan 2014-08-12 7:51 ` Amir Vadai 2014-08-12 8:57 ` Gavin Shan 2014-08-12 9:04 ` Amir Vadai
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.