Page MenuHomeFreeBSD
Differential D36562

riscv: better CTR messages in pmap_enter_l2()
ClosedPublic
Actions

Authored by mhorne on Sep 14 2022, 2:19 PM.
Tags
None
Referenced Files
F167939969: D36562.id.diff
Tue, Aug 25, 1:01 PM
F167914511: D36562.diff
Tue, Aug 25, 10:04 AM
Unknown Object (File)
Fri, Aug 21, 2:01 PM
Unknown Object (File)
Fri, Aug 21, 9:45 AM
Unknown Object (File)
Thu, Aug 20, 7:52 AM
Unknown Object (File)
Thu, Aug 13, 6:04 PM
Unknown Object (File)
Thu, Aug 13, 1:26 PM
Unknown Object (File)
Wed, Aug 12, 10:16 PM
View All Files
Subscribers
imp
jhb
jrtc27

Details

Reviewers
markj
jhb
Group Reviewers
riscv
Commits
rG3fd429a7ba29: riscv: better CTR messages in pmap_enter_l2()
rGdd18b62cec62: riscv: better CTR messages in pmap_enter_l2()
Summary

Disambiguate the failure cases.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 47377
Build 44264: arc lint + arc unit

Event Timeline

mhorne created this revision.Sep 14 2022, 2:19 PM
Herald added a subscriber: imp. · View Herald TranscriptSep 14 2022, 2:19 PM
mhorne requested review of this revision.Sep 14 2022, 2:19 PM
Harbormaster completed remote builds in B47377: Diff 110546.Sep 14 2022, 2:19 PM
jrtc27 added a subscriber: jrtc27.Sep 14 2022, 2:25 PM
jrtc27 added inline comments.
sys/riscv/riscv/pmap.c
3238

couldn't above, could not here?

jhb accepted this revision as: jhb.Sep 30 2022, 9:41 PM
jhb added a subscriber: jhb.

I don't care strongly, but I would maybe reword these further, e.g.

"pmap_enter_l2: failed to allocate PT page for va %#lx in pmap %p"

That is, substitute the more detailed explanation for "failure" in the original logs.

This revision is now accepted and ready to land.Sep 30 2022, 9:41 PM
Closed by commit rGdd18b62cec62: riscv: better CTR messages in pmap_enter_l2() (authored by mhorne). · Explain WhyOct 6 2022, 10:06 PM
This revision was automatically updated to reflect the committed changes.
mhorne added a commit: rGdd18b62cec62: riscv: better CTR messages in pmap_enter_l2().
mhorne added a commit: rG3fd429a7ba29: riscv: better CTR messages in pmap_enter_l2().Oct 9 2022, 5:04 PM

Revision Contents
Changeset List

PathSize
sys/
riscv/
riscv/
12 lines

Diff 110546

View Options

sys/riscv/riscv/pmap.c

Loading...