Skip to Content.
Sympa Menu

ndt-dev - [ndt-dev] [ndt] r556 committed - Added comments to explain algorithms that used constants as best...

Subject: NDT-DEV email list created

List archive

[ndt-dev] [ndt] r556 committed - Added comments to explain algorithms that used constants as best...


Chronological Thread 
  • From:
  • To:
  • Subject: [ndt-dev] [ndt] r556 committed - Added comments to explain algorithms that used constants as best...
  • Date: Mon, 29 Aug 2011 16:13:18 +0000

Revision: 556
Author:

Date: Mon Aug 29 09:12:49 2011
Log: Added comments to explain algorithms that used constants as best
the reason could be determined.

jeff


http://code.google.com/p/ndt/source/detail?r=556

Modified:
/branches/kkumar_code_organize/Applet/Tcpbw100.java

=======================================
--- /branches/kkumar_code_organize/Applet/Tcpbw100.java Mon Aug 29 08:36:48 2011
+++ /branches/kkumar_code_organize/Applet/Tcpbw100.java Mon Aug 29 09:12:49 2011
@@ -3284,7 +3284,7 @@
// Add connection details to statistics pane and
email text

// Is the connection receiver limited?
- if (rwintime > .015) {
+ if (rwintime > NDTConstants.BUFFER_LIMITED) {
_txtStatistics.append(_resBundDisplayMsgs
.getString("thisConnIs")
+ " "
@@ -3313,7 +3313,7 @@
}

// Is the connection sender limited?
- if (sendtime > .015) {
+ if (sendtime > NDTConstants.BUFFER_LIMITED) {
_txtStatistics.append(_resBundDisplayMsgs
.getString("thisConnIs")
+ " "
@@ -3336,6 +3336,9 @@
}

// Is the connection network limited?
+ // If the congestion window is limited more than 0.5%
+ // of the time, NDT claims the connection is network
+ // limited.
if (cwndtime > .005) {
_txtStatistics.append(_resBundDisplayMsgs
.getString("thisConnIs")
@@ -3352,6 +3355,9 @@


// Is the loss excessive?
+ // If the link speed is less than a T3, and loss
+ // is greater than .01 percent, loss is determined
+ // to be excessive.
if ((spd < 4) && (loss > .01)) {
// If packet loss rate > 1%

_txtStatistics.append(_resBundDisplayMsgs.getString("excLoss")
@@ -3397,6 +3403,9 @@
if (_iMaxRwinRcvd <
NDTConstants.TCP_MAX_RECV_WIN_SIZE)
_iWinScaleRcvd = 0; //Max rec window size
lesser than TCP's max value,
// so, no scaling
requested
+ // According to RFC1323, Section 2.3 the max valid value of _iWinScaleRcvd is 14.
+ // Unclear why NDT uses 20 for this, but leaving for now in case this is a web100
+ // error value of some kind. (Revisit after Methodology document written.)
if ((_iWinScaleRcvd == 0) || (_iWinScaleRcvd > 20))

_txtStatistics.append(_resBundDisplayMsgs.getString("off")
+ "\n");


  • [ndt-dev] [ndt] r556 committed - Added comments to explain algorithms that used constants as best..., ndt, 08/29/2011

Archive powered by MHonArc 2.6.16.

Top of Page