public inbox for linux-arch@vger.kernel.org
 help / color / mirror / Atom feed
From: "David S. Miller" <davem@davemloft.net>
To: James.Bottomley@steeleye.com
Cc: linux-arch@vger.kernel.org
Subject: Re: Potential problem with the page_mapping macro and flush_dcache_page()
Date: Sun, 19 Mar 2006 13:00:09 -0800 (PST)	[thread overview]
Message-ID: <20060319.130009.102283514.davem@davemloft.net> (raw)
In-Reply-To: <1142801589.3240.51.camel@mulgrave.il.steeleye.com>

From: James Bottomley <James.Bottomley@SteelEye.com>
Date: Sun, 19 Mar 2006 14:53:08 -0600

> Actually, this is my specific worry.  On parisc, the coherence index
> associated with the DMA is that of the kernel mapping, not the user
> mappings, so DMA only flushes the kernel cache for the line, not the
> user one.  If nothing happens to purge the user line, DMA to anon pages
> won't work on parisc either.

You have physically indexed caches, thus cache coherency transactions
should purge all cache lines with that same physical TAG.

If not, you have much larger problems.

> I do think we need to begin considering it.  The user space device
> managers, like hal, are starting to use SG_IO to interrogate devices,
> making it much more prevalent than before.  It's got to be only a matter
> of time before this issue bites something like hal.
> 
> Is the principle that get_user_pages() should return coherent pages OK?
> If so, I think the fix is to have get_user_pages do a specific user page
> flush for anonymous pages.

Direct I/O is another problematic case.

  reply	other threads:[~2006-03-19 21:00 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-03-19 17:15 Potential problem with the page_mapping macro and flush_dcache_page() James Bottomley
2006-03-19 20:33 ` David S. Miller
2006-03-19 20:53   ` James Bottomley
2006-03-19 21:00     ` David S. Miller [this message]
2006-03-19 21:04       ` James Bottomley
2006-03-19 21:23         ` James Bottomley
2006-03-19 21:26           ` David S. Miller
2006-03-19 23:50             ` James Bottomley
2006-03-20  1:15               ` James Bottomley
2006-03-19 21:25         ` David S. Miller
2006-03-19 21:29           ` James Bottomley
2006-03-19 22:01             ` David S. Miller
2006-03-19 23:39               ` James Bottomley
2006-03-19 23:42                 ` David S. Miller
2006-03-23  6:20             ` Andrew Morton
2006-03-23 14:13               ` James Bottomley

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=20060319.130009.102283514.davem@davemloft.net \
    --to=davem@davemloft.net \
    --cc=James.Bottomley@steeleye.com \
    --cc=linux-arch@vger.kernel.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