[GitHub] [incubator-nuttx] no1wudi edited a comment on pull request #3626: libc: Move pthread stuff to userspace

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

no1wudi edited a comment on pull request #3626:
URL: https://github.com/apache/incubator-nuttx/pull/3626#issuecomment-851760149


   > @no1wudi Has this change been verified on a PROTECTED and a KERNEL build? I think that is essential to know if some of these changes and the comments are correct. If any of your comments are correct, then those builds are currently broken.
   
   These changes had beed verified on stm32 and k210, I'll continue this work according to the latest suggestions from @xiaoxiang781216 and @patacongo :
   
   1. Move pthread_exit & pthread_create to USERSPACE to unify the convention that how to drop to userspace from kernel side like this:
     `regs[REG_EPC]      = (uintptr_t)USERSPACE->pthread_startup`
     `regs[REG_EPC]      = (uintptr_t)USERSPACE->pthread_exit`
   2. Move tls destructor to user space (task_info_s)
   3. Add SYS_pthread_exit syscall like SYS_pthread_create
   4. Move TCB_FLAG_CANCEL_DOING/TCB_FLAG_CANCEL_PENDING to tls_info_s
   5. Move task_setcancelstate to libc
   6. Minor fixes/changes from comments
   
   


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