LinuxKPI: sysfs: use a sysctl_ctx_list
Needs ReviewPublic
Actions

Authored by bz on May 4 2025, 10:08 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Oct 9, 3:27 PM

	Unknown Object (File)
	Sep 14 2025, 6:32 AM

	Unknown Object (File)
	Sep 13 2025, 5:32 PM

	Unknown Object (File)
	Sep 12 2025, 11:17 AM

	Unknown Object (File)
	Sep 4 2025, 12:36 PM

	Unknown Object (File)
	Aug 28 2025, 4:33 AM

	Unknown Object (File)
	Aug 8 2025, 6:06 PM

	Unknown Object (File)
	Jul 4 2025, 3:42 AM

View All 25 Files

Subscribers

emaste

imp

Details

Reviewers

None

Group Reviewers

linuxkpi

Summary

While sysctl_remove_oid() may recusrively try to delete all
sysctl nodes, if one is still referenced it may panic (bd3cdc3105aae).
Use a sysctl_ctx_list to nicely group them all together
and also to release them -- at beast -- with a leak.

Sponsored by: The FreeBSD Foundation
MFC after: 3 days

Test Plan

Just putting this out. Still not sure it's the best idea and
fully correct. Done while debugging some other crashes.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 63879
Build 60763: arc lint + arc unit

Event Timeline

bz created this revision.May 4 2025, 10:08 PM

Herald added subscribers: emaste, imp. · View Herald TranscriptMay 4 2025, 10:08 PM

bz requested review of this revision.May 4 2025, 10:08 PM

Harbormaster completed remote builds in B63879: Diff 154811.May 4 2025, 10:08 PM

Revision Contents
Changeset List

Path

Size

sys/

compat/

linuxkpi/

common/

include/

linux/

kobject.h

1 line

sysfs.h

48 lines

Diff 154811

View Options

sys/compat/linuxkpi/common/include/linux/kobject.h