diff options
author | Glenn Kasten <gkasten@google.com> | 2012-02-28 12:30:08 -0800 |
---|---|---|
committer | Glenn Kasten <gkasten@google.com> | 2012-02-28 16:38:09 -0800 |
commit | 480b46802bef1371d5caa16ad5454fce04769c57 (patch) | |
tree | 50bc53de16466d73d177b8f19ff96e8b9774b730 | |
parent | a3873833d518e032138cf70188b6f33cd7acec3d (diff) | |
download | frameworks_av-480b46802bef1371d5caa16ad5454fce04769c57.zip frameworks_av-480b46802bef1371d5caa16ad5454fce04769c57.tar.gz frameworks_av-480b46802bef1371d5caa16ad5454fce04769c57.tar.bz2 |
Shorten thread names
prctl(PR_SET_NAME) limits to 15 characters. Before we had names like
"Binder Thread #" and the counter was cut off :-( Also remove redundant
"thread" at end of name; it's always a thread.
Change-Id: I1f99c2730ba0787ed9b59c15914356cddf698e2f
-rw-r--r-- | media/libmedia/AudioRecord.cpp | 2 | ||||
-rw-r--r-- | media/libmedia/AudioTrack.cpp | 2 | ||||
-rw-r--r-- | services/audioflinger/AudioFlinger.cpp | 4 | ||||
-rw-r--r-- | services/audioflinger/AudioFlinger.h | 2 | ||||
-rw-r--r-- | services/audioflinger/AudioPolicyService.cpp | 4 |
5 files changed, 7 insertions, 7 deletions
diff --git a/media/libmedia/AudioRecord.cpp b/media/libmedia/AudioRecord.cpp index a4068ff..dd73a8e 100644 --- a/media/libmedia/AudioRecord.cpp +++ b/media/libmedia/AudioRecord.cpp @@ -307,7 +307,7 @@ status_t AudioRecord::start() pid_t tid; if (t != 0) { mReadyToRun = WOULD_BLOCK; - t->run("ClientRecordThread", ANDROID_PRIORITY_AUDIO); + t->run("AudioRecord", ANDROID_PRIORITY_AUDIO); tid = t->getTid(); // pid_t is unknown until run() ALOGV("getTid=%d", tid); if (tid == -1) { diff --git a/media/libmedia/AudioTrack.cpp b/media/libmedia/AudioTrack.cpp index 74c97ed..4890f05 100644 --- a/media/libmedia/AudioTrack.cpp +++ b/media/libmedia/AudioTrack.cpp @@ -370,7 +370,7 @@ void AudioTrack::start() android_atomic_and(~CBLK_DISABLED_ON, &cblk->flags); pid_t tid; if (t != 0) { - t->run("AudioTrackThread", ANDROID_PRIORITY_AUDIO); + t->run("AudioTrack", ANDROID_PRIORITY_AUDIO); tid = t->getTid(); // pid_t is unknown until run() ALOGV("getTid=%d", tid); if (tid == -1) { diff --git a/services/audioflinger/AudioFlinger.cpp b/services/audioflinger/AudioFlinger.cpp index 462c2fa..b7ddbb5 100644 --- a/services/audioflinger/AudioFlinger.cpp +++ b/services/audioflinger/AudioFlinger.cpp @@ -1464,7 +1464,7 @@ AudioFlinger::PlaybackThread::PlaybackThread(const sp<AudioFlinger>& audioFlinge mMasterVolume(audioFlinger->masterVolumeSW_l()), mLastWriteTime(0), mNumWrites(0), mNumDelayedWrites(0), mInWrite(false) { - snprintf(mName, kNameLength, "AudioOut_%d", id); + snprintf(mName, kNameLength, "AudioOut_%X", id); readOutputParameters(); @@ -4791,7 +4791,7 @@ AudioFlinger::RecordThread::RecordThread(const sp<AudioFlinger>& audioFlinger, // mBytesRead is only meaningful while active, and so is cleared in start() // (but might be better to also clear here for dump?) { - snprintf(mName, kNameLength, "AudioIn_%d", id); + snprintf(mName, kNameLength, "AudioIn_%X", id); readInputParameters(); } diff --git a/services/audioflinger/AudioFlinger.h b/services/audioflinger/AudioFlinger.h index e908d3f..457bd98 100644 --- a/services/audioflinger/AudioFlinger.h +++ b/services/audioflinger/AudioFlinger.h @@ -548,7 +548,7 @@ private: Vector< sp<EffectChain> > mEffectChains; uint32_t mDevice; // output device for PlaybackThread // input + output devices for RecordThread - static const int kNameLength = 32; + static const int kNameLength = 16; // prctl(PR_SET_NAME) limit char mName[kNameLength]; sp<IPowerManager> mPowerManager; sp<IBinder> mWakeLockToken; diff --git a/services/audioflinger/AudioPolicyService.cpp b/services/audioflinger/AudioPolicyService.cpp index 987b039..753b1d2 100644 --- a/services/audioflinger/AudioPolicyService.cpp +++ b/services/audioflinger/AudioPolicyService.cpp @@ -69,7 +69,7 @@ AudioPolicyService::AudioPolicyService() // start tone playback thread mTonePlaybackThread = new AudioCommandThread(String8("")); // start audio commands thread - mAudioCommandThread = new AudioCommandThread(String8("ApmCommandThread")); + mAudioCommandThread = new AudioCommandThread(String8("ApmCommand")); /* instantiate the audio policy manager */ rc = hw_get_module(AUDIO_POLICY_HARDWARE_MODULE_ID, &module); @@ -638,7 +638,7 @@ void AudioPolicyService::AudioCommandThread::onFirstRef() if (mName != "") { run(mName.string(), ANDROID_PRIORITY_AUDIO); } else { - run("AudioCommandThread", ANDROID_PRIORITY_AUDIO); + run("AudioCommand", ANDROID_PRIORITY_AUDIO); } } |