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

Re: [dm-devel] RDAC path checker status change messages




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.


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



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