Differential D51064

pfsync: Allocate and initialize buckets before attaching the interface
ClosedPublic
Actions

Authored by zlei on Jun 26 2025, 2:04 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Apr 12, 12:23 AM

	Unknown Object (File)
	Fri, Apr 3, 11:27 PM

	Unknown Object (File)
	Wed, Mar 25, 3:09 AM

	Unknown Object (File)
	Sun, Mar 22, 3:58 PM

	Unknown Object (File)
	Sun, Mar 22, 6:41 AM

	Unknown Object (File)
	Sun, Mar 22, 1:06 AM

	Unknown Object (File)
	Sat, Mar 21, 1:57 AM

	Unknown Object (File)
	Fri, Mar 20, 7:32 AM

View All 59 Files

Subscribers

vegeta_tuxpowered.net

Details

Reviewers

Commits

rG0a5e8108bf20: pfsync: Allocate and initialize buckets before attaching the interface
rG8cc376735c65: pfsync: Allocate and initialize buckets before attaching the interface
rGedc307eca9a9: pfsync: Allocate and initialize buckets before attaching the interface

Summary

This prevents a potential race that the ioctl threads see NULL or
uninitialized buckets.

Fixes: 4fc65bcbe3fb pfsync: Performance improvement
MFC after: 1 week

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

zlei created this revision.Jun 26 2025, 2:04 PM

Herald added subscribers: vegeta_tuxpowered.net, glebius, melifaro and 2 others. · View Herald TranscriptJun 26 2025, 2:04 PM

zlei requested review of this revision.Jun 26 2025, 2:04 PM

kp accepted this revision.Jun 27 2025, 11:50 AM

This revision is now accepted and ready to land.Jun 27 2025, 11:50 AM

Closed by commit rGedc307eca9a9: pfsync: Allocate and initialize buckets before attaching the interface (authored by zlei). · Explain WhyJun 28 2025, 3:49 PM

This revision was automatically updated to reflect the committed changes.

zlei added a commit: rGedc307eca9a9: pfsync: Allocate and initialize buckets before attaching the interface.

zlei added a commit: rG8cc376735c65: pfsync: Allocate and initialize buckets before attaching the interface.Jul 7 2025, 10:09 AM

zlei added a commit: rG0a5e8108bf20: pfsync: Allocate and initialize buckets before attaching the interface.Jul 8 2025, 10:06 AM

Revision Contents
Changeset List

Path

Size

sys/

netpfil/

pf/

33 lines

Diff 157724

sys/netpfil/pf/if_pfsync.c

Loading...