Commit 0d6cd689 authored by Eric Dumazet's avatar Eric Dumazet Committed by Jakub Kicinski

net: proc: speedup /proc/net/netstat

Use cache friendly helpers to better use cpu caches
while reading /proc/net/netstat

Tested on a platform with 256 threads (AMD Rome)

Before: 305 usec spent in netstat_seq_show()
After: 130 usec spent in netstat_seq_show()
Signed-off-by: default avatarEric Dumazet <edumazet@google.com>
Link: https://lore.kernel.org/r/20210128162145.1703601-1-eric.dumazet@gmail.comSigned-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent df610cd9
...@@ -464,30 +464,52 @@ static int snmp_seq_show(struct seq_file *seq, void *v) ...@@ -464,30 +464,52 @@ static int snmp_seq_show(struct seq_file *seq, void *v)
*/ */
static int netstat_seq_show(struct seq_file *seq, void *v) static int netstat_seq_show(struct seq_file *seq, void *v)
{ {
int i; const int ip_cnt = ARRAY_SIZE(snmp4_ipextstats_list) - 1;
const int tcp_cnt = ARRAY_SIZE(snmp4_net_list) - 1;
struct net *net = seq->private; struct net *net = seq->private;
unsigned long *buff;
int i;
seq_puts(seq, "TcpExt:"); seq_puts(seq, "TcpExt:");
for (i = 0; snmp4_net_list[i].name; i++) for (i = 0; i < tcp_cnt; i++)
seq_printf(seq, " %s", snmp4_net_list[i].name); seq_printf(seq, " %s", snmp4_net_list[i].name);
seq_puts(seq, "\nTcpExt:"); seq_puts(seq, "\nTcpExt:");
for (i = 0; snmp4_net_list[i].name; i++) buff = kzalloc(max(tcp_cnt * sizeof(long), ip_cnt * sizeof(u64)),
seq_printf(seq, " %lu", GFP_KERNEL);
snmp_fold_field(net->mib.net_statistics, if (buff) {
snmp4_net_list[i].entry)); snmp_get_cpu_field_batch(buff, snmp4_net_list,
net->mib.net_statistics);
for (i = 0; i < tcp_cnt; i++)
seq_printf(seq, " %lu", buff[i]);
} else {
for (i = 0; i < tcp_cnt; i++)
seq_printf(seq, " %lu",
snmp_fold_field(net->mib.net_statistics,
snmp4_net_list[i].entry));
}
seq_puts(seq, "\nIpExt:"); seq_puts(seq, "\nIpExt:");
for (i = 0; snmp4_ipextstats_list[i].name; i++) for (i = 0; i < ip_cnt; i++)
seq_printf(seq, " %s", snmp4_ipextstats_list[i].name); seq_printf(seq, " %s", snmp4_ipextstats_list[i].name);
seq_puts(seq, "\nIpExt:"); seq_puts(seq, "\nIpExt:");
for (i = 0; snmp4_ipextstats_list[i].name; i++) if (buff) {
seq_printf(seq, " %llu", u64 *buff64 = (u64 *)buff;
snmp_fold_field64(net->mib.ip_statistics,
snmp4_ipextstats_list[i].entry, memset(buff64, 0, ip_cnt * sizeof(u64));
offsetof(struct ipstats_mib, syncp))); snmp_get_cpu_field64_batch(buff64, snmp4_ipextstats_list,
net->mib.ip_statistics,
offsetof(struct ipstats_mib, syncp));
for (i = 0; i < ip_cnt; i++)
seq_printf(seq, " %llu", buff64[i]);
} else {
for (i = 0; i < ip_cnt; i++)
seq_printf(seq, " %llu",
snmp_fold_field64(net->mib.ip_statistics,
snmp4_ipextstats_list[i].entry,
offsetof(struct ipstats_mib, syncp)));
}
kfree(buff);
seq_putc(seq, '\n'); seq_putc(seq, '\n');
mptcp_seq_show(seq); mptcp_seq_show(seq);
return 0; return 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment