Page MenuHomeFreeBSD

pf: qid and pqid can be uint16_t
ClosedPublic

Authored by kp on Sep 7 2021, 2:37 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Oct 20, 1:00 AM
Unknown Object (File)
Sun, Oct 19, 8:13 PM
Unknown Object (File)
Sun, Sep 28, 6:11 AM
Unknown Object (File)
Aug 31 2025, 1:43 AM
Unknown Object (File)
Aug 24 2025, 12:27 PM
Unknown Object (File)
Aug 3 2025, 4:00 PM
Unknown Object (File)
Jul 26 2025, 5:03 PM
Unknown Object (File)
Jul 26 2025, 4:21 PM

Details

Summary

tag2name() returns a uint16_t, so we don't need to use uint32_t for the
qid (or pqid). This reduces the size of struct pf_kstate slightly. That
in turn buys us space to add extra fields for dummynet later.

Happily these fields are not exposed to user space (there are user space
versions of them, but they can just stay uint32_t), so there's no ABI
breakage in modifying this.

MFC after: 1 week
Sponsored by: Rubicon Communications, LLC ("Netgate")

Diff Detail

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