Page MenuHomePhabricator

Limit memory usage of `ssh-exec` during large pull operations
ClosedPublic

Authored by epriestley on Dec 16 2013, 7:57 PM.
Tags
None
Referenced Files
F18780439: D7776.id.diff
Sun, Oct 12, 2:35 PM
F18779305: D7776.diff
Sat, Oct 11, 1:20 PM
F18672739: D7776.id.diff
Thu, Sep 25, 8:09 AM
F18672692: D7776.id17592.diff
Thu, Sep 25, 8:05 AM
F18672691: D7776.id17583.diff
Thu, Sep 25, 8:05 AM
F18672609: D7776.diff
Thu, Sep 25, 7:54 AM
F18629826: D7776.diff
Sep 16 2025, 10:01 AM
F18585336: D7776.id.diff
Sep 11 2025, 3:03 PM
Subscribers

Details

Summary

Fixes T4241. Ref T4206. See T4241 for a description here. Generally, when we connect a fat pipe (git-upload-pack) to a narrow one (git over SSH) we currently read limitless data into memory. Instead, throttle reads until writes catch up. This is now possible because of the previous changes in this sequence.

Test Plan
  • Ran git clone and git push on the entire Wine repository.
  • Observed CPU and memory usage.
  • Memory usage was constant and low, CPU usage was high only during I/O (which is expected, since we have to actually do work, although thre might be room to further reduce this).

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

src/infrastructure/ssh/PhabricatorSSHPassthruCommand.php
141

Adding this check might fix T4206.