plugins/lockstep: make socket path not positional & parse bool arg
Signed-off-by: Mahmoud Mandour <ma.mandourr@gmail.com> Reviewed-by: Alex Bennée <alex.bennee@linaro.org> Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Message-Id: <20210730135817.17816-6-ma.mandourr@gmail.com>
This commit is contained in:
parent
8a3eab66e6
commit
b18a0cad81
@ -319,22 +319,35 @@ QEMU_PLUGIN_EXPORT int qemu_plugin_install(qemu_plugin_id_t id,
|
||||
int argc, char **argv)
|
||||
{
|
||||
int i;
|
||||
|
||||
if (!argc || !argv[0]) {
|
||||
qemu_plugin_outs("Need a socket path to talk to other instance.");
|
||||
return -1;
|
||||
}
|
||||
g_autofree char *sock_path = NULL;
|
||||
|
||||
for (i = 0; i < argc; i++) {
|
||||
char *p = argv[i];
|
||||
if (strcmp(p, "verbose") == 0) {
|
||||
verbose = true;
|
||||
} else if (!setup_unix_socket(argv[0])) {
|
||||
qemu_plugin_outs("Failed to setup socket for communications.");
|
||||
g_autofree char **tokens = g_strsplit(p, "=", 2);
|
||||
|
||||
if (g_strcmp0(tokens[0], "verbose") == 0) {
|
||||
if (!qemu_plugin_bool_parse(tokens[0], tokens[1], &verbose)) {
|
||||
fprintf(stderr, "boolean argument parsing failed: %s\n", p);
|
||||
return -1;
|
||||
}
|
||||
} else if (g_strcmp0(tokens[0], "sockpath") == 0) {
|
||||
sock_path = tokens[1];
|
||||
} else {
|
||||
fprintf(stderr, "option parsing failed: %s\n", p);
|
||||
return -1;
|
||||
}
|
||||
}
|
||||
|
||||
if (sock_path == NULL) {
|
||||
fprintf(stderr, "Need a socket path to talk to other instance.\n");
|
||||
return -1;
|
||||
}
|
||||
|
||||
if (!setup_unix_socket(sock_path)) {
|
||||
fprintf(stderr, "Failed to setup socket for communications.\n");
|
||||
return -1;
|
||||
}
|
||||
|
||||
our_id = id;
|
||||
|
||||
qemu_plugin_register_vcpu_tb_trans_cb(id, vcpu_tb_trans);
|
||||
|
@ -286,7 +286,7 @@ communicate over::
|
||||
|
||||
./sparc-softmmu/qemu-system-sparc -monitor none -parallel none \
|
||||
-net none -M SS-20 -m 256 -kernel day11/zImage.elf \
|
||||
-plugin ./contrib/plugins/liblockstep.so,arg=lockstep-sparc.sock \
|
||||
-plugin ./contrib/plugins/liblockstep.so,sockpath=lockstep-sparc.sock \
|
||||
-d plugin,nochain
|
||||
|
||||
which will eventually report::
|
||||
|
Loading…
Reference in New Issue
Block a user