Author: zothar
Date: 2006-05-15 21:50:55 +0000 (Mon, 15 May 2006)
New Revision: 8706
Modified:
trunk/freenet/src/freenet/clients/http/DarknetConnectionsToadlet.java
trunk/freenet/src/freenet/node/PeerManager.java
trunk/freenet/src/freenet/node/PeerNode.java
Log:
Rename the 'backed off' concept to 'routing backed off' to hopefully avoid
further confusion like mine when I committed r8687. This commit should contain
no functional changes.
Modified: trunk/freenet/src/freenet/clients/http/DarknetConnectionsToadlet.java
===================================================================
--- trunk/freenet/src/freenet/clients/http/DarknetConnectionsToadlet.java
2006-05-15 09:05:29 UTC (rev 8705)
+++ trunk/freenet/src/freenet/clients/http/DarknetConnectionsToadlet.java
2006-05-15 21:50:55 UTC (rev 8706)
@@ -108,9 +108,9 @@
String[][] messageTypesRows = new String[peerNodes.length][];
for(int i=0;i<peerNodes.length;i++) {
PeerNode pn = peerNodes[i];
- long backedOffUntil = pn.getBackedOffUntil();
- boolean backedOffNow = (now < backedOffUntil);
- int backoff = (int)(Math.max(backedOffUntil - now, 0));
+ long routingBackedOffUntil =
pn.getRoutingBackedOffUntil();
+ boolean routingBackedOffNow = (now <
routingBackedOffUntil);
+ int backoff = (int)(Math.max(routingBackedOffUntil -
now, 0));
long idle = pn.lastReceivedPacketTime();
// Elements must be HTML encoded.
@@ -121,7 +121,7 @@
Object status;
if(pn.isConnected()) {
status = CONNECTED;
- if(backedOffNow) {
+ if(routingBackedOffNow) {
status = BACKED_OFF;
}
} else if(pn.hasCompletedHandshake() &&
!Version.checkGoodVersion(pn.getVersion())) {
@@ -136,7 +136,7 @@
row[3] = pn.getDetectedPeer() != null ?
HTMLEncoder.encode(pn.getDetectedPeer().toString()) : "(address unknown)";
row[4] = HTMLEncoder.encode(pn.getVersion());
row[5] = new Double(pn.getLocation().getValue());
- row[6] = backoff/1000 + "/" +
pn.getBackoffLength()/1000;
+ row[6] = backoff/1000 + "/" +
pn.getRoutingBackoffLength()/1000;
if (idle == -1) row[7] = " ";
else row[7] = new Long((now - idle) / 60000);
row[8] = "<input type=\"checkbox\"
name=\"delete_node_"+pn.hashCode()+"\" />";
Modified: trunk/freenet/src/freenet/node/PeerManager.java
===================================================================
--- trunk/freenet/src/freenet/node/PeerManager.java 2006-05-15 09:05:29 UTC
(rev 8705)
+++ trunk/freenet/src/freenet/node/PeerManager.java 2006-05-15 21:50:55 UTC
(rev 8706)
@@ -395,8 +395,8 @@
Logger.minor(this, "Skipping (not connected): "+p.getPeer());
continue;
}
- if(p.isBackedOff()) {
- Logger.minor(this, "Skipping (backed off): "+p.getPeer());
+ if(p.isRoutingBackedOff()) {
+ Logger.minor(this, "Skipping (routing backed off):
"+p.getPeer());
continue;
}
count++;
Modified: trunk/freenet/src/freenet/node/PeerNode.java
===================================================================
--- trunk/freenet/src/freenet/node/PeerNode.java 2006-05-15 09:05:29 UTC
(rev 8705)
+++ trunk/freenet/src/freenet/node/PeerNode.java 2006-05-15 21:50:55 UTC
(rev 8706)
@@ -1106,7 +1106,7 @@
public String getStatus() {
return
- (isConnected ? "CONNECTED " : "DISCONNECTED") + " " +
getPeer()+" "+myName+" "+currentLocation.getValue()+" "+getVersion()+" backoff:
"+backoffLength+" ("+(Math.max(backedOffUntil -
System.currentTimeMillis(),0))+")";
+ (isConnected ? "CONNECTED " : "DISCONNECTED") + " " +
getPeer()+" "+myName+" "+currentLocation.getValue()+" "+getVersion()+" backoff:
"+routingBackoffLength+" ("+(Math.max(routingBackedOffUntil -
System.currentTimeMillis(),0))+")";
}
public String getFreevizOutput() {
@@ -1221,26 +1221,26 @@
return hashCode;
}
- private final Object backoffSync = new Object();
+ private final Object routingBackoffSync = new Object();
- public boolean isBackedOff() {
- synchronized(backoffSync) {
- if(System.currentTimeMillis() < backedOffUntil) {
- Logger.minor(this, "Is backed off");
+ public boolean isRoutingBackedOff() {
+ synchronized(routingBackoffSync) {
+ if(System.currentTimeMillis() < routingBackedOffUntil) {
+ Logger.minor(this, "Routing is backed off");
return true;
} else return false;
}
}
- long backedOffUntil = -1;
- /** Initial nominal backoff length */
- final int INITIAL_BACKOFF_LENGTH = 5000;
+ long routingBackedOffUntil = -1;
+ /** Initial nominal routing backoff length */
+ final int INITIAL_ROUTING_BACKOFF_LENGTH = 5000;
/** Double every time */
- final int BACKOFF_MULTIPLIER = 2;
+ final int ROUTING_BACKOFF_MULTIPLIER = 2;
/** Maximum: 24 hours */
- final int MAX_BACKOFF_LENGTH = 24*60*60*1000;
- /** Current nominal backoff length */
- int backoffLength = INITIAL_BACKOFF_LENGTH;
+ final int MAX_ROUTING_BACKOFF_LENGTH = 24*60*60*1000;
+ /** Current nominal routing backoff length */
+ int routingBackoffLength = INITIAL_ROUTING_BACKOFF_LENGTH;
/**
* Got a local RejectedOverload.
@@ -1248,36 +1248,36 @@
*/
public void localRejectedOverload() {
Logger.minor(this, "Local rejected overload on "+this);
- synchronized(backoffSync) {
+ synchronized(routingBackoffSync) {
long now = System.currentTimeMillis();
// Don't back off any further if we are already backed
off
- if(now > backedOffUntil) {
- backoffLength = backoffLength *
BACKOFF_MULTIPLIER;
- if(backoffLength > MAX_BACKOFF_LENGTH)
- backoffLength = MAX_BACKOFF_LENGTH;
- int x = node.random.nextInt(backoffLength);
- backedOffUntil = now + x;
- Logger.minor(this, "Backing off:
backoffLength="+backoffLength+", until "+x+"ms on "+getPeer());
+ if(now > routingBackedOffUntil) {
+ routingBackoffLength = routingBackoffLength *
ROUTING_BACKOFF_MULTIPLIER;
+ if(routingBackoffLength >
MAX_ROUTING_BACKOFF_LENGTH)
+ routingBackoffLength =
MAX_ROUTING_BACKOFF_LENGTH;
+ int x =
node.random.nextInt(routingBackoffLength);
+ routingBackedOffUntil = now + x;
+ Logger.minor(this, "Backing off:
routingBackoffLength="+routingBackoffLength+", until "+x+"ms on "+getPeer());
} else {
- Logger.minor(this, "Ignoring
localRejectedOverload: "+(backedOffUntil-now)+"ms remaining on backoff on
"+getPeer());
+ Logger.minor(this, "Ignoring
localRejectedOverload: "+(routingBackedOffUntil-now)+"ms remaining on routing
backoff on "+getPeer());
}
}
}
/**
* Didn't get RejectedOverload.
- * Reset backoff.
+ * Reset routing backoff.
*/
public void successNotOverload() {
Logger.minor(this, "Success not overload on "+this);
- synchronized(backoffSync) {
+ synchronized(routingBackoffSync) {
long now = System.currentTimeMillis();
// Don't un-backoff if still backed off
- if(now > backedOffUntil) {
- backoffLength = INITIAL_BACKOFF_LENGTH;
- Logger.minor(this, "Resetting backoff on
"+getPeer());
+ if(now > routingBackedOffUntil) {
+ routingBackoffLength =
INITIAL_ROUTING_BACKOFF_LENGTH;
+ Logger.minor(this, "Resetting routing backoff
on "+getPeer());
} else {
- Logger.minor(this, "Ignoring
successNotOverload: "+(backedOffUntil-now)+"ms remaining on backoff on
"+getPeer());
+ Logger.minor(this, "Ignoring
successNotOverload: "+(routingBackedOffUntil-now)+"ms remaining on routing
backoff on "+getPeer());
}
}
}
@@ -1356,12 +1356,12 @@
return myName;
}
- public int getBackoffLength() {
- return this.backoffLength;
+ public int getRoutingBackoffLength() {
+ return this.routingBackoffLength;
}
- public long getBackedOffUntil() {
- return backedOffUntil;
+ public long getRoutingBackedOffUntil() {
+ return routingBackedOffUntil;
}
public boolean hasCompletedHandshake() {