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)
	Wed, Sep 4, 2:17 PM

	Unknown Object (File)
	Sat, Aug 10, 12:01 AM

	Unknown Object (File)
	Fri, Aug 9, 1:34 AM

	Unknown Object (File)
	Fri, Aug 9, 1:32 AM

	Unknown Object (File)
	Fri, Aug 9, 1:32 AM

	Unknown Object (File)
	Fri, Aug 9, 1:32 AM

	Unknown Object (File)
	Aug 4 2024, 4:26 AM

	Unknown Object (File)
	Aug 2 2024, 2:38 PM

View All 28 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