[dm-devel] Patch to minimize noise with snapshots in emc_clariion checker

Levy, Jerome jerome.levy at emc.com
Mon Apr 22 20:34:14 UTC 2013


Patch to stop emc_clariion_checker from logging messages when probes 
to snapshot LUNs occur. Notification is still available if logging is 
turned up (see condlog()) but normal probing of snapshots will no 
longer produce status messages. Path functionality on snapshot probes 
is unchanged.

Signed-off-by: Jerry Levy <jerome.levy at emc.com>

--- ./libmultipath/checkers/emc_clariion.c      2013-04-22 15:50:19.081555624 -0400
+++ ./libmultipath/checkers/emc_clariion.c.orig 2013-04-22 14:07:36.966548097 -0400
@@ -15,7 +15,6 @@
 #include "../libmultipath/sg_include.h"
 #include "libsg.h"
 #include "checkers.h"
-#include "debug.h"
 
 #define INQUIRY_CMD     0x12
 #define INQUIRY_CMDLEN  6
@@ -199,8 +198,9 @@ int libcheck_check (struct checker * c)
                                 * 02/04/03 not 05/25/01 on read.
                                 */
                                SET_INACTIVE_SNAP(c);
-                               condlog(3, "emc_clariion_checker: Active path to "
-                                       "inactive snapshot WWN %s.", wwnstr);
+                               MSG(c, "emc_clariion_checker: Active "
+                                       "path to inactive snapshot WWN %s.",
+                                       wwnstr);
                        } else
                                MSG(c, "emc_clariion_checker: Read "
                                        "error for WWN %s.  Sense data are "
@@ -219,8 +219,9 @@ int libcheck_check (struct checker * c)
        } else {
                if (IS_INACTIVE_SNAP(c)) {
                        hexadecimal_to_ascii(ct->wwn, wwnstr);
-                       condlog(3, "emc_clariion_checker: Passive path to "
-                               "inactive snapshot WWN %s.", wwnstr);
+                       MSG(c, "emc_clariion_checker: Passive "
+                               "path to inactive snapshot WWN %s.",
+                               wwnstr);
                        ret = PATH_DOWN;
                } else {
                        MSG(c,





More information about the dm-devel mailing list