Skip to content

testing/ostest: Replace up_tls_info with tls_get_info#1173

Merged
pkarashchenko merged 1 commit into
apache:masterfrom
xiaoxiang781216:tls
May 30, 2022
Merged

testing/ostest: Replace up_tls_info with tls_get_info#1173
pkarashchenko merged 1 commit into
apache:masterfrom
xiaoxiang781216:tls

Conversation

@xiaoxiang781216

@xiaoxiang781216 xiaoxiang781216 commented May 27, 2022

Copy link
Copy Markdown
Contributor

Summary

follow the upcoming kernel side change

Impact

Testing

follow the kernel side change

Signed-off-by: Xiang Xiao <xiaoxiang@xiaomi.com>
Comment thread testing/ostest/tls.c Outdated
@pkarashchenko

Copy link
Copy Markdown
Contributor

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__).

@xiaoxiang781216

Copy link
Copy Markdown
Contributor Author

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:
apache/nuttx@c9b5292
This patch need merge first otherwise the above PR can't pass CI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants