Differential D39538

Add new privilege PRIV_KDB_SET_BACKEND
ClosedPublic
Actions

Authored by stevek on Apr 12 2023, 5:30 PM.

Tags

None

Referenced Files

	F143198643: D39538.id120225.diff
	Tue, Jan 27, 7:53 AM

	F143158398: D39538.diff
	Mon, Jan 26, 5:46 PM

	Unknown Object (File)
	Sun, Jan 25, 3:43 PM

	Unknown Object (File)
	Sun, Jan 25, 9:22 AM

	Unknown Object (File)
	Tue, Jan 20, 8:21 PM

	Unknown Object (File)
	Sat, Jan 3, 1:29 PM

	Unknown Object (File)
	Dec 18 2025, 5:28 AM

	Unknown Object (File)
	Dec 13 2025, 12:11 AM

View All 71 Files

Subscribers

Details

Reviewers

Commits

rG4819e5aeda4e: Add new privilege PRIV_KDB_SET_BACKEND

Summary

Check for PRIV_KDB_SET_BACKEND before allowing a thread to change
the KDB backend.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 50869
Build 47760: arc lint + arc unit

Event Timeline

stevek created this revision.Apr 12 2023, 5:30 PM

Herald added a subscriber: imp. · View Herald TranscriptApr 12 2023, 5:30 PM

stevek requested review of this revision.Apr 12 2023, 5:30 PM

Harbormaster completed remote builds in B50869: Diff 120225.Apr 12 2023, 5:30 PM

sjg accepted this revision.Apr 12 2023, 5:45 PM

sjg added a subscriber: sjg.

sjg added inline comments.

sys/sys/priv.h
526	I know this is updated via D39537 so ignoring

This revision is now accepted and ready to land.Apr 12 2023, 5:45 PM

emaste added a subscriber: emaste.Apr 12 2023, 7:18 PM

emaste added inline comments.Apr 12 2023, 7:22 PM

sys/sys/priv.h
519	I might expand this to "Kernel debugger privileges" (and in particular I might expect privileges related to e.g. ddb(4) to also land in here).
526	The tree should always be in a usable state so this should really update here and again in D39537

stevek added inline comments.Apr 12 2023, 7:48 PM

sys/sys/priv.h
519	Sure, makes sense. Maybe PRIV_DBG_SET_BACKEND for the name instead?
526	Yes, this one will come first. I have both commits in separate feature branches in my git tree.

Closed by commit rG4819e5aeda4e: Add new privilege PRIV_KDB_SET_BACKEND (authored by stevek). · Explain WhyApr 16 2023, 6:38 PM

This revision was automatically updated to reflect the committed changes.

stevek added a commit: rG4819e5aeda4e: Add new privilege PRIV_KDB_SET_BACKEND.

Revision Contents
Changeset List

Path

Size

sys/

kern/

6 lines

sys/

5 lines

Diff 120225

sys/kern/subr_kdb.c

Loading...

sys/sys/priv.h

Loading...