From: Arnd Bergmann <arnd@arndb.de>
To: michael@ellerman.id.au
Cc: linuxppc-dev@ozlabs.org, paulus@samba.org,
cbe-oss-dev@ozlabs.org, GBIRAN@il.ibm.com
Subject: Re: [PATCH] powerpc/cell/axon-msi: retry on missing interrupt
Date: Fri, 21 Nov 2008 15:35:42 +0100 [thread overview]
Message-ID: <200811211535.42987.arnd@arndb.de> (raw)
In-Reply-To: <1227228636.7948.15.camel@localhost>
On Friday 21 November 2008, Michael Ellerman wrote:
> By incrementing the offset we're dropping the irq. Would it be better to
> just return, and hope that the next time we come in the MSI will have
> landed in the fifo and then we can deliver it? It might be late, really
> late I guess, but that might be better then dropping it altogether.
If this happens, I'd rather treat it as an obvious failure case and make
the device driver hang rather than trying to fix up in case another MSI
comes in. The 100 microseconds should cover any case of really really
late, and if we don't get a valid interrupt here (e.g. because a card
still sends an interrupt after the device driver has unmapped the virq)
looking again at the same position would mean we don't move on to the
next valid one (or have to make the error case more complex).
Also, skipping an invalid interrupt is consistent with what we do right
now. Remember that we can't reliably tell the difference between
'invalid interrupt' and 'no interrupt at all'.
Arnd <><
prev parent reply other threads:[~2008-11-21 14:35 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-11-17 16:10 [PATCH] powerpc/cell/axon-msi: retry on missing interrupt Arnd Bergmann
2008-11-18 13:11 ` Giora Biran
2008-11-18 20:18 ` Arnd Bergmann
2008-11-21 0:50 ` Michael Ellerman
2008-11-21 14:35 ` Arnd Bergmann [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200811211535.42987.arnd@arndb.de \
--to=arnd@arndb.de \
--cc=GBIRAN@il.ibm.com \
--cc=cbe-oss-dev@ozlabs.org \
--cc=linuxppc-dev@ozlabs.org \
--cc=michael@ellerman.id.au \
--cc=paulus@samba.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox