Differential D45326

riscv: Don't handle missing kernel L3 pages
ClosedPublic
Actions

Authored by mhorne on Thu, May 23, 7:53 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Jun 14, 6:43 PM

	Unknown Object (File)
	Sun, Jun 9, 4:45 AM

	Unknown Object (File)
	Thu, May 30, 5:17 PM

	Unknown Object (File)
	Thu, May 30, 10:46 AM

	Unknown Object (File)
	Tue, May 28, 6:29 PM

	Unknown Object (File)
	Mon, May 27, 6:04 PM

	Unknown Object (File)
	Sun, May 26, 6:23 PM

Subscribers

Details

Reviewers

markj
br
jrtc27
jhb

Group Reviewers

Commits

rG3ff981587ff8: riscv: Don't handle missing kernel L3 pages

Summary

This code path should never be hit, if it does it means we did not
bootstrap correctly. Turn it into a panic like we do on amd64 and arm64.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

mhorne created this revision.Thu, May 23, 7:53 PM

Herald added a subscriber: imp. · View Herald TranscriptThu, May 23, 7:53 PM

mhorne requested review of this revision.Thu, May 23, 7:53 PM

Harbormaster completed remote builds in B57880: Diff 139009.Thu, May 23, 7:53 PM

mhorne added a parent revision: D45325: riscv: adjust physmem reservation.Thu, May 23, 7:53 PM

mhorne added a child revision: D45327: riscv: rework page table bootstrap.

I have a vague memory that this was needed at some point, but after digging for a while I can't find it. It'll be pretty obvious if the premise of the commit is wrong, so this seems ok.

This revision is now accepted and ready to land.Mon, May 27, 2:07 PM

jhb accepted this revision.Thu, Jun 6, 5:24 PM

Closed by commit rG3ff981587ff8: riscv: Don't handle missing kernel L3 pages (authored by mhorne). · Explain WhyFri, Jun 14, 6:02 PM

This revision was automatically updated to reflect the committed changes.

mhorne added a commit: rG3ff981587ff8: riscv: Don't handle missing kernel L3 pages.

Revision Contents
Changeset List

Path

Size

sys/

riscv/

riscv/

43 lines

Diff 139859

sys/riscv/riscv/pmap.c

Loading...