commit 466d7580f2461c7065f420475ef14c05fbf7417e
author Treehugger Robot <treehugger-gerrit@google.com> Sat Mar 04 16:56:20 2023 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Sat Mar 04 16:56:20 2023 +0000
tree 7140637ace4a545e9837edf0ca3b0b8468bf4d4e
parent 25a92407b1676635dede1a72e83cf37eab8393c8
parent b0106f8ce2070630c632d8e4de4f56164d54c210

Merge "Fix PvmfwDebugPolicyHostTests#testRamdump"

virtualizationmanager/src/debug_config.rs

diff --git a/virtualizationmanager/src/debug_config.rs b/virtualizationmanager/src/debug_config.rs
index a4ec419..eda854a 100644
--- a/virtualizationmanager/src/debug_config.rs
+++ b/virtualizationmanager/src/debug_config.rs
@@ -40,19 +40,13 @@
 pub fn is_ramdump_needed(config: &VirtualMachineConfig) -> bool {
     let enabled_in_dp =
         get_debug_policy_bool("/proc/device-tree/avf/guest/common/ramdump").unwrap_or_default();
-    let (protected, debuggable) = match config {
-        VirtualMachineConfig::RawConfig(config) => {
+    let debuggable = match config {
+        VirtualMachineConfig::RawConfig(_) => {
             // custom VMs are considered debuggable for flexibility
-            (config.protectedVm, true)
+            true
         }
-        VirtualMachineConfig::AppConfig(config) => {
-            (config.protectedVm, config.debugLevel == DebugLevel::FULL)
-        }
+        VirtualMachineConfig::AppConfig(config) => config.debugLevel == DebugLevel::FULL,
     };
 
-    if protected {
-        enabled_in_dp
-    } else {
-        enabled_in_dp || debuggable
-    }
+    enabled_in_dp || debuggable
 }