[dm-devel] [PATCH] multipathd/main.c: delete redundant judgment before LOG_MSG()
Christophe Varoqui
christophe.varoqui at opensvc.com
Wed Oct 19 06:00:10 UTC 2016
Merged.
On Tue, Oct 18, 2016 at 7:02 PM, Benjamin Marzinski <bmarzins at redhat.com>
wrote:
> On Tue, Oct 18, 2016 at 05:11:45PM +0800, li.minghui7 at zte.com.cn wrote:
> > From: "li.minghui" <li.minghui7 at zte.com.cn>
> >
> > Problem:the LOG_MSG() doesn't note the path info "path offline" when the
> > path is in offline with the pp->offline = 1.
> >
> > Reasons:There is no need to judge the strlen(b) again before calling
> > LOG_MSG(a, b) because there exists the judgmnet of strlen(b) in
> > defination of LOG_MSG(a,b).
>
> ACK
>
> -Ben
>
> > ---
> > multipathd/main.c | 9 +++------
> > 1 file changed, 3 insertions(+), 6 deletions(-)
> >
> > diff --git a/multipathd/main.c b/multipathd/main.c
> > index b6eb696..b147ca5 100644
> > --- a/multipathd/main.c
> > +++ b/multipathd/main.c
> > @@ -1454,8 +1454,7 @@ void repair_path(struct path * pp)
> > return;
> >
> > checker_repair(&pp->checker);
> > - if (strlen(checker_message(&pp->checker)))
> > - LOG_MSG(1, checker_message(&pp->checker));
> > + LOG_MSG(1, checker_message(&pp->checker));
> > }
> >
> > /*
> > @@ -1590,8 +1589,7 @@ check_path (struct vectors * vecs, struct path *
> pp, int ticks)
> > int oldstate = pp->state;
> > pp->state = newstate;
> >
> > - if (strlen(checker_message(&pp->checker)))
> > - LOG_MSG(1, checker_message(&pp->checker));
> > + LOG_MSG(1, checker_message(&pp->checker));
> >
> > /*
> > * upon state change, reset the checkint
> > @@ -1703,8 +1701,7 @@ check_path (struct vectors * vecs, struct path *
> pp, int ticks)
> > pp->tick = pp->checkint;
> > }
> > }
> > - else if (newstate == PATH_DOWN &&
> > - strlen(checker_message(&pp->checker))) {
> > + else if (newstate == PATH_DOWN) {
> > int log_checker_err;
> >
> > conf = get_multipath_config();
> > --
> > 2.8.1.windows.1
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/dm-devel/attachments/20161019/bb61b9ef/attachment.htm>
More information about the dm-devel
mailing list