kvm pause now uses the git option parser to parse parameters.
Added option to pause instance by PID and pause all instances.
Improved usage message.
Signed-off-by: Sasha Levin <levinsasha928@xxxxxxxxx>
---
tools/kvm/builtin-pause.c | 48 +++++++++++++++++++++++++++++++-------------
1 files changed, 34 insertions(+), 14 deletions(-)
diff --git a/tools/kvm/builtin-pause.c b/tools/kvm/builtin-pause.c
index 7a6a6c7..1c3ec44 100644
--- a/tools/kvm/builtin-pause.c
+++ b/tools/kvm/builtin-pause.c
@@ -1,22 +1,40 @@
-#include <stdio.h>
-#include <string.h>
-#include <signal.h>
-
#include <kvm/util.h>
#include <kvm/kvm-cmd.h>
#include <kvm/builtin-pause.h>
#include <kvm/kvm.h>
#include <kvm/parse-options.h>
+#include <stdio.h>
+#include <string.h>
+#include <signal.h>
+
+static bool all;
+static u64 instance_pid;
+static const char *instance_name;
+
static const char * const pause_usage[] = {
- "kvm pause <instance name>",
+ "kvm pause [--all] [-n name] [-p pid]",
NULL
};
static const struct option pause_options[] = {
+ OPT_GROUP("General options:"),
+ OPT_BOOLEAN('a', "all", &all, "Pause all instances"),
+ OPT_STRING('n', "name", &instance_name, "name", "Instance name"),
+ OPT_U64('p', "pid", &instance_pid, "Instance pid"),
OPT_END()
};
+static void parse_pause_options(int argc, const char **argv)
+{
+ while (argc != 0) {
+ argc = parse_options(argc, argv, pause_options, pause_usage,
+ PARSE_OPT_STOP_AT_NON_OPTION);
+ if (argc != 0)
+ kvm_pause_help();
+ }
+}
+
void kvm_pause_help(void)
{
usage_with_options(pause_usage, pause_options);
@@ -29,18 +47,20 @@ static int do_pause(const char *name, int pid)
int kvm_cmd_pause(int argc, const char **argv, const char *prefix)
{
- int pid;
+ parse_pause_options(argc, argv);
- if (argc != 1)
+ if (all)
+ return kvm__enumerate_instances(do_pause);
+
+ if (instance_name == NULL &&
+ instance_pid == 0)
kvm_pause_help();
- if (strcmp(argv[0], "all") == 0) {
- return kvm__enumerate_instances(do_pause);
- }
+ if (instance_name)
+ instance_pid = kvm__get_pid_by_instance(argv[0]);
- pid = kvm__get_pid_by_instance(argv[0]);
- if (pid < 0)
- die("Failed locating instance name");
+ if (instance_pid <= 0)
+ die("Failed locating instance");
- return kill(pid, SIGUSR2);
+ return kill(instance_pid, SIGUSR2);
}
--
1.7.6
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
