virtualization.lists.linux-foundation.org archive mirror
 help / color / mirror / Atom feed
From: Jeremy Fitzhardinge <jeremy@goop.org>
To: Zachary Amsden <zach@vmware.com>
Cc: Chris Wright <chrisw@sous-sol.org>,
	Virtualization Mailing List <virtualization@lists.osdl.org>
Subject: Re: how set_pte_at()'s vaddr and ptep args relate
Date: Tue, 07 Nov 2006 15:42:19 -0800	[thread overview]
Message-ID: <455119DB.2000704@goop.org> (raw)
In-Reply-To: <455117C0.2030202@vmware.com>

Zachary Amsden wrote:
> Anything where you implicitly defer pagetable updates is far too 
> vulnerable to bugs.  We played with several such schemes before, and 
> although they could be made to work for a shadow mode hypervisor, 
> getting them to work for both shadow and direct mode, with performance 
> opportunities for everyone was just too risky and a burden on the 
> Linux mm code.

Yep.

> There is no architectural rule about tlb flush that I am aware of, 
> however, most cores will allow you to do NP->P transitions without a 
> flush.  YMMV.  I believe the Linux use is fine.

Hm, I was under the impression there's an actual architectural guarantee 
there, but I don't know chapter&verse.

> Good.  It does not seem worth the effort.  I do have the code to make 
> it work, but it is really ugly.  If some user comes screaming for it 
> later, we can always add it back. 
I'm working on linear pagetables, so that ptes can be allocated from 
anywhere any be directly accessable.  This eliminates the need for 
CONFIG_HIGHPTE, and it also simplifies a lot of the pagetable walking.  
Manipulating other processes's pagetables would still need kmap (or a 
second window for cross-process pagetable manipulation), but I should 
think that's pretty rare.

    J

  reply	other threads:[~2006-11-07 23:42 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-11-07 19:57 how set_pte_at()'s vaddr and ptep args relate Jeremy Fitzhardinge
2006-11-07 22:19 ` Zachary Amsden
2006-11-07 22:38   ` Jeremy Fitzhardinge
2006-11-07 23:33     ` Zachary Amsden
2006-11-07 23:42       ` Jeremy Fitzhardinge [this message]
2006-11-07 23:59         ` Zachary Amsden
2006-11-08  0:15           ` Jeremy Fitzhardinge
2006-11-08  0:19             ` Zachary Amsden
2006-11-08  8:34             ` Keir Fraser
2006-11-08 19:59               ` Jeremy Fitzhardinge
2006-11-08 20:18                 ` Jeremy Fitzhardinge
2006-11-08 23:17                   ` Keir Fraser
2006-11-08 23:25                     ` Jeremy Fitzhardinge
2006-11-09  8:29                       ` Keir Fraser
2006-11-09  9:15                         ` Zachary Amsden

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=455119DB.2000704@goop.org \
    --to=jeremy@goop.org \
    --cc=chrisw@sous-sol.org \
    --cc=virtualization@lists.osdl.org \
    --cc=zach@vmware.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;
as well as URLs for NNTP newsgroup(s).