wangchdo commented on code in PR #17642:
URL: https://github.com/apache/nuttx/pull/17642#discussion_r2664687227
##########
sched/hrtimer/hrtimer_start.c:
##########
@@ -139,10 +88,24 @@ int hrtimer_start(FAR hrtimer_t *hrtimer,
hrtimer->expired = hrtimer_gettime() + ns;
}
+ /* Ensure expiration time does not overflow */
+
+ DEBUGASSERT(hrtimer->expired != UINT64_MAX);
Review Comment:
Done
##########
sched/hrtimer/hrtimer_process.c:
##########
@@ -69,63 +70,76 @@
void hrtimer_process(uint64_t now)
{
FAR hrtimer_t *hrtimer;
- uint64_t expired;
- uint32_t period = 0;
irqstate_t flags;
+ hrtimer_cb func;
+ uint64_t expired;
+ uint64_t period = 0;
Review Comment:
Done
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
