[GitHub] [mynewt-core] kasjer commented on a change in pull request #2112: kernel/os: Fix sanity interval handling

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

kasjer commented on a change in pull request #2112: kernel/os: Fix sanity interval handling
URL: https://github.com/apache/mynewt-core/pull/2112#discussion_r350666137
 
 

 ##########
 File path: kernel/os/src/os.c
 ##########
 @@ -111,7 +112,7 @@ os_idle_task(void *arg)
         ++g_os_idle_ctr;
 
         now = os_time_get();
-        if (OS_TIME_TICK_GT(now, sanity_last + sanity_itvl_ticks)) {
+        if (OS_TIME_TICK_GEQ(now, sanity_last + sanity_itvl_ticks)) {
 
 Review comment:
   this changes sanity check frequency by one tick.
   I think it could be left as it was except that line 139 could be change to
   sanity_to_next = 0;

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services