Modify

Opened 6 years ago

Last modified 6 weeks ago

#3575 assigned Bug

TCP functions slow and TCPCloseSocket with shutdown

Reported by: ripdad Owned by: Jon
Milestone: Component: AutoIt
Version: 3.3.14.2 Severity: None
Keywords: Cc:

Description

Here is a list of problems that I found with TCP in AutoIt, using v3.3.10 through v3.3.15, while transferring a 10mb string with TCPSend to TCPRecv. Please keep in mind, these are my observations using computers available to me and I have yet to test it on Windows 10. But be patient, it will come around eventually.

  1. TCPRecv will not detect a TCPSend -> TCPCloseSocket. It will detect a TCPShutdown. But, not if TCPCloseSocket is used first.
  1. TCPAccept speed is about 108ms, while v3.3.8.1 is about 15ms.
  1. TCPSend speed is about 200 to 650ms, while v3.3.8.1 is about 28 to 90ms. The higher the string is in size, the more exponential the time becomes. This leads to an unresponsive GUI.

Note:
I don't believe TCPSend is the problem, because I can use v3.3.8.1 to send and v3.3.14 to receive and it still has the same results. Yet, TCPRecv speed is about ~1ms. So, I don't know what's going on here.

  1. When TCPSend is paused, TCPRecv speed is about 108ms, while v3.3.8.1 is about 15ms.

Note:
This test simulates a delay on a server, which can be up to 10 seconds or more.

  1. Completion time of the 10mb string is about 28 to 64 seconds and v3.3.8.1 is about 7 to 10 seconds, depending on what computer is used for the test.

In conclusion, the TCP functions do indeed work, with the exception of #1. This is about timing issues within the code. The major issue is #3 - but all these problems combined just makes it worse.

It does seem to me, that a 100ms timer is in place with each of these functions, or possibly something that is common to all of them. If this is true, then it should not be any more than 10ms or none at all. I don't know if fixing that would solve it all, but it's worth a shot in my opinion.

You should have a way to benchmark TCP response times, to verify my statements.

The other TCP functions seem to be working fine. (ie: TCPConnect, TCPListen, etc.)

Attachments (0)

Change History (9)

comment:1 Changed 6 years ago by ripdad

I found the solution to the slow problem and was right about the 100ms timer.
Starting with v3.3.10.0, you must use Opt('TCPTimeout', 10) in your script
to behave like v3.3.8.1.

The #1 problem still remains.

comment:2 Changed 6 years ago by ripdad

In reality, that should act as a Timeout timer and not a Sleep timer.
So, it's still a bug in my opinion.

In the meantime, Opt('TCPTimeout', 10) is a workaround for it.

comment:3 follow-up: Changed 6 years ago by Jpm

Sorry to ask so late, can i have repro scripts so I can analyse and try to find a solution
THanks

comment:4 in reply to: ↑ 3 Changed 5 years ago by Jos

Replying to Jpm:

Sorry to ask so late, can i have repro scripts so I can analyse and try to find a solution
THanks

JP, It looks like we need to do a shudown() before the closesocket() in F_TCPCloseSocket() to stop traffic from being accepted. Read this post and the next 4 posts: https://www.autoitscript.com/forum/topic/196642-tcpsend-bug/?do=findComment&comment=1410377

Jos

comment:5 Changed 3 years ago by Jpm

  • Resolution set to Works For Me
  • Status changed from new to closed

sorry for this very late updating
under the current beta every thing work fine.
#1 is OK
#3 is even a little be better

comment:6 Changed 3 years ago by Jpm

  • Resolution Works For Me deleted
  • Status changed from closed to reopened

comment:7 Changed 3 years ago by Jpm

  • Owner set to Jpm
  • Status changed from reopened to assigned

Re thinking of the pb I think it can be useful to add a TCPShutdownSocket(listensocket,1)
which can be called before TCPCloseSocket(listensocket)
this can be combined with an extension of TCPCloseSocket(listensocket, 1)
Addition sent to Jon

Version 0, edited 3 years ago by Jpm (next)

comment:8 Changed 3 years ago by Jpm

  • Summary changed from TCP functions slow to TCP functions slow and TCPCloseSocket with shutdown

comment:9 Changed 6 weeks ago by Jpm

  • Owner changed from Jpm to Jon

Fix sent to Jon

Guidelines for posting comments:

  • You cannot re-open a ticket but you may still leave a comment if you have additional information to add.
  • In-depth discussions should take place on the forum.

For more information see the full version of the ticket guidelines here.

Add Comment

Modify Ticket

Action
as assigned The owner will remain Jon.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.