Skip to content

input: fixed threaded input processor initialization - #10402

Merged
edsiper merged 3 commits into
masterfrom
merge-leonardo-fix-for-9835
May 30, 2025
Merged

input: fixed threaded input processor initialization #10402
edsiper merged 3 commits into
masterfrom
merge-leonardo-fix-for-9835

Conversation

@edsiper

@edsiper edsiper commented May 29, 2025

Copy link
Copy Markdown
Member

This is a rebased PR of #10350 on top of master branch (trying to workaround some CI issues)


Fluent Bit is licensed under Apache 2.0, by submitting this pull request I understand that this code will be released under the terms of that license.

Leonardo Alminana added 2 commits May 29, 2025 09:27
Signed-off-by: Eduardo Silva <eduardo@chronosphere.io>
Signed-off-by: Eduardo Silva <eduardo@chronosphere.io>
Signed-off-by: Eduardo Silva <eduardo@chronosphere.io>
@edsiper

edsiper commented May 30, 2025

Copy link
Copy Markdown
Member Author

last CI issue is not in our side , github runner is exiting right away.

@drbugfinder-work

Copy link
Copy Markdown
Contributor

Hi @leonardo-albertovich, @edsiper!

This PR implements the threaded input event-loop binding discussed in #9835. The direction is right, but in practice it appears insufficient and introduces a regression with timer/notification handling in threaded inputs.

After this change, timer/notification events in the per-thread loop can remain effectively always ready, causing a tight epoll_wait(0) spin and pushing CPU usage to ~100%. See #10529
Because of this hot-loop behavior we currently can’t reliably validate whether the original problem in #9835 is actually fixed under load.

@leonardo-albertovich could you provide a follow-up that completes the fix so timer/notification events in threaded inputs don’t trigger the non-stop epoll_wait(0) loop? Once available, we’ll retest and confirm #9835 without reintroducing the 100% CPU issue from #10529.

cc @lecaros

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants