From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1MgpLR-0007aJ-9i
	for qemu-devel@nongnu.org; Thu, 27 Aug 2009 20:29:33 -0400
Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43)
	id 1MgpLM-0007Qr-8l
	for qemu-devel@nongnu.org; Thu, 27 Aug 2009 20:29:32 -0400
Received: from [199.232.76.173] (port=55424 helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1MgpLM-0007QZ-19
	for qemu-devel@nongnu.org; Thu, 27 Aug 2009 20:29:28 -0400
Received: from mx20.gnu.org ([199.232.41.8]:10821)
	by monty-python.gnu.org with esmtps (TLS-1.0:RSA_AES_256_CBC_SHA1:32)
	(Exim 4.60) (envelope-from <anthony@codemonkey.ws>)
	id 1MgpKl-0006Lz-3i
	for qemu-devel@nongnu.org; Thu, 27 Aug 2009 20:28:51 -0400
Received: from qw-out-1920.google.com ([74.125.92.148])
	by mx20.gnu.org with esmtp (Exim 4.60)
	(envelope-from <anthony@codemonkey.ws>) id 1MgpKk-0001Mp-7M
	for qemu-devel@nongnu.org; Thu, 27 Aug 2009 20:28:50 -0400
Received: by qw-out-1920.google.com with SMTP id 5so347829qwc.4
	for <qemu-devel@nongnu.org>; Thu, 27 Aug 2009 17:28:48 -0700 (PDT)
Message-ID: <4A9724BE.9080107@codemonkey.ws>
Date: Thu, 27 Aug 2009 19:28:46 -0500
From: Anthony Liguori <anthony@codemonkey.ws>
MIME-Version: 1.0
Subject: Re: [Qemu-devel] [PATCH] Switch to disable SDL zoom
References: <1250698389-23005-1-git-send-email-toni@tyndur.org>
In-Reply-To: <1250698389-23005-1-git-send-email-toni@tyndur.org>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Antoine Kaufmann <toni@tyndur.org>
Cc: qemu-devel@nongnu.org

Hi Antonie,

Antoine Kaufmann wrote:
> Command line switch to disable SDL zoom, because zoom is quite ugly with
> window managers that use fixed layouts (like awesomewm). Especially
> because there is no possibility to reset the window to the "correct"
> size.
>   

I think this is a valuable feature but I'd prefer to not add another 
-sdl-<feature> option.  In fact, I'd like to get rid of the existing ones.

I'd suggest that you look at introducing a new -display option.  The 
syntax would look like this:

-display sdl,frame=off,scaling=off
-display vnc,address=:3,tls

If you use the recently introduced QemuOptions framework, it should be 
very straight forward.  I would accept an initial patch that just 
converted sdl as vnc is a bit more difficult.

The advantages of this approach are numerous beyond just aesthetics.  
This would allow these display properties to be manipulated via the -set 
option which means that eventually, they'll be tunable via a host 
configuration file.  It also puts together some interfaces for 
supporting multiple graphics cards for guests and multiple display backends.

Regards,

Anthony Liguori