From: David Brownell <david-b@pacbell.net>
To: "Woodruff, Richard" <r-woodruff2@ti.com>
Cc: Russell King <rmk@arm.linux.org.uk>,
Tony Lindgren <tony@atomide.com>,
"Hiremath, Vaibhav" <hvaibhav@ti.com>,
"Shah, Hardik" <hardik.shah@ti.com>,
"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>
Subject: Re: Spurious interrupt warning
Date: Sun, 11 Jan 2009 01:04:25 -0800 [thread overview]
Message-ID: <200901110104.25312.david-b@pacbell.net> (raw)
In-Reply-To: <13B9B4C6EF24D648824FF11BE8967162036D48171F@dlee02.ent.ti.com>
On Wednesday 07 January 2009, Woodruff, Richard wrote:
> I don't mind using a read back dependency. I don't like it but
> that may be necessary in some cases for a given device.
>
> What I don't like is constantly hitting them and getting a broken
> system. How many of these are out there? If we can remove most
> of them in current software with SO then that isn't a bad tradeoff.
The same issue comes up with PCI drivers though ... and there,
it's routinely accepted that critical paths in (some) drivers
need to issue readbacks to flush the relevant posted writes.
That's *especially* true in IRQ paths, and when DMA engines
update state in shared memory.
Which says to me that grumbling about strongly ordered regions
is more of a developer discipline/training problem than any
kind of real technical issue. Not unlike how to do locking
right; or satisfy coding style guidelines; or follow correct
procedures to merge patches; or synchronize between several
execution contexts; or ... lots of other things that waste
time when folk don't bother doing them right at first.
Good drivers are not easy to do, and this is just one of the
many ways that shows up. Fortunately *this* class of problem
tends to be easy to address by a careful code audit.
- Dave
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2009-01-11 9:04 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-01-06 9:54 Spurious interrupt warning Shah, Hardik
2009-01-06 11:08 ` Tony Lindgren
2009-01-06 11:12 ` Hiremath, Vaibhav
2009-01-06 11:19 ` Tony Lindgren
2009-01-06 14:05 ` Woodruff, Richard
2009-01-07 8:26 ` Tony Lindgren
2009-01-07 16:13 ` Woodruff, Richard
2009-01-07 17:13 ` Russell King
2009-01-07 17:24 ` Woodruff, Richard
2009-01-11 9:04 ` David Brownell [this message]
2009-01-11 14:18 ` Woodruff, Richard
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=200901110104.25312.david-b@pacbell.net \
--to=david-b@pacbell.net \
--cc=hardik.shah@ti.com \
--cc=hvaibhav@ti.com \
--cc=linux-omap@vger.kernel.org \
--cc=r-woodruff2@ti.com \
--cc=rmk@arm.linux.org.uk \
--cc=tony@atomide.com \
/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