Page MenuHomeFreeBSD
Differential D56014

rge: make rx_process_limit a sysctl tunable
ClosedPublic
Actions

Authored by chris.longros_gmail.com on Sat, Mar 21, 10:11 AM.
Tags
None
Referenced Files
F151181176: D56014.id.diff
Mon, Apr 6, 3:44 PM
Unknown Object (File)
Mon, Apr 6, 10:32 AM
Unknown Object (File)
Sun, Apr 5, 7:08 PM
Unknown Object (File)
Sat, Apr 4, 2:26 AM
Unknown Object (File)
Mon, Mar 30, 8:08 PM
Unknown Object (File)
Mon, Mar 30, 6:04 PM
Unknown Object (File)
Sun, Mar 29, 9:57 AM
Unknown Object (File)
Sat, Mar 28, 4:12 PM
View All 13 Files
Subscribers
imp
ziaee

Details

Reviewers
adrian
ziaee
Group Reviewers
manpages
Commits
rG930a790c2abb: rge: make rx_process_limit a sysctl tunable
Summary

The number of packets processed per interrupt was hardcoded to 16.
Add a per-interface sysctl dev.rge.%d.rx_process_limit tunable so
users can adjust this value at runtime.

Signed-off-by: Christos Longros <chris.longros@gmail.com>

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 71622
Build 68505: arc lint + arc unit

Event Timeline

chris.longros_gmail.com created this revision.Sat, Mar 21, 10:11 AM
Herald added subscribers: ziaee, imp. · View Herald TranscriptSat, Mar 21, 10:11 AM
chris.longros_gmail.com requested review of this revision.Sat, Mar 21, 10:11 AM
Harbormaster completed remote builds in B71622: Diff 174079.Sat, Mar 21, 10:11 AM
chris.longros_gmail.com added a reviewer: adrian.Sat, Mar 21, 10:14 AM
ziaee accepted this revision.Sat, Mar 21, 7:29 PM
ziaee added a reviewer: manpages.

Ok from manpages. No comment on the source change, but I appreciate including robust and concise documentation with the proposed change!

This revision is now accepted and ready to land.Sat, Mar 21, 7:30 PM
adrian accepted this revision.Wed, Mar 25, 1:27 AM
Closed by commit rG930a790c2abb: rge: make rx_process_limit a sysctl tunable (authored by chris.longros_gmail.com, committed by adrian). · Explain WhyWed, Mar 25, 1:30 AM
This revision was automatically updated to reflect the committed changes.
adrian added a commit: rG930a790c2abb: rge: make rx_process_limit a sysctl tunable.

Revision Contents
Changeset List

PathSize
share/
man/
man4/
5 lines
sys/
dev/
rge/
3 lines
5 lines
2 lines

Diff 174079

View Options

share/man/man4/rge.4

Loading...
View Options

sys/dev/rge/if_rge.c

Loading...
View Options

sys/dev/rge/if_rge_sysctl.c

Loading...
View Options

sys/dev/rge/if_rgevar.h

Loading...