From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=53671 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1PzbRm-0000KW-Aw for qemu-devel@nongnu.org; Tue, 15 Mar 2011 17:06:31 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1PzbRk-00041T-Qy for qemu-devel@nongnu.org; Tue, 15 Mar 2011 17:06:30 -0400 Received: from mail-iw0-f173.google.com ([209.85.214.173]:40833) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1PzbRk-0003va-Nv for qemu-devel@nongnu.org; Tue, 15 Mar 2011 17:06:28 -0400 Received: by iwl42 with SMTP id 42so1120144iwl.4 for ; Tue, 15 Mar 2011 14:05:42 -0700 (PDT) Message-ID: <4D7FD49C.7070108@codemonkey.ws> Date: Tue, 15 Mar 2011 16:05:32 -0500 From: Anthony Liguori MIME-Version: 1.0 Subject: Re: [Qemu-devel] KVM call minutes for Mar 15 References: <20110315145346.GH20456@x200.localdomain> <4D7F93AB.4050207@codemonkey.ws> <20110315190606.GM20456@x200.localdomain> In-Reply-To: <20110315190606.GM20456@x200.localdomain> 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: Chris Wright Cc: libvir-list@redhat.com, Jiri Denemark , Chris Lalancette , qemu-devel@nongnu.org, kvm@vger.kernel.org On 03/15/2011 02:06 PM, Chris Wright wrote: > * Anthony Liguori (anthony@codemonkey.ws) wrote: >> On 03/15/2011 09:53 AM, Chris Wright wrote: >>> QAPI > >>> - c library implementation is critical to have unit tests and test >>> driven development >>> - thread safe? >>> - no shared state, no statics. >>> - threading model requires lock for the qmp session >>> - licensiing? >>> - LGPL >>> - forwards/backwards compat? >>> - designed with that in mind see wiki: >>> >>> http://wiki.qemu.org/Features/QAPI >> One neat feature of libqmp is that once libvirt has a better QMP >> passthrough interface, we can create a QmpSession that uses libvirt. >> >> It would look something like: >> >> QmpSession *libqmp_session_new_libvirt(virDomainPtr dom); > Looks like you mean this? > > -> request QmpSession -> > client libvirt > <- return QmpSession<- > > client -> QmpSession -> QMP -> QEMU Maybe, your ASCII art confuses me :-) QmpSession is just a wrapper around a transport. It can be an fd that you read() and write() JSON strings to, but it's just as easy to read and write through JSON strings via virQemuMonitorCommand() or whatever the interface currently is. > So bypassing libvirt completely to actually use the session? > > Currently, it's more like: > > client -> QemuMonitorCommand -> libvirt -> QMP -> QEMU It's not bypassing. It's an API on top of a libvirt command. FWIW, the code generator could be trivially modified to generate a libvirt style API if there's any interest. So instead of: void qmp_block_passwd(QmpSession *sess, const char *device, const char *password, Error **errp); It would be: int virQemuBlockPasswd(virDomainPtr dom, const char *device, const char *password); But I'm not sure that's really that useful. >> The QmpSession returned by this call can then be used with all of >> the libqmp interfaces. This means we can still exercise our test >> suite with a guest launched through libvirt. It also should make >> the libvirt pass through interface a bit easier to consume by third >> parties. > This sounds like it's something libvirt folks should be involved with. > At the very least, this mode is there now and considered basically > unstable/experimental/developer use: > > "Qemu monitor command '%s' executed; libvirt results may be unpredictable!" > > So likely some concern about making it easier to use, esp. assuming > that third parties above are mgmt apps, not just developers. To be clear, there's no real support needed from libvirt here other than the passthrough. How that interface is supported in libvirt is more or less orthogonal to libqmp. libqmp is a C API to QMP. It can speak QMP over whatever transports speak QMP. If you can speak QMP to libvirt, then it's only natural to bridge libqmp to libvirt. Regards, Anthony Liguori > thanks, > -chris >