[lvm-devel] [PATCH 2 of 4] LVM: cleanup 2

Jonathan Brassow jbrassow at redhat.com
Mon Jan 10 20:56:38 UTC 2011


Patch name: lvm-cleanup-2.patch

The check for segtype_is_virtual is done at the top of the function,
no need to do it again at the bottom.


Index: LVM2/lib/metadata/lv_manip.c
===================================================================
--- LVM2.orig/lib/metadata/lv_manip.c
+++ LVM2/lib/metadata/lv_manip.c
@@ -1557,8 +1557,7 @@ struct alloc_handle *allocate_extents(st
 			       parallel_areas)))
 		return_NULL;
 
-	if (!segtype_is_virtual(segtype) &&
-	    !_allocate(ah, vg, lv, 1, allocatable_pvs)) {
+	if (!_allocate(ah, vg, lv, 1, allocatable_pvs)) {
 		alloc_destroy(ah);
 		return_NULL;
 	}




More information about the lvm-devel mailing list