[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

[lvm-devel] [PATCH LVM2] (9/12) lvconvert to use build_parallel_areas_from_lv()



This patch uncomments the use of build_parallel_areas_from_lv() by lvconvert.
Based on other patches, it works now.


$ diffstat -p1 09.lvconvert.patch
 tools/lvconvert.c |   13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)


Thanks,
-- 
Jun'ichi Nomura, NEC Corporation of America
diff -X dontdiff -urp LVM2.08.anywhere/tools/lvconvert.c LVM2.09.lvconvert/tools/lvconvert.c
--- LVM2.08.anywhere/tools/lvconvert.c	2006-10-13 09:58:47.000000000 -0400
+++ LVM2.09.lvconvert/tools/lvconvert.c	2006-10-13 21:20:59.000000000 -0400
@@ -281,15 +281,10 @@ static int lvconvert_mirrors(struct cmd_
 			if (lp->mirrors == existing_mirrors) {
 				if (!seg->log_lv && !arg_count(cmd, corelog_ARG)) {
 					/* No disk log present, add one. */
-					/* FIXME: Why doesn't this work?  Without
-					   it, we will probably put the log on the
-					   same device as a mirror leg.
-					  if (!(parallel_areas = build_parallel_areas_from_lv(cmd, lv))) {
-					  stack;
-					  return 0;
-					  }
-					*/
-					parallel_areas = NULL;
+					if (!(parallel_areas = build_parallel_areas_from_lv(cmd, lv))) {
+						stack;
+						return 0;
+					}
 					if (!lv_mirror_percent(cmd, lv, 0, &sync_percent, NULL)) {
 						log_error("Unable to determine mirror sync status.");
 						return 0;

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]