summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorZdenek Kabelac <zkabelac@redhat.com>2017-03-08 14:28:28 +0100
committerZdenek Kabelac <zkabelac@redhat.com>2017-03-10 19:33:01 +0100
commitd11b8eef893b9bd05b2e923f8637e527df16ac01 (patch)
treec34f528231f6472cf4cdbd2a5a1155a5ccc162ed
parent52935b5834b6592acab1e68df18b9779de54033f (diff)
downloadlvm2-d11b8eef893b9bd05b2e923f8637e527df16ac01.tar.gz
cleanup: easier code
-rw-r--r--lib/metadata/thin_manip.c20
1 files changed, 8 insertions, 12 deletions
diff --git a/lib/metadata/thin_manip.c b/lib/metadata/thin_manip.c
index ce5b95cf3..d1259b430 100644
--- a/lib/metadata/thin_manip.c
+++ b/lib/metadata/thin_manip.c
@@ -781,20 +781,16 @@ int lv_is_thin_origin(const struct logical_volume *lv, unsigned int *snap_count)
if (snap_count)
*snap_count = 0;
- if (!lv_is_thin_volume(lv) ||
- dm_list_empty(&lv->segs_using_this_lv))
- return 0;
+ if (lv_is_thin_volume(lv))
+ dm_list_iterate_items(segl, &lv->segs_using_this_lv)
+ if (segl->seg->origin == lv) {
+ r = 1;
+
+ if (!snap_count)
+ break;/* not interested in number of snapshots */
- dm_list_iterate_items(segl, &lv->segs_using_this_lv) {
- if (segl->seg->origin == lv) {
- r = 1;
- if (snap_count)
(*snap_count)++;
- else
- /* not interested in number of snapshots */
- break;
- }
- }
+ }
return r;
}