[Qemu-devel] [PATCH 2/4] hmp: introduce 'writeconfig' command

2017-10-23 Thread Vadim Galitsyn
Add 'writeconfig' command for HMP monitor. This command is a sibling of QMP command 'writeconfig'. This is a simple way to keep track of current state of VM after series of hotplugs and/or hotunplugs of different devices. Signed-off-by: Vadim Galitsyn <vadim.galit...@profitbricks.com> Sign

[Qemu-devel] [PATCH 3/4] tests: test-hmp: extend with 'writeconfig' command

2017-10-23 Thread Vadim Galitsyn
Extend list of test cases with 'writeconfig' command. Signed-off-by: Vadim Galitsyn <vadim.galit...@profitbricks.com> Cc: Dr. David Alan Gilbert <dgilb...@redhat.com> Cc: qemu-devel@nongnu.org --- tests/test-hmp.c | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/test-hmp.c

[Qemu-devel] [PATCH 4/4] tests: test-hmp: print command execution result

2017-10-23 Thread Vadim Galitsyn
. Signed-off-by: Vadim Galitsyn <vadim.galit...@profitbricks.com> Cc: Dr. David Alan Gilbert <dgilb...@redhat.com> Cc: qemu-devel@nongnu.org --- tests/test-hmp.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/tests/test-hmp.c b/tests/test-hmp.c index 8e21eee61c

[Qemu-devel] QMP, HMP: introduce 'writeconfig' command

2017-10-23 Thread Vadim Galitsyn
Hi Guys, This thread is a continuation of discussion started in http://lists.nongnu.org/archive/html/qemu-devel/2017-02/msg03182.html. This series introduces ‘writeconfig’ command support for QMP and HMP monitors. This functionality might be useful for live migration for cases when guest

[Qemu-devel] [PATCH 1/4] qmp: introduce 'writeconfig' command

2017-10-23 Thread Vadim Galitsyn
Add support for `writeconfig' command for QMP monitor. This is a simple way to keep track of current state of VM after series of hotplugs and/or hotunplugs of different devices. Signed-off-by: Vadim Galitsyn <vadim.galit...@profitbricks.com> Signed-off-by: Eduardo Otubo <e

Re: [Qemu-devel] [PATCH v7 2/3] qmp: introduce query-memory-size-summary command

2017-09-14 Thread Vadim Galitsyn
:26 PM, Dr. David Alan Gilbert < dgilb...@redhat.com> wrote: > * Igor Mammedov (imamm...@redhat.com) wrote: > > On Tue, 29 Aug 2017 17:30:21 +0200 > > Vadim Galitsyn <vadim.galit...@profitbricks.com> wrote: > > > > > Add a new query-memory-size-summary

Re: [Qemu-devel] [PATCH v7 0/3] hmp, qmp: 'info memory_size_summary', 'query-memory-size-summary', 'info numa' updates

2017-09-14 Thread Vadim Galitsyn
Igor, David, Thank you! Best regards, Vadim On Thu, Sep 14, 2017 at 12:09 PM, Dr. David Alan Gilbert < dgilb...@redhat.com> wrote: > * Igor Mammedov (imamm...@redhat.com) wrote: > > On Thu, 14 Sep 2017 11:35:36 +0200 > > Vadim Galitsyn <vadim.galit...@profitbricks.com&g

Re: [Qemu-devel] [PATCH v7 0/3] hmp, qmp: 'info memory_size_summary', 'query-memory-size-summary', 'info numa' updates

2017-09-14 Thread Vadim Galitsyn
Hi Guys, Could you please let me know if you have an update on this topic? Thank you, Vadim On Tue, Aug 29, 2017 at 5:30 PM, Vadim Galitsyn < vadim.galit...@profitbricks.com> wrote: > Hi Guys, > > Sorry for the delay. This is continuation of > http://lists.nongnu.org/archi

[Qemu-devel] [PATCH v7 2/3] qmp: introduce query-memory-size-summary command

2017-08-29 Thread Vadim Galitsyn
UG enabled, no value is reported. Signed-off-by: Vasilis Liaskovitis <vasilis.liaskovi...@profitbricks.com> Signed-off-by: Mohammed Gamal <mohammed.ga...@profitbricks.com> Signed-off-by: Eduardo Otubo <eduardo.ot...@profitbricks.com> Signed-off-by: Vadim Galitsyn <vadim.gali

[Qemu-devel] [PATCH v7 1/3] hmp: extend "info numa" with hotplugged memory information

2017-08-29 Thread Vadim Galitsyn
Report amount of hotplugged memory in addition to total amount per NUMA node. Signed-off-by: Vadim Galitsyn <vadim.galit...@profitbricks.com> Cc: Eduardo Habkost <ehabk...@redhat.com> Cc: David Hildenbrand <da...@redhat.com> Cc: qemu-devel@nongnu.org --- include/qemu/typedefs

[Qemu-devel] [PATCH v7 0/3] hmp, qmp: 'info memory_size_summary', 'query-memory-size-summary', 'info numa' updates

2017-08-29 Thread Vadim Galitsyn
Hi Guys, Sorry for the delay. This is continuation of http://lists.nongnu.org/archive/html/qemu-devel/2017-08/msg02937.html. I tried to update all the things according to your input regarding to v6 series. I am attaching all the versions history here in cover letter. Best regards, Vadim v7:

[Qemu-devel] [PATCH v7 3/3] hmp: introduce 'info memory_size_summary' command

2017-08-29 Thread Vadim Galitsyn
fitbricks.com> Signed-off-by: Vadim Galitsyn <vadim.galit...@profitbricks.com> Reviewed-by: Eugene Crosser <evgenii.cherkas...@profitbricks.com> Cc: Dr. David Alan Gilbert <dgilb...@redhat.com> Cc: Markus Armbruster <arm...@redhat.com> Cc: Igor Mammedov <imamm...@redhat

Re: [Qemu-devel] [PATCH v5 2/3] qmp: introduce query-memory-size-summary command

2017-08-16 Thread Vadim Galitsyn
Hi Guys, Thanks again! Just sent patch v6 which takes into account comments you left. Best regards, Vadim On Wed, Aug 16, 2017 at 8:01 AM, Markus Armbruster <arm...@redhat.com> wrote: > Vadim Galitsyn <vadim.galit...@profitbricks.com> writes: > > > Hi Guys, > &g

[Qemu-devel] [PATCH v6 2/3] qmp: introduce query-memory-size-summary command

2017-08-16 Thread Vadim Galitsyn
d. Signed-off-by: Vasilis Liaskovitis <vasilis.liaskovi...@profitbricks.com> Signed-off-by: Mohammed Gamal <mohammed.ga...@profitbricks.com> Signed-off-by: Eduardo Otubo <eduardo.ot...@profitbricks.com> Signed-off-by: Vadim Galitsyn <vadim.galit...@profitbricks.com> Reviewed-b

[Qemu-devel] [PATCH v6 1/3] Extend "info numa" with hotplugged memory information

2017-08-16 Thread Vadim Galitsyn
Report amount of hotplugged memory in addition to total amount per NUMA node. Signed-off-by: Vadim Galitsyn <vadim.galit...@profitbricks.com> Cc: Eduardo Habkost <ehabk...@redhat.com> Cc: David Hildenbrand <da...@redhat.com> Cc: qemu-devel@nongnu.org --- include/qemu/typedefs

[Qemu-devel] [PATCH v6 3/3] hmp: introduce 'info memory_size_summary' command

2017-08-16 Thread Vadim Galitsyn
s not have CONFIG_MEM_HOTPLUG enabled, no value is reported. Signed-off-by: Vasilis Liaskovitis <vasilis.liaskovi...@profitbricks.com> Signed-off-by: Mohammed Gamal <mohammed.ga...@profitbricks.com> Signed-off-by: Eduardo Otubo <eduardo.ot...@profitbricks.com> Signed-off-by:

Re: [Qemu-devel] [PATCH v5 3/3] hmp: introduce 'info memory-size-summary' command

2017-08-15 Thread Vadim Galitsyn
Hi Eric, Thank you for the input. I will update it with the next version. Btw, most of HMP "info *" commands use '-' instead of '_' in names =) Best regards, Vadim On Fri, Jul 28, 2017 at 8:27 PM, Eric Blake <ebl...@redhat.com> wrote: > On 07/28/2017 07:10 AM, V

Re: [Qemu-devel] [PATCH v5 2/3] qmp: introduce query-memory-size-summary command

2017-08-15 Thread Vadim Galitsyn
ank you in advance, Vadim On Tue, Aug 15, 2017 at 9:51 AM, Igor Mammedov <imamm...@redhat.com> wrote: > On Fri, 28 Jul 2017 14:10:43 +0200 > Vadim Galitsyn <vadim.galit...@profitbricks.com> wrote: > > > Command above provides the following memory information in bytes: &

[Qemu-devel] [PATCH v5 1/3] Extend "info numa" with hotplugged memory information

2017-07-28 Thread Vadim Galitsyn
Report amount of hotplugged memory in addition to total amount per NUMA node. Signed-off-by: Vadim Galitsyn <vadim.galit...@profitbricks.com> Cc: Eduardo Habkost <ehabk...@redhat.com> Cc: David Hildenbrand <da...@redhat.com> Cc: qemu-devel@nongnu.org --- include/qemu/typedefs

Re: [Qemu-devel] [PATCH v4] hmp, qmp: introduce memory-size-summary commands

2017-07-28 Thread Vadim Galitsyn
Hi Guys, This thread is a continuation of discussion from: http://lists.nongnu.org/archive/html/qemu-devel/2017-07/msg01565.html I will post changes list here in cover letter. v5: * hmp: Updated description and '.help' message for 'info memory-size-summary' command. * hmp: Removed '-'

[Qemu-devel] [PATCH v5 3/3] hmp: introduce 'info memory-size-summary' command

2017-07-28 Thread Vadim Galitsyn
arget does not have CONFIG_MEM_HOTPLUG enabled, no value is reported. Signed-off-by: Vasilis Liaskovitis <vasilis.liaskovi...@profitbricks.com> Signed-off-by: Mohammed Gamal <mohammed.ga...@profitbricks.com> Signed-off-by: Eduardo Otubo <eduardo.ot...@profitbricks.com> S

[Qemu-devel] [PATCH v5 2/3] qmp: introduce query-memory-size-summary command

2017-07-28 Thread Vadim Galitsyn
s reported. Signed-off-by: Vasilis Liaskovitis <vasilis.liaskovi...@profitbricks.com> Signed-off-by: Mohammed Gamal <mohammed.ga...@profitbricks.com> Signed-off-by: Eduardo Otubo <eduardo.ot...@profitbricks.com> Signed-off-by: Vadim Galitsyn <vadim.galit...@profitbricks.c

Re: [Qemu-devel] [PATCH v4] hmp, qmp: introduce memory-size-summary commands

2017-07-07 Thread Vadim Galitsyn
* Markus Armbruster (arm...@redhat.com) wrote: > >> Sorry for the late review, got a bit overwhelmed... > >> > >> Vadim Galitsyn <vadim.galit...@profitbricks.com> writes: > >> > >> > Commands above provide the following memory information in bytes:

Re: [Qemu-devel] [PATCH v3] hmp, qmp: introduce "info memory" and "query-memory" commands

2017-06-30 Thread Vadim Galitsyn
n 2017 17:21:06 +0200 > > Vadim Galitsyn <vadim.galit...@profitbricks.com> wrote: > > > > >> +void hmp_info_memory(Monitor *mon, const QDict *qdict) > >> +{ > >> +Error *err = NULL; > >> +MemoryInfo *info = qmp_query_memory(); > &g

[Qemu-devel] [PATCH v4] hmp, qmp: introduce memory-size-summary commands

2017-06-30 Thread Vadim Galitsyn
y: Vasilis Liaskovitis <vasilis.liaskovi...@profitbricks.com> Signed-off-by: Mohammed Gamal <mohammed.ga...@profitbricks.com> Signed-off-by: Eduardo Otubo <eduardo.ot...@profitbricks.com> Signed-off-by: Vadim Galitsyn <vadim.galit...@profitbricks.com> Reviewed-b

[Qemu-devel] [PATCH v4] hmp, qmp: introduce memory-size-summary commands

2017-06-30 Thread Vadim Galitsyn
Hi Guys, This thread is a continuation of discussion started at http://lists.nongnu.org/archive/html/qemu-devel/2017-06/msg05972.html. Vadim

Re: [Qemu-devel] [PATCH v2] hmp, qmp: introduce "info memory" and "query-memory" commands

2017-06-22 Thread Vadim Galitsyn
negotiated, I will come up with test scenario and test case itself. Thank you, Vadim On Tue, Jun 20, 2017 at 4:10 PM, Markus Armbruster <arm...@redhat.com> wrote: > "Dr. David Alan Gilbert" <dgilb...@redhat.com> writes: > > > * Vadim Galitsyn (vadim.galit...@profitb

Re: [Qemu-devel] [PATCH] hmp, qmp: introduce "info memory" and "query-memory" commands

2017-06-22 Thread Vadim Galitsyn
ide information about hot-plugged memory per NUMA node and current patch will utilize this functionality. Actual "info memory" command (as well as "query-memory") will provide information as you suggested. Thank you, Vadim On Mon, Jun 19, 2017 at 9:47 AM, David Hildenbrand <

[Qemu-devel] [PATCH v3] hmp, qmp: introduce "info memory" and "query-memory" commands

2017-06-14 Thread Vadim Galitsyn
: Mohammed Gamal <mohammed.ga...@profitbricks.com> Signed-off-by: Eduardo Otubo <eduardo.ot...@profitbricks.com> Signed-off-by: Vadim Galitsyn <vadim.galit...@profitbricks.com> Reviewed-by: Eugene Crosser <evgenii.cherkas...@profitbricks.com> Cc: Dr. David Alan Gilbert <dgil

Re: [Qemu-devel] [PATCH v2] hmp, qmp: introduce "info memory" and "query-memory" commands

2017-06-14 Thread Vadim Galitsyn
this topic is for another mainling list. Thank you, Vadim On Wed, Jun 14, 2017 at 11:22 AM, Dr. David Alan Gilbert < dgilb...@redhat.com> wrote: > * Vadim Galitsyn (vadim.galit...@profitbricks.com) wrote: > > Commands above provide the following memory information in bytes: > >

[Qemu-devel] [PATCH v2] hmp, qmp: introduce "info memory" and "query-memory" commands

2017-06-13 Thread Vadim Galitsyn
: Mohammed Gamal <mohammed.ga...@profitbricks.com> Signed-off-by: Eduardo Otubo <eduardo.ot...@profitbricks.com> Signed-off-by: Vadim Galitsyn <vadim.galit...@profitbricks.com> Reviewed-by: Eugene Crosser <evgenii.cherkas...@profitbricks.com> Cc: Dr. David Alan Gilbert <dgil

[Qemu-devel] [PATCH] hmp, qmp: introduce "info memory" and "query-memory" commands

2017-06-13 Thread Vadim Galitsyn
Hello, Some time ago we came up with the idea to provide the following patch. The story started in https://lists.gnu.org/archive/html/qemu-devel/2012-07/msg01472.html. For our production environment, we need QEMU to report the amount of guest memory, taking into account hot-plugged memory.

[Qemu-devel] [PATCH] hmp, qmp: introduce "info memory" and "query-memory" commands

2017-06-13 Thread Vadim Galitsyn
: Mohammed Gamal <mohammed.ga...@profitbricks.com> Signed-off-by: Eduardo Otubo <eduardo.ot...@profitbricks.com> Signed-off-by: Vadim Galitsyn <vadim.galit...@profitbricks.com> Reviewed-by: Eugene Crosser <evgenii.cherkas...@profitbricks.com> Cc: Dr. David Alan Gilbert <dgil