[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20061231.124017.85689231.davem@davemloft.net>
Date: Sun, 31 Dec 2006 12:40:17 -0800 (PST)
From: David Miller <davem@...emloft.net>
To: miklos@...redi.hu
Cc: rmk+lkml@....linux.org.uk, arjan@...radead.org, torvalds@...l.org,
linux-kernel@...r.kernel.org, linux-arch@...r.kernel.org,
akpm@...l.org
Subject: Re: fuse, get_user_pages, flush_anon_page, aliasing caches and all
that again
From: Miklos Szeredi <miklos@...redi.hu>
Date: Sun, 31 Dec 2006 13:24:53 +0100
> > I'm willing to do that - and I guess this means we can probably do this
> > instead of walking the list of VMAs for the shared mapping, thereby
> > hitting both anonymous and shared mappings with the same code?
>
> But for the get_user_pages() case there's no point, is there? The VMA
> and the virtual address is already available, so trying to find it
> again through RMAP doesn't much make sense.
>
> Users of get_user_pages() don't care about any other mappings (maybe
> ptrace does, I don't know) only about one single user mapping and one
> kernel mapping.
>
> So using flush_dcache_page() there is an overkill, trying to teach it
> about anonymous pages is not the real solution, flush_dcache_page()
> was never meant to be used on anything but file mapped pages.
Even in the ptrace() case, you do want to flush all the other VMA's
that might be out there with an aliased cached copy in the cpu cache.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists