Re: [PATCH] Log auto-recovery of ring only once

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

Please merge this patch to fix the following defect.
http://www.spinics.net/lists/corosync/msg03316.html

Regards,
Kazunori INOUE


2014-10-02 21:09 GMT+09:00 Jan Friesse <jfriesse@xxxxxxxxxx>:
> Make sure to log auto-recovery of ring only once. Every
> MESSAGE_TYPE_RING_TEST_ACTIVATE receive is logged, but with lower
> priority and more detailed information.
>
> Signed-off-by: Jan Friesse <jfriesse@xxxxxxxxxx>
> ---
>  exec/totemrrp.c |   14 +++++++++++---
>  1 files changed, 11 insertions(+), 3 deletions(-)
>
> diff --git a/exec/totemrrp.c b/exec/totemrrp.c
> index 95a789e..6817395 100644
> --- a/exec/totemrrp.c
> +++ b/exec/totemrrp.c
> @@ -1909,15 +1909,23 @@ void rrp_deliver_fn (
>                 }
>         } else
>         if (hdr->type == MESSAGE_TYPE_RING_TEST_ACTIVATE) {
> -               log_printf (
> -                       rrp_instance->totemrrp_log_level_notice,
> -                       "Automatically recovered ring %d", hdr->ring_number);
>
>                 if (hdr->endian_detector != ENDIAN_LOCAL) {
>                         test_active_msg_endian_convert(hdr, &tmp_msg);
>                         hdr = &tmp_msg;
>                 }
>
> +               log_printf (
> +                       rrp_instance->totemrrp_log_level_debug,
> +                       "Received ring test activate message for ring %d sent by node %u",
> +                       hdr->ring_number,
> +                       hdr->nodeid_activator);
> +
> +               if (rrp_instance->stats.faulty[deliver_fn_context->iface_no]) {
> +                       log_printf (rrp_instance->totemrrp_log_level_notice,
> +                           "Automatically recovered ring %d", hdr->ring_number);
> +               }
> +
>                 totemrrp_ring_reenable (rrp_instance, deliver_fn_context->iface_no);
>                 if (hdr->nodeid_activator != rrp_instance->my_nodeid) {
>                         totemnet_token_send (
> --
> 1.7.1
>
> _______________________________________________
> discuss mailing list
> discuss@xxxxxxxxxxxx
> http://lists.corosync.org/mailman/listinfo/discuss
_______________________________________________
discuss mailing list
discuss@xxxxxxxxxxxx
http://lists.corosync.org/mailman/listinfo/discuss



[Index of Archives]     [Linux Clusters]     [Corosync Project]     [Linux USB Devel]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Linux Kernel]     [Linux SCSI]     [X.Org]

  Powered by Linux