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.

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

[Backport v3.7-branch] lib: os: p4wq: add a done handler and use it to free up memory #85677

Merged
merged 2 commits into from
Feb 21, 2025

Conversation

zephyrbot
Copy link
Collaborator

@zephyrbot zephyrbot commented Feb 12, 2025

Backport b55b9ae~2..b55b9ae from #84322.

Fixes #84384

Add an optional handler to the p4wq to give the submitting code
(e.g. rtio workq) a possibility execute code after the work was
succesfully executed.

Signed-off-by: Florian Weber <Florian.Weber@live.de>
(cherry picked from commit 093b29f)
This commit fixes the bug that the memory of the work request is
freed up in the work handler, before it is not needed anymore by the p4wq.
This is fixed now, by using the new done_handler in the p4wq
for freeing up that memory.

Signed-off-by: Florian Weber <Florian.Weber@live.de>
(cherry picked from commit b55b9ae)
@zephyrbot zephyrbot added Backport Backport PR and backport failure issues area: RTIO area: Base OS Base OS Library (lib/os) labels Feb 12, 2025
@fabiobaltieri fabiobaltieri added this to the v3.7.2 milestone Feb 17, 2025
@nashif nashif merged commit 4d48909 into v3.7-branch Feb 21, 2025
28 of 29 checks passed
@nashif nashif deleted the backport-84322-to-v3.7-branch branch February 21, 2025 01:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Base OS Base OS Library (lib/os) area: RTIO Backport Backport PR and backport failure issues
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

6 participants