Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

build: remove dynamic-pthread-stacks patch #10

Open
cfriedt opened this issue Dec 13, 2020 · 2 comments
Open

build: remove dynamic-pthread-stacks patch #10

cfriedt opened this issue Dec 13, 2020 · 2 comments
Assignees
@cfriedt
Labels
build documentation Improvements or additions to documentation enhancement New feature or request
Milestone
Beta

Comments

@cfriedt
Copy link
Owner

cfriedt commented Dec 13, 2020

Once zephyrproject-rtos/zephyr#29029 is merged, we can remove zephyr-dynamic-pthread-stacks.patch from the build process. This ticket is here to track the upstream ticket and ensure it gets merged eventually.

The README.rst file should be updated to reflect the patches that must be applied.
@cfriedt cfriedt added build documentation Improvements or additions to documentation labels Dec 13, 2020
@cfriedt cfriedt self-assigned this Dec 13, 2020
@cfriedt cfriedt added this to the Beta milestone Dec 13, 2020
@cfriedt cfriedt added the enhancement New feature or request label Dec 13, 2020
@cfriedt cfriedt mentioned this issue Mar 3, 2021
Merged
@cfriedt
Copy link
Owner Author

cfriedt commented Mar 3, 2021 •
edited
Loading

The upstream fix also depends on zephyrproject-rtos/zephyr#26999 where the goal is to create a k_alloc_thread_stack() system call. With that system call in place, the pthread change is trivial.

cfriedt/issue/26999/add-a-stack-allocation-api aims to fix zephyrproject-rtos/zephyr#26999 but is currently WIP.

@cfriedt
Copy link
Owner Author

cfriedt commented Mar 3, 2021

Once zephyrproject-rtos/zephyr#29029 is merged, we can remove zephyr-dynamic-pthread-stacks.patch from the build process. This ticket is here to track the upstream ticket and ensure it gets merged eventually.

That PR has been moved to zephyrproject-rtos/zephyr#31603 and is currently WIP

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

@cfriedt cfriedt

Labels
build documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Milestone
Beta
Development

No branches or pull requests
1 participant
@cfriedt