Page MenuHomeFreeBSD

pf: add COMPAT_FREEBSD12 for DIOCKEEPCOUNTERS
ClosedPublic

Authored by kp on Nov 15 2021, 9:17 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Aug 6, 3:51 PM
Unknown Object (File)
Tue, Aug 5, 5:03 AM
Unknown Object (File)
Mon, Jul 21, 1:51 AM
Unknown Object (File)
Fri, Jul 18, 12:15 PM
Unknown Object (File)
Wed, Jul 16, 12:06 AM
Unknown Object (File)
Tue, Jul 15, 9:41 PM
Unknown Object (File)
Jul 7 2025, 7:45 AM
Unknown Object (File)
Jul 6 2025, 2:21 AM

Details

Summary

DIOCKEEPCOUNTERS used to overlap with DIOCGIFSPEEDV0, which has been
fixed in 14, but remains in stable/12 and stable/13.
Support the old, overlapping, call under COMPAT_FREEBSD12.

Diff Detail

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

Event Timeline

kp requested review of this revision.Nov 15 2021, 9:17 PM
sys/netpfil/pf/pf_ioctl.c
4668

Given you've merged it to stable/13, I think it probably should be COMPAT_FREEBSD13 and use _FREEBSD13 as the suffix. If we don't have a COMPAT_FREEBSD13 option yet, that can be added in a commit before this one (and I can help with that if needed)

(The option in the commit log still says 12, but I realize if you updated it locally, the phab description doesn't auto-update so this might just be stale in phab)

This revision is now accepted and ready to land.Nov 16 2021, 5:49 PM
In D33001#745526, @jhb wrote:

(The option in the commit log still says 12, but I realize if you updated it locally, the phab description doesn't auto-update so this might just be stale in phab)

That's exactly correct, yes. I updated the commit message locally after pushing the update. In part because git-arc uses the commit message to match commits to reviews.

This revision was automatically updated to reflect the committed changes.