public inbox for linux-xfs@vger.kernel.org
 help / color / mirror / Atom feed
* Re: Interaction between Xen and XFS: stray RW mappings
       [not found] ` <20071014225618.GN23367404@sgi.com>
@ 2007-10-15  9:36   ` Andi Kleen
  0 siblings, 0 replies; only message in thread
From: Andi Kleen @ 2007-10-15  9:36 UTC (permalink / raw)
  To: David Chinner
  Cc: Jeremy Fitzhardinge, xfs, Xen-devel, Linux Kernel Mailing List,
	Mark Williamson, Morten Bøgeskov, xfs-masters,
	nickpiggin

David Chinner <dgc@sgi.com> writes:
>
> And yes, we delay unmapping pages until we have a batch of them
> to unmap. vmap and vunmap do not scale, so this is batching helps
> alleviate some of the worst of the problems.

You're keeping vmaps around for already freed pages?

That will be a big problem for proper PAT support, which needs
to track all mappings to memory. It's not just a problem for Xen.

In fact I suspect it is already broken with DRM or AGP for example which
can use UC and WC mappings -- if you keep the mapping around and
DRM or AGP turns the page in another mapping uncacheable you're
creating an illegal cache attribute alias. These are known to occasionally
create cache corruptions on several x86s; giving ___VERY___ hard to debug
bugs once a blue moon.

Probably it'll require some generic VM batching mechanism where
Xen or PAT code can hook into the list or force unmap the mappings
as needed.

Definitely needs to be fixed if true. You're lucky that Xen caught it
in time.

-Andi

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2007-10-15  9:37 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <470FA7C3.90404@goop.org>
     [not found] ` <20071014225618.GN23367404@sgi.com>
2007-10-15  9:36   ` Interaction between Xen and XFS: stray RW mappings Andi Kleen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox