Skip to content

Commit 3067e8e

Browse files
committed
Count invalid echos received per peer
1 parent 43cce6c commit 3067e8e

4 files changed

Lines changed: 13 additions & 0 deletions

File tree

ice-adapter/src/main/java/com/faforever/iceadapter/debug/DebugWindow.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -184,6 +184,7 @@ public void peerConnectivityUpdate(Peer peer) {
184184
p.averageRtt.set(connectivityChecker.map(PeerConnectivityCheckerModule::getAverageRTT).orElse(-1.0f).intValue());
185185
p.lastReceived.set(connectivityChecker.map(PeerConnectivityCheckerModule::getLastPacketReceived).map(last -> System.currentTimeMillis() - last).orElse(-1L).intValue());
186186
p.echosReceived.set(connectivityChecker.map(PeerConnectivityCheckerModule::getEchosReceived).orElse(-1L).intValue());
187+
p.echosReceived.set(connectivityChecker.map(PeerConnectivityCheckerModule::getEchosReceived).orElse(-1L).intValue());
187188
});
188189
}
189190
});
@@ -213,6 +214,7 @@ public static class DebugPeer {
213214
public SimpleIntegerProperty averageRtt = new SimpleIntegerProperty(-1);
214215
public SimpleIntegerProperty lastReceived = new SimpleIntegerProperty(-1);
215216
public SimpleIntegerProperty echosReceived = new SimpleIntegerProperty(-1);
217+
public SimpleIntegerProperty invalidEchosReceived = new SimpleIntegerProperty(-1);
216218
public SimpleStringProperty localCandidate = new SimpleStringProperty("");
217219
public SimpleStringProperty remoteCandidate = new SimpleStringProperty("");
218220

@@ -276,8 +278,14 @@ public int getEchosReceived() {
276278
return echosReceived.get();
277279
}
278280

281+
public int getInvalidEchosReceived() {
282+
return invalidEchosReceived.get();
283+
}
284+
279285
public SimpleIntegerProperty echosReceivedProperty() { return echosReceived; }
280286

287+
public SimpleIntegerProperty invalidEchosReceivedProperty() { return invalidEchosReceived; }
288+
281289
public String getLocalCandidate() {
282290
return localCandidate.get();
283291
}

ice-adapter/src/main/java/com/faforever/iceadapter/debug/DebugWindowController.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@ public class DebugWindowController {
3737
public TableColumn rttColumn;
3838
public TableColumn lastColumn;
3939
public TableColumn echosRcvColumn;
40+
public TableColumn invalidEchosRcvColumn;
4041
public TableColumn localCandColumn;
4142
public TableColumn remoteCandColumn;
4243

@@ -66,6 +67,7 @@ private void initialize() {
6667
rttColumn.setCellValueFactory(new PropertyValueFactory<>("averageRtt"));
6768
lastColumn.setCellValueFactory(new PropertyValueFactory<>("lastReceived"));
6869
echosRcvColumn.setCellValueFactory(new PropertyValueFactory<>("echosReceived"));
70+
invalidEchosRcvColumn.setCellValueFactory(new PropertyValueFactory<>("invalidEchosReceived"));
6971
localCandColumn.setCellValueFactory(new PropertyValueFactory<>("localCandidate"));
7072
remoteCandColumn.setCellValueFactory(new PropertyValueFactory<>("remoteCandidate"));
7173

ice-adapter/src/main/java/com/faforever/iceadapter/ice/PeerConnectivityCheckerModule.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ public class PeerConnectivityCheckerModule {
2424
@Getter private float averageRTT = 0.0f;
2525
@Getter private long lastPacketReceived;
2626
@Getter private long echosReceived = 0;
27+
@Getter private long invalidEchosReceived = 0;
2728

2829
public PeerConnectivityCheckerModule(PeerIceModule ice) {
2930
this.ice = ice;
@@ -73,6 +74,7 @@ void echoReceived(byte[] data, int offset, int length) {
7374

7475
if (length != 9) {
7576
log.trace("Received echo of wrong length, length: {}", length);
77+
invalidEchosReceived++;
7678
}
7779

7880
int rtt = (int) (System.currentTimeMillis() - Longs.fromByteArray(Arrays.copyOfRange(data, offset + 1, length)));

ice-adapter/src/main/resources/debugWindow.fxml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,7 @@
6363
<TableColumn fx:id="rttColumn" prefWidth="100.0" text="average rtt" />
6464
<TableColumn fx:id="lastColumn" prefWidth="100.0" text="last recv" />
6565
<TableColumn fx:id="echosRcvColumn" prefWidth="100.0" text="echos recv" />
66+
<TableColumn fx:id="invalidEchosRcvColumn" prefWidth="100.0" text="invalid echos recv" />
6667
</columns>
6768
</TableView>
6869

0 commit comments

Comments
 (0)