Skip to content

Backport work queue builder #87

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.

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

Open
wants to merge 2 commits into
base: v4.1-branch
Choose a base branch
from

Conversation

bjarki-andreasen
Copy link

@bjarki-andreasen bjarki-andreasen commented Apr 11, 2025

Backport work queue builder, see #86

The k_work_q has been extended with a new feature, the work timeout,
which comes with a new field to k_work_queue_config, work_timeout_ms,
which must be initialized.

Signed-off-by: Bjarki Arge Andreasen <[email protected]>
Need to point to zephyr PR to align zephyr API with WorkQueueBuilder.

Signed-off-by: Bjarki Arge Andreasen <[email protected]>
@d3zd3z d3zd3z added the DNM Do not merge label Apr 15, 2025
@d3zd3z
Copy link
Collaborator

d3zd3z commented Apr 15, 2025

Want to make sure this gets a DNM, as this definitely cannot be merged until the referenced PR is merged into Zephyr. Even then, we have to be careful because the Rust code is frequently used with earlier versions of Zephyr.

@bjarki-andreasen
Copy link
Author

bjarki-andreasen commented Apr 24, 2025

Want to make sure this gets a DNM, as this definitely cannot be merged until the referenced PR is merged into Zephyr. Even then, we have to be careful because the Rust code is frequently used with earlier versions of Zephyr.

This is not possible though, given zephyr is running CI building this module, so they have to be merged together. If we remove rust from upstream CI, then this can wait, but for now, its blocking zephyr...

If we merge zephyr PR, all zephyr PRs will start failing given Rust build will fail...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DNM Do not merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants