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)
	Thu, Mar 19, 5:30 PM

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

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

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

	Unknown Object (File)
	Sun, Mar 15, 1:57 PM

	Unknown Object (File)
	Sun, Mar 15, 6:03 AM

	Unknown Object (File)
	Sat, Mar 14, 6:09 PM

	Unknown Object (File)
	Sat, Mar 14, 11:49 AM

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