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
Unknown Object (File)
Fri, Dec 20, 6:13 AM
Unknown Object (File)
Fri, Dec 13, 4:31 AM
Unknown Object (File)
Sun, Dec 8, 4:07 PM
Unknown Object (File)
Fri, Dec 6, 4:35 PM
Unknown Object (File)
Tue, Dec 3, 8:25 AM
Unknown Object (File)
Nov 26 2024, 5:33 PM
Unknown Object (File)
Nov 22 2024, 2:24 PM
Unknown Object (File)
Nov 19 2024, 1:47 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.