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
F162595444: D31873.diff
Tue, Jul 14, 9:47 PM
Unknown Object (File)
Mon, Jul 13, 2:37 AM
Unknown Object (File)
Sat, Jul 4, 2:29 PM
Unknown Object (File)
Sat, Jul 4, 5:54 AM
Unknown Object (File)
Thu, Jun 18, 2:35 PM
Unknown Object (File)
Jun 7 2026, 11:09 PM
Unknown Object (File)
Apr 26 2026, 7:51 AM
Unknown Object (File)
Apr 19 2026, 12:54 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