From dc2ce71313cef15efa2ba0addfb0f4115af80ca6 Mon Sep 17 00:00:00 2001 From: Jonathan Brassow Date: Wed, 20 Feb 2013 15:03:58 -0600 Subject: [PATCH] clean-up: Remove a FIXME question that has been settled It is ok for us to use the shorthand 'lv_is_virtual' to detect error targets in a RAID LV when searching for candidates for device replacement. --- lib/metadata/raid_manip.c | 1 - 1 file changed, 1 deletion(-) diff --git a/lib/metadata/raid_manip.c b/lib/metadata/raid_manip.c index f716664bd..b1ada5cd1 100644 --- a/lib/metadata/raid_manip.c +++ b/lib/metadata/raid_manip.c @@ -1675,7 +1675,6 @@ int lv_raid_replace(struct logical_volume *lv, return 0; } - // FIXME: safe to use 'virtual' as a substitute for 'error'? if (lv_is_virtual(seg_lv(raid_seg, s)) || lv_is_virtual(seg_metalv(raid_seg, s)) || _lv_is_on_pvs(seg_lv(raid_seg, s), remove_pvs) ||