diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2015-10-25 20:41:19 +0100 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2015-10-25 21:09:31 +0100 |
commit | f898cf7539149a54a121b0aae59076c2d647bb37 (patch) | |
tree | 30a710d6acc1c42b238d5ad863baed10f4acf4db | |
parent | 844b00958492224c448b30f89ae5532bb6591fd2 (diff) | |
download | lvm2-f898cf7539149a54a121b0aae59076c2d647bb37.tar.gz |
dev_manager: no flush for extension
Recognize the target only 'extends' and do not enforce
'flush' in this case. Only the size reduction
still requires flush (so disables usage of no_flush flag).
If some other targets do require flush before suspend,
they have to explicitly ask for it.
-rw-r--r-- | WHATS_NEW | 1 | ||||
-rw-r--r-- | lib/activate/dev_manager.c | 2 |
2 files changed, 2 insertions, 1 deletions
@@ -1,5 +1,6 @@ Version 2.02.133 - ====================================== + Enforce flush for suspend only when volume size is reduced. Enable code which detects the need of flush during suspend. Ensure --use-policy will resize volume to fit below threshold. Correct percentage evaluation when checking thin-pool over threshold. diff --git a/lib/activate/dev_manager.c b/lib/activate/dev_manager.c index 1caa2d77a..dc6d2fd29 100644 --- a/lib/activate/dev_manager.c +++ b/lib/activate/dev_manager.c @@ -3296,7 +3296,7 @@ static int _tree_action(struct dev_manager *dm, const struct logical_volume *lv, if (!dm_tree_preload_children(root, dlid, DLID_SIZE)) goto_out; - if (dm_tree_node_size_changed(root)) + if ((dm_tree_node_size_changed(root) < 0)) dm->flush_required = 1; if (action == ACTIVATE) { |