simon-git: putty (master): Ben Harris

Commits to Tartarus CVS repository. tartarus-commits at lists.tartarus.org
Sat Apr 9 17:33:37 BST 2016


TL;DR:
  5c42f97 Switch to flow-control-based SFTP uploading.

Repository:     git://git.tartarus.org/simon/putty.git
On the web:     http://tartarus.org/~simon-git/gitweb/?p=putty.git
Branch updated: master
Committer:      Ben Harris <bjh21 at bjh21.me.uk>
Date:           2016-04-09 17:33:37

commit 5c42f97b68aa843693088d398ced48d03922ed9a
web diff http://tartarus.org/~simon-git/gitweb/?p=putty.git;a=commitdiff;h=5c42f97b68aa843693088d398ced48d03922ed9a;hp=cb366681859e000a06660139fd335fb6355f075d
Author: Ben Harris <bjh21 at bjh21.me.uk>
Date:   Sat Apr 9 00:24:12 2016 +0100

    Switch to flow-control-based SFTP uploading.
    
    Formerly PuTTY's SFTP code would transmit (or buffer) a megabyte of data
    before even starting to look for acknowledgements, but wouldn't allow
    there to be more than a megabyte of unacknowledged data at a time.  Now,
    instead, it pays attention to whether the transmit path is blocked, and
    transmits iff it isn't.
    
    This should mean that SFTP goes faster over long fat pipes, and also
    doesn't end up buffering so much over thin ones.
    
    I practice, I tend to run into other performance limitations (such as
    TCP or SSH-2 windows) before this enhancement looks particularly good,
    but with an artificial lag of 250 ms on the loopback interface this
    patch almost doubles my upload speed, so I think it's worthwhile.

 pscp.c  |    4 ++++
 psftp.c |    4 ++++
 sftp.c  |    2 +-
 sftp.h  |    4 ++++
 4 files changed, 13 insertions(+), 1 deletion(-)



More information about the tartarus-commits mailing list