From d35d4cb5175f75e503ee9da0b67ffbe22e8d63ab Mon Sep 17 00:00:00 2001 From: Michael Roth Date: Fri, 13 Apr 2012 21:07:36 -0500 Subject: [PATCH 1/2] qemu-ga: generate missing stubs for fsfreeze MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit When linux-specific commands (including guest-fsfreeze-*) were consolidated under defined(__linux__), we forgot to account for the case where defined(__linux__) && !defined(FIFREEZE). As a result stubs are no longer being generated on linux hosts that don't have FIFREEZE support. Fix this. Tested-by: Andreas Färber Reviewed-by: Luiz Capitulino --- qga/commands-posix.c | 46 ++++++++++++++++++++++++-------------------- 1 file changed, 25 insertions(+), 21 deletions(-) diff --git a/qga/commands-posix.c b/qga/commands-posix.c index faf970dff3..087c3af7ff 100644 --- a/qga/commands-posix.c +++ b/qga/commands-posix.c @@ -881,27 +881,6 @@ error: #else /* defined(__linux__) */ -GuestFsfreezeStatus qmp_guest_fsfreeze_status(Error **err) -{ - error_set(err, QERR_UNSUPPORTED); - - return 0; -} - -int64_t qmp_guest_fsfreeze_freeze(Error **err) -{ - error_set(err, QERR_UNSUPPORTED); - - return 0; -} - -int64_t qmp_guest_fsfreeze_thaw(Error **err) -{ - error_set(err, QERR_UNSUPPORTED); - - return 0; -} - void qmp_guest_suspend_disk(Error **err) { error_set(err, QERR_UNSUPPORTED); @@ -925,6 +904,31 @@ GuestNetworkInterfaceList *qmp_guest_network_get_interfaces(Error **errp) #endif +#if !defined(CONFIG_FSFREEZE) + +GuestFsfreezeStatus qmp_guest_fsfreeze_status(Error **err) +{ + error_set(err, QERR_UNSUPPORTED); + + return 0; +} + +int64_t qmp_guest_fsfreeze_freeze(Error **err) +{ + error_set(err, QERR_UNSUPPORTED); + + return 0; +} + +int64_t qmp_guest_fsfreeze_thaw(Error **err) +{ + error_set(err, QERR_UNSUPPORTED); + + return 0; +} + +#endif + /* register init/cleanup routines for stateful command groups */ void ga_command_state_init(GAState *s, GACommandState *cs) { From 4bdd04165a50899d8700d8009a3e044f88ade45f Mon Sep 17 00:00:00 2001 From: Michael Roth Date: Tue, 17 Apr 2012 11:28:27 -0500 Subject: [PATCH 2/2] qemu-ga: fix help output Reviewed-by: Luiz Capitulino --- qemu-ga.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/qemu-ga.c b/qemu-ga.c index d6f786e50d..74a1b02c68 100644 --- a/qemu-ga.c +++ b/qemu-ga.c @@ -117,12 +117,13 @@ static gboolean register_signal_handlers(void) static void usage(const char *cmd) { printf( -"Usage: %s -c \n" +"Usage: %s [-m -p ] []\n" "QEMU Guest Agent %s\n" "\n" " -m, --method transport method: one of unix-listen, virtio-serial, or\n" " isa-serial (virtio-serial is the default)\n" -" -p, --path device/socket path (%s is the default for virtio-serial)\n" +" -p, --path device/socket path (the default for virtio-serial is:\n" +" %s)\n" " -l, --logfile set logfile path, logs to stderr by default\n" " -f, --pidfile specify pidfile (default is %s)\n" " -v, --verbose log extra debugging information\n" @@ -131,7 +132,7 @@ static void usage(const char *cmd) #ifdef _WIN32 " -s, --service service commands: install, uninstall\n" #endif -" -b, --blacklist comma-separated list of RPCs to disable (no spaces, \"?\"" +" -b, --blacklist comma-separated list of RPCs to disable (no spaces, \"?\"\n" " to list available RPCs)\n" " -h, --help display this help and exit\n" "\n"