Lazy unmount postinstall if it fails to unmount.
If postinstall forks a new child that's not killed when killing the main
postinstall process or if postinstall makes any other running process to
hold a file descriptor open in the mounted filesystem, the filesystem is
busy an we can't unmount /postinstall. Since the postinstall mountpoint is
fix in Android, we need to force a lazy unmount in order for the process
to succeed in a future run.
This case can only occur when canceling an update during postinstall (for
example if the update was retried from the server) and would otherwise
clear itself after a few unmount retries if the process using the fd
stops using it.
Bug: 31021934
Test: Added native tests to excersice this case.
Change-Id: I69404658f481082b944f2ea6077e9a754a690ae6
diff --git a/common/utils.cc b/common/utils.cc
index 17f3e09..1e04b61 100644
--- a/common/utils.cc
+++ b/common/utils.cc
@@ -686,14 +686,24 @@
}
bool UnmountFilesystem(const string& mountpoint) {
- for (int num_retries = 0; ; ++num_retries) {
+ int num_retries = 1;
+ for (;; ++num_retries) {
if (umount(mountpoint.c_str()) == 0)
+ return true;
+ if (errno != EBUSY || num_retries >= kUnmountMaxNumOfRetries)
break;
-
- TEST_AND_RETURN_FALSE_ERRNO(errno == EBUSY &&
- num_retries < kUnmountMaxNumOfRetries);
usleep(kUnmountRetryIntervalInMicroseconds);
}
+ if (errno == EINVAL) {
+ LOG(INFO) << "Not a mountpoint: " << mountpoint;
+ return false;
+ }
+ PLOG(WARNING) << "Error unmounting " << mountpoint << " after " << num_retries
+ << " attempts. Lazy unmounting instead, error was";
+ if (umount2(mountpoint.c_str(), MNT_DETACH) != 0) {
+ PLOG(ERROR) << "Lazy unmount failed";
+ return false;
+ }
return true;
}