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
F103628436: D31873.diff
Wed, Nov 27, 8:42 AM
Unknown Object (File)
Fri, Nov 22, 3:53 PM
Unknown Object (File)
Thu, Nov 14, 8:56 AM
Unknown Object (File)
Fri, Nov 8, 1:16 AM
Unknown Object (File)
Oct 21 2024, 1:40 PM
Unknown Object (File)
Oct 5 2024, 10:22 PM
Unknown Object (File)
Sep 24 2024, 7:25 PM
Unknown Object (File)
Sep 24 2024, 4:48 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