From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49159) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ePqRC-0002Gu-5O for qemu-devel@nongnu.org; Fri, 15 Dec 2017 08:54:13 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ePqR9-0003Ej-4R for qemu-devel@nongnu.org; Fri, 15 Dec 2017 08:54:06 -0500 Received: from mx1.redhat.com ([209.132.183.28]:42466) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1ePqR8-0003De-Tk for qemu-devel@nongnu.org; Fri, 15 Dec 2017 08:54:03 -0500 Date: Fri, 15 Dec 2017 21:53:53 +0800 From: Fam Zheng Message-ID: <20171215135353.GF15187@lemon> References: <20171205055200.16305-1-peterx@redhat.com> <20171205055200.16305-13-peterx@redhat.com> <20171213171934.GC8317@stefanha-x1.localdomain> <20171215094010.GB15187@lemon> <20171215132642.GH26982@stefanha-x1.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171215132642.GH26982@stefanha-x1.localdomain> Subject: Re: [Qemu-devel] [RFC v5 12/26] qmp: negociate QMP capabilities List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: Peter Xu , Laurent Vivier , Juan Quintela , Markus Armbruster , qemu-devel@nongnu.org, mdroth@linux.vnet.ibm.com, marcandre.lureau@redhat.com, Stefan Hajnoczi , Paolo Bonzini , "Dr . David Alan Gilbert" On Fri, 12/15 13:26, Stefan Hajnoczi wrote: > > > QEMU always offers the 'oob' capability, even if the monitor does not > > > support it. Should it send 'oob' only when mon->use_io_thr to make > > > things easier for clients? > > > > So should we firstly agree on whether the capabilities is on the current monitor > > connection or QEMU as a whole? > > It's more flexible to allow per-connection capabilities. Is there a > reason against it? No, I just think either way we should document it. So if we define it per-connection, like you said, "oob" shouldn't be sent in the greeting message, and patch 11 need to be updated. Fam