porting/npl/linux: Handle EINTR from sem_timedwait

It's been observed that `sem_timedwait()` can return `EINTR` even if `sigaction()` specified `SA_RESTART`. Not sure what the deal with that is but handling it is simple enough.
apache · Sep 6, 2024 · 1709b23 · 1709b23
1 parent f8d6061
commit 1709b23
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/porting/npl/linux/src/os_sem.c b/porting/npl/linux/src/os_sem.c
@@ -71,9 +71,14 @@ ble_npl_sem_pend(struct ble_npl_sem *sem, uint32_t timeout)
  wait.tv_sec += timeout / 1000;
  wait.tv_nsec += (timeout % 1000) * 1000000;
 
- err = sem_timedwait(&sem->lock, &wait);
- if (err && errno == ETIMEDOUT) {
- return BLE_NPL_TIMEOUT;
+ while ((err = sem_timedwait(&sem->lock, &wait)) != 0) {
+ switch (errno) {
+ case EINTR:
+ continue;
+ case ETIMEDOUT:
+ return BLE_NPL_TIMEOUT;
+ }
+ break;
  }
  }