You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2022/03/28 13:49:04 UTC

[GitHub] [incubator-nuttx] GUIDINGLI opened a new pull request #5888: workqueue: add work_foreach support

GUIDINGLI opened a new pull request #5888:
URL: https://github.com/apache/incubator-nuttx/pull/5888


   
   ## Summary
   
   workqueue: add work_foreach support
   
   ## Impact
   
   workqueue new API
   
   ## Testing
   
   VELA


-- 
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



[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5888: workqueue: add work_foreach support

Posted by GitBox <gi...@apache.org>.
pkarashchenko commented on a change in pull request #5888:
URL: https://github.com/apache/incubator-nuttx/pull/5888#discussion_r836622285



##########
File path: include/nuttx/wqueue.h
##########
@@ -292,6 +292,10 @@ struct work_notifier_s
   worker_t worker;     /* The worker function to schedule */
 };
 
+/* This is the callback type used by work_foreach() */
+
+typedef CODE void (*work_foreach_t)(int tid, FAR void *arg);

Review comment:
       ok. got it. it is "task id"




-- 
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



[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5888: workqueue: add work_foreach support

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 merged pull request #5888:
URL: https://github.com/apache/incubator-nuttx/pull/5888


   


-- 
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



[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5888: workqueue: add work_foreach support

Posted by GitBox <gi...@apache.org>.
pkarashchenko commented on a change in pull request #5888:
URL: https://github.com/apache/incubator-nuttx/pull/5888#discussion_r836621544



##########
File path: include/nuttx/wqueue.h
##########
@@ -292,6 +292,10 @@ struct work_notifier_s
   worker_t worker;     /* The worker function to schedule */
 };
 
+/* This is the callback type used by work_foreach() */
+
+typedef CODE void (*work_foreach_t)(int tid, FAR void *arg);

Review comment:
       what `tid` is about? Should it be `pid`?




-- 
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