Take notice of 'discard-enable' in xenstore

When enabling discard in the frontend, both the 'feature-discard' and
'discard-enable' flags should be taken into account.

Signed-off-by: Andreas Kinzler <ml-ak@posteo.de>
[Added commit comment]
Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

diff --git a/src/xenvbd/frontend.c b/src/xenvbd/frontend.c
index e65a3b09fc790aa2feb29fe0793c6b4cd9909590..83a0f5e14e4889a2d9517d87510da737968b32cf 100644 (file)
--- a/src/xenvbd/frontend.c
+++ b/src/xenvbd/frontend.c
@@ -833,6 +833,9 @@ FrontendReadDiskInfo(
     )
 {
     BOOLEAN Changed = FALSE;
+    BOOLEAN Discard;
+    BOOLEAN DiscardFeature = FALSE;
+    BOOLEAN DiscardEnable = TRUE;
 
     Changed |= FrontendReadFeature(Frontend,
                                    "feature-barrier",
@@ -840,9 +843,17 @@ FrontendReadDiskInfo(
     Changed |= FrontendReadFeature(Frontend,
                                    "feature-flush-cache",
                                    &Frontend->DiskInfo.FlushCache);
-    Changed |= FrontendReadFeature(Frontend,
-                                   "feature-discard",
-                                   &Frontend->DiskInfo.Discard);
+
+    // discard related
+    FrontendReadFeature(Frontend,
+                        "feature-discard",
+                        &DiscardFeature);
+    FrontendReadFeature(Frontend,
+                        "discard-enable",
+                        &DiscardEnable);
+    Discard = DiscardFeature && DiscardEnable;
+    Changed |= (Discard != Frontend->DiskInfo.Discard);
+    Frontend->DiskInfo.Discard = Discard;
     Changed |= FrontendReadFeature(Frontend,
                                    "discard-secure",
                                    &Frontend->DiskInfo.DiscardSecure);