commit eac6c476e252ad9943b4854acf0163a861cb8aec
author Chenbo Feng <fengc@google.com> Mon Feb 05 15:06:23 2018 -0800
committer Chenbo Feng <fengc@google.com> Wed Feb 28 19:07:12 2018 -0800
tree f6f364f310c66e3f7562062008e67497f8d1be7e
parent f3205bee6f6287658b93e77ac3e18345870c984f

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

libbpf/BpfNetworkStatsTest.cpp

diff --git a/libbpf/BpfNetworkStatsTest.cpp b/libbpf/BpfNetworkStatsTest.cpp
index 951bda9..eb20e0e 100644
--- a/libbpf/BpfNetworkStatsTest.cpp
+++ b/libbpf/BpfNetworkStatsTest.cpp
@@ -143,18 +143,10 @@
 }
 
 TEST_F(BpfNetworkStatsHelperTest, TestGetUidStatsTotal) {
-    StatsValue value1 = {.rxTcpBytes = TEST_BYTES0,
-                         .rxTcpPackets = TEST_PACKET0,
-                         .txTcpBytes = TEST_BYTES1,
-                         .txTcpPackets = TEST_PACKET1,
-                         .rxUdpPackets = 0,
-                         .rxUdpBytes = 0,
-                         .txUdpPackets = 0,
-                         .txUdpBytes = 0,
-                         .rxOtherBytes = TEST_BYTES2,
-                         .rxOtherPackets = TEST_PACKET2,
-                         .txOtherBytes = TEST_BYTES3,
-                         .txOtherPackets = TEST_PACKET3};
+    StatsValue value1 = {.rxBytes = TEST_BYTES0,
+                         .rxPackets = TEST_PACKET0,
+                         .txBytes = TEST_BYTES1,
+                         .txPackets = TEST_PACKET1,};
     populateFakeStats(TEST_UID1, 0, IFACE0, TEST_COUNTERSET0, &value1, mFakeUidStatsMap);
     populateFakeStats(TEST_UID1, 0, IFACE0, TEST_COUNTERSET1, &value1, mFakeUidStatsMap);
     populateFakeStats(TEST_UID2, 0, IFACE0, TEST_COUNTERSET1, &value1, mFakeUidStatsMap);
@@ -220,18 +212,10 @@
     const char* iface = "lo";
     int ifaceIndex = if_nametoindex(iface);
     ASSERT_LT(0, ifaceIndex);
-    StatsValue value1 = {.rxTcpBytes = TEST_BYTES0,
-                         .rxTcpPackets = TEST_PACKET0,
-                         .txTcpBytes = TEST_BYTES1,
-                         .txTcpPackets = TEST_PACKET1,
-                         .rxUdpPackets = 0,
-                         .rxUdpBytes = 0,
-                         .txUdpPackets = 0,
-                         .txUdpBytes = 0,
-                         .rxOtherBytes = TEST_BYTES2,
-                         .rxOtherPackets = TEST_PACKET2,
-                         .txOtherBytes = TEST_BYTES3,
-                         .txOtherPackets = TEST_PACKET3};
+    StatsValue value1 = {.rxBytes = TEST_BYTES0,
+                         .rxPackets = TEST_PACKET0,
+                         .txBytes = TEST_BYTES1,
+                         .txPackets = TEST_PACKET1,};
     populateFakeStats(0, 0, 0, COUNTERSETS_LIMIT, &value1, mFakeTagStatsMap);
     populateFakeStats(TEST_UID1, TEST_TAG, ifaceIndex, TEST_COUNTERSET0, &value1, mFakeTagStatsMap);
     populateFakeStats(TEST_UID1, TEST_TAG, ifaceIndex + 1, TEST_COUNTERSET0, &value1,
@@ -259,18 +243,10 @@
     const char* iface = "lo";
     int ifaceIndex = if_nametoindex(iface);
     ASSERT_LT(0, ifaceIndex);
-    StatsValue value1 = {.rxTcpBytes = TEST_BYTES0,
-                         .rxTcpPackets = TEST_PACKET0,
-                         .txTcpBytes = TEST_BYTES1,
-                         .txTcpPackets = TEST_PACKET1,
-                         .rxUdpPackets = 0,
-                         .rxUdpBytes = 0,
-                         .txUdpPackets = 0,
-                         .txUdpBytes = 0,
-                         .rxOtherBytes = TEST_BYTES2,
-                         .rxOtherPackets = TEST_PACKET2,
-                         .txOtherBytes = TEST_BYTES3,
-                         .txOtherPackets = TEST_PACKET3};
+    StatsValue value1 = {.rxBytes = TEST_BYTES0,
+                         .rxPackets = TEST_PACKET0,
+                         .txBytes = TEST_BYTES1,
+                         .txPackets = TEST_PACKET1,};
     populateFakeStats(0, 0, 0, COUNTERSETS_LIMIT, &value1, mFakeUidStatsMap);
     populateFakeStats(TEST_UID1, 0, ifaceIndex, TEST_COUNTERSET0, &value1, mFakeUidStatsMap);
     populateFakeStats(TEST_UID1, 0, ifaceIndex + 1, TEST_COUNTERSET0, &value1, mFakeUidStatsMap);