Re: [Spice-devel] Delay in Gimp when using qxl driver

2012-01-18 Thread Soeren Sandmann
> Yes, so here is another timing from ssh login on the VM : > > PerfTop: 686 irqs/sec kernel: 0.4% exact: 0.0% [1000Hz > cpu-clock], (all, 1 CPU) > -- > > samples pcnt funct

Re: [Spice-devel] RFC: Render support in the protocol

2011-07-11 Thread Soeren Sandmann
Yonit Halperin writes: > Hi, > It looks like Render overlaps QXL_DRAW_ROP3. Maybe QXLRop3 can be > exploited? I'm not sure it's possible to wedge the Render semantics into QXLRop3. Here are some differences between the two commands: - Render supports full affine transformations, not just scalin

Re: [Spice-devel] RFC: Render support in the protocol

2011-07-07 Thread Soeren Sandmann
Hi, > If FILTER only has 2 possible values, it would fit on a single bit, > did you use 2 because you know it will have to be extended soon? I don't know of any specific plans, but there has been some talk about adding support for bicubic filtering, so that's why I added the extra bit. There are

[Spice-devel] RFC: Render support in the protocol

2011-07-06 Thread Soeren Sandmann
RENDER is a very commonly used X extension that currently is not supported very well in SPICE. This leads to the QXL X driver having to read back pixels from the device, render in software, then transmit an image. This is currently the biggest performance issue with SPICE for X. The specification

Re: [Spice-devel] [PATCH xf86-video-qxl 0/6] use spice-protocol

2011-02-11 Thread Soeren Sandmann
Hi, Both these patches and the enum typedefs look good to me - I'll pull them once the spice-protocol patches land. thanks, Soren ___ Spice-devel mailing list Spice-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/spice-devel

Re: [Spice-devel] [PATCH xf86-drv-qxl F14-branch] Don't access the qxl device when our vt is not focussed

2010-10-12 Thread Soeren Sandmann
Hans de Goede writes: > Trying to do so causes the X-server to segfault as it has > dropped iopl permission, which is, erm, undesirable :) I don't think the function should be called in_vga_mode() anymore. Maybe just rename it to have_vt() or something like that - other than that, it looks good

Re: [Spice-devel] leaking drawables

2010-09-08 Thread Soeren Sandmann
Gerd Hoffmann writes: >Hi, > > > This happens for several consecutive resource releases. Looking at what > > actually gets released by the driver we see that the resources are freed > > in the same order as they were release, its just that a chunk of > > resources are missing here and there.

Re: [Spice-devel] [PATCH 00/30] Convert cairo canvas to use pixman

2010-02-21 Thread Soeren Sandmann
Izik Eidus writes: > Hi, before we commit this, another thing we should do is get pixman dll > for windows, without this the whole windows build will be broken. > > Soren any experience in building pixman on windows? I know that pixman is being built on Windows with MSVC, because it is used in

Re: [Spice-devel] [PATCH 14/30] Convert cairo canvas copy bits to pixman

2010-02-21 Thread Soeren Sandmann
Alexander Larsson writes: > --- > common/cairo_canvas.c | 254 > - > 1 files changed, 41 insertions(+), 213 deletions(-) > > diff --git a/common/cairo_canvas.c b/common/cairo_canvas.c > index d0307d8..109a07c 100644 > --- a/common/cairo_canvas.c

Re: [Spice-devel] [PATCH 11/30] Convert cairo canvas draw_opaque() to using pixman

2010-02-21 Thread Soeren Sandmann
Alexander Larsson writes: > +static void scale_image (CairoCanvas *canvas, > + pixman_region32_t *region, > + SPICE_ADDRESS src_bitmap, > + int src_x, int src_y, > + int src_width, int src_height, > +

Re: [Spice-devel] [PATCH 10/30] Convert cairo canvas draw_fill() to using pixman

2010-02-21 Thread Soeren Sandmann
Alexander Larsson writes: > +/* round down X to even 32 pixels (i.e. uint32_t) */ > +extents.x1 = extents.x1 & ~(0x1f); > + > +mask_data = (uint32_t *)((uint8_t *)mask_data + mask_stride * extents.y1 > + extents.x1 / 32); > +mask_x -= extents.x1; > +mask_y -= extents.y1; > +

Re: [Spice-devel] [PATCH 03/30] Add line rasterizer

2010-02-21 Thread Soeren Sandmann
Alexander Larsson writes: > On Fri, 2010-02-19 at 10:43 +0200, Izik Eidus wrote: > > On Thu, 18 Feb 2010 21:58:29 +0100 > > Alexander Larsson wrote: > > > > > This is based on the X11 line drawers in mi/mizerline.c etc > > > --- > > > client/Makefile.am |1 + > > > client/lines.cpp