Differential D38360

jail: Use atomic(9) instead of CK atomics
ClosedPublic
Actions

Authored by markj on Feb 2 2023, 10:41 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Nov 24, 11:41 AM

	Unknown Object (File)
	Sun, Nov 16, 2:54 PM

	Unknown Object (File)
	Thu, Nov 13, 4:44 AM

	Unknown Object (File)
	Tue, Nov 11, 11:06 PM

	Unknown Object (File)
	Sat, Nov 8, 9:52 AM

	Unknown Object (File)
	Oct 29 2025, 2:08 AM

	Unknown Object (File)
	Oct 28 2025, 6:35 AM

	Unknown Object (File)
	Oct 26 2025, 5:28 PM

Subscribers

Details

Reviewers

glebius
mjg

Group Reviewers

Commits

rG27202b98dc2d: jail: Use atomic(9) instead of CK atomics

Summary

There's no reason to use one over the other here, let's prefer the
interface that's used elsewhere in the kernel.

No functional change intended.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markj created this revision.Feb 2 2023, 10:41 PM

Herald added a subscriber: imp. · View Herald TranscriptFeb 2 2023, 10:41 PM

markj requested review of this revision.Feb 2 2023, 10:41 PM

Harbormaster completed remote builds in B49496: Diff 116371.Feb 2 2023, 10:41 PM

markj added a parent revision: D38359: inpcb: Deduplicate some assertions.Feb 2 2023, 10:41 PM

markj added a child revision: D38361: inpcb: Assert against wildcard addrs in in_pcblookup_hash_locked().

My motivation to use these primitives was his: "The network epoch protects some ck lists, now it is going to also protect some pointers, so let's use ck primitive for that.". I don't have any strong opinion on this. So I neither approve nor block this change. No objection from me.

mjg accepted this revision.Feb 3 2023, 6:11 PM

This revision is now accepted and ready to land.Feb 3 2023, 6:11 PM

Closed by commit rG27202b98dc2d: jail: Use atomic(9) instead of CK atomics (authored by markj). · Explain WhyFeb 7 2023, 8:12 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG27202b98dc2d: jail: Use atomic(9) instead of CK atomics.

Revision Contents
Changeset List

Path

Size

sys/

kern/

4 lines

Diff 116737

sys/kern/kern_jail.c

Loading...