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

Zdenek Kabelac zkabelac at redhat.com
Tue Jan 11 10:32:10 UTC 2011


Dne 10.1.2011 21:56, Jonathan Brassow napsal(a):
> 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;
>  	}
> 

Ack

Zdenek




More information about the lvm-devel mailing list