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

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

 



Kazunori,


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

I've completely forget this patch :) It's in master now.

> 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
> 

_______________________________________________
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