Paths

Table of Contentst

kinst: exclude cpu_switch
ClosedPublic
Actions

Authored by christos on Jul 12 2023, 1:25 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Apr 14, 9:10 PM

	Unknown Object (File)
	Tue, Apr 14, 1:06 AM

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

	Unknown Object (File)
	Wed, Apr 8, 8:14 PM

	Unknown Object (File)
	Mar 19 2026, 5:30 PM

	Unknown Object (File)
	Mar 18 2026, 3:37 AM

	Unknown Object (File)
	Mar 17 2026, 5:11 PM

	Unknown Object (File)
	Mar 16 2026, 9:59 PM

View All Files

Subscribers

imp

Details

Reviewers

markj

Commits

rGeb1413c9a6c7: kinst: exclude cpu_switch

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

christos created this revision.Jul 12 2023, 1:25 PM

Herald added a subscriber: imp. · View Herald TranscriptJul 12 2023, 1:25 PM

christos requested review of this revision.Jul 12 2023, 1:25 PM

Harbormaster completed remote builds in B52577: Diff 124540.Jul 12 2023, 1:25 PM

cpu_switch() was able to be traced in arm64 and riscv, and as is expected, resulted in crashes.

markj added inline comments.Jul 13 2023, 5:53 PM

sys/cddl/dev/kinst/kinst.c
99	Please add a comment explaining why: cpu_switch() modifies the value of `curthread`, but kinst_invop() assumes that this won't happen.

Explain why

Harbormaster completed remote builds in B52619: Diff 124614.Jul 13 2023, 6:05 PM

Approved.

This revision is now accepted and ready to land.Jul 13 2023, 6:34 PM

Closed by commit rGeb1413c9a6c7: kinst: exclude cpu_switch (authored by christos). · Explain WhyJul 19 2023, 3:00 PM

This revision was automatically updated to reflect the committed changes.

christos added a commit: rGeb1413c9a6c7: kinst: exclude cpu_switch.

Revision Contents
Changeset List

Path

Size

sys/

cddl/

dev/

kinst/

kinst.c

7 lines

Diff 124855

View Options

sys/cddl/dev/kinst/kinst.c

kinst: exclude cpu_switchClosedPublicActions