pf: Fix definitions of pf_pfil_*_hooked
ClosedPublic
Actions

Authored by markj on Dec 13 2022, 2:37 PM.

Tags

None

Referenced Files

	F97563522: D37684.diff
	Mon, Sep 30, 2:21 AM

	Unknown Object (File)
	Tue, Sep 17, 2:51 PM

	Unknown Object (File)
	Sun, Sep 15, 12:44 AM

	Unknown Object (File)
	Sun, Sep 15, 12:44 AM

	Unknown Object (File)
	Tue, Sep 10, 10:16 AM

	Unknown Object (File)
	Mon, Sep 9, 7:46 PM

	Unknown Object (File)
	Mon, Sep 9, 6:15 AM

	Unknown Object (File)
	Wed, Sep 4, 4:59 PM

Subscribers

Details

Reviewers

kp
mjg

Commits

rG57cc96f49e1f: pf: Fix definitions of pf_pfil_*_hooked

Summary

This use of "volatile" in the vnet definitions doesn't have any effect.
VNET_DEFINE_STATE(volatile int, ...) should work, but let's avoid using
"volatile" altogether and convert to atomic_load/atomic_store.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 48702
Build 45588: arc lint + arc unit

Event Timeline

markj created this revision.Dec 13 2022, 2:37 PM

Herald added subscribers: glebius, melifaro, farrokhi, imp. · View Herald TranscriptDec 13 2022, 2:37 PM

markj requested review of this revision.Dec 13 2022, 2:37 PM

Harbormaster completed remote builds in B48702: Diff 114099.Dec 13 2022, 2:37 PM

maybe bool it while at it?

Switch to bool

Harbormaster completed remote builds in B48703: Diff 114100.Dec 13 2022, 2:51 PM

mjg accepted this revision.Dec 13 2022, 2:54 PM

This revision is now accepted and ready to land.Dec 13 2022, 2:54 PM

kp accepted this revision.Dec 14 2022, 4:13 AM

Closed by commit rG57cc96f49e1f: pf: Fix definitions of pf_pfil_*_hooked (authored by markj). · Explain WhyDec 14 2022, 2:52 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG57cc96f49e1f: pf: Fix definitions of pf_pfil_*_hooked.

Revision Contents
Changeset List

Path

Size

sys/

netpfil/

pf/

pf_ioctl.c

20 lines

Diff 114099

View Options

pf: Fix definitions of pf_pfil_*_hookedClosedPublicActions