[dm-devel] 答复: Re: [PATCH 1/2] multipath-tools: replace static and define default values for config options

[zhou.guichun at zte.com.cn]

I am terrible sorry for brought inconvenience for you because of my wrong 
operition.I 
have fix this wrong. Excuse me for my  unintentional fault!

 
周桂春  Zhou GuiChun
软件开发  SoftWare Development
虚拟化长沙开发部  NIV Changsha Development Dept



湖南省长沙市岳麓区尖山路39号中电软件园八楼

T: +86 731-82057106    M: +86 13657402573
E: zhou.guichun at zte.com.cn
www.zte.com.cn




发件人:         Xose Vazquez Perez <xose.vazquez at gmail.com>
收件人:         zhou.guichun at zte.com.cn, 10184522 at zte.com.cn, 
10072500 at zte.com.cn, 10074136 at zte.com.cn, 
抄送:   Benjamin Marzinski <bmarzins at redhat.com>, Hannes Reinecke 
<hare at suse.de>, Christophe Varoqui <christophe.varoqui at opensvc.com>, 
device-mapper development <dm-devel at redhat.com>
日期:   2016/10/27 03:17
主题:   Re: [PATCH 1/2] multipath-tools: replace static and define default 
values for config options



On 10/26/2016 04:33 AM, zhou.guichun at zte.com.cn wrote:
> From: Xose Vazquez Perez <xose.vazquez at gmail.com>

This patch already was applied: 
http://git.opensvc.com/gitweb.cgi?p=multipath-tools/.git;a=commitdiff;h=906c098284afc9573d710ba25580b6919352a3ff

This looks like an internal e-mail, please fix your mailer.

> Replace "const" by PRIO_CONST.
> Replace static force_sync and partition_delim values, and define
> DEFAULT_FORCE_SYNC and DEFAULT_PARTITION_DELIM
> 
> Cc: Benjamin Marzinski <bmarzins at redhat.com>
> Cc: Hannes Reinecke <hare at suse.de>
> Cc: Christophe Varoqui <christophe.varoqui at opensvc.com>
> Cc: device-mapper development <dm-devel at redhat.com>
> Signed-off-by: Xose Vazquez Perez <xose.vazquez at gmail.com>
> ---
>  libmultipath/config.c   | 4 ++--
>  libmultipath/defaults.h | 4 +++-
>  2 files changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/libmultipath/config.c b/libmultipath/config.c
> index 92e4deb..ed51afb 100644
> --- a/libmultipath/config.c
> +++ b/libmultipath/config.c
> @@ -606,8 +606,8 @@ load_config (char * file)
>                conf->fast_io_fail = DEFAULT_FAST_IO_FAIL;
>                conf->retain_hwhandler = DEFAULT_RETAIN_HWHANDLER;
>                conf->detect_prio = DEFAULT_DETECT_PRIO;
> -              conf->force_sync = 0;
> -              conf->partition_delim = NULL;
> +              conf->force_sync = DEFAULT_FORCE_SYNC;
> +              conf->partition_delim = DEFAULT_PARTITION_DELIM;
>                conf->processed_main_config = 0;
>                conf->find_multipaths = DEFAULT_FIND_MULTIPATHS;
>                conf->uxsock_timeout = DEFAULT_REPLY_TIMEOUT;
> diff --git a/libmultipath/defaults.h b/libmultipath/defaults.h
> index e9581a6..9af9a9a 100644
> --- a/libmultipath/defaults.h
> +++ b/libmultipath/defaults.h
> @@ -28,11 +28,13 @@
>  #define DEFAULT_RETRIGGER_DELAY               10
>  #define DEFAULT_RETRIGGER_TRIES               3
>  #define DEFAULT_UEV_WAIT_TIMEOUT 30
> -#define DEFAULT_PRIO                          "const"
> +#define DEFAULT_PRIO                          PRIO_CONST
>  #define DEFAULT_PRIO_ARGS             ""
>  #define DEFAULT_CHECKER                               TUR
>  #define DEFAULT_FLUSH                                 FLUSH_DISABLED
>  #define DEFAULT_USER_FRIENDLY_NAMES USER_FRIENDLY_NAMES_OFF
> +#define DEFAULT_FORCE_SYNC            0
> +#define DEFAULT_PARTITION_DELIM               NULL
> 
>  #define DEFAULT_CHECKINT              5
>  #define MAX_CHECKINT(a)                               (a << 2)
> 



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/dm-devel/attachments/20161027/14e98d53/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: image/gif
Size: 6015 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/dm-devel/attachments/20161027/14e98d53/attachment.gif>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: image/gif
Size: 2054 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/dm-devel/attachments/20161027/14e98d53/attachment-0001.gif>