Re: [Spice-devel] "qxl: add optinal 64bit vram bar" patch significantly slows down Spice

2012-04-16 Thread Avi Kivity
On 04/16/2012 03:43 PM, Gerd Hoffmann wrote: > Hi, > >> > Looks like both the BAR and it's backing region were initialized to 4k. >> > The number 4096 does show up in the patch, probably it sneaked into the >> > BAR somehow. > Yea, one page is the minimum size, that shouldn't have been the defau

Re: [Spice-devel] "qxl: add optinal 64bit vram bar" patch significantly slows down Spice

2012-04-16 Thread Avi Kivity
On 04/16/2012 12:39 PM, Gerd Hoffmann wrote: > On 04/06/12 15:54, Hans de Goede wrote: > > Hi Gerd, > > > > While cherry picking some spice patches into my own qemu-kvm-1.0 branch, > > so that we can add them to Fedora-17 I noticed a significant slowdown > > after I was done cherry picking. Invest

Re: [Spice-devel] "qxl: add optinal 64bit vram bar" patch significantly slows down Spice

2012-04-16 Thread Avi Kivity
On 04/16/2012 03:02 PM, Hans de Goede wrote: > > Could it be that the patch somehow causes seabios and/or Xorg to > see the region as different wrt being cachable / uncachable or > some such? In general we ignore cacheability in kvm. Most likely a but in the memory core caused the region to be in

Re: [Spice-devel] "qxl: add optinal 64bit vram bar" patch significantly slows down Spice

2012-04-16 Thread Avi Kivity
On 04/16/2012 12:39 PM, Gerd Hoffmann wrote: > On 04/06/12 15:54, Hans de Goede wrote: > > Hi Gerd, > > > > While cherry picking some spice patches into my own qemu-kvm-1.0 branch, > > so that we can add them to Fedora-17 I noticed a significant slowdown > > after I was done cherry picking. Invest

Re: [Spice-devel] [PATCH 00/13] KVM: MMU: fast page fault

2012-04-10 Thread Avi Kivity
On 04/09/2012 09:26 PM, Xiao Guangrong wrote: > Yes, if Xwindow is not enabled, the benefit is limited. :) I'm more interested in migration. We could optimize the framebuffer by disabling dirty logging when VNC/Spice is not connected (which should usually be the case), or when the SDL window is m

Re: [Spice-devel] client_migrate_info - do we need a new command?

2011-12-14 Thread Avi Kivity
On 12/13/2011 08:19 PM, Anthony Liguori wrote: > In our call today, Avi asked that we evaluate whether the interface > for client_migrate_info is the Right Interface before we introduce a > new command to work around the fact that async commands are broken. > > I looked into this today and here's w

Re: [Spice-devel] [Qemu-devel] viewing continuous guest virtual memory as continuous in qemu

2011-10-03 Thread Avi Kivity
On 10/03/2011 10:37 AM, Alon Levy wrote: > > Hi, > won't there be an overhead for rendering on a non continuous > surface? Will it be worthwhile comparing to not creating the > surface? If I use a scatter-gather list there is overhead of allocating and copying the surface whenever I want to

Re: [Spice-devel] [Qemu-devel] viewing continuous guest virtual memory as continuous in qemu

2011-10-02 Thread Avi Kivity
On 10/02/2011 04:31 PM, Alon Levy wrote: On Sun, Oct 02, 2011 at 03:24:36PM +0200, Alon Levy wrote: > Hi, > Converting qemu's ram allocation to a mmap and using remap_file_pages seems like it could work. Any ideas why it wouldn't? It's linux-specific. Also, does it work on anonymous memory?

Re: [Spice-devel] [Qemu-devel] paravirtual mouse/tablet

2011-01-13 Thread Avi Kivity
On 01/13/2011 07:19 PM, Gerd Hoffmann wrote: On 01/13/11 17:18, Avi Kivity wrote: On 01/13/2011 12:19 PM, Gerd Hoffmann wrote: Hi, Just throwing a quick writeup into the ring to kickstart the design discussion ;) typedef struct qemu_pvtablet_message { uint32_t size; /* whole message size

Re: [Spice-devel] [Qemu-devel] paravirtual mouse/tablet

2011-01-13 Thread Avi Kivity
On 01/13/2011 12:19 PM, Gerd Hoffmann wrote: Hi, Just throwing a quick writeup into the ring to kickstart the design discussion ;) typedef struct qemu_pvtablet_message { uint32_t size;/* whole message size */ uint32_t type;/* qemu_pvtablet_type */ uin

Re: [Spice-devel] [Qemu-devel] paravirtual mouse/tablet

2011-01-13 Thread Avi Kivity
On 01/13/2011 05:52 PM, Anthony Liguori wrote: /* host->guest, sent before any other events */ typedef struct qemu_pvtablet_init { uint32_t res_x; /* x axis resolution */ uint32_t res_y; /* y axis resolution */ uint32_t features; /* qemu_pvtablet_features */ uint32_t available