<div dir="ltr">Merged.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 18, 2016 at 7:02 PM, Benjamin Marzinski <span dir="ltr"><<a href="mailto:bmarzins@redhat.com" target="_blank">bmarzins@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Oct 18, 2016 at 05:11:45PM +0800, <a href="mailto:li.minghui7@zte.com.cn">li.minghui7@zte.com.cn</a> wrote:<br>
> From: "li.minghui" <<a href="mailto:li.minghui7@zte.com.cn">li.minghui7@zte.com.cn</a>><br>
><br>
> Problem:the LOG_MSG() doesn't note the path info "path offline" when the<br>
> path is in offline with the pp->offline = 1.<br>
><br>
> Reasons:There is no need to judge the strlen(b) again before calling<br>
> LOG_MSG(a, b) because there exists the judgmnet of strlen(b) in<br>
> defination of LOG_MSG(a,b).<br>
<br>
</span>ACK<br>
<br>
-Ben<br>
<div class="HOEnZb"><div class="h5"><br>
> ---<br>
>  multipathd/main.c | 9 +++------<br>
>  1 file changed, 3 insertions(+), 6 deletions(-)<br>
><br>
> diff --git a/multipathd/main.c b/multipathd/main.c<br>
> index b6eb696..b147ca5 100644<br>
> --- a/multipathd/main.c<br>
> +++ b/multipathd/main.c<br>
> @@ -1454,8 +1454,7 @@ void repair_path(struct path * pp)<br>
>               return;<br>
><br>
>       checker_repair(&pp->checker);<br>
> -     if (strlen(checker_message(&pp-><wbr>checker)))<br>
> -             LOG_MSG(1, checker_message(&pp->checker))<wbr>;<br>
> +     LOG_MSG(1, checker_message(&pp->checker))<wbr>;<br>
>  }<br>
><br>
>  /*<br>
> @@ -1590,8 +1589,7 @@ check_path (struct vectors * vecs, struct path * pp, int ticks)<br>
>               int oldstate = pp->state;<br>
>               pp->state = newstate;<br>
><br>
> -             if (strlen(checker_message(&pp-><wbr>checker)))<br>
> -                     LOG_MSG(1, checker_message(&pp->checker))<wbr>;<br>
> +             LOG_MSG(1, checker_message(&pp->checker))<wbr>;<br>
><br>
>               /*<br>
>                * upon state change, reset the checkint<br>
> @@ -1703,8 +1701,7 @@ check_path (struct vectors * vecs, struct path * pp, int ticks)<br>
>                       pp->tick = pp->checkint;<br>
>               }<br>
>       }<br>
> -     else if (newstate == PATH_DOWN &&<br>
> -              strlen(checker_message(&pp-><wbr>checker))) {<br>
> +     else if (newstate == PATH_DOWN) {<br>
>               int log_checker_err;<br>
><br>
>               conf = get_multipath_config();<br>
> --<br>
> 2.8.1.windows.1<br>
</div></div></blockquote></div><br></div>