diff mbox

libmultipath: fix unit to seconds in log message for checker timeout

Message ID 20170627151147.11776-1-mwilck@suse.com (mailing list archive)
State Not Applicable, archived
Delegated to: Mike Snitzer
Headers show

Commit Message

Martin Wilck June 27, 2017, 3:11 p.m. UTC
checker_timeout is in seconds, not milliseconds, since 905281da
"Specify checker_timeout in seconds". Fix the log messages.

Signed-off-by: Martin Wilck <mwilck@suse.com>
---
 libmultipath/propsel.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Christophe Varoqui Aug. 3, 2017, 6:44 a.m. UTC | #1
Merged,
Thanks.

On Tue, Jun 27, 2017 at 5:11 PM, Martin Wilck <mwilck@suse.com> wrote:

> checker_timeout is in seconds, not milliseconds, since 905281da
> "Specify checker_timeout in seconds". Fix the log messages.
>
> Signed-off-by: Martin Wilck <mwilck@suse.com>
> ---
>  libmultipath/propsel.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/libmultipath/propsel.c b/libmultipath/propsel.c
> index 27f39517..4ac7edcd 100644
> --- a/libmultipath/propsel.c
> +++ b/libmultipath/propsel.c
> @@ -333,11 +333,11 @@ out:
>                                 pp->dev, c->timeout);
>         }
>         else if (sysfs_get_timeout(pp, &c->timeout) > 0)
> -               condlog(3, "%s: checker timeout = %u ms (setting: kernel
> sysfs)",
> +               condlog(3, "%s: checker timeout = %u s (setting: kernel
> sysfs)",
>                                 pp->dev, c->timeout);
>         else {
>                 c->timeout = DEF_TIMEOUT;
> -               condlog(3, "%s: checker timeout = %u ms (setting:
> multipath internal)",
> +               condlog(3, "%s: checker timeout = %u s (setting: multipath
> internal)",
>                                 pp->dev, c->timeout);
>         }
>         return 0;
> --
> 2.13.1
>
>
--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel
diff mbox

Patch

diff --git a/libmultipath/propsel.c b/libmultipath/propsel.c
index 27f39517..4ac7edcd 100644
--- a/libmultipath/propsel.c
+++ b/libmultipath/propsel.c
@@ -333,11 +333,11 @@  out:
 				pp->dev, c->timeout);
 	}
 	else if (sysfs_get_timeout(pp, &c->timeout) > 0)
-		condlog(3, "%s: checker timeout = %u ms (setting: kernel sysfs)",
+		condlog(3, "%s: checker timeout = %u s (setting: kernel sysfs)",
 				pp->dev, c->timeout);
 	else {
 		c->timeout = DEF_TIMEOUT;
-		condlog(3, "%s: checker timeout = %u ms (setting: multipath internal)",
+		condlog(3, "%s: checker timeout = %u s (setting: multipath internal)",
 				pp->dev, c->timeout);
 	}
 	return 0;