[Spice-commits] Changes to 'rebase/spice-next'

[Gerd Hoffmann]

New branch 'rebase/spice-next' available with the following commits:
commit b6366c77e218f1dd1cbcd8a81059dca666fc7414
Author: Hans de Goede <hdegoede at redhat.com>
Date:   Fri Apr 5 11:30:24 2013 +0200

    spice-qemu-char: vmc_write: Don't write more bytes then we're asked too
    
    This one took me eons to debug, but I've finally found it now, oh well.
    
    The usage of the MIN macro in this line:
        last_out = MIN(len, qemu_chr_be_can_write(scd->chr));
    
    Causes qemu_chr_be_can_write to be called *twice*, since the MIN macro
    evaluates its arguments twice (bad MIN macro, bad!). And the result of
    the call can change between the 2 calls since the guest may have consumed
    some data from the virtio ringbuffer between the calls!
    
    When this happens it is possible for qemu_chr_be_can_write to return less
    then len in the call made for the comparision, and then to return more then
    len in the actual call for the return-value of MIN, after which we will end
    up writing len data + some extra garbage, not good.
    
    This patch fixes this by only calling qemu_chr_be_can_write once.
    
    Signed-off-by: Hans de Goede <hdegoede at redhat.com>
    Signed-off-by: Gerd Hoffmann <kraxel at redhat.com>

commit 786aa6974d85f2024a4469d41c2a9a7c2630453e
Author: Alon Levy <alevy at redhat.com>
Date:   Fri Apr 5 11:30:23 2013 +0200

    spice-qemu-char: Remove intermediate buffer
    
    virtio-serial's buffer is valid when it calls us, and we don't
    access it otherwise: vmc_read is only called in response to wakeup,
    or else we set datalen=0 and throttle. Then vmc_read is called back,
    we return 0 (not accessing the buffer) and set the timer to unthrottle.
    
    Also make datalen int and not ssize_t (to fit spice_chr_write signature).
    
    HdG: Update to apply to spice-qemu-char with new gio-channel based
    flowcontrol support.
    
    Signed-off-by: Hans de Goede <hdegoede at redhat.com>
    Signed-off-by: Gerd Hoffmann <kraxel at redhat.com>

commit d22307cc00038b1bc0538fc430075eeecff5b6c1
Author: Hans de Goede <hdegoede at redhat.com>
Date:   Fri Apr 5 11:30:22 2013 +0200

    spice-qemu-char: Add watch support
    
    Signed-off-by: Hans de Goede <hdegoede at redhat.com>
    Signed-off-by: Gerd Hoffmann <kraxel at redhat.com>

commit 36e03056efb9d320ae770386f191561419fb43d0
Author: Hans de Goede <hdegoede at redhat.com>
Date:   Fri Apr 5 11:30:21 2013 +0200

    spice-qemu-char: Remove #ifdef-ed code for old spice-server compat
    
    We now require spice-server to be >= 0.12.0 so this is no longer needed.
    
    Signed-off-by: Hans de Goede <hdegoede at redhat.com>
    Signed-off-by: Gerd Hoffmann <kraxel at redhat.com>

commit 294c2c27412db3730e6484fdf1b1132b211a6935
Author: Hans de Goede <hdegoede at redhat.com>
Date:   Fri Apr 5 11:30:20 2013 +0200

    virtio-console: Remove any pending watches on close
    
    Signed-off-by: Hans de Goede <hdegoede at redhat.com>
    Signed-off-by: Gerd Hoffmann <kraxel at redhat.com>

commit 8596f293b580dc47ff8bb2ab681a9e11a95121fb
Author: Hans de Goede <hdegoede at redhat.com>
Date:   Fri Apr 5 11:30:19 2013 +0200

    virtio-console: Also throttle when less was written then requested
    
    This is necessary so that we get properly woken up to write the rest.
    
    This patch also changes the len argument to the have_data callback, to
    avoid doing an unsigned signed comparison.
    
    Signed-off-by: Hans de Goede <hdegoede at redhat.com>
    Acked-by: Amit Shah <amit.shah at redhat.com>
    Signed-off-by: Gerd Hoffmann <kraxel at redhat.com>

commit d30d32f965d66a7561f6af2996050db402ead914
Author: Alon Levy <alevy at redhat.com>
Date:   Wed Mar 13 17:58:35 2013 +0200

    spice: (32 bit only) fix surface cmd tracking destruction
    
    No change for 64 bit arches, but for 32 bit previously we zeroed half
    the surfaces cmd array, instead of all of it.
    
    Signed-off-by: Alon Levy <alevy at redhat.com>
    Signed-off-by: Gerd Hoffmann <kraxel at redhat.com>

commit cec6db97dbbf5fcfdacc8c6ff95005861cad8520
Author: Gerd Hoffmann <kraxel at redhat.com>
Date:   Thu Apr 4 10:15:34 2013 +0200

    qxl: add 2000x2000 and 2048x2048 video modes

commit d4bcb199fb15f9f079ef280e7e6f9ccdfaa49ced
Author: Gerd Hoffmann <kraxel at redhat.com>
Date:   Fri Mar 15 11:53:47 2013 +0100

    qxl: add 4k + 8k resolutions
    
    Signed-off-by: Gerd Hoffmann <kraxel at redhat.com>