From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1N0I3w-0004E9-4H for qemu-devel@nongnu.org; Tue, 20 Oct 2009 12:59:56 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1N0I3r-0004AQ-2h for qemu-devel@nongnu.org; Tue, 20 Oct 2009 12:59:55 -0400 Received: from [199.232.76.173] (port=56313 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1N0I3q-0004AE-SN for qemu-devel@nongnu.org; Tue, 20 Oct 2009 12:59:50 -0400 Received: from mail-fx0-f211.google.com ([209.85.220.211]:55771) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1N0I3q-0001Pj-DI for qemu-devel@nongnu.org; Tue, 20 Oct 2009 12:59:50 -0400 Received: by fxm7 with SMTP id 7so6306456fxm.34 for ; Tue, 20 Oct 2009 09:59:48 -0700 (PDT) Message-ID: <4ADDEC7B.1000103@codemonkey.ws> Date: Tue, 20 Oct 2009 11:59:39 -0500 From: Anthony Liguori MIME-Version: 1.0 Subject: Re: [Qemu-devel] [PATCH] new SDL keyboard shortcuts to start and stop VM References: <4ADCE635.4060604@codemonkey.ws> <4ADD9388.8040206@redhat.com> In-Reply-To: <4ADD9388.8040206@redhat.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Kevin Wolf Cc: Mulyadi Santosa , qemu-devel@nongnu.org Kevin Wolf wrote: > Am 20.10.2009 00:20, schrieb Anthony Liguori: > >> Mulyadi Santosa wrote: >> >>> IMO, it would be faster if we provide keyboard shortcuts that will >>> stop and resume VM execution right from SDL guest interface, rather >>> than switching to console monitor first and type "s" or "c" >>> respectively. >>> >>> >> Is this really common of an operation that you would need an escape key >> for it? >> >> Why are you so frequently stopping and continuing a guest? >> > > Why are you all trying to explain to him that actually he doesn't want > to have this feature? I could have used it, too, at times (stop the > guest to have enough time to attach gdb, for example). There are other > ways to do it (although they are not as simple) and I used them, but > that doesn't make this feature less useful. > > Does it take anything away for you? Or do you have plans to use those > keys otherwise? If not, why not add a feature that some might find > useful, even though others don't? > There is such a thing as feature bloat. It leads to very confusing behavior for users. It also increases the testing matrix. As it turns out, there's a better way to do what he's looking for that requires no changes. Had we just taken this patch, then that's another feature that has to be tested for SDL whenever there's a change there. It also means there will be differing behavior for VNC so it probably needs to be supported there. But then for something like libvirt, it's not going to expect that something else pauses/starts a VM. Features are not free. They have long term maintenance costs so we should consider whether a feature really offers value. Regards, Anthony Liguori