From: Aurelien Jarno <aurelien@aurel32.net>
To: Stefan Weil <weil@mail.berlios.de>
Cc: Stefano Stabellini <stefano.stabellini@eu.citrix.com>,
aurel32@debian.org, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [BUG] Changing SDL full screen mode no longer works
Date: Thu, 19 Nov 2009 14:08:19 +0100 [thread overview]
Message-ID: <4B054343.3040107@aurel32.net> (raw)
In-Reply-To: <4B04FA73.60003@mail.berlios.de>
Stefan Weil a écrit :
> Stefan Weil schrieb:
>> Stefano Stabellini schrieb:
>>> Hi all,
>>> currently vga always resizes the screen when vga_hw_invalidate is called
>>> while this is not required and all the other graphic emulators don't.
>>> This patch fixes it, making vga invalidate behaviour consistent with the
>>> other emulated devices.
>>>
>>> Signed-off-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
>> This rather old patch breaks switching to and from full screen mode
>> using SDL and alt-ctrl-f key.
>>
>> Please revert or fix it. The patch is also part of stable-0.11, so I
>> expect the same problem there (but did not test it).
>>
>> Found by git bisect, tested on several linux installations.
>>
>
> Debian qemu 0.11.0-5 is suffering from this bug, too.
>
> So 0bd8246bfec1dfb2eb959f52db535572c0260f4c
> should be reverted or fixed not only for master but
> also for stable-0.11.
>
Before reverting that into stable-0.11, we should take time to
understand the problem and have a correct fix (which might be a revert).
--
Aurelien Jarno GPG: 1024D/F1BCDB73
aurelien@aurel32.net http://www.aurel32.net
prev parent reply other threads:[~2009-11-19 13:08 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-08-03 15:14 [Qemu-devel] [PATCH] vga: do not resize the screen on hw_invalidate Stefano Stabellini
2009-11-18 19:53 ` Stefan Weil
2009-11-19 7:57 ` [Qemu-devel] [BUG] Changing SDL full screen mode no longer works Stefan Weil
2009-11-19 13:08 ` Aurelien Jarno [this message]
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=4B054343.3040107@aurel32.net \
--to=aurelien@aurel32.net \
--cc=aurel32@debian.org \
--cc=qemu-devel@nongnu.org \
--cc=stefano.stabellini@eu.citrix.com \
--cc=weil@mail.berlios.de \
/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.