Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit ab02745f authored by Alistair Strachan's avatar Alistair Strachan Committed by Jorge E. Moreira
Browse files

Fall back to CLOCK_BOOTTIME if CLOCK_BOOTTIME_ALARM fails 

If the cuttlefish device does not have an rtc device (such as the crosvm
VMM) the bt osi layer can promote crashes due to it not being able to
create a CLOCK_BOOTTIME_ALARM timer. Bring back a fallback but enable it
at runtime instead of compile time.

Bug: 126955943
Test: run with cuttlefish
Change-Id: I3ab0282b3e8fde776aa7b37d5772c8f62cf957bf
parent e1856f49

system/osi/src/alarm.cc

+5 −7
Original line number Diff line number Diff line
@@ -114,12 +114,6 @@ struct alarm_t { 
int64_t TIMER_INTERVAL_FOR_WAKELOCK_IN_MS = 3000;

static const clockid_t CLOCK_ID = CLOCK_BOOTTIME;



#if (KERNEL_MISSING_CLOCK_BOOTTIME_ALARM == TRUE)

static const clockid_t CLOCK_ID_ALARM = CLOCK_BOOTTIME;

#else

static const clockid_t CLOCK_ID_ALARM = CLOCK_BOOTTIME_ALARM;

#endif



// This mutex ensures that the |alarm_set|, |alarm_cancel|, and alarm callback

// functions execute serially and not concurrently. As a result, this mutex

// also protects the |alarms| list.
@@ -327,7 +321,11 @@ static bool lazy_initialize(void) { 
  if (!timer_create_internal(CLOCK_ID, &timer)) goto error;

  timer_initialized = true;



  if (!timer_create_internal(CLOCK_ID_ALARM, &wakeup_timer)) goto error;

  if (!timer_create_internal(CLOCK_BOOTTIME_ALARM, &wakeup_timer)) {

    if (!timer_create_internal(CLOCK_BOOTTIME, &wakeup_timer)) {

      goto error;

    }

  }

  wakeup_timer_initialized = true;



  alarm_expired = semaphore_new(0);