Re: [PATCH] multipath_tools: add alias while printing checker_message

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

 



You're right,
Applied

On mar., 2010-04-13 at 09:21 -0600, Moger, Babu wrote:
> This patch adds alias while printing checker messages. 
> Example of current Checker message. 
> 
> Apr 11 04:03:53 mymachine multipathd: sde: rdac checker reports path is down
> 
> Most of the time "sde" is meaningless when debugging the past failures. 
> 
> This patch add alias before the checker message..  
> Example of the new message..
> 
> Apr 12 16:55:54 mymachine multipathd: mpathb: sde - rdac checker reports path is down
> 
> Signed-off-by: Babu Moger <babu.moger@xxxxxxx>
> Reviewed-by: Vijay Chauhan <Vijay.Chauhan@xxxxxxx>
> 
> ---
> --- multipath-tools/multipathd/main.c.orig	2010-04-12 13:58:27.000000000 -0500
> +++ multipath-tools/multipathd/main.c	2010-04-12 14:01:08.000000000 -0500
> @@ -61,7 +61,7 @@
>  #define CMDSIZE 160
>  
>  #define LOG_MSG(a,b) \
> -	if (strlen(b)) condlog(a, "%s: %s", pp->dev, b);
> +	if (strlen(b)) condlog(a, "%s: %s - %s", pp->mpp->alias, pp->dev, b);
>  
>  pthread_cond_t exit_cond = PTHREAD_COND_INITIALIZER;
>  pthread_mutex_t exit_mutex = PTHREAD_MUTEX_INITIALIZER;
> 
> 
> 
> 
> --
> dm-devel mailing list
> dm-devel@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/dm-devel


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