mirror of
git://sourceware.org/git/lvm2.git
synced 2025-03-10 16:58:47 +03:00
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.
This commit is contained in:
parent
bd0ee420b5
commit
dc2ce71313
@ -1675,7 +1675,6 @@ int lv_raid_replace(struct logical_volume *lv,
|
|||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
// FIXME: safe to use 'virtual' as a substitute for 'error'?
|
|
||||||
if (lv_is_virtual(seg_lv(raid_seg, s)) ||
|
if (lv_is_virtual(seg_lv(raid_seg, s)) ||
|
||||||
lv_is_virtual(seg_metalv(raid_seg, s)) ||
|
lv_is_virtual(seg_metalv(raid_seg, s)) ||
|
||||||
_lv_is_on_pvs(seg_lv(raid_seg, s), remove_pvs) ||
|
_lv_is_on_pvs(seg_lv(raid_seg, s), remove_pvs) ||
|
||||||
|
Loading…
x
Reference in New Issue
Block a user