Page MenuHomeFreeBSD

Use new FIB4 KPI in ip_tryforward()
ClosedPublic

Authored by ae on Nov 15 2016, 9:21 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 20, 2:35 PM
Unknown Object (File)
Sat, Apr 20, 2:04 PM
Unknown Object (File)
Thu, Mar 28, 6:10 AM
Unknown Object (File)
Mar 25 2024, 12:46 AM
Unknown Object (File)
Mar 15 2024, 1:37 PM
Unknown Object (File)
Mar 15 2024, 1:25 PM
Unknown Object (File)
Dec 27 2023, 7:43 PM
Unknown Object (File)
Dec 27 2023, 7:35 PM
Subscribers

Details

Summary

ip_tryforward() is used in run to completion forwarding scenario. Current rtalloc API has serious problem on system with many CPU cores and, especially, when network cards has many concurrent working queues. Lock contention on the rtentry's mutex kills the performance.
This patch changes ip_tryforward() to use new FIB4 KPI. The main benefit this KPI gets from holding all needed information in the on-stack allocated structure, so there is no need to hold rtentry mutex, thus there is no lock contention.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 5951
Build 6232: arc lint + arc unit

Event Timeline

ae retitled this revision from to Use new FIB4 KPI in ip_tryforward().
ae updated this object.
ae edited the test plan for this revision. (Show Details)
ae added reviewers: melifaro, olivier.
ae set the repository for this revision to rS FreeBSD src repository - subversion.
olivier edited edge metadata.

I can't understand the code, but regarding only the performance benefit it's great :-)

Improvement on a 8 cores Intel Xeon with Chelsio, full details:

graph.png (768×1 px, 54 KB)

Improvement on a 4cores Atom, full detail :

graph.png (768×1 px, 21 KB)

Improvement on a 8 cores Atom with Chelsio, full details:

graph.png (1×1 px, 47 KB)

This revision is now accepted and ready to land.Nov 16 2016, 6:39 AM
This revision was automatically updated to reflect the committed changes.