Re: Multiple MSI, take 3

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Eric W. Biederman <ebiederm@...>
Cc: Suresh Siddha <suresh.b.siddha@...>, Matthew Wilcox <matthew@...>, linux-pci@vger.kernel.org <linux-pci@...>, linux-kernel@vger.kernel.org <linux-kernel@...>, grundler@parisc-linux.org <grundler@...>, mingo@elte.hu <mingo@...>, tglx@linutronix.de <tglx@...>, jgarzik@pobox.com <jgarzik@...>, linux-ide@vger.kernel.org <linux-ide@...>, jbarnes@virtuousgeek.org <jbarnes@...>, rdunlap@xenotime.net <rdunlap@...>, mtk.manpages@gmail.com <mtk.manpages@...>
Date: Saturday, July 12, 2008 - 3:36 am

> It is edge triggered so it won't refire when unmasked (especially if we don't know).

Well, we are smarter than that. soft-masking is a know well-solved
problem. We just latch that something happened while masked and refire
when unmasked. Not terribly hard. We already do that in various
situations to mask edge interrupts.


Masking in HW is totally optional. I don't mask in HW on cell for
example, the HW just can't.


We only re-fire if it actually occured while "masked", that should take
care that we never fire once too much, no ?


There must be some way of knowing what work is to do (ie, whether a DMA
q entry is completed, some kind of done bit, etc...). There generally is
at least, so that even in that case, spurrious MSIs are mostly a non
issue, but I don't think we have them here.
 

I still mostly dislike the new approach, I prefer Matthew's original one
with SW masking of the MSIs. For example, if you have the MSIs be 'one'
interrupt, then you hit all of the logic in the IRQ core to make sure
only one happens at once. Might not be what you want, and -will- cause
some to be dropped... not nice.

Ben.


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

Messages in current thread:
Multiple MSI, take 3, Matthew Wilcox, (Thu Jul 10, 8:57 pm)
Re: Multiple MSI, take 3, Jike Song, (Fri Sep 26, 1:30 am)
Re: Multiple MSI, take 3, Matthew Wilcox, (Sat Sep 27, 3:04 pm)
Re: Multiple MSI, take 3, Eric W. Biederman, (Fri Jul 11, 6:06 am)
Re: Multiple MSI, take 3, Suresh Siddha, (Fri Jul 11, 5:59 pm)
Re: Multiple MSI, take 3, Eric W. Biederman, (Fri Jul 11, 6:59 pm)
Re: Multiple MSI, take 3, Benjamin Herrenschmidt, (Fri Jul 11, 11:52 pm)
Re: Multiple MSI, take 3, Eric W. Biederman, (Sat Jul 12, 12:41 am)
Re: Multiple MSI, take 3, Benjamin Herrenschmidt, (Sat Jul 12, 3:36 am)
Re: Multiple MSI, take 3, Eric W. Biederman, (Sun Jul 13, 6:30 pm)
Re: Multiple MSI, take 3, Benjamin Herrenschmidt, (Sun Jul 13, 6:44 pm)
Re: Multiple MSI, take 3, Eric W. Biederman, (Sun Jul 13, 7:29 pm)
Re: Multiple MSI, take 3, Benjamin Herrenschmidt, (Sun Jul 13, 8:17 pm)
Re: Multiple MSI, take 3, David Miller, (Sun Jul 13, 8:44 pm)
Re: Multiple MSI, take 3, Eric W. Biederman, (Sun Jul 13, 10:03 pm)
Re: Multiple MSI, take 3, David Miller, (Sun Jul 13, 11:19 pm)
Re: Multiple MSI, take 3, Suresh Siddha, (Fri Jul 11, 7:15 pm)
Re: Multiple MSI, take 3, Eric W. Biederman, (Fri Jul 11, 7:59 pm)
Re: Multiple MSI, take 3, Matthew Wilcox, (Fri Jul 11, 6:23 am)
Re: Multiple MSI, take 3, Eric W. Biederman, (Fri Jul 11, 7:34 am)
Re: Multiple MSI, take 3, Matthew Wilcox, (Fri Jul 11, 8:17 am)
Re: Multiple MSI, take 3, Matthew Wilcox, (Fri Jul 11, 11:10 am)
Re: Multiple MSI, take 3, Eric W. Biederman, (Fri Jul 11, 7:05 am)
Re: Multiple MSI, take 3, David Miller, (Fri Jul 11, 6:32 am)
Re: Multiple MSI, take 3, Matthew Wilcox, (Fri Jul 11, 6:41 am)
[PATCH] Rewrite MSI-HOWTO, Matthew Wilcox, (Thu Jul 10, 9:00 pm)
Re: [PATCH] Rewrite MSI-HOWTO, Grant Grundler, (Fri Sep 26, 2:42 am)
[PATCH] PCI: Add support for multiple MSI, Matthew Wilcox, (Thu Jul 10, 8:59 pm)
Re: [PATCH] PCI: Add support for multiple MSI, Hidetoshi Seto, (Fri Jul 11, 4:28 am)
Re: [PATCH] PCI: Add support for multiple MSI, Benjamin Herrenschmidt, (Fri Jul 11, 11:45 pm)
Re: [PATCH] PCI: Add support for multiple MSI, Matthew Wilcox, (Fri Jul 11, 5:45 am)
[PATCH] AHCI: Request multiple MSIs, Matthew Wilcox, (Thu Jul 10, 9:00 pm)
[PATCH] PCI MSI: Replace 'type' with 'is_msix', Matthew Wilcox, (Thu Jul 10, 8:59 pm)
[PATCH] x86-64: Support for multiple MSIs, Matthew Wilcox, (Thu Jul 10, 9:00 pm)
Re: [PATCH] x86-64: Support for multiple MSIs, Kenji Kaneshige, (Fri Jul 11, 12:50 am)
Re: [PATCH] x86-64: Support for multiple MSIs, Matthew Wilcox, (Fri Jul 11, 4:50 am)
Re: [PATCH] x86-64: Support for multiple MSIs, Kenji Kaneshige, (Sun Jul 13, 9:08 pm)