Page MenuHomeFreeBSD

iflib: emulate counters in netmap mode
ClosedPublic

Authored by franco_opnsense.org on Aug 16 2021, 7:53 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 2, 11:32 PM
Unknown Object (File)
Feb 10 2024, 6:38 AM
Unknown Object (File)
Dec 30 2023, 8:58 AM
Unknown Object (File)
Dec 27 2023, 9:18 PM
Unknown Object (File)
Dec 20 2023, 3:56 AM
Unknown Object (File)
Dec 19 2023, 12:17 PM
Unknown Object (File)
Dec 19 2023, 7:22 AM
Unknown Object (File)
Dec 10 2023, 1:35 PM

Details

Summary

When iflib devices are in netmap mode the driver
counters are no longer updated making it look from
userspace tools that traffic has stopped.

Submitted by: Stephan de Wit
PR: 252850

Diff Detail

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

Event Timeline

erj added a subscriber: erj.

This seems fine to me.

This revision is now accepted and ready to land.Aug 16 2021, 3:55 PM

Good idea, thanks! I added just a little comment.

sys/net/iflib.c
1082

I think this should be pkt_len, which is the cumulative size of a multi-descriptor packet, whereas len is the size of the last descriptor only.

This revision now requires changes to proceed.Aug 17 2021, 11:22 AM

fixed partial length on tx

franco_opnsense.org added inline comments.
sys/net/iflib.c
1082

Nice catch, thanks! I changed it to pi.ipi_len where pkg_len is stored to avoid the dependency on an internal variable.

This revision is now accepted and ready to land.Aug 17 2021, 11:49 AM

Appreciate the reviews :) Unfortunately I'm not a committer so is someone willing to help out? Thanks in advance.