wangchdo commented on code in PR #17573:
URL: https://github.com/apache/nuttx/pull/17573#discussion_r2654653729


##########
sched/hrtimer/hrtimer_cancel.c:
##########


Review Comment:
   in interrup context, it will be spin loop:
   
   ```
     bool cansleep = !up_interrupt_context() &&
                     !is_idle_task(this_task());
   ```



##########
include/nuttx/hrtimer.h:
##########
@@ -97,11 +97,14 @@ struct hrtimer_node_s
 
 struct hrtimer_s
 {
-  hrtimer_node_t          node;    /* RB-tree node for sorted insertion */
-  enum hrtimer_state_e    state;   /* Current timer state */
-  hrtimer_cb              func;    /* Expiration callback function */
-  FAR void               *arg;     /* Argument passed to callback */
-  uint64_t                expired; /* Absolute expiration time (ns) */
+  hrtimer_node_t  node;        /* RB-tree node for sorted insertion */

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]

Reply via email to