Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

AlarmManagerService: explain some timerfd_create() failures

Browse Source 
When AlarmManagerService is broken, starting the framework fails in
weird and inconsistent ways.  So if we detect your kernel is missing the
required timerfd support, give up immediately and explain why.

Bug: 28357356

Test: manually break timerfd_create syscall, check adb logcat during boot
Change-Id: Iaa153fca38e9cd8e801f3e71c3e1ae2351907828
Signed-off-by: Greg Hackmann <ghackmann@google.com>
This commit is contained in:
Greg Hackmann
2016-05-02 13:22:51 -07:00
parent d7151c06f1
commit a0126e0857
1 changed files with 27 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

29
services/core/jni/com_android_server_AlarmManagerService.cpp
View File

@@ -298,6 +298,32 @@ static int wall_clock_rtc()
return -1;
}
static void log_timerfd_create_error(clockid_t id)
{
if (errno == EINVAL) {
switch (id) {
case CLOCK_REALTIME_ALARM:
case CLOCK_BOOTTIME_ALARM:
ALOGE("kernel missing required commits:");
ALOGE("https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=6cffe00f7d4e24679eae6b7aae4caaf915288256");
ALOGE("https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=11ffa9d6065f344a9bd769a2452f26f2f671e5f8");
LOG_ALWAYS_FATAL("kernel does not support timerfd_create() with alarm timers");
break;
case CLOCK_BOOTTIME:
ALOGE("kernel missing required commit:");
ALOGE("https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=4a2378a943f09907fb1ae35c15de917f60289c14");
LOG_ALWAYS_FATAL("kernel does not support timerfd_create(CLOCK_BOOTTIME)");
break;
default:
break;
}
}
ALOGE("timerfd_create(%u) failed: %s", id, strerror(errno));
}
static jlong android_server_AlarmManagerService_init(JNIEnv*, jobject)
{
int epollfd;
@@ -313,8 +339,7 @@ static jlong android_server_AlarmManagerService_init(JNIEnv*, jobject)
for (size_t i = 0; i < fds.size(); i++) {
fds[i] = timerfd_create(android_alarm_to_clockid[i], 0);
if (fds[i] < 0) {
ALOGE("timerfd_create(%u) failed: %s", android_alarm_to_clockid[i],
strerror(errno));
log_timerfd_create_error(android_alarm_to_clockid[i]);
close(epollfd);
for (size_t j = 0; j < i; j++) {
close(fds[j]);