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)
Sun, Aug 3, 4:00 PM
Unknown Object (File)
Sat, Jul 26, 5:03 PM
Unknown Object (File)
Sat, Jul 26, 4:21 PM
Unknown Object (File)
Sat, Jul 26, 12:03 AM
Unknown Object (File)
Fri, Jul 25, 3:27 PM
Unknown Object (File)
Fri, Jul 25, 7:51 AM
Unknown Object (File)
Tue, Jul 22, 8:39 PM
Unknown Object (File)
Jul 13 2025, 11:28 AM

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