From: "Michael S. Tsirkin" <mst@redhat.com>
To: Anthony Liguori <anthony@codemonkey.ws>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH RFC] vga: flag vga ram for notifiers
Date: Thu, 31 Mar 2011 20:49:41 +0200 [thread overview]
Message-ID: <20110331184940.GA25688@redhat.com> (raw)
In-Reply-To: <4D94C916.6080709@codemonkey.ws>
On Thu, Mar 31, 2011 at 01:33:58PM -0500, Anthony Liguori wrote:
> On 03/31/2011 12:43 PM, Michael S. Tsirkin wrote:
> >Currently, vga cards that allocate vga ram,
> >register it as regular ram. When this happens
> >a lot, vhost need to get notified and flush
> >its memory tables, which is slow.
> >
> >This was observed with cirrus vga.
> >
> >As a solution, add an explicit flag when
> >registering vga ram, vhost-net can simply ignore it.
> >
> >Long term, we might be able to use this API
> >to avoid the need to request
> >dirty loggin from devices explicitly.
> >
> >Tested: with cirrus vga only.
> >
> >Signed-off-by: Michael S. Tsirkin<mst@redhat.com>
>
> Treating vga specially is not the right approach.
>
> You want to treat real RAM specially and only make that visible to
> vhost. See http://wiki.qemu.org/Features/RamAPI
That seems like a dead project? And VGa is unhandled there.
> There is nothing special about VGA.
It is special in that guest can control host virtual to
guest physical mappings. In this VGA is similar to
IO rather than RAM.
--
MST
next prev parent reply other threads:[~2011-03-31 18:50 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-03-31 17:43 [Qemu-devel] [PATCH RFC] vga: flag vga ram for notifiers Michael S. Tsirkin
2011-03-31 18:33 ` Anthony Liguori
2011-03-31 18:49 ` Michael S. Tsirkin [this message]
2011-03-31 19:01 ` Anthony Liguori
2011-03-31 19:07 ` Michael S. Tsirkin
2011-03-31 19:26 ` Anthony Liguori
2011-03-31 20:03 ` Michael S. Tsirkin
2011-03-31 20:21 ` Anthony Liguori
2011-03-31 19:17 ` Alex Williamson
2011-03-31 19:18 ` Peter Maydell
2011-03-31 19:29 ` Anthony Liguori
2011-03-31 20:12 ` Peter Maydell
2011-03-31 20:23 ` Anthony Liguori
2011-03-31 21:32 ` Peter Maydell
2011-03-31 21:38 ` Michael S. Tsirkin
2011-03-31 21:49 ` Anthony Liguori
2011-03-31 23:42 ` Michael S. Tsirkin
2011-04-01 0:44 ` Anthony Liguori
2011-04-01 1:07 ` Michael S. Tsirkin
2011-04-01 7:12 ` Peter Maydell
2011-04-01 11:21 ` Michael S. Tsirkin
2011-03-31 21:26 ` Michael S. Tsirkin
2011-03-31 21:32 ` Anthony Liguori
2011-03-31 21:37 ` Michael S. Tsirkin
2011-03-31 21:48 ` Anthony Liguori
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=20110331184940.GA25688@redhat.com \
--to=mst@redhat.com \
--cc=anthony@codemonkey.ws \
--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 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.