Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Build thread pool based on memory, not cpus #33

Open
robertdfrench opened this issue Jun 29, 2021 · 1 comment
Open

Build thread pool based on memory, not cpus #33

robertdfrench opened this issue Jun 29, 2021 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@robertdfrench
Copy link
Owner

The limiting factor for concurrency in our case is not cpus (these apps are nearly alway I/O bound), but the amount of memory we'd like to dedicate to portunusd itself. If this can be expressed in the config file (or if a reasonable default can be inferred from the zone's memory allocation) then we can set the threadpool size to S = floor(M/T), where M is the aformentioned memory limit, and T is the per-thread stack size.

@robertdfrench robertdfrench added the enhancement New feature or request label Jun 29, 2021
@robertdfrench robertdfrench added this to the Load Balancing milestone Jun 29, 2021
@robertdfrench robertdfrench self-assigned this Jun 29, 2021
@robertdfrench
Copy link
Owner Author

See also #14 and #6

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant