Skip to content

options: add --nice process-priority controls - #1139

Open
steadytao wants to merge 2 commits into
masterfrom
nice-process-priorities
Open

steadytao wants to merge 2 commits into
masterfrom
nice-process-priorities

Conversation

@steadytao

@steadytao steadytao commented Oct 23, 2026 •

Copy link
Copy Markdown
Member

#1011 and the chain before it established proposal(s) and option-design discussion. This PR implements the feature in a bit of a cleaner manner after reviewing that work.

Summary

Adds --nice and --no-nice to control CPU and I/O scheduling priorities for local and remote rsync processes.

Settings can target the local process, remote process or both. CPU and I/O values compose independently and later settings replace only the component they specify. Unsupported or denied local changes produce a warning without terminating the transfer. Daemon clients cannot raise a worker above its inherited priorities. rrsync accepts only local priority reductions. Local-only settings work with older peers while remote settings require peer support.

@steadytao

steadytao commented Oct 23, 2026 •

Copy link
Copy Markdown
Member Author

There is probably a bit more work to be done on this but it is quite late for me and I spent far more time on this than I had thought haha. @michaelmess-de, please review the revised semantics and platform behaviour. Will come back to this over the weekend.

Comment thread testsuite/niceness_test.py
Comment thread main.c
Comment thread options.c
Comment thread rsync.1.md Outdated
@steadytao
steadytao force-pushed the nice-process-priorities branch from e89f2be to 5b1749b Compare October 24, 2026 13:59
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.

2 participants