From: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
To: Avi Kivity <avi@redhat.com>
Cc: "qemu-devel@nongnu.org" <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] [6989] Fix display breakage when resizing the screen (v2) (Avi Kivity)
Date: Tue, 07 Apr 2009 10:32:03 +0100 [thread overview]
Message-ID: <49DB1D93.40908@eu.citrix.com> (raw)
In-Reply-To: <49DB1CA6.4000307@redhat.com>
Avi Kivity wrote:
> I found the real cause - vga_screen_dump() left values for the current
> display resolution in s->last_width and s->last_height. This caused the
> next vga_draw_graphic() to render to the 640x400 display surface it had
> without resizing it.
>
> The bug is triggered if you manage to screendump after a screen
> resolution change, but before the next bitblit or console refresh. If
> no vnc or sdl client is connected, that's just the next bitblt so the
> window is considerably longer.
This explains why it is so hard to reproduce.
> I'll follow up with a patch. Anthony, please revert 6989 before
> applying it. Thanks to Stefano for pointing out the problems with the
> patch.
>
No problem :)
next prev parent reply other threads:[~2009-04-07 9:32 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-04-05 18:41 [Qemu-devel] [6989] Fix display breakage when resizing the screen (v2) (Avi Kivity) Anthony Liguori
2009-04-06 14:21 ` Stefano Stabellini
2009-04-06 14:29 ` Anthony Liguori
2009-04-06 14:33 ` Stefano Stabellini
2009-04-06 14:44 ` Avi Kivity
2009-04-07 1:44 ` Ryan Harper
2009-04-07 9:28 ` Avi Kivity
2009-04-07 9:32 ` Stefano Stabellini [this message]
2009-04-06 14:41 ` Avi Kivity
2009-04-06 14:53 ` Avi Kivity
2009-04-06 17:35 ` Stefano Stabellini
2009-04-06 17:46 ` Avi Kivity
2009-04-06 18:01 ` Stefano Stabellini
2009-04-06 18:32 ` Avi Kivity
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=49DB1D93.40908@eu.citrix.com \
--to=stefano.stabellini@eu.citrix.com \
--cc=avi@redhat.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.