From: Anthony Liguori <aliguori@linux.vnet.ibm.com>
To: Ian Pratt <m+Ian.Pratt@cl.cam.ac.uk>
Cc: xen-devel@lists.xensource.com, "Cui, Dexuan" <dexuan.cui@intel.com>
Subject: Re: [Patch] Qemu map cache
Date: Tue, 05 Dec 2006 14:48:09 -0600 [thread overview]
Message-ID: <4575DB09.7000105@linux.vnet.ibm.com> (raw)
In-Reply-To: <8A87A9A84C201449A0C56B728ACF491E04EDB4@liverpoolst.ad.cl.cam.ac.uk>
Ian Pratt wrote:
>> Have you considered doing something similar for mainline QEMU? The
>> reason I ask is that V2E pulls in all of the dynamic translation code.
>> My initial reaction is that doing map cache will require a significant
>> amount change to the dynamic translation bits since we can no longer
>> make the assumption that memory can be accessed directly. I don't
>>
> fully
>
>> have my head around it yet, but this may involve lots of nastiness
>>
> with
>
>> keeping track of which TB's reference what memory and invalidating
>>
> those
>
>> TBs when map cache references are invalidated. The QEMU TLB may
>> simplify some of this but I'm not entirely sure.
>>
>> Have you given this any thought?
>>
>
> Being able to invalidate (sections of) qemu's mappings (at least
> asynchronously) is essential to allow the balloon driver to work for HVM
> guests.
To be able to change portions of the physical memory mapping right? You
don't strictly need a map cache for this (you can simply remap portions
of the address space). You really only need the map cache for > 2GB
guests (which admittedly could be a ballooned down guest that started
out > 2GB).
> V2E is going to have to bite the bullet on this one.
>
> Of course, in a 64b environment the map cache can be direct mapped, but
> you still need the ability to do invalidates. BTW: I'm comfortable if
> V2E only works on 64b.
We may be able to work around this using one of QEMU's TLB. If the map
cache goes in for 3.0.4, then we can look at just supporting 64 bit for
3.0.5 and fixing 32 bit post 3.0.5 (if that's necessary). Sound like a
reasonable plan?
Regards,
Anthony Liguori
> Sooner or latter there's going to be some new
> feature which isn't supported on Yonah...
>
> Ian
>
next prev parent reply other threads:[~2006-12-05 20:48 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-12-04 17:33 [Patch] Qemu map cache Cui, Dexuan
2006-12-04 18:26 ` Anthony Liguori
2006-12-04 22:58 ` Anthony Liguori
2006-12-05 20:28 ` Ian Pratt
2006-12-05 20:48 ` Anthony Liguori [this message]
2006-12-06 0:33 ` Ian Pratt
-- strict thread matches above, loose matches on Subject: below --
2006-12-07 2:08 Cui, Dexuan
2006-12-07 10:35 ` Keir Fraser
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=4575DB09.7000105@linux.vnet.ibm.com \
--to=aliguori@linux.vnet.ibm.com \
--cc=dexuan.cui@intel.com \
--cc=m+Ian.Pratt@cl.cam.ac.uk \
--cc=xen-devel@lists.xensource.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.