[PATCH] new monitor func status

[PATCH] new monitor func status

[Philipp Wehrheim]

Hey,

attached is a small patch that adds the new info subcommand - status.

The status indicates if the VM is running or paused this info makes
life for (stateless) KVM frontends easier.
Since the info is available inside kvm why not make it available?

Please consider applying.
flip

--

diff --git a/qemu/monitor.c b/qemu/monitor.c
index 3948aae..ee8476b 100644
--- a/qemu/monitor.c
+++ b/qemu/monitor.c
@@ -1418,6 +1418,15 @@ static void do_inject_nmi(int cpu_index)
 }
 #endif

+static void do_info_status(void)
+{
+    if (vm_running)
+       term_printf("VM status: running\n");
+    else
+       term_printf("VM status: paused\n");
+}
+
+
 static void do_balloon(int value)
 {
     ram_addr_t target = value;
@@ -1582,6 +1591,8 @@ static const term_cmd_t info_cmds[] = {
       "", "show capture information" },
     { "snapshots", "", do_info_snapshots,
       "", "show the currently saved VM snapshots" },
+    { "status", "", do_info_status,
+      "", "show the current VM status (running|paused)" },
     { "pcmcia", "", pcmcia_info,
       "", "show guest PCMCIA status" },
     { "mice", "", do_info_mice,

Re: [PATCH] new monitor func status

[Avi Kivity]

Philipp Wehrheim wrote:
> Hey,
>
> attached is a small patch that adds the new info subcommand - status.
>
> The status indicates if the VM is running or paused this info makes
> life for (stateless) KVM frontends easier.
> Since the info is available inside kvm why not make it available?
>
>   

Please post to qemu-devel@nongnu.org; this functionality is common to 
kvm and qemu.

-- 
error compiling committee.c: too many arguments to function

Re: [PATCH] new monitor func status

[Philipp Wehrheim]

Ill send the patch to the qemu-devel list

thanks

Avi Kivity schrieb:
> Philipp Wehrheim wrote:
>> Hey,
>>
>> attached is a small patch that adds the new info subcommand - status.
>>
>> The status indicates if the VM is running or paused this info makes
>> life for (stateless) KVM frontends easier.
>> Since the info is available inside kvm why not make it available?
>>
>>   
> 
> Please post to qemu-devel@nongnu.org; this functionality is common to
> kvm and qemu.
>