diff options
author | Matt Macy <mmacy@FreeBSD.org> | 2018-05-23 21:02:14 +0000 |
---|---|---|
committer | Matt Macy <mmacy@FreeBSD.org> | 2018-05-23 21:02:14 +0000 |
commit | 4f6c66cc9c75c857341b6ca6ca92d4808be8d3b5 (patch) | |
tree | 4bfcf94f0a6ca1b0584e050e141fcc97051aad44 /sys/netinet/netdump | |
parent | 630ba2c514d723e8c55b5a513b541b614338047f (diff) | |
download | src-4f6c66cc9c75c857341b6ca6ca92d4808be8d3b5.tar.gz src-4f6c66cc9c75c857341b6ca6ca92d4808be8d3b5.zip |
UDP: further performance improvements on tx
Cumulative throughput while running 64
netperf -H $DUT -t UDP_STREAM -- -m 1
on a 2x8x2 SKL went from 1.1Mpps to 2.5Mpps
Single stream throughput increases from 910kpps to 1.18Mpps
Baseline:
https://people.freebsd.org/~mmacy/2018.05.11/udpsender2.svg
- Protect read access to global ifnet list with epoch
https://people.freebsd.org/~mmacy/2018.05.11/udpsender3.svg
- Protect short lived ifaddr references with epoch
https://people.freebsd.org/~mmacy/2018.05.11/udpsender4.svg
- Convert if_afdata read lock path to epoch
https://people.freebsd.org/~mmacy/2018.05.11/udpsender5.svg
A fix for the inpcbhash contention is pending sufficient time
on a canary at LLNW.
Reviewed by: gallatin
Sponsored by: Limelight Networks
Differential Revision: https://reviews.freebsd.org/D15409
Notes
Notes:
svn path=/head/; revision=334118
Diffstat (limited to 'sys/netinet/netdump')
-rw-r--r-- | sys/netinet/netdump/netdump_client.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/netinet/netdump/netdump_client.c b/sys/netinet/netdump/netdump_client.c index c6174b9c1fec..a662fa86e9c7 100644 --- a/sys/netinet/netdump/netdump_client.c +++ b/sys/netinet/netdump/netdump_client.c @@ -1069,7 +1069,7 @@ netdump_configure(struct netdump_conf *conf, struct thread *td) return (EINVAL); } IFNET_RLOCK_NOSLEEP(); - TAILQ_FOREACH(ifp, &V_ifnet, if_link) { + CK_STAILQ_FOREACH(ifp, &V_ifnet, if_link) { if (strcmp(ifp->if_xname, conf->ndc_iface) == 0) break; } |