pf: Fix definitions of pf_pfil_*_hooked
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F175227635: D37684.id114141.diff
	Fri, Oct 9, 6:32 AM

	F175227048: D37684.id114100.diff
	Fri, Oct 9, 6:26 AM

	F175223402: D37684.id114099.diff
	Fri, Oct 9, 5:48 AM

	F175204870: D37684.diff
	Fri, Oct 9, 2:17 AM

	Unknown Object (File)
	Tue, Oct 6, 12:00 AM

	Unknown Object (File)
	Mon, Oct 5, 7:54 PM

	Unknown Object (File)
	Mon, Oct 5, 7:35 PM

	Unknown Object (File)
	Mon, Oct 5, 7:25 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 Not Applicable

Unit

Tests Not Applicable

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

24 lines

Diff 114141

View Options

pf: Fix definitions of pf_pfil_*_hookedClosedPublicActions