Skip to content

Conversation

ycsin
Copy link
Member

@ycsin ycsin commented Jun 14, 2024

Check if an address is in the thread stack only when CONFIG_THREAD_STACK_INFO is enabled, since otherwise the stack_info will not be available.

This fixes compilation error when CONFIG_THREAD_STACK_INFO is explicitly disabled.

fixes #74296

@ycsin ycsin added bug The issue is a bug, or the PR is fixing a bug area: RISCV RISCV Architecture (32-bit & 64-bit) Hotfix Fix for issues blocking development, i.e. CI issues, tests failing in CI, etc. Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. labels Jun 14, 2024
Check if an address is in the thread stack only when
`CONFIG_THREAD_STACK_INFO` is enabled, since otherwise the
`stack_info` will not be available.

This fixes compilation error when `CONFIG_THREAD_STACK_INFO`
is explicitly disabled.

Signed-off-by: Yong Cong Sin <ycsin@meta.com>
@ycsin ycsin force-pushed the pr/fix_riscv_stack_trace_kthread_bound branch from 833c1aa to 578a8e5 Compare June 14, 2024 04:56
@nashif nashif merged commit bb66d11 into zephyrproject-rtos:main Jun 14, 2024
@ycsin ycsin deleted the pr/fix_riscv_stack_trace_kthread_bound branch June 14, 2024 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: RISCV RISCV Architecture (32-bit & 64-bit) bug The issue is a bug, or the PR is fixing a bug Hotfix Fix for issues blocking development, i.e. CI issues, tests failing in CI, etc. Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

tests/posix/common/ portability.posix.common.static_stack failing to build for riscv
4 participants