[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5421: sched/wdog: fix adding one extra tick to any delay

[GitBox <gi...@apache.org>]

pkarashchenko commented on pull request #5421:
URL: https://github.com/apache/incubator-nuttx/pull/5421#issuecomment-1030686625


   > I can't find the most recent discussions.  I can't remember if they were email discussions or PR discussions.  There are many other discussions with people are confused themselves into trying to remove that +1 which is just totally wrong.
   > 
   > Perhaps we should put some comments there to prevent this from happening yet again in the future?
   
   Yes. Comment would be fine. But without this change the POSIX timers and timerfd are running with errors and that also seems not to be fine. I agree that things like sleep or delay may have +1 (actually +N) accuracy, but I do not expect timers to have so bad accuracy especially for periodic timers. Let's not merge this change and I will think how timers can be fixed.


-- 
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: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org