[PATCH v3] libmultipath: minimize noise with snapshots in emc_clariion checker

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

 



Corrections noted. Sorry for the inconvenience... changing include and inverting diff.
Thanks, Mike!!!!
--- 

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


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


--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel




[Index of Archives]     [DM Crypt]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Packaging]     [Fedora SELinux]     [Yosemite Discussion]     [KDE Users]     [Fedora Docs]

  Powered by Linux