Revert "raid: do not enforce flushing of raids when it is not required"

This reverts commit ce5ea07411d57a2bd276d8d29e46ce30b957f657.
More thinking needed.

diff --git a/WHATS_NEW b/WHATS_NEW
index 0aa23046db1266825154b9683543a2b0d3ceca49..aa3960b3122bdc041abbdf5ee5dc18c14cffacae 100644 (file)
--- a/WHATS_NEW
+++ b/WHATS_NEW
@@ -1,6 +1,5 @@
 Version 2.03.11 - 
 ==================================
-  Allow raid extension without flushing raid LV first.
   Use _rmeta and _rimage as origin only devices for table loading.
   Switch code base to use flexible array syntax.
   Fix 64bit math when calculation cachevol size.

diff --git a/lib/activate/activate.c b/lib/activate/activate.c
index c5adc50658006f85272a4ad5eec2cd6cd28e7e2f..0c6bd1a9a34d33def000b25f109882273cee55bc 100644 (file)
--- a/lib/activate/activate.c
+++ b/lib/activate/activate.c
@@ -2177,7 +2177,7 @@ static int _lv_suspend(struct cmd_context *cmd, const char *lvid_s,
         * TODO: Relax this limiting condition further */
        if (!flush_required &&
            (lv_is_pvmove(lv) || pvmove_lv ||
-            (!lv_is_raid(lv) && !lv_is_mirror(lv) && !lv_is_thin_pool(lv) && !lv_is_thin_volume(lv)))) {
+            (!lv_is_mirror(lv) && !lv_is_thin_pool(lv) && !lv_is_thin_volume(lv)))) {
                log_debug("Requiring flush for LV %s.", display_lvname(lv));
                flush_required = 1;
        }