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.

diff --git a/WHATS_NEW b/WHATS_NEW
index 33d11a46dd05ccd5a0cc2437b5efc475483655b0..80a198a03638823795e816a059a09b5927a105a2 100644 (file)
--- a/WHATS_NEW
+++ b/WHATS_NEW
@@ -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 1caa2d77a311ec2c1b75797631989b31cf7871e5..dc6d2fd29f61dd9a6d25e4cb239c875622c92695 100644 (file)
--- 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) {