Re: [PATCH 1/2] msi: Invert the sense of the MSI enables.

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Roland Dreier
Date: Friday, May 25, 2007 - 1:25 pm

> > In addition to PCI INTx compatible interrupt emulation, PCI Express
 > > requires support of MSI or MSI-X or both. 
 > Which suggests that INTx support is required.
 > 
 > I do not find any wording that suggest the opposite.
 > I do see it stated that it is intended to EOL support for INTx at
 > some point.
 > 
 > Where did you see it mentioned that INTx was optional?

I don't see any requirement that a device that generates MSI
interrupts must also be able to signal the same interrupts via INTx.
The spec explicitly says:

    "All PCI Express device Functions that are capable of generating
    interrupts must support MSI or MSI-X or both."

but there is no corresponding explicit requirement that legacy INTx
mode be supported, so it certainly seems permitted for a device not to
generate INTx interrupts.  In fact as you alluded to, the spec says,

    "The legacy INTx emulation mechanism may be deprecated in a future
    version of this specification."

and I wouldn't think the intention would be for one version of the
spec to *require* something that is planned on being deprecated later.

And the Pathscale guys were pretty confident that their device was
compliant with the PCIe spec.

 - R.
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 1/2] msi: Invert the sense of the MSI enables., Eric W. Biederman, (Thu May 24, 9:19 pm)
[PATCH 2/2] msi: Add support for the Intel chipsets that ..., Eric W. Biederman, (Thu May 24, 9:26 pm)
Re: [PATCH 1/2] msi: Invert the sense of the MSI enables., Andrew Morton, (Thu May 24, 9:31 pm)
Re: [PATCH 1/2] msi: Invert the sense of the MSI enables., Michael Ellerman, (Thu May 24, 10:14 pm)
Re: [PATCH 1/2] msi: Invert the sense of the MSI enables., Eric W. Biederman, (Thu May 24, 10:20 pm)
Re: [PATCH 1/2] msi: Invert the sense of the MSI enables., Grant Grundler, (Thu May 24, 10:44 pm)
Re: [PATCH 1/2] msi: Invert the sense of the MSI enables., Eric W. Biederman, (Thu May 24, 10:57 pm)
Re: [PATCH 1/2] msi: Invert the sense of the MSI enables., Eric W. Biederman, (Thu May 24, 10:59 pm)
Re: [PATCH 2/2] msi: Add support for the Intel chipsets t ..., Eric W. Biederman, (Thu May 24, 11:10 pm)
Re: [PATCH 1/2] msi: Invert the sense of the MSI enables., David Miller, (Thu May 24, 11:40 pm)
Re: [PATCH 1/2] msi: Invert the sense of the MSI enables., Eric W. Biederman, (Fri May 25, 8:17 am)
Re: [PATCH 1/2] msi: Invert the sense of the MSI enables., Roland Dreier, (Fri May 25, 8:40 am)
Re: [PATCH 1/2] msi: Invert the sense of the MSI enables., Roland Dreier, (Fri May 25, 8:42 am)
Re: [PATCH 1/2] msi: Invert the sense of the MSI enables., Eric W. Biederman, (Fri May 25, 9:10 am)
Re: [PATCH 2/2] msi: Add support for the Intel chipsets t ..., Eric W. Biederman, (Fri May 25, 9:52 am)
RE: [PATCH 1/2] msi: Invert the sense of the MSI enables., David Schwartz, (Fri May 25, 1:09 pm)
Re: [PATCH 1/2] msi: Invert the sense of the MSI enables., Jonathan Lundell, (Fri May 25, 1:16 pm)
Re: [PATCH 1/2] msi: Invert the sense of the MSI enables., Roland Dreier, (Fri May 25, 1:25 pm)
Re: [PATCH 1/2] msi: Invert the sense of the MSI enables., Eric W. Biederman, (Fri May 25, 2:06 pm)
Re: [PATCH 1/2] msi: Invert the sense of the MSI enables., Roland Dreier, (Fri May 25, 2:17 pm)
Re: [PATCH 1/2] msi: Invert the sense of the MSI enables., Grant Grundler, (Fri May 25, 11:43 pm)
Re: [PATCH 1/2] msi: Invert the sense of the MSI enables., Grant Grundler, (Fri May 25, 11:52 pm)