attn robey - paramiko readv [again]

[Robey Pointer]

On 19 Apr 2006, at 6:18, Robert Collins wrote:

> Hi Robey,
> 	I'm working on making knits the default in the test suite, and I'm
> finding a set of lockups I can only put down to race conditions with
> readv: if I step into the readv routine, it works, if I call it, it
> locks up in recv in this thread.
>
> Would a newer paramiko (using 1.5.2 at the moment) help ? Or is it a
> StubServer issue ?

You can't be using 1.5.2!  It didn't have readv. :)

Try 1.5.4 -- even though it was in 1.5.3 it wasn't quite working  
right (and was undocumented).

I've made one tweak since then that's only in the bzr branch*, but it  
would only affect rapidly doing multiple readv's on the same file, or  
a prefetch followed immediately by a readv (readv is implemented as a  
form of prefetch).  Is it possible that's what's happening?  If so,  
the bzr branch should fix it.

[But if you do a prefetch followed-by readv, you will get terrible  
performance anyway.  Maybe I should have readv notice that situation  
and just use the prefetched data in that case.  Hmm.]

I'll need to make a 1.5.5 soon anyway, because Alexander helped me  
find a bunch of Windows issues in the sftp unit tests.

robey

* <http://www.lag.net/paramiko/bzr/paramiko>