Re: Patch to minimize noise with snapshots in emc_clariion checker

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

 



On Mon, Apr 22 2013 at  4:34pm -0400,
Levy, Jerome <jerome.levy@xxxxxxx> wrote:

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

Hmm, something happened with your patch... this is actually the inverse
of what you wanted ;)

Also, the #include should be: #include "../libmultipath/debug.h" 

Please send v2 and use a subject like:
[PATCH v2] libmultipath: minimize noise with snapshots in emc_clariion checker

(Also wouldn't hurt to cc Christophe Varoqui <christophe.varoqui@xxxxxxxxx>)

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