Merge "Start primary storage move outside lock." into nyc-dev (a3290363) · Commits · e / os / android_frameworks_base

services/core/java/com/android/server/MountService.java

+12 −8

Original line number	Diff line number	Diff line
		@@ -2030,6 +2030,9 @@ class MountService extends IMountService.Stub
		enforcePermission(android.Manifest.permission.MOUNT_UNMOUNT_FILESYSTEMS);
		waitForReady();

		final VolumeInfo from;
		final VolumeInfo to;

		synchronized (mLock) {
		if (Objects.equals(mPrimaryStorageUuid, volumeUuid)) {
		throw new IllegalArgumentException("Primary storage already at " + volumeUuid);
		@@ -2049,10 +2052,11 @@ class MountService extends IMountService.Stub
		onMoveStatusLocked(MOVE_STATUS_COPY_FINISHED);
		onMoveStatusLocked(PackageManager.MOVE_SUCCEEDED);
		mHandler.obtainMessage(H_RESET).sendToTarget();
		return;

		} else {
		final VolumeInfo from = findStorageForUuid(mPrimaryStorageUuid);
		final VolumeInfo to = findStorageForUuid(volumeUuid);
		from = findStorageForUuid(mPrimaryStorageUuid);
		to = findStorageForUuid(volumeUuid);

		if (from == null) {
		Slog.w(TAG, "Failing move due to missing from volume " + mPrimaryStorageUuid);
		@@ -2063,6 +2067,8 @@ class MountService extends IMountService.Stub
		onMoveStatusLocked(PackageManager.MOVE_FAILED_INTERNAL_ERROR);
		return;
		}
		}
		}

		try {
		mConnector.execute("volume", "move_storage", from.id, to.id);
		@@ -2070,8 +2076,6 @@ class MountService extends IMountService.Stub
		throw e.rethrowAsParcelableException();
		}
		}
		}
		}

		@Override
		public int[] getStorageUsers(String path) {