From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45628) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bsSAw-0007vG-B6 for qemu-devel@nongnu.org; Fri, 07 Oct 2016 06:14:48 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bsSAr-0006WD-9g for qemu-devel@nongnu.org; Fri, 07 Oct 2016 06:14:45 -0400 Received: from mx1.redhat.com ([209.132.183.28]:44176) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bsSAr-0006Vp-2J for qemu-devel@nongnu.org; Fri, 07 Oct 2016 06:14:41 -0400 Received: from int-mx14.intmail.prod.int.phx2.redhat.com (int-mx14.intmail.prod.int.phx2.redhat.com [10.5.11.27]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id C1A973D94B for ; Fri, 7 Oct 2016 10:14:39 +0000 (UTC) Date: Fri, 7 Oct 2016 12:14:37 +0200 From: Kevin Wolf Message-ID: <20161007101437.GD4589@noname.redhat.com> References: <87eg3ulq9n.fsf@dusky.pond.sub.org> <20161005174358.GD11921@work-vm> <0d193d70-9e6e-c91a-01ad-926851a544e2@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <0d193d70-9e6e-c91a-01ad-926851a544e2@redhat.com> Subject: Re: [Qemu-devel] Monitor brain dump List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: "Dr. David Alan Gilbert" , Markus Armbruster , qemu-devel@nongnu.org, Luiz Capitulino Am 06.10.2016 um 13:07 hat Paolo Bonzini geschrieben: > On 05/10/2016 19:43, Dr. David Alan Gilbert wrote: > > > Speaking of the pocket calculator: my recommendation would be "nuke from > > > orbit". It adds surprising corner cases to the HMP language, and > > > provides next to no value. > > > > Huh, didn't realise that existed - I assume you mean get_expr and friends? > > yep sounds nukable > > We've been suggesting commands like "x/10i $pc-20" to people who > reported emulation failures in KVM, so there is some benefit in that. Yes, there are some operations that it supports that I think are really rarely used (I don't think I've ever used modulo in the monitor), but using the register variables like $pc or calculating offsets (i.e. sums and possibly multiplications) is fairly common and useful for debugging guests. Looking at git blame, this seems to be something that just works and is hardly ever touched, so why does leaving it there hurt us? Any examples of the surprising corner cases? Can they be addressed without removing the useful parts of it? > My hunch is that it would be a drop in the sea if monitor.c were > refactored properly. Now that QMP middle mode is gone it should be much > easier. That, too. Kevin