> Sent: Thursday, May 29, 2003 2:55 PM
> To: Nguyen, Tom L
> Cc: Jeff Garzik; Nakajima, Jun;
> Subject: RE: RFC Proposal to enable MSI support in Linux kernel
> > I am working on an updated patch to 2.5.70 to incorporate the comments
> > from you and Jeff Garzik. Regarding the variable renaming of irq, I am
> > open to any suggestions. Please make a recommendation and I will
> > incorporate it into the next update.
> Well essentially, if the function is being passed a vector, lets call the
> parameter 'vector', right now you have to look at the code paths careful
> to determine wether it's an irq number or a vector masquerading as an irq
Not so fast. This part is #ifdef CONFIG_ACPI_PCI and legacy IRQs can be overridden by Interrupt Source Overrides. Since they can change polarity (as far as the spec is concerned), I think we should fix the code. Hint: Look at the mp_override_legacy_irq().Quote:
> > Regarding platform_legacy_irq, I have not seen an edge-triggered
> > interrupt failure caused by platform_legacy_irq.
> So we can go with Jeff's suggestion there and remove the unecessary check
> for wether to assign level or edge?
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/