ExternalVendorCode/genode
1
0
Fork 0
mirror of https://github.com/genodelabs/genode.git synced 2025-01-18 18:56:29 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

timer: clamp one-shot timeouts to avoid overflow

Browse Source 
Issue #2579
This commit is contained in:
Martin Stein 2017-11-10 10:54:23 +01:00 committed by Christian Helmuth
parent f3dafbf5a6
commit 5d39acd3c3
1 changed files with 13 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
repos/os/src/drivers/timer/include/session_component.h
View File

@ -58,7 +58,19 @@ class Timer::Session_component : public Genode::Rpc_object<Session>,
********************/
void trigger_once(unsigned us) override {
_timeout.schedule_one_shot(Microseconds(us), *this); }
/*
* FIXME Workaround for the problem that Alarm scheduler may
* categorize big timeouts into the wrong time counter
* period due to its outdated internal time. This is needed
* only because the Alarm framework solely takes absolute
* time values on one-shot timeouts. and thus As soon as the
* Alarm framework takes solely relative time values, please
* remove this.
*/
Microseconds typed_us(us > ~0UL >> 1 ? ~0UL >> 1 : us);
_timeout.schedule_one_shot(typed_us, *this);
}
void trigger_periodic(unsigned us) override {
_timeout.schedule_periodic(Microseconds(us), *this); }