Simplify the traffic stats entry struct
Since the framework API no longer support traffic stats detail such as
rxTcpPacket number or rxTcpBytes, The eBPF program and the native helper
functions no longer need to store those information as well. Removing
them from the struct StatsValue can save some space per stats entry and
reduce the total size of stats map.
Bug: 30950746
Test: run cts -m CtsNetTestCases -t android.net.cts.TrafficStatsTest
Change-Id: I70c24b762ecc9d58fc4a3ac48a7944416eff7c81
diff --git a/server/TrafficController.cpp b/server/TrafficController.cpp
index 9535efe..e099845 100644
--- a/server/TrafficController.cpp
+++ b/server/TrafficController.cpp
@@ -376,18 +376,10 @@
curKey = nextKey;
continue;
}
- removedStatsTotal.rxTcpPackets += old_stats.rxTcpPackets;
- removedStatsTotal.rxTcpBytes += old_stats.rxTcpBytes;
- removedStatsTotal.txTcpPackets += old_stats.txTcpPackets;
- removedStatsTotal.txTcpBytes += old_stats.txTcpBytes;
- removedStatsTotal.rxUdpPackets += old_stats.rxUdpPackets;
- removedStatsTotal.rxUdpBytes += old_stats.rxUdpBytes;
- removedStatsTotal.txUdpPackets += old_stats.txUdpPackets;
- removedStatsTotal.txUdpBytes += old_stats.txUdpBytes;
- removedStatsTotal.rxOtherPackets += old_stats.rxOtherPackets;
- removedStatsTotal.rxOtherBytes += old_stats.rxOtherBytes;
- removedStatsTotal.txOtherPackets += old_stats.txOtherPackets;
- removedStatsTotal.txOtherBytes += old_stats.txOtherBytes;
+ removedStatsTotal.rxPackets += old_stats.rxPackets;
+ removedStatsTotal.rxBytes += old_stats.rxBytes;
+ removedStatsTotal.txPackets += old_stats.txPackets;
+ removedStatsTotal.txBytes += old_stats.txBytes;
} else {
curKey = nextKey;
}