Re: [Qemu-devel] [RFC 3/7] qxl: introduce QXLCookie

2012-02-20 Thread Gerd Hoffmann
+if (cookie == NULL) { +cookie = qxl_cookie_new(QXL_COOKIE_TYPE_IO, +QXL_IO_UPDATE_AREA_ASYNC, +0); +} Automagic cookie creation is still there. I think when cookie is NULL you should just

Re: [Qemu-devel] [RFC 3/7] qxl: introduce QXLCookie

2012-02-20 Thread Alon Levy
On Mon, Feb 20, 2012 at 11:56:58AM +0100, Gerd Hoffmann wrote: +if (cookie == NULL) { +cookie = qxl_cookie_new(QXL_COOKIE_TYPE_IO, +QXL_IO_UPDATE_AREA_ASYNC, +0); +} Automagic cookie

Re: [Qemu-devel] [RFC 3/7] qxl: introduce QXLCookie

2012-02-20 Thread Gerd Hoffmann
Hi, Which spice server version added async support? IIRC this is 0.8.2? I'm tempted to raise the minimal supported version to 0.8.latest and zap a bunch of #ifdefs from the code. What do you think? How does that work? raise the minimal supported version in configure? Yes. Then zap

[Qemu-devel] [RFC 3/7] qxl: introduce QXLCookie

2012-02-19 Thread Alon Levy
Will be used in the next patch. Signed-off-by: Alon Levy al...@redhat.com --- hw/qxl-render.c|2 +- hw/qxl.c | 65 +-- hw/qxl.h |2 +- ui/spice-display.c | 26 ++-- ui/spice-display.h | 12