LVS
lvs-devel
Google
 
Web LinuxVirtualServer.org

Re: [PATCH] ipvsadm: Fix list daemon to show backup daemon

To: "Ryan O'Hara" <rohara@xxxxxxxxxx>
Subject: Re: [PATCH] ipvsadm: Fix list daemon to show backup daemon
Cc: lvs-devel@xxxxxxxxxxxxxxx
From: Julian Anastasov <ja@xxxxxx>
Date: Fri, 6 Jun 2014 10:24:09 +0300 (EEST)
        Hello,

On Tue, 20 May 2014, Ryan O'Hara wrote:

> The list_daemon function in ipvsadm.c will show the master daemon
> twice, but never the backup daemon. This patch replaces the redundant
> check for IP_VS_STATE_MASTER with a check for IP_VS_STATE_BACKUP, then
> prints the appropriate message.
> 
> Signed-off-by: Ryan O'Hara <rohara@xxxxxxxxxx>

        In case you still wait for ack:

Acked-by: Julian Anastasov <ja@xxxxxx>

> ---
>  ipvsadm.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/ipvsadm.c b/ipvsadm.c
> index 4c83a8b..755827f 100644
> --- a/ipvsadm.c
> +++ b/ipvsadm.c
> @@ -1732,8 +1732,8 @@ static void list_daemon(void)
>               if (u[i].state & IP_VS_STATE_MASTER)
>                       printf("master sync daemon (mcast=%s, syncid=%d)\n",
>                              u[i].mcast_ifn, u[i].syncid);
> -             if (u[i].state & IP_VS_STATE_MASTER)
> -                     printf("master sync daemon (mcast=%s, syncid=%d)\n",
> +             if (u[i].state & IP_VS_STATE_BACKUP)
> +                     printf("backup sync daemon (mcast=%s, syncid=%d)\n",
>                              u[i].mcast_ifn, u[i].syncid);
>       }
>       free(u);
> -- 
> 1.9.0

Regards

--
Julian Anastasov <ja@xxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe lvs-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

<Prev in Thread] Current Thread [Next in Thread>