eclipse-iceoryx / eclipse-iceoryx/iceoryx

Deprecate posix timer

Open
#337 54 comments 0 reactions 1 assignee Claimed by @shankaranarayanan-c View on GitHub
refactoring
Dominant language
C++
Stars
2.2k
Forks
492
Avg merge
18h 57m
Merged PRs (30d)
1

Description

## Brief feature description

Deprecate posix timer from iceoryx code base.

## Detailed information

Posix timer has several issues (concurrency, truncation) and is pretty heavy weight and a bit slow. As iceorxy needs to be very responsive the posix timer should be removed and replaced with a lightweight stopwatch.

This will also close #190 as it'll be obsolete.

Tasks
- [x] create `DeadlineTimer` or similar and replace this part of the `PosixTimer` where applicable
- [x] create an object that creates a thread and peridically executes a task
- [ ] create a small wrapper around a `posix::semaphore` which can be used for `concurrent::PeriodicTask`
- [x] replace the keep alive message from a `std::chrono` timestamp with a `units::Duration` from a `timespec` initialized with `clock_gettime` with `CLOCK_REALTIME` -> maybe better to use a different issue for this

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.