sergiotarxz/nuttx
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fix inaccurate time conversion. Remove MSEC_PER_TICK and convert uint32_t to uin64_t.

Browse Source
This commit is contained in:
Gregory Nutt 2014-08-11 11:14:40 -06:00
parent 51da249d0e
commit b4bbe354fd
1 changed files with 17 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
sched/sched/sched_timerexpiration.c
View File

@ -290,8 +290,13 @@ static unsigned int sched_timer_process(unsigned int ticks, bool noswitches)
static void sched_timer_start(unsigned int ticks) static void sched_timer_start(unsigned int ticks)
{ {
uint32_t msecs; #ifdef CONFIG_HAVE_LONG_LONG
uint32_t secs; uint64_t usecs;
uint64_t secs;
#else
uint64_t usecs;
uint64_t secs;
#endif
uint32_t nsecs; uint32_t nsecs;
int ret; int ret;
@ -308,11 +313,18 @@ static void sched_timer_start(unsigned int ticks)
/* Convert ticks to a struct timespec that up_timer_start() can /* Convert ticks to a struct timespec that up_timer_start() can
* understand. * understand.
*
* REVISIT: Calculations may not have acceptable ragne if uint64_t
* is not supported(?)
*/ */
msecs = TICK2MSEC(ticks); #ifdef CONFIG_HAVE_LONG_LONG
secs = msecs / MSEC_PER_SEC; usecs = TICK2USEC((uint64_t)ticks);
nsecs = (msecs - (secs * MSEC_PER_SEC)) * NSEC_PER_MSEC; #else
usecs = TICK2USEC((uint64_t)ticks);
#endif
secs = usecs / USEC_PER_SEC;
nsecs = (usecs - (secs * USEC_PER_SEC)) * NSEC_PER_MSEC;
ts.tv_sec = (time_t)secs; ts.tv_sec = (time_t)secs;
ts.tv_nsec = (long)nsecs; ts.tv_nsec = (long)nsecs;