areguera / rpms / cockpit

Forked from rpms/cockpit 5 years ago
Clone

Blame SOURCES/0001-storaged-Drop-VDO-async-option-8679.patch

87e9e1
From 64315aad40a4a525611c2d66cff1c214c29ed44a Mon Sep 17 00:00:00 2001
87e9e1
From: Martin Pitt <martinpitt@users.noreply.github.com>
87e9e1
Date: Mon, 26 Feb 2018 08:55:27 +0100
87e9e1
Subject: [PATCH] storaged: Drop VDO async option (#8679)
87e9e1
87e9e1
The current VDO version will auto detect the correct setting to use for
87e9e1
write policy (either sync or async).  Explicitly setting the option
87e9e1
disable the auto feature and set VDO to async mode, which may get
87e9e1
into an unsupported configuration later on.
87e9e1
87e9e1
As this is also an option that is quite hard to explain to users and we
87e9e1
don't/can't give them the information that they need to do an informed
87e9e1
decision, just remove the option completely and rely on automatic
87e9e1
detection.
87e9e1
---
87e9e1
 pkg/storaged/client.js      | 2 --
87e9e1
 pkg/storaged/vdos-panel.jsx | 5 -----
87e9e1
 2 files changed, 7 deletions(-)
87e9e1
87e9e1
diff --git a/pkg/storaged/client.js b/pkg/storaged/client.js
87e9e1
index 54f7e4e..761605c 100644
87e9e1
--- a/pkg/storaged/client.js
87e9e1
+++ b/pkg/storaged/client.js
87e9e1
@@ -695,8 +695,6 @@
87e9e1
                 args.push("--deduplication", options.deduplication? "enabled" : "disabled");
87e9e1
             if (options.emulate_512 !== undefined)
87e9e1
                 args.push("--emulate512", options.emulate_512? "enabled" : "disabled");
87e9e1
-            if (options.asynchronous !== undefined)
87e9e1
-                args.push("--writePolicy", options.asynchronous? "async" : "sync");
87e9e1
             return cmd(args);
87e9e1
         }
87e9e1
 
87e9e1
diff --git a/pkg/storaged/vdos-panel.jsx b/pkg/storaged/vdos-panel.jsx
87e9e1
index 035760d..529b08f 100644
87e9e1
--- a/pkg/storaged/vdos-panel.jsx
87e9e1
+++ b/pkg/storaged/vdos-panel.jsx
87e9e1
@@ -98,10 +98,6 @@ export class VDOsPanel extends React.Component {
87e9e1
                                 Title: _("Deduplication"),
87e9e1
                                 Value: true
87e9e1
                               },
87e9e1
-                              { CheckBox: "asynchronous",
87e9e1
-                                Title: _("Transfer data asynchronously"),
87e9e1
-                                Value: false,
87e9e1
-                              },
87e9e1
                               { CheckBox: "emulate_512",
87e9e1
                                 Title: _("Use 512 Byte emulation"),
87e9e1
                                 Value: false
87e9e1
@@ -124,7 +120,6 @@ export class VDOsPanel extends React.Component {
87e9e1
                                                             index_mem: vals.index_mem,
87e9e1
                                                             compression: vals.compression,
87e9e1
                                                             deduplication: vals.deduplication,
87e9e1
-                                                            asynchronous: vals.asynchronous,
87e9e1
                                                             emulate_512: vals.emulate_512
87e9e1
                                                         });
87e9e1
                                                     });
87e9e1
-- 
87e9e1
2.14.3
87e9e1