Set Realtime Audio Thread priority on Android

base/threading/platform_thread_posix.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/threading/platform_thread.h"
 
 #include <errno.h>
 #include <sched.h>
 
 #include "base/lazy_instance.h"
@@ skipping 12 matching lines @@
 #if defined(OS_LINUX)
 #include <sys/prctl.h>
 #include <sys/resource.h>
 #include <sys/syscall.h>
 #include <sys/time.h>
 #include <unistd.h>
 #endif
 
 #if defined(OS_ANDROID)
 #include "base/android/jni_android.h"
+#include "jni/ThreadUtils_jni.h"
 #endif
 
 // TODO(bbudge) Use time.h when NaCl toolchain supports _POSIX_TIMERS
 #if defined(OS_NACL)
 #include <sys/nacl_syscalls.h>
 #endif
 
 namespace base {
 
 #if defined(OS_MACOSX)
@@ skipping 224 matching lines @@
 
 // static
 void PlatformThread::Join(PlatformThreadHandle thread_handle) {
   // Joining another thread may block the current thread for a long time, since
   // the thread referred to by |thread_handle| may still be running long-lived /
   // blocking tasks.
   base::ThreadRestrictions::AssertIOAllowed();
   pthread_join(thread_handle, NULL);
 }
 
-#if !defined(OS_MACOSX)
-// Mac OS X uses lower-level mach APIs.
-
+#if !defined(OS_MACOSX) && !defined(OS_ANDROID)
+// Mac OS X uses lower-level mach APIs and Android uses Java APIs.
 // static
 void PlatformThread::SetThreadPriority(PlatformThreadHandle, ThreadPriority) {
   // TODO(crogers): Implement, see http://crbug.com/116172
 }
 #endif
 
+#if defined(OS_ANDROID)
+bool RegisterThreadUtils(JNIEnv* env) {
+    return RegisterNativesImpl(env);

[bulach, 2013/01/09 12:57:58]

nit: unindent -2

[Wei James(wistoch), 2013/01/10 08:30:05]

fixed. thanks

+}
+
+void PlatformThread::SetThreadPriority(PlatformThreadHandle,
+                                       ThreadPriority priority) {
+  // Use Android SDK API to set priority for real time audio thread and it will
+  // set the thread to corresponding cgroups.

[bulach, 2013/01/09 12:57:58]

this comment is a bit spurious, since it's just mapping
"kThreadPriority_RealtimeAudio" to "Java_ThreadUtils_setThreadPriorityAudio"..

[Wei James(wistoch), 2013/01/10 08:30:05]

removed, thanks

+  if (priority == kThreadPriority_RealtimeAudio) {
+    JNIEnv* env = base::android::AttachCurrentThread();
+    Java_ThreadUtils_setAudioThreadPriority(env, PlatformThread::CurrentId());

[bulach, 2013/01/09 12:57:58]

nit: as above, maybe rename to ..._setThreadPriorityAudio

[Wei James(wistoch), 2013/01/10 08:30:05]

fixed. thanks

+  } else {
+    NOTREACHED() << "Unknown thread priority.";
+  }
+}
+#endif

[bulach, 2013/01/09 12:57:58]

nit:  // defined(OS_ANDROID)

[Wei James(wistoch), 2013/01/10 08:30:05]

fixed. thanks

+
 }  // namespace base