All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Teigland <teigland@sourceware.org>
To: lvm-devel@redhat.com
Subject: master - scan: put dev back on caller's list
Date: Mon, 23 Apr 2018 09:51:21 -0400	[thread overview]
Message-ID: <201804231351.w3NDpLd9032420@lists01.pubmisc.prod.ext.phx2.redhat.com> (raw)

Gitweb:        https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=c2b10daf694bea490044b752209a416f886373ea
Commit:        c2b10daf694bea490044b752209a416f886373ea
Parent:        e7670d333855ecccb21b30bc6b4525d6bafaa940
Author:        David Teigland <teigland@redhat.com>
AuthorDate:    Thu Feb 15 14:17:51 2018 -0600
Committer:     David Teigland <teigland@redhat.com>
CommitterDate: Fri Apr 20 11:22:47 2018 -0500

scan: put dev back on caller's list

Commit 6e442875613915e506440e59a290b56756df2521 missed
adding devs back to caller's list.
---
 lib/label/label.c |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/lib/label/label.c b/lib/label/label.c
index 38db733..c35818a 100644
--- a/lib/label/label.c
+++ b/lib/label/label.c
@@ -513,6 +513,7 @@ static int _scan_list(struct dm_list *devs, int *failed)
 			if (!_scan_dev_open(devl->dev)) {
 				log_debug_devs("%s: Failed to open device.", dev_name(devl->dev));
 				dm_list_del(&devl->list);
+				dm_list_add(&done_devs, &devl->list);
 				scan_failed_count++;
 				continue;
 			}
@@ -561,8 +562,8 @@ static int _scan_list(struct dm_list *devs, int *failed)
 	if (!dm_list_empty(devs))
 		goto scan_more;
 
-	log_debug_devs("Scanned %d devices: %d for lvm, %d failed.",
-			dm_list_size(&done_devs), scan_lvm_count, scan_failed_count);
+	log_debug_devs("Scanned devices: %d lvm, %d failed.",
+			scan_lvm_count, scan_failed_count);
 
 	if (failed)
 		*failed = scan_failed_count;



             reply	other threads:[~2018-04-23 13:51 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-23 13:51 David Teigland [this message]
2018-04-23 13:55 master - scan: put dev back on caller's list David Teigland

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201804231351.w3NDpLd9032420@lists01.pubmisc.prod.ext.phx2.redhat.com \
    --to=teigland@sourceware.org \
    --cc=lvm-devel@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.