From: "Glauber Costa" <glommer@gmail.com>
To: qemu-devel@nongnu.org
Cc: Glauber Costa <glommer@redhat.com>, aliguori@us.ibm.com
Subject: Re: [Qemu-devel] vga optmization
Date: Mon, 3 Nov 2008 16:18:35 -0200 [thread overview]
Message-ID: <5d6222a80811031018w4a84e966t50f25e7c4ddb34f2@mail.gmail.com> (raw)
In-Reply-To: <490F3F64.30204@bellard.org>
On Mon, Nov 3, 2008 at 4:13 PM, Fabrice Bellard <fabrice@bellard.org> wrote:
> Glauber Costa wrote:
>> Hi guys,
>>
>> this is a port of current kvm vga memory optimization to our new
>> infrastructure proposed by anthony. It's goal is to use as few
>> kvm specific hooks as possible. In fact, the only one I'm relying
>> on is enabling/disabling of logging. The rest, is pretty much general.
>>
>> We map the linear frame buffer area as RAM, and then use dirty tracking
>> to decide whether or not to update it. To be consistent with qemu,
>> this version, differently from upstream kvm, tracks memory based on its
>> physical address, represented by vram_offset, instead of vram_ptr, or
>> any other construct.
>>
>> Let me know what you think
>
> Why don't you modify the lower level QEMU dirty bits handling functions
> to be consistent with the KVM dirty bits ? By doing that you can avoid
> patching the device drivers and have smaller code. The fact that KVM use
> physical memory addresses is not a problem if you can convert the ram
> addresses to physical memory addresses (in most cases there is only one
> physical address corresponding to one RAM address).
Not here. We have to map vga linear frame buffer as RAM. So the option
is to always do that. That's pretty much the only device specific
thing we're doing here
About physical memory, exactly: code in kvm userspace does it a little
bit differently,
and it is _this_ code that does it the qemu way. So it seems we're in agreement.
--
Glauber Costa.
"Free as in Freedom"
http://glommer.net
"The less confident you are, the more serious you have to act."
next prev parent reply other threads:[~2008-11-03 18:18 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-11-03 17:31 [Qemu-devel] vga optmization Glauber Costa
2008-11-03 17:43 ` Stefano Stabellini
2008-11-03 17:52 ` Glauber Costa
2008-11-03 18:06 ` Stefano Stabellini
2008-11-03 18:03 ` Blue Swirl
2008-11-03 18:14 ` Glauber Costa
2008-11-03 18:41 ` Blue Swirl
2008-11-03 18:47 ` Glauber Costa
2008-11-03 18:13 ` Fabrice Bellard
2008-11-03 18:18 ` Glauber Costa [this message]
2008-11-04 7:23 ` Avi Kivity
2008-11-04 9:31 ` andrzej zaborowski
2008-11-04 11:40 ` Stefano Stabellini
2008-11-04 13:43 ` Glauber Costa
2008-11-04 14:51 ` Avi Kivity
2008-11-04 14:52 ` Anthony Liguori
2008-11-04 14:55 ` Glauber Costa
2008-11-04 15:13 ` Stefano Stabellini
2008-11-04 20:42 ` Avi Kivity
2008-11-04 20:51 ` Anthony Liguori
2008-11-04 15:01 ` Stefano Stabellini
2008-11-04 20:28 ` Glauber Costa
2008-11-04 20:40 ` Anthony Liguori
2008-11-05 14:42 ` Stefano Stabellini
2008-11-07 11:15 ` Glauber Costa
2008-11-07 11:33 ` Stefano Stabellini
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=5d6222a80811031018w4a84e966t50f25e7c4ddb34f2@mail.gmail.com \
--to=glommer@gmail.com \
--cc=aliguori@us.ibm.com \
--cc=glommer@redhat.com \
--cc=qemu-devel@nongnu.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;
as well as URLs for NNTP newsgroup(s).