This is the mail archive of the
mailing list for the Cygwin project.
Re: rsync over ssh hang issue understood
Darryl Miles wrote:
* The outstanding byte count needs to be protected by a mutex.
Are you familiar with the Interlocked* family of functions? Depending on
what exactly you need to do with the value, a mutex may be unnecessary.
Do these function perform buslock prefixed increment/add with memory
operand. As per Linux kernel /usr/src/linux/include/asm-i386/atomic.h.
Its an atomic add operation that would be needed, but I dont think
that will be the only thing that will need intra-thread protection. The
close() case flag setting might need to be involved too as well as other
things, like we need serialization.
One point from my last post was:
> Select justs need to lookup the outstanding by count and update the
write_ready state based on the threshold.
I need to revise that, in that all decrement operations on the
outstanding byte count need to re-test for the new decremented value
ends is below the threshold to allow more IO to occur. If so it should
_consider_ waking up all threads waiting on the fd under select(2).
That consideration would be along the lines of:
* If there is blocked IO already waiting behind the serialization gate
to come through, then this instance of the io completion routine is not
responsible for waking up select(2). That responsibility passes to the
next IO after it has come through the gate and called WriteFileEx() and
then added its byte count to the outstanding byte count as necessary.
* If there is no blocked IO waiting at the gate to come through, then
its the responsibility of this completion routine to wake up select waiters.
This is to stop false writable events from waking up select then we
already have an IO queued up and blocked.
Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple
Problem reports: http://cygwin.com/problems.html