From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MJ2yE-0003PT-Ag for qemu-devel@nongnu.org; Tue, 23 Jun 2009 06:11:18 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MJ2y9-0003Mt-DG for qemu-devel@nongnu.org; Tue, 23 Jun 2009 06:11:17 -0400 Received: from [199.232.76.173] (port=51418 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MJ2y9-0003MY-0w for qemu-devel@nongnu.org; Tue, 23 Jun 2009 06:11:13 -0400 Received: from mx20.gnu.org ([199.232.41.8]:61890) by monty-python.gnu.org with esmtps (TLS-1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1MJ2y8-0003w9-K9 for qemu-devel@nongnu.org; Tue, 23 Jun 2009 06:11:12 -0400 Received: from lizzard.sbs.de ([194.138.37.39]) by mx20.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1MJ2y7-0002bz-Or for qemu-devel@nongnu.org; Tue, 23 Jun 2009 06:11:12 -0400 Message-ID: <4A40AA3D.2050807@siemens.com> Date: Tue, 23 Jun 2009 12:11:09 +0200 From: Jan Kiszka MIME-Version: 1.0 References: <20090623013005.39e27923@doriath> <4A409A4D.8030605@siemens.com> <20090623100428.GD6881@redhat.com> In-Reply-To: <20090623100428.GD6881@redhat.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] Re: [PATCH 11/11] QMP: Command-line flag to enable control mode List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Daniel P. Berrange" Cc: aliguori@us.ibm.com, ehabkost@redhat.com, dlaor@redhat.com, qemu-devel@nongnu.org, Luiz Capitulino , avi@redhat.com Daniel P. Berrange wrote: > On Tue, Jun 23, 2009 at 11:03:09AM +0200, Jan Kiszka wrote: >> Luiz Capitulino wrote: >>> This change adds a flag called 'control' to the already existing >>> '-monitor' command-line option. This flag can be used to enable >>> control mode. >>> >>> Its syntax is: >>> >>> qemu [...] -monitor control, >>> >>> Where is a chardev (excluding 'vc', for obvious reasons). >>> >>> For example: >>> >>> $ qemu [...] -monitor control,tcp:localhost:4444,server >>> >>> Will run QEMU in control mode, waiting for a client TCP connection >>> on localhost port 4444. >>> >>> Signed-off-by: Luiz Capitulino >> At this chance, I would vote for "[PATCH 12/11] Allow multiple -monitor >> instances". I think Anthony posted such a patch before. Now we should >> really include this as your extension may block the stand-alone -monitor >> switch for the control channel, preventing to additionally set up one >> (or more) for debugging purposes. > > If we support multiple monitors, it might be desirable to allow the > app owning the main 'control' monitor channel to be able to indicate > that additional monitor channels are read-only. eg, so libvirt could > allow the user to connect to the monitor to run 'info' commands for > debug support without risk of having state changed behind its back Couldn't libvirt deal with update given we provide them as events? Otherwise, your suggestion makes sense, definitely as long as it would wreck libvirt's internal house keeping or for commands that are not synchronizable. Jan -- Siemens AG, Corporate Technology, CT SE 2 Corporate Competence Center Embedded Linux