Re: [Qemu-devel] [PATCH] vmware_vga: do not cache depth and bypp

[Gerd Hoffmann <kraxel@redhat.com>]

On 03/29/13 17:28, Alex DAMIAN wrote:
> From: Alexandru DAMIAN <alexandru.damian@intel.com>
> 
> Do not cache depth and bypp information in the device state.
> 
> This resolves a bug where Xorg video-vmare driver refuses
> to start up because the depth value read is the one cached from the
> device start (default 32 from ui/console.c) and it is not consistent
> with the graphical console depth, which may be different from
> the default depth.

Does it actually work?  /me posted a simliar patch and according to Jan
(Cc'ed) it doesn't fix the issue.

> @@ -1113,7 +1111,6 @@ static const VMStateDescription vmstate_vmware_vga_internal = {
>      .minimum_version_id_old = 0,
>      .post_load = vmsvga_post_load,
>      .fields      = (VMStateField[]) {
> -        VMSTATE_INT32_EQUAL(depth, struct vmsvga_state_s),

This breaks live migration.

cheers,
  Gerd