From af9d667ccf3e24058214cf4cc0a8aa8bc5100e3c Mon Sep 17 00:00:00 2001 From: Kenny Root Date: Fri, 8 Oct 2010 09:21:39 -0700 Subject: OBB: rearrange to be entirely asynchronous Rearrange structure of MountService handling of OBBs to be entirely asynchronous so we don't rely on locking as much. We still need the locking to support dumpsys which has been improved to output all the data structures for OBBs. Added more tests to cover more of the error return codes. Oh and fix a logic inversion bug. Change-Id: I34f541192dbbb1903b24825889b8fa8f43e6e2a9 --- libs/storage/IMountService.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'libs/storage/IMountService.cpp') diff --git a/libs/storage/IMountService.cpp b/libs/storage/IMountService.cpp index 3ad9319..f36e2a3 100644 --- a/libs/storage/IMountService.cpp +++ b/libs/storage/IMountService.cpp @@ -430,13 +430,14 @@ public: } void mountObb(const String16& filename, const String16& key, - const sp& token) + const sp& token, int32_t nonce) { Parcel data, reply; data.writeInterfaceToken(IMountService::getInterfaceDescriptor()); data.writeString16(filename); data.writeString16(key); data.writeStrongBinder(token->asBinder()); + data.writeInt32(nonce); if (remote()->transact(TRANSACTION_mountObb, data, &reply) != NO_ERROR) { LOGD("mountObb could not contact remote\n"); return; @@ -448,7 +449,7 @@ public: } } - void unmountObb(const String16& filename, const bool force, const sp& token) + void unmountObb(const String16& filename, const bool force) { Parcel data, reply; data.writeInterfaceToken(IMountService::getInterfaceDescriptor()); -- cgit v1.1