[GitHub] [incubator-nuttx] hartmannathan commented on pull request #4035: work_queue: schedule the work queue using the timer mechanism

[GitBox]

hartmannathan commented on pull request #4035:
URL: https://github.com/apache/incubator-nuttx/pull/4035#issuecomment-874330013


   Hi @Donny9 do you intend to continue refining this PR?
   
   Currently there are some compile errors:
   
   ```
   wqueue/work_usrthread.c: In function 'work_process':
   352
   Error: wqueue/work_usrthread.c:160:29: error: 'volatile struct work_s' has 
no member named 'qtime'
   353
     160 |       elapsed = ctick - work->qtime;
   354
         |                             ^~
   355
   Error: wqueue/work_usrthread.c:161:26: error: 'volatile struct work_s' has 
no member named 'delay'
   356
     161 |       if (elapsed >= work->delay)
   357
         |                          ^~
   358
   Error: wqueue/work_usrthread.c:215:49: error: 'volatile struct work_s' has 
no member named 'dq'; did you mean 'sq'?
   359
     215 |               work = (FAR struct work_s *)work->dq.flink;
   360
         |                                                 ^~
   361
         |                                                 sq
   ```
   
   It looks like that's because you changed only the kernel worker queues, not 
the user work queue that can be used by applications in Kernel or Protected 
builds.
   


-- 
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-unsubscr...@nuttx.apache.org

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