Use csr_read() to read sstatus instead of inline assembly.
ClosedPublic
Actions

Authored by jhb on Feb 5 2020, 12:15 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Apr 15, 1:19 AM

	Unknown Object (File)
	Feb 22 2024, 3:04 PM

	Unknown Object (File)
	Dec 11 2023, 5:49 AM

	Unknown Object (File)
	Nov 8 2023, 9:17 AM

	Unknown Object (File)
	Nov 6 2023, 6:47 AM

	Unknown Object (File)
	Nov 3 2023, 6:38 AM

	Unknown Object (File)
	Nov 2 2023, 5:23 AM

	Unknown Object (File)
	Oct 7 2023, 8:10 AM

View All 22 Files

Subscribers

imp

Details

Reviewers

br
mhorne

Commits

rS357594: Use csr_read() to read sstatus instead of inline assembly.

Summary

While here, remove a local variable to avoid the CSR read in non-debug
kernels.

Test Plan

booted kernels under spike and qemu

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhb created this revision.Feb 5 2020, 12:15 AM

Harbormaster completed remote builds in B29175: Diff 67804.Feb 5 2020, 12:15 AM

mhorne accepted this revision.Feb 5 2020, 8:09 PM

This revision is now accepted and ready to land.Feb 5 2020, 8:09 PM

Closed by commit rS357594: Use csr_read() to read sstatus instead of inline assembly. (authored by jhb). · Explain WhyFeb 5 2020, 8:32 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rS357594: Use csr_read() to read sstatus instead of inline assembly..

Herald added a subscriber: imp. · View Herald TranscriptFeb 5 2020, 8:32 PM

Revision Contents
Changeset List

Path

Size

head/

sys/

riscv/

trap.c

12 lines

Diff 67828

View Options

head/sys/riscv/riscv/trap.c

Use csr_read() to read sstatus instead of inline assembly.ClosedPublicActions