Re: [Qemu-devel] QEMU patch to allow VM introspection via libvmi

2015-10-27 Thread Valerio Aimale
On 10/27/15 10:11 AM, Markus Armbruster wrote: [...] Eduardo, I did try this approach. It takes 2 line changes in exec.c: comment the unlink out, and making sure MAP_SHARED is used when -mem-path and -mem-prealloc are given. It works beautifully, and libvmi accesses are fast. However, the VM is

Re: [Qemu-devel] QEMU patch to allow VM introspection via libvmi

2015-10-27 Thread Valerio Aimale
On 10/27/15 9:00 AM, Markus Armbruster wrote: Valerio Aimale writes: On 10/26/15 11:52 AM, Eduardo Habkost wrote: I was trying to advocate the use of a shared mmap'ed region. The sharing would be two-ways (RW for both) between the QEMU virtualizer and the libvmi process. I envision

Re: [Qemu-devel] QEMU patch to allow VM introspection via libvmi

2015-10-27 Thread Valerio Aimale
On 10/27/15 9:18 AM, Valerio Aimale wrote: I did not try to mmap'ing to a file on a RAMdisk. Without physical disk I/O, the VM might run faster. I did try with the file on a ramdisk $ sudo mount -o size=3G -t tmpfs none /ramdisk $ /usr/local/bin/qemu-system-x86_64 -name Windows

Re: [Qemu-devel] QEMU patch to allow VM introspection via libvmi

2015-10-27 Thread Valerio Aimale
On 10/26/15 11:52 AM, Eduardo Habkost wrote: I was trying to advocate the use of a shared mmap'ed region. The sharing would be two-ways (RW for both) between the QEMU virtualizer and the libvmi process. I envision that there could be a QEMU command line argument, such as "--mmap-guest-memory "

Re: [Qemu-devel] QEMU patch to allow VM introspection via libvmi

2015-10-26 Thread Valerio Aimale
On 10/26/15 3:09 AM, Markus Armbruster wrote: [...] Eduardo, I think it would be a common rule of politeness not to pass any judgement on a person that you don't know, but for some texts in a mailing list. I think I understand how mmap() works, and very well. Participating is this discussion h

Re: [Qemu-devel] QEMU patch to allow VM introspection via libvmi

2015-10-23 Thread Valerio Aimale
On 10/23/15 12:55 PM, Eduardo Habkost wrote: On Thu, Oct 22, 2015 at 03:51:28PM -0600, Valerio Aimale wrote: On 10/22/15 3:47 PM, Eduardo Habkost wrote: On Thu, Oct 22, 2015 at 01:57:13PM -0600, Valerio Aimale wrote: On 10/22/15 1:12 PM, Eduardo Habkost wrote: On Wed, Oct 21, 2015 at 12:54

Re: [Qemu-devel] QEMU patch to allow VM introspection via libvmi

2015-10-23 Thread Valerio Aimale
On 10/23/15 8:56 AM, Eric Blake wrote: On 10/23/2015 08:44 AM, Valerio Aimale wrote: Libvmi dependence on virsh is so strict, that libvmi does not even know if the QEMU VM has an open qmp unix socket or inet socket, to send commands through. Thus, libvmi sends qmp commands (to query

Re: [Qemu-devel] QEMU patch to allow VM introspection via libvmi

2015-10-23 Thread Valerio Aimale
On 10/23/15 2:18 AM, Daniel P. Berrange wrote: [...] It can't today, but just because it creates a temporary file inside mem-path and unlinks it immediately after opening a file descriptor. We could make memory-backend-file also accept a full filename as argument, or add a mechanism to let QEMU

Re: [Qemu-devel] QEMU patch to allow VM introspection via libvmi

2015-10-23 Thread Valerio Aimale
On 10/23/15 12:35 AM, Markus Armbruster wrote: Eduardo Habkost writes: On Wed, Oct 21, 2015 at 12:54:23PM +0200, Markus Armbruster wrote: Valerio Aimale writes: [...] There's also a similar patch, floating around the internet, the uses shared memory, instead of sockets, as inter-pr

Re: [Qemu-devel] QEMU patch to allow VM introspection via libvmi

2015-10-22 Thread Valerio Aimale
On 10/22/15 3:47 PM, Eduardo Habkost wrote: On Thu, Oct 22, 2015 at 01:57:13PM -0600, Valerio Aimale wrote: On 10/22/15 1:12 PM, Eduardo Habkost wrote: On Wed, Oct 21, 2015 at 12:54:23PM +0200, Markus Armbruster wrote: Valerio Aimale writes: [...] There's also a similar patch, flo

Re: [Qemu-devel] QEMU patch to allow VM introspection via libvmi

2015-10-22 Thread Valerio Aimale
On 10/22/15 2:03 PM, Eric Blake wrote: On 10/22/2015 01:57 PM, Valerio Aimale wrote: pmemmap would return the following json { 'success' : 'true', 'map_filename' : '/tmp/QEM_mmap_1234567' } In general, it is better if the client controls the

Re: [Qemu-devel] QEMU patch to allow VM introspection via libvmi

2015-10-22 Thread Valerio Aimale
On 10/22/15 1:12 PM, Eduardo Habkost wrote: On Wed, Oct 21, 2015 at 12:54:23PM +0200, Markus Armbruster wrote: Valerio Aimale writes: [...] There's also a similar patch, floating around the internet, the uses shared memory, instead of sockets, as inter-process communication between l

Re: [Qemu-devel] QEMU patch to allow VM introspection via libvmi

2015-10-22 Thread Valerio Aimale
On 10/21/15 4:54 AM, Markus Armbruster wrote: Valerio Aimale writes: On 10/19/15 1:52 AM, Markus Armbruster wrote: Valerio Aimale writes: On 10/16/15 2:15 AM, Markus Armbruster wrote: vale...@aimale.com writes: All- I've produced a patch for the current QEMU HEAD, for libv

Re: [Qemu-devel] QEMU patch to allow VM introspection via libvmi

2015-10-22 Thread Valerio Aimale
On 10/22/15 5:50 AM, Markus Armbruster wrote: Valerio Aimale writes: On 10/21/15 4:54 AM, Markus Armbruster wrote: Valerio Aimale writes: On 10/19/15 1:52 AM, Markus Armbruster wrote: Valerio Aimale writes: On 10/16/15 2:15 AM, Markus Armbruster wrote: vale...@aimale.com writes

Re: [Qemu-devel] QEMU patch to allow VM introspection via libvmi

2015-10-21 Thread Valerio Aimale
On 10/21/15 4:54 AM, Markus Armbruster wrote: Valerio Aimale writes: On 10/19/15 1:52 AM, Markus Armbruster wrote: Valerio Aimale writes: On 10/16/15 2:15 AM, Markus Armbruster wrote: vale...@aimale.com writes: All- I've produced a patch for the current QEMU HEAD, for libv

Re: [Qemu-devel] [PATCH] QEMU patch for libvmi to introspect QEMU/kvm virtual machines. Usually this patch is distributed with libvmi, but, it might be more useful to have it in the QEMU source perman

2015-10-21 Thread Valerio Aimale
Valerio On 10/19/15 3:33 PM, Eric Blake wrote: On 10/15/2015 05:44 PM, vale...@aimale.com wrote: From: Valerio Aimale Long subject line, and no message body. Remember, you want the subject line to be a one-line short summary of 'what', then the commit body message for 'why&

Re: [Qemu-devel] QEMU patch to allow VM introspection via libvmi

2015-10-19 Thread Valerio Aimale
On 10/19/15 1:52 AM, Markus Armbruster wrote: Valerio Aimale writes: On 10/16/15 2:15 AM, Markus Armbruster wrote: vale...@aimale.com writes: All- I've produced a patch for the current QEMU HEAD, for libvmi to introspect QEMU/KVM VMs. Libvmi has patches for the old qeum-kvm fork, i

Re: [Qemu-devel] QEMU patch to allow VM introspection via libvmi

2015-10-16 Thread Valerio Aimale
On 10/16/15 2:15 AM, Markus Armbruster wrote: vale...@aimale.com writes: All- I've produced a patch for the current QEMU HEAD, for libvmi to introspect QEMU/KVM VMs. Libvmi has patches for the old qeum-kvm fork, inside its source tree: https://github.com/libvmi/libvmi/tree/master/tools/qemu-k