[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/5] pr-helper: fix --socket-path default in hel
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH 1/5] pr-helper: fix --socket-path default in help |
Date: |
Tue, 26 Jun 2018 13:13:49 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.8.0 |
On 06/26/2018 12:40 PM, Paolo Bonzini wrote:
> Currently --help shows "(default '(null)')" for the -k/--socket-path
> option. Fix it by getting the default path in /var/run.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> scsi/qemu-pr-helper.c | 15 ++++++++-------
> 1 file changed, 8 insertions(+), 7 deletions(-)
>
> diff --git a/scsi/qemu-pr-helper.c b/scsi/qemu-pr-helper.c
> index d0f83176e1..4057cf355c 100644
> --- a/scsi/qemu-pr-helper.c
> +++ b/scsi/qemu-pr-helper.c
> @@ -74,8 +74,16 @@ static int uid = -1;
> static int gid = -1;
> #endif
>
> +static void compute_default_paths(void)
> +{
> + if (!socket_path) {
> + socket_path =
> qemu_get_local_state_pathname("run/qemu-pr-helper.sock");
> + }
> +}
> +
> static void usage(const char *name)
> {
> + compute_default_paths();
> (printf) (
> "Usage: %s [OPTIONS] FILE\n"
> "Persistent Reservation helper program for QEMU\n"
> @@ -845,13 +853,6 @@ static const char *socket_activation_validate_opts(void)
> return NULL;
> }
>
> -static void compute_default_paths(void)
> -{
> - if (!socket_path) {
> - socket_path =
> qemu_get_local_state_pathname("run/qemu-pr-helper.sock");
> - }
> -}
> -
> static void termsig_handler(int signum)
> {
> atomic_cmpxchg(&state, RUNNING, TERMINATE);
>