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, Sep 6, 9:41 AM
Unknown Object (File)
Aug 30 2024, 8:53 PM
Unknown Object (File)
Aug 30 2024, 8:42 PM
Unknown Object (File)
Aug 30 2024, 8:42 PM
Unknown Object (File)
Aug 30 2024, 8:16 PM
Unknown Object (File)
Aug 30 2024, 3:34 AM
Unknown Object (File)
Aug 26 2024, 1:02 PM
Unknown Object (File)
Aug 26 2024, 10:06 AM
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.