Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Tool to tune system parameters like PoH service priority #7155

Merged
merged 6 commits into from
Dec 3, 2019

Conversation

pgarg66
Copy link
Contributor

@pgarg66 pgarg66 commented Nov 26, 2019

Problem

The PoH service thread scheduling policy and priority cannot be tuned unless we use a shell script. Using the shell script is not a viable option for external nodes.

Summary of Changes

Added a new daemon sys_tuner that runs at a higher privilege. It listens on a local domain socket. When validator boots up, the poh_service connects to the socket. It triggers the sys_tuner to look up poh_service's thread ID, and adjust its priority and scheduling policy.

Fixes #6482

@codecov
Copy link

codecov bot commented Nov 27, 2019

Codecov Report

Merging #7155 into master will decrease coverage by 0.1%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master   #7155     +/-   ##
========================================
- Coverage    72.8%   72.6%   -0.2%     
========================================
  Files         234     235      +1     
  Lines       49719   49787     +68     
========================================
- Hits        36196   36193      -3     
- Misses      13523   13594     +71

mvines
mvines previously approved these changes Nov 27, 2019
Copy link
Contributor

@mvines mvines left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Move

# Reference: https://medium.com/@CameronSparr/increase-os-udp-buffers-to-improve-performance-51d167bb1360
sysctl_write net.core.rmem_max 134217728
sysctl_write net.core.rmem_default 134217728
sysctl_write net.core.wmem_max 134217728
sysctl_write net.core.wmem_default 134217728
into sys-tuner as a follow-up?

It would be good to add some documentation about solana-sys-tuner into book/src/book/src/running-validator/validator-start.md

net/remote/remote-node.sh Show resolved Hide resolved
sys-tuner/src/lib.rs Outdated Show resolved Hide resolved
sys-tuner/Cargo.toml Outdated Show resolved Hide resolved
sys-tuner/src/main.rs Show resolved Hide resolved
sys-tuner/src/main.rs Show resolved Hide resolved
sys-tuner/src/main.rs Outdated Show resolved Hide resolved
sys-tuner/src/main.rs Show resolved Hide resolved
sys-tuner/src/main.rs Outdated Show resolved Hide resolved
sys-tuner/src/main.rs Outdated Show resolved Hide resolved
@mergify mergify bot dismissed mvines’s stale review December 2, 2019 04:29

Pull request has been modified.

@pgarg66 pgarg66 merged commit 076e384 into solana-labs:master Dec 3, 2019
@pgarg66 pgarg66 deleted the sys-tuner branch December 3, 2019 00:46
carllin added a commit to carllin/solana that referenced this pull request Dec 14, 2019
carllin added a commit to carllin/solana that referenced this pull request Dec 14, 2019
carllin added a commit to carllin/solana that referenced this pull request Dec 16, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PoH service takes variable amount of time to generate hashes
2 participants