<tt><font size=2>Hello Bart,</font></tt>
<br><font size=2 face="sans-serif">I also think </font><tt><font size=2>that
we do not need init_path_check_interval()</font></tt><font size=2 face="sans-serif">，</font>
<br><font size=2 face="sans-serif">since pp->checkint is </font><tt><font size=2>initialized
in configure() too.</font></tt>
<br>
<br><font size=2 face="sans-serif">Cheers,</font>
<br><font size=2 face="sans-serif">Tang<br>
</font>
<br>
<br>
<br>
<br><font size=1 color=#5f5f5f face="sans-serif">发件人:    
    </font><font size=1 face="sans-serif">Bart Van Assche
<bart.vanassche@sandisk.com></font>
<br><font size=1 color=#5f5f5f face="sans-serif">收件人:    
    </font><font size=1 face="sans-serif"><tang.junhui@zte.com.cn>,
<christophe.varoqui@opensvc.com>, </font>
<br><font size=1 color=#5f5f5f face="sans-serif">抄送:    
   </font><font size=1 face="sans-serif">bart.vanassche@sandisk.com,
dm-devel@redhat.com</font>
<br><font size=1 color=#5f5f5f face="sans-serif">日期:    
    </font><font size=1 face="sans-serif">2016/10/11
04:41</font>
<br><font size=1 color=#5f5f5f face="sans-serif">主题:    
   </font><font size=1 face="sans-serif">Re: [dm-devel]
[PATCH 00/15] Fifteen multipath-tools patches</font>
<br><font size=1 color=#5f5f5f face="sans-serif">发件人:    
   </font><font size=1 face="sans-serif">dm-devel-bounces@redhat.com</font>
<br>
<hr noshade>
<br>
<br>
<br><tt><font size=2>On 10/07/2016 07:05 PM, tang.junhui@zte.com.cn wrote:<br>
> "0007-multipathd-Fix-a-data-race.patch"<br>
><br>
> This patch solved the data race problem,<br>
><br>
> but the assignment for paths check interval has no effect,<br>
><br>
> since there is no path in vecs when calling<br>
> init_path_check_interval(vecs) in child(),<br>
><br>
> I think it is better to call<br>
>  init_path_check_interval(vecs) at reconfigure() or configure()<br>
><br>
> after the paths has created in vecs .<br>
<br>
Hello Tang,<br>
<br>
If init_path_check_interval() doesn't have any effect then I propose to
<br>
remove that function. pp->checkint is namely already initialized in
<br>
cli_add_path().<br>
<br>
Bart.<br>
<br>
--<br>
dm-devel mailing list<br>
dm-devel@redhat.com<br>
</font></tt><a href="https://www.redhat.com/mailman/listinfo/dm-devel"><tt><font size=2>https://www.redhat.com/mailman/listinfo/dm-devel</font></tt></a><tt><font size=2><br>
</font></tt>
<br>