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/05/19 16:25:36 UTC

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new issue, #6301: Implement spinlock on top of atomic api and remove up_testset

xiaoxiang781216 opened a new issue, #6301:
URL: https://github.com/apache/incubator-nuttx/issues/6301

   See the discussion here: https://github.com/apache/incubator-nuttx/pull/6283


-- 
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.apache.org

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

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on issue #6301: Implement spinlock on top of atomic api and remove up_testset

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on issue #6301:
URL: https://github.com/apache/incubator-nuttx/issues/6301#issuecomment-1141854926

   Yes, in your case not only up_testset, but also atomic_xxx need rewrite for cxd56xx. The issue description is still right, it's better to use atomic api directly, but implement atomic api has different approach:
   
   1. boolchain's bultin
   2. libs/libc/machine/arc_atomic. for single core
   3. chip special one(e.g. cxd56xx)


-- 
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] masayuki2009 commented on issue #6301: Implement spinlock on top of atomic api and remove up_testset

Posted by GitBox <gi...@apache.org>.
masayuki2009 commented on issue #6301:
URL: https://github.com/apache/incubator-nuttx/issues/6301#issuecomment-1141534250

   @xiaoxiang781216 
   For example, cxd56xx needs to use up_testset API because the SoC has a restriction to use ldrex and strex.
   
   


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