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