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/28 16:03:55 UTC

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on pull request #1173: testing/ostest: Replace up_tls_info with tls_get_info

xiaoxiang781216 commented on PR #1173:
URL: https://github.com/apache/incubator-nuttx-apps/pull/1173#issuecomment-1140289355

   > Could you please share the upcoming kernel side changes? Now we have
   > 
   > ```
   > #ifndef up_tls_info
   > #  if defined(CONFIG_TLS_ALIGNED) && !defined(__KERNEL__)
   > #    define up_tls_info() TLS_INFO((uintptr_t)up_getsp())
   > #  else
   > #    define up_tls_info() tls_get_info()
   > #  endif
   > #endif
   > ```
   > 
   > so this change covers only the `else` case of `if defined(CONFIG_TLS_ALIGNED) && !defined(__KERNEL__)`.
   
   Here is patch:
   https://github.com/apache/incubator-nuttx/pull/6337/commits/c9b5292676feea98bbb1901d14236d1a0dc7e3a2
   This patch need merge first otherwise the above PR can't pass CI.


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