Re: RDAC path checker status change messages

[Benjamin Marzinski <bmarzins@redhat.com>]

On Wed, Jun 24, 2009 at 04:19:00PM -0400, Charlie Brady wrote:
>
> On Wed, 24 Jun 2009, Chandra Seetharaman wrote:
>
>> We do not want to print these messages as they will be flooding
>> your /var/log/messages even when everything is good.
>
> Agreed.
>
>> We want to print these messages only to get User's attention.
>
> Agreed.
>
>> If your concern is that the last message is "down", then we should be
>> having a static variable and print up or ghost message only once (when
>> we toggle from down to up/ghost).
>
> You do have a static variable, so that messages are only printed when the 
> status changes. However, because of the missing MSG() calls when the status 
> is PATH_UP and PATH_GHOST, then the string printed is "path down", when it 
> should be "path up" or "path ghost".
>
> I believe that the patch is good. The MSG() calls will change the message 
> string, and then the higher layer will notice the status change, and print 
> the message.
>
> Please check the code and correct me if I am wrong.

You are correct. Looks fine by me.
I'll add this for RHEL 5.5. It will make it into Centos after that.

-Ben

>
>>
>> chandra
>> On Tue, 2009-06-23 at 15:50 -0400, Charlie Brady wrote:
>>> It seems to me that this patch is required, to fix a problem with "path
>>> down" messages being logged when the path transitions to up (and
>>> presumably ghost).
>>>
>>> --- libcheckers/rdac.c.orig     2009-06-23 15:38:05.371422000 -0400
>>> +++ libcheckers/rdac.c  2009-06-23 15:39:28.020843000 -0400
>>> @@ -105,5 +105,12 @@
>>>                  return PATH_DOWN;
>>>          }
>>>
>>> -       return ((inq.avtcvp & 0x1) ? PATH_UP : PATH_GHOST);
>>> +       if (inq.avtcvp & 0x1) {
>>> +               MSG(c, MSG_RDAC_UP);
>>> +               return PATH_UP;
>>> +       }
>>> +       else {
>>> +               MSG(c, MSG_RDAC_GHOST);
>>> +               return PATH_GHOST;
>>> +       }
>>>   }
>>>
>>> --
>>> dm-devel mailing list
>>> dm-devel@redhat.com
>>> https://www.redhat.com/mailman/listinfo/dm-devel
>>
>> --
>> dm-devel mailing list
>> dm-devel@redhat.com
>> https://www.redhat.com/mailman/listinfo/dm-devel
>>
>
> --
> dm-devel mailing list
> dm-devel@redhat.com
> https://www.redhat.com/mailman/listinfo/dm-devel