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

iv_work: add support for submitting work_items from within worker threads #25

Closed

Conversation

bazsi
Copy link
Contributor

@bazsi bazsi commented Apr 24, 2022

This is a request-for-comments PR against iv_work to allow the submission of tasks from a worker thread and not just from the thread that owns the iv_work pool.

@MrAnno
Copy link

MrAnno commented May 13, 2022

This is a really interesting addition that we could build upon in syslog-ng.

I wonder how this approach fits into the ivykis world.

The behavior of the completion callback might be surprising when submitting jobs from a worker, but it does what is currently documented.

@bazsi bazsi force-pushed the iv-work-pool-support-for-slave-work-items branch from 3b1b1f6 to f09619f Compare May 14, 2023 14:47
…eads

This patch adds iv_work_pool_submit_continuation() function that allows the
submission of work items from any of the worker threads.

The idea is to make it possible for work item handlers to decide if some of
their tasks is best carried out by a separate work item, to be executed in
parallel.

Signed-off-by: Balazs Scheidler <bazsi77@gmail.com>
@bazsi bazsi force-pushed the iv-work-pool-support-for-slave-work-items branch from f09619f to 6773fa4 Compare May 19, 2023 21:24
@bazsi bazsi force-pushed the iv-work-pool-support-for-slave-work-items branch from 4e6d4f3 to 6773fa4 Compare October 14, 2023 05:09
@buytenh
Copy link
Owner

buytenh commented Jan 26, 2024

I've rebased this commit onto master and merged it as 7263f81,
where it is part of the v0.43 release. Thank you!

@buytenh buytenh closed this Jan 26, 2024
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.

3 participants