From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1JzB6e-0000qH-Ua for qemu-devel@nongnu.org; Thu, 22 May 2008 09:45:21 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1JzB6d-0000mh-2P for qemu-devel@nongnu.org; Thu, 22 May 2008 09:45:20 -0400 Received: from [199.232.76.173] (port=37772 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1JzB6c-0000mI-Rd for qemu-devel@nongnu.org; Thu, 22 May 2008 09:45:18 -0400 Received: from mail.windriver.com ([147.11.1.11]:43828 helo=mail.wrs.com) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1JzB6c-0002BO-9b for qemu-devel@nongnu.org; Thu, 22 May 2008 09:45:18 -0400 Received: from ALA-MAIL03.corp.ad.wrs.com (ala-mail03 [147.11.57.144]) by mail.wrs.com (8.13.6/8.13.6) with ESMTP id m4MDjEGL027412 for ; Thu, 22 May 2008 06:45:14 -0700 (PDT) Message-ID: <483578EB.1070403@windriver.com> Date: Thu, 22 May 2008 08:45:15 -0500 From: Jason Wessel MIME-Version: 1.0 Subject: Re: [Qemu-devel] Re: [PATCH 2/5] gdbstub: gdb pass-through qemu monitor support References: <1210860693-22245-1-git-send-email-jason.wessel@windriver.com> <1210860693-22245-2-git-send-email-jason.wessel@windriver.com> <1210860693-22245-3-git-send-email-jason.wessel@windriver.com> <20080515221716.GC27300@edgar.se.axis.com> <483180AD.40704@windriver.com> <291f35090805210558l3eae915ch5937c311f37f91bd@mail.gmail.com> <483455E3.1080706@windriver.com> <48357404.2040608@web.de> In-Reply-To: <48357404.2040608@web.de> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Jan Kiszka wrote: > Jason Wessel wrote: > >> Maxim Gorbachyov wrote: >> >>> On Mon, May 19, 2008 at 5:29 PM, Jason Wessel >>> wrote: >>> >>> [cut] >>> >>> >>>> Seems reasonable too, see newly attached patch. >>>> >>>> Obviously you need the new monitor patch applied prior to this one. >>>> This patch was updated against the prior patch. >>>> >>>> >>> It was noticed [by Jan Kiszka and me] that the output on the monitor >>> console goes both to the local as well as the remote one - but only if >>> you issue a command via gdb. When you type in the command locally (ie. >>> on the qemu side), that output is not forwarded to the gdb frontend. >>> Is this behavior desired? >>> >>> >>> >>> >> That was certainly the intended behavior when I created the patch. >> There is no reason to feed the monitor output up to the debugger that >> may or may not be attached. >> > > That's not what we were confused by. > > >> The behavior could get altered to check if >> the debugger is in the "continue" state and then send the information >> from the monitor, but it seemed of little value to do so. >> > > Rather the contrary: What is the point of mirroring the output you see > in gdb to the local monitor console? > > That happens to have little to do with the gdb implementation. It is a function of the monitor mux. The monitor mux always writes out to all the output channels automatically. In the case of gdb sending input for which you only want to receive the response it would be plausible to add a "focus" parameter to the monitor mux code to indicate which "channel" to send the output to. The monitor mux could also be changed to simply shift the output focus to the most recent place it received an input, and if the focus was -1 to start it would broadcast every where, the way it does today. It is a trivial enough change, if this is what you are looking for. Jason.