From: Hannes Reinecke Subject: [PATCH 12/29] Minor fixes for priority handling Date: Mon, 15 Jul 2013 15:00:13 +0200 Message-ID: <1373893230-26077-13-git-send-email-hare@suse.de> References: <1373893230-26077-1-git-send-email-hare@suse.de> Reply-To: device-mapper development Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1373893230-26077-1-git-send-email-hare@suse.de> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dm-devel-bounces@redhat.com Errors-To: dm-devel-bounces@redhat.com To: Christophe Varoqui Cc: dm-devel@redhat.com List-Id: dm-devel.ids When no prio handler was selected we should be setting the priority to 'PRIO_UNDEF'. Also fixup a typo in the logging message when selecting the default prioritizer. Signed-off-by: Hannes Reinecke --- libmultipath/discovery.c | 1 + libmultipath/propsel.c | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/libmultipath/discovery.c b/libmultipath/discovery.c index fcb8e4f..a482c53 100644 --- a/libmultipath/discovery.c +++ b/libmultipath/discovery.c @@ -957,6 +957,7 @@ get_prio (struct path * pp) select_prio(pp); if (!prio_selected(p)) { condlog(3, "%s: no prio selected", pp->dev); + pp->priority = PRIO_UNDEF; return 1; } } diff --git a/libmultipath/propsel.c b/libmultipath/propsel.c index 65c2c7f..cea8c1c 100644 --- a/libmultipath/propsel.c +++ b/libmultipath/propsel.c @@ -447,7 +447,7 @@ select_prio (struct path * pp) prio_get(p, DEFAULT_PRIO, DEFAULT_PRIO_ARGS); condlog(3, "%s: prio = %s (internal default)", pp->dev, DEFAULT_PRIO); - condlog(3, "%s: prio = %s (internal default)", + condlog(3, "%s: prio args = %s (internal default)", pp->dev, DEFAULT_PRIO_ARGS); return 0; } -- 1.7.10.4