diff options
author | Jeff Sharkey <jsharkey@android.com> | 2020-04-06 16:51:58 -0600 |
---|---|---|
committer | Jeff Sharkey <jsharkey@android.com> | 2020-04-07 10:00:43 -0600 |
commit | 564929dd2fd0e77e32fc4e7868c707ff2865e696 (patch) | |
tree | 6875270332f83189a2eaaf3c08eeb7a91c4e0988 /jni/com_android_providers_media_FuseDaemon.cpp | |
parent | e3650648b32a0b91489d11705cb151b87a32c8a3 (diff) | |
download | MediaProvider-564929dd2fd0e77e32fc4e7868c707ff2865e696.tar.gz |
Additional tracing and performance improvements.
When debugging is enabled, emit the method name being called for all
outgoing Binder transactions. This identified invalidateFuseDentry()
was calling StorageManager APIs too aggressively, so we introduce
sCachedVolumePathToId to answer its questions from a local cache.
Add tracing showing begin/end of SQLite transactions, along with the
overall scope of the transaction. This identified that insertFile()
was doing getParent() work with a separate transaction, so we merge
that into the main transaction.
Bug: 152005629
Test: atest --test-mapping packages/providers/MediaProvider
Change-Id: I61e62ed773f4dc1a10acf7033848197d54d2ecf5
Diffstat (limited to 'jni/com_android_providers_media_FuseDaemon.cpp')
-rw-r--r-- | jni/com_android_providers_media_FuseDaemon.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/jni/com_android_providers_media_FuseDaemon.cpp b/jni/com_android_providers_media_FuseDaemon.cpp index 66aea9b06..ae1c17526 100644 --- a/jni/com_android_providers_media_FuseDaemon.cpp +++ b/jni/com_android_providers_media_FuseDaemon.cpp @@ -99,7 +99,6 @@ void com_android_providers_media_FuseDaemon_invalidate_fuse_dentry_cache(JNIEnv* } bool com_android_providers_media_FuseDaemon_is_fuse_thread(JNIEnv* env, jclass clazz) { - LOG(VERBOSE) << "Checking if FUSE thread..."; return pthread_getspecific(fuse::MediaProviderWrapper::gJniEnvKey) != nullptr; } |