Page MenuHomeFreeBSD

cxgbe(4): Always use the per-VI callout to read interface stats.
ClosedPublic

Authored by np on Apr 1 2021, 8:43 AM.
Tags
None
Referenced Files
F103392957: D29527.id86666.diff
Sun, Nov 24, 10:51 AM
Unknown Object (File)
Fri, Nov 15, 11:29 PM
Unknown Object (File)
Oct 23 2024, 7:09 AM
Unknown Object (File)
Oct 16 2024, 11:42 PM
Unknown Object (File)
Oct 4 2024, 12:00 AM
Unknown Object (File)
Oct 3 2024, 6:51 PM
Unknown Object (File)
Oct 2 2024, 4:00 AM
Unknown Object (File)
Oct 1 2024, 6:57 PM
Subscribers

Details

Summary

There is no change in the source of the stats (t4_get_port_stats or
t4_get_vi_stats) but the per-port callout is gone.

Sponsored by: Chelsio Communications

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

np requested review of this revision.Apr 1 2021, 8:43 AM
sys/dev/cxgbe/t4_main.c
6284

This seems a bit odd as it means you will drop/pickup this lock several times. Maybe make the acquire of this lock in vi_refresh_stats() conditional on IS_VF instead and move the assertion here?

np marked an inline comment as done.Apr 1 2021, 5:57 PM

Acquire reg_lock around read_vf_stat instead of in a loop in the function.

This revision is now accepted and ready to land.Apr 1 2021, 9:20 PM