diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2012-03-04 17:36:23 +0000 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2012-03-04 17:36:23 +0000 |
commit | 462de06d9640c237a23ad4ab2c0a5529387868d6 (patch) | |
tree | 84880cc788022034fe6aed06a19a0167f14a7085 /libdm | |
parent | b3103ef328eaeff816ce8e44331f8a9a164dc3e2 (diff) | |
download | lvm2-462de06d9640c237a23ad4ab2c0a5529387868d6.tar.gz lvm2-462de06d9640c237a23ad4ab2c0a5529387868d6.tar.xz lvm2-462de06d9640c237a23ad4ab2c0a5529387868d6.zip |
Return success for deactivation of thin pool
if the thin_check fail on thin pool - still return successful deactivation,
since lvremove would currently fail.
TODO: find some way to not run check with lvremove.
Diffstat (limited to 'libdm')
-rw-r--r-- | libdm/libdm-deptree.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/libdm/libdm-deptree.c b/libdm/libdm-deptree.c index bbd2b9f8..4dead759 100644 --- a/libdm/libdm-deptree.c +++ b/libdm/libdm-deptree.c @@ -1591,7 +1591,10 @@ static int _dm_tree_deactivate_children(struct dm_tree_node *dnode, if (child->callback && !child->callback(child, DM_NODE_CALLBACK_DEACTIVATED, child->callback_data)) - r = 0; // FIXME: _node_clear_table() without callback ? + stack; + // FIXME: We need to let lvremove pass, + // so for now deactivation ignores check result + //r = 0; // FIXME: _node_clear_table() without callback ? if (dm_tree_node_num_children(child, 0) && !_dm_tree_deactivate_children(child, uuid_prefix, uuid_prefix_len, level + 1)) |