Skip to content

Flb procs draft #2

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
Draft
wants to merge 19 commits into
base: master
Choose a base branch
from
Draft

Flb procs draft #2

wants to merge 19 commits into from

Conversation

hsmatulis
Copy link
Owner

Prototype of sample fluent-bit multi-threading implementation

@hsmatulis
Simple multithreading implementation for flb
a720f4c
@hsmatulis
Add flag to make output plugin use worker threadsy
0193b36
@hsmatulis
Make stackdriver thread safe
e695995
@hsmatulis
Move metrics counting to the main thread
bc1c80a
@hsmatulis
Add locks to upstream code
250ae60
@hsmatulis
Fix mutex in wrong place
6738b14
@hsmatulis
Change the way threads return to be compatible with multithreading
5211a23
@hsmatulis
Make error message clearer
0a67ee3
@hsmatulis
Fix issues with printing
df8f038
@hsmatulis
Fix assigning threads to wrong worker
c5106c1
@hsmatulis
Increase synchronization guarantees
b9f3e85
@hsmatulis
Fix checking bool incorrectly
f790ec6
@hsmatulis
Add more trace statements for debugging
42ea472
@hsmatulis
Make print statements into trace statements
7814bbd
@hsmatulis
Prevent locking when sending TLS handshake
f8b9d23 
This could cause a hang:
Worker #1 gets list lock,
-> Worker #1 coro #1 sends TLS handshake
-> Worker #1 coro #1 yields
-> Worker #1 coro #2 tries to acquire list lock
-> deadlock :(
@hsmatulis
Update mbed config to allow contexts to be used in multiple threads
4f08089
@hsmatulis
Set the default number of workers to 3
958ddc0
@hsmatulis
Fix issue mbed tls init race condition
c934445 
Initializing an ssl context in a child thread is not considered
thread safe for mbed.
@hsmatulis
Add locking to mbed ssl setup
5ceb985
@github-actions
Copy link

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the Stale label Mar 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Stale

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@hsmatulis