Revert "Reboot only userspace to kick FUSE flag change into place"

This reverts commit 41cd8dec12.

Reason for revert: b/148423904

IN ADDITION this change adds FuseDaemonHostTest to presubmit only when
StorageManagerService.java has been modified. This should help catch
similar problems in the future.

Change-Id: Ib6864c6686bef801eddbfa7b129431c5f1372b21
This commit is contained in:
Shafik Nassar
2020-01-28 18:35:25 +00:00
parent 41cd8dec12
commit 07dcbbc8d4
2 changed files with 6 additions and 9 deletions

View File

@@ -1701,15 +1701,8 @@ class StorageManagerService extends IStorageManager.Stub
if (mIsFuseEnabled != settingsFuseFlag) {
Slog.i(TAG, "Toggling persist.sys.fuse to " + settingsFuseFlag);
SystemProperties.set(PROP_FUSE, Boolean.toString(settingsFuseFlag));
PowerManager powerManager = mContext.getSystemService(PowerManager.class);
if (powerManager.isRebootingUserspaceSupported()) {
// Perform userspace reboot to kick policy into place
powerManager.reboot(PowerManager.REBOOT_USERSPACE);
} else {
// Perform hard reboot to kick policy into place
powerManager.reboot("fuse_prop");
}
// Perform hard reboot to kick policy into place
mContext.getSystemService(PowerManager.class).reboot("fuse_prop");
}
}

View File

@@ -36,6 +36,10 @@
}
],
"file_patterns": ["NotificationManagerService\\.java"]
},
{
"name": "FuseDaemonHostTest",
"file_patterns": ["StorageManagerService\\.java"]
}
]
}